]> www.infradead.org Git - users/jedix/linux-maple.git/commitdiff
dax: remove access to page->index
authorMatthew Wilcox (Oracle) <willy@infradead.org>
Mon, 16 Dec 2024 15:53:55 +0000 (15:53 +0000)
committerAndrew Morton <akpm@linux-foundation.org>
Fri, 28 Feb 2025 01:00:02 +0000 (17:00 -0800)
This looks like a complete mess (why are we setting page->index at page
fault time?), but I no longer care about DAX, and there's no reason to let
DAX hold us back from removing page->index.

Link: https://lkml.kernel.org/r/20241216155408.8102-1-willy@infradead.org
Signed-off-by: Matthew Wilcox (Oracle) <willy@infradead.org>
Reviewed-by: Jane Chu <jane.chu@oracle.com>
Reviewed-by: Dan Williams <dan.j.williams@intel.com>
Cc: Alistair Popple <apopple@nvidia.com>
Cc: Dave Jiang <dave.jiang@intel.com>
Cc: Vishal Verma <vishal.l.verma@intel.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
drivers/dax/device.c

index 6d74e62bbee0d37125ebc9decc1d342449bbb150..bc871a34b9cd2d3a12a88bd5aa61f194e39fb86a 100644 (file)
@@ -89,14 +89,13 @@ static void dax_set_mapping(struct vm_fault *vmf, pfn_t pfn,
                        ALIGN_DOWN(vmf->address, fault_size));
 
        for (i = 0; i < nr_pages; i++) {
-               struct page *page = pfn_to_page(pfn_t_to_pfn(pfn) + i);
+               struct folio *folio = pfn_folio(pfn_t_to_pfn(pfn) + i);
 
-               page = compound_head(page);
-               if (page->mapping)
+               if (folio->mapping)
                        continue;
 
-               page->mapping = filp->f_mapping;
-               page->index = pgoff + i;
+               folio->mapping = filp->f_mapping;
+               folio->index = pgoff + i;
        }
 }