On Fri, Jul 24, 2020 at 10:13:52AM +0100, Mark Rutland wrote:

On Fri, Jul 24, 2020 at 01:46:18PM +0530, Anshuman Khandual wrote:

quoted

On 07/24/2020 08:38 AM, Kefeng Wang wrote:

quoted

quoted

Reported-by: Hulk Robot <redacted>
Signed-off-by: Kefeng Wang <redacted>
---
  arch/arm64/kernel/setup.c | 1 +
  1 file changed, 1 insertion(+)

diff --git a/arch/arm64/kernel/setup.c b/arch/arm64/kernel/setup.c
index c793276ec7ad9..3aea05fbb9998 100644
--- a/arch/arm64/kernel/setup.c
+++ b/arch/arm64/kernel/setup.c

@@ -275,6 +275,7 @@ static int __init reserve_memblock_reserved_regions(void)

  arch_initcall(reserve_memblock_reserved_regions);
    u64 __cpu_logical_map[NR_CPUS] = { [0 ... NR_CPUS-1] = INVALID_HWID };
+EXPORT_SYMBOL(__cpu_logical_map);

If modules are using cpu_logical_map(), this looks sane ot me, but I
wonder if we should instead turn cpu_logical_map() into a C wrapper in
smp.c, or at least mark __cpu_logical_map as __ro_after_init lest
someone have the bright idea to fiddle with it.

I'd go for a C wrapper and also change a couple of instances where we
assign a value directly to cpu_logical_map(cpu).

-- 
Catalin

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