x86/mm: Check for pfn instead of page in vmalloc_sync_one()
authorJoerg Roedel <jroedel@suse.de>
Fri, 19 Jul 2019 18:46:50 +0000 (20:46 +0200)
committerThomas Gleixner <tglx@linutronix.de>
Mon, 22 Jul 2019 08:18:30 +0000 (10:18 +0200)
Do not require a struct page for the mapped memory location because it
might not exist. This can happen when an ioremapped region is mapped with
2MB pages.

Fixes: 5d72b4fba40ef ('x86, mm: support huge I/O mapping capability I/F')
Signed-off-by: Joerg Roedel <jroedel@suse.de>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Reviewed-by: Dave Hansen <dave.hansen@linux.intel.com>
Link: https://lkml.kernel.org/r/20190719184652.11391-2-joro@8bytes.org
arch/x86/mm/fault.c

index 6c46095..e64173d 100644 (file)
@@ -183,7 +183,7 @@ static inline pmd_t *vmalloc_sync_one(pgd_t *pgd, unsigned long address)
        if (!pmd_present(*pmd))
                set_pmd(pmd, *pmd_k);
        else
-               BUG_ON(pmd_page(*pmd) != pmd_page(*pmd_k));
+               BUG_ON(pmd_pfn(*pmd) != pmd_pfn(*pmd_k));
 
        return pmd_k;
 }