drm/etnaviv: switch to PFN mappings
authorLucas Stach <l.stach@pengutronix.de>
Fri, 9 Sep 2022 09:20:44 +0000 (11:20 +0200)
committerJaehoon Chung <jh80.chung@samsung.com>
Wed, 13 Mar 2024 06:58:55 +0000 (15:58 +0900)
There is no reason to use page based mappings, as the established
mappings are special driver mappings anyways and should not be
handled like normal pages.

Be consistent with what other drivers do and use raw PFN based
mappings.

Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
(cherry picked from commit 8304fb3b0e7d56aa04c19eb239f0d7ebca793fe0)
Signed-off-by: Jaehoon Chung <jh80.chung@samsung.com>
drivers/gpu/drm/etnaviv/etnaviv_gem.c

index e3dea85451c621fd79cb5a4c985e3011ddba6b96..706aa3ff941b5ce535ea2d13d08c333cefa222f8 100644 (file)
@@ -130,8 +130,7 @@ static int etnaviv_gem_mmap_obj(struct etnaviv_gem_object *etnaviv_obj,
 {
        pgprot_t vm_page_prot;
 
-       vma->vm_flags &= ~VM_PFNMAP;
-       vma->vm_flags |= VM_MIXEDMAP;
+       vma->vm_flags |= VM_PFNMAP;
 
        vm_page_prot = vm_get_page_prot(vma->vm_flags);
 
@@ -176,7 +175,8 @@ static vm_fault_t etnaviv_gem_fault(struct vm_fault *vmf)
        struct vm_area_struct *vma = vmf->vma;
        struct drm_gem_object *obj = vma->vm_private_data;
        struct etnaviv_gem_object *etnaviv_obj = to_etnaviv_bo(obj);
-       struct page **pages, *page;
+       struct page **pages;
+       unsigned long pfn;
        pgoff_t pgoff;
        int err;
 
@@ -200,12 +200,12 @@ static vm_fault_t etnaviv_gem_fault(struct vm_fault *vmf)
        /* We don't use vmf->pgoff since that has the fake offset: */
        pgoff = (vmf->address - vma->vm_start) >> PAGE_SHIFT;
 
-       page = pages[pgoff];
+       pfn = page_to_pfn(pages[pgoff]);
 
        VERB("Inserting %p pfn %lx, pa %lx", (void *)vmf->address,
-            page_to_pfn(page), page_to_pfn(page) << PAGE_SHIFT);
+            pfn, pfn << PAGE_SHIFT);
 
-       return vmf_insert_page(vma, vmf->address, page);
+       return vmf_insert_pfn(vma, vmf->address, pfn);
 }
 
 int etnaviv_gem_mmap_offset(struct drm_gem_object *obj, u64 *offset)