sparc64: kern_addr_valid regression
authorbob picco <bob.picco@oracle.com>
Fri, 10 Mar 2017 19:31:19 +0000 (14:31 -0500)
committerDavid S. Miller <davem@davemloft.net>
Tue, 28 Mar 2017 04:20:23 +0000 (21:20 -0700)
I encountered this bug when using /proc/kcore to examine the kernel. Plus a
coworker inquired about debugging tools. We computed pa but did
not use it during the maximum physical address bits test. Instead we used
the identity mapped virtual address which will always fail this test.

I believe the defect came in here:
[bpicco@zareason linus.git]$ git describe --contains bb4e6e85daa52
v3.18-rc1~87^2~4
.

Signed-off-by: Bob Picco <bob.picco@oracle.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
arch/sparc/mm/init_64.c

index 3328043e990c2b61e1994e439383f22853d2431d..0cda653ae007645fa01f05b4c40518332159a6ac 100644 (file)
@@ -1567,7 +1567,7 @@ bool kern_addr_valid(unsigned long addr)
        if ((long)addr < 0L) {
                unsigned long pa = __pa(addr);
 
-               if ((addr >> max_phys_bits) != 0UL)
+               if ((pa >> max_phys_bits) != 0UL)
                        return false;
 
                return pfn_valid(pa >> PAGE_SHIFT);