x86/pgtable: drop pXd_none() checks from pXd_free_pYd_table()
authorWill Deacon <[email protected]>
Fri, 28 Dec 2018 08:37:45 +0000 (00:37 -0800)
committerLinus Torvalds <[email protected]>
Fri, 28 Dec 2018 20:11:50 +0000 (12:11 -0800)
The core code already has a check for pXd_none(), so remove it from the
architecture implementation.

Link: http://lkml.kernel.org/r/[email protected]
Signed-off-by: Will Deacon <[email protected]>
Acked-by: Thomas Gleixner <[email protected]>
Reviewed-by: Toshi Kani <[email protected]>
Cc: Chintan Pandya <[email protected]>
Cc: Toshi Kani <[email protected]>
Cc: Michal Hocko <[email protected]>
Cc: Ingo Molnar <[email protected]>
Cc: "H. Peter Anvin" <[email protected]>
Cc: Sean Christopherson <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
arch/x86/mm/pgtable.c

index 59274e2c1ac44c0fb2fb4c004e3e64484b305335..e95a7d6ac8f896b6244f1cde54115b49fab4d2b2 100644 (file)
@@ -811,9 +811,6 @@ int pud_free_pmd_page(pud_t *pud, unsigned long addr)
        pte_t *pte;
        int i;
 
-       if (pud_none(*pud))
-               return 1;
-
        pmd = (pmd_t *)pud_page_vaddr(*pud);
        pmd_sv = (pmd_t *)__get_free_page(GFP_KERNEL);
        if (!pmd_sv)
@@ -855,9 +852,6 @@ int pmd_free_pte_page(pmd_t *pmd, unsigned long addr)
 {
        pte_t *pte;
 
-       if (pmd_none(*pmd))
-               return 1;
-
        pte = (pte_t *)pmd_page_vaddr(*pmd);
        pmd_clear(pmd);