arm64: mmu: drop pXd_present() checks from pXd_free_pYd_table()
authorWill Deacon <will.deacon@arm.com>
Fri, 28 Dec 2018 08:37:42 +0000 (00:37 -0800)
committerLinus Torvalds <torvalds@linux-foundation.org>
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/1544120495-17438-3-git-send-email-will.deacon@arm.com
Signed-off-by: Will Deacon <will.deacon@arm.com>
Cc: Chintan Pandya <cpandya@codeaurora.org>
Cc: Toshi Kani <toshi.kani@hpe.com>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: Michal Hocko <mhocko@suse.com>
Cc: "H. Peter Anvin" <hpa@zytor.com>
Cc: Ingo Molnar <mingo@elte.hu>
Cc: Sean Christopherson <sean.j.christopherson@intel.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
arch/arm64/mm/mmu.c

index da513a1..13b8036 100644 (file)
@@ -1003,10 +1003,8 @@ int pmd_free_pte_page(pmd_t *pmdp, unsigned long addr)
 
        pmd = READ_ONCE(*pmdp);
 
-       if (!pmd_present(pmd))
-               return 1;
        if (!pmd_table(pmd)) {
-               VM_WARN_ON(!pmd_table(pmd));
+               VM_WARN_ON(1);
                return 1;
        }
 
@@ -1026,10 +1024,8 @@ int pud_free_pmd_page(pud_t *pudp, unsigned long addr)
 
        pud = READ_ONCE(*pudp);
 
-       if (!pud_present(pud))
-               return 1;
        if (!pud_table(pud)) {
-               VM_WARN_ON(!pud_table(pud));
+               VM_WARN_ON(1);
                return 1;
        }