Call page_folio() a little earlier so we can use folio_mapping()
instead of page_mapping(), saving a call to compound_head().

Signed-off-by: Matthew Wilcox (Oracle) <wi...@infradead.org>
---
 mm/userfaultfd.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/mm/userfaultfd.c b/mm/userfaultfd.c
index d9dcc7d71a39..e6486923263c 100644
--- a/mm/userfaultfd.c
+++ b/mm/userfaultfd.c
@@ -180,9 +180,9 @@ int mfill_atomic_install_pte(pmd_t *dst_pmd,
        pte_t _dst_pte, *dst_pte;
        bool writable = dst_vma->vm_flags & VM_WRITE;
        bool vm_shared = dst_vma->vm_flags & VM_SHARED;
-       bool page_in_cache = page_mapping(page);
        spinlock_t *ptl;
-       struct folio *folio;
+       struct folio *folio = page_folio(page);
+       bool page_in_cache = folio_mapping(folio);
 
        _dst_pte = mk_pte(page, dst_vma->vm_page_prot);
        _dst_pte = pte_mkdirty(_dst_pte);
@@ -212,7 +212,6 @@ int mfill_atomic_install_pte(pmd_t *dst_pmd,
        if (!pte_none_mostly(ptep_get(dst_pte)))
                goto out_unlock;
 
-       folio = page_folio(page);
        if (page_in_cache) {
                /* Usually, cache pages are already added to LRU */
                if (newly_allocated)
-- 
2.43.0


Reply via email to