Re: [PATCH] arm: 9016/2: Make symbol 'tmp_pmd_table' static

I'm sorry for making such a stupid mistake. There was only one patch committed before(5615f69bc209 "ARM: 9016/2: Initialize the mapping of KASan shadow memory"), and I used the same subject by mistake.

Thanks for your correction, I will revise the subject and resend it. How about using "arm: mm: kasan_init" in the subject?


On 2021/3/27 18:20, Russell King - ARM Linux admin wrote:

Why do you have 9016/2 in the subject line? That's an identifier from
the patch system which shouldn't be in the subject line.

If you want to refer to something already committed, please do so via
the sha1 git hash and quote the first line of the commit description
within ("...") in the body of your commit description.

Thanks.

On Sat, Mar 27, 2021 at 04:30:18PM +0800, Shixin Liu wrote:

quoted

Symbol 'tmp_pmd_table' is not used outside of kasan_init.c and only used
when CONFIG_ARM_LPAE enabled. So marks it static and add it into CONFIG_ARM_LPAE.

Signed-off-by: Shixin Liu <redacted>
---
 arch/arm/mm/kasan_init.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/arch/arm/mm/kasan_init.c b/arch/arm/mm/kasan_init.c
index 9c348042a724..3a06d3b51f97 100644
--- a/arch/arm/mm/kasan_init.c
+++ b/arch/arm/mm/kasan_init.c

@@ -27,7 +27,9 @@
 
 static pgd_t tmp_pgd_table[PTRS_PER_PGD] __initdata __aligned(PGD_SIZE);
 
-pmd_t tmp_pmd_table[PTRS_PER_PMD] __page_aligned_bss;
+#ifdef CONFIG_ARM_LPAE
+static pmd_t tmp_pmd_table[PTRS_PER_PMD] __page_aligned_bss;
+#endif
 
 static __init void *kasan_alloc_block(size_t size)
 {

-- 
2.25.1

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel