ion: use vm_insert_pfn for faulted pages
Most ion userspace mappings are created with remap_pfn_range. Use vm_insert_pfn instead of vm_insert_page to make faulted cached mappings look more like uncached mappings. Signed-off-by: Colin Cross <ccross@android.com> Signed-off-by: John Stultz <john.stultz@linaro.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
e460bc5e43
commit
462be0c616
1 changed files with 6 additions and 3 deletions
|
@ -890,14 +890,15 @@ static void ion_buffer_sync_for_device(struct ion_buffer *buffer,
|
||||||
int ion_vm_fault(struct vm_area_struct *vma, struct vm_fault *vmf)
|
int ion_vm_fault(struct vm_area_struct *vma, struct vm_fault *vmf)
|
||||||
{
|
{
|
||||||
struct ion_buffer *buffer = vma->vm_private_data;
|
struct ion_buffer *buffer = vma->vm_private_data;
|
||||||
|
unsigned long pfn;
|
||||||
int ret;
|
int ret;
|
||||||
|
|
||||||
mutex_lock(&buffer->lock);
|
mutex_lock(&buffer->lock);
|
||||||
ion_buffer_page_dirty(buffer->pages + vmf->pgoff);
|
ion_buffer_page_dirty(buffer->pages + vmf->pgoff);
|
||||||
|
|
||||||
BUG_ON(!buffer->pages || !buffer->pages[vmf->pgoff]);
|
BUG_ON(!buffer->pages || !buffer->pages[vmf->pgoff]);
|
||||||
ret = vm_insert_page(vma, (unsigned long)vmf->virtual_address,
|
|
||||||
ion_buffer_page(buffer->pages[vmf->pgoff]));
|
pfn = page_to_pfn(ion_buffer_page(buffer->pages[vmf->pgoff]));
|
||||||
|
ret = vm_insert_pfn(vma, (unsigned long)vmf->virtual_address, pfn);
|
||||||
mutex_unlock(&buffer->lock);
|
mutex_unlock(&buffer->lock);
|
||||||
if (ret)
|
if (ret)
|
||||||
return VM_FAULT_ERROR;
|
return VM_FAULT_ERROR;
|
||||||
|
@ -956,6 +957,8 @@ static int ion_mmap(struct dma_buf *dmabuf, struct vm_area_struct *vma)
|
||||||
}
|
}
|
||||||
|
|
||||||
if (ion_buffer_fault_user_mappings(buffer)) {
|
if (ion_buffer_fault_user_mappings(buffer)) {
|
||||||
|
vma->vm_flags |= VM_IO | VM_PFNMAP | VM_DONTEXPAND |
|
||||||
|
VM_DONTDUMP;
|
||||||
vma->vm_private_data = buffer;
|
vma->vm_private_data = buffer;
|
||||||
vma->vm_ops = &ion_vma_ops;
|
vma->vm_ops = &ion_vma_ops;
|
||||||
ion_vm_open(vma);
|
ion_vm_open(vma);
|
||||||
|
|
Loading…
Add table
Reference in a new issue