Message ID | 20220406145802.538416-6-mpe@ellerman.id.au (mailing list archive) |
---|---|
State | New |
Headers | show |
Series | [1/6] powerpc: Fix virt_addr_valid() for 64-bit Book3E & 32-bit | expand |
Context | Check | Description |
---|---|---|
snowpatch_ozlabs/github-powerpc_ppctests | success | Successfully ran 8 jobs. |
snowpatch_ozlabs/github-powerpc_selftests | success | Successfully ran 8 jobs. |
snowpatch_ozlabs/github-powerpc_clang | success | Successfully ran 7 jobs. |
snowpatch_ozlabs/github-powerpc_sparse | success | Successfully ran 4 jobs. |
diff --git a/arch/powerpc/mm/mem.c b/arch/powerpc/mm/mem.c index 4d221d033804..81e9d948a8e8 100644 --- a/arch/powerpc/mm/mem.c +++ b/arch/powerpc/mm/mem.c @@ -305,6 +305,13 @@ void __init mem_init(void) MODULES_VADDR, MODULES_END); #endif #endif /* CONFIG_PPC32 */ + + // Check virt_addr_valid() works as expected + WARN_ON(!virt_addr_valid(PAGE_OFFSET)); + WARN_ON(virt_addr_valid(PAGE_OFFSET - 1)); + WARN_ON(virt_addr_valid(high_memory)); + WARN_ON(virt_addr_valid(VMALLOC_START)); + WARN_ON(virt_addr_valid(VMALLOC_END - 1)); } void free_initmem(void)
We've had several bugs now with virt_addr_valid() being wrong, so lets add some always-enabled boot time checks that it behaves as expected. Signed-off-by: Michael Ellerman <mpe@ellerman.id.au> --- arch/powerpc/mm/mem.c | 7 +++++++ 1 file changed, 7 insertions(+)