Re: [PATCH v7 10/14] arm64/numa: define numa_distance as array to simplify code

From: Will Deacon
Date: Fri Aug 26 2016 - 11:29:20 EST


On Wed, Aug 24, 2016 at 03:44:49PM +0800, Zhen Lei wrote:
> 1. MAX_NUMNODES is base on CONFIG_NODES_SHIFT, the default value of the
> latter is very small now.
> 2. Suppose the default value of MAX_NUMNODES is enlarged to 64, so the
> size of numa_distance is 4K, it's still acceptable if run the Image
> on other processors.
> 3. It will make function __node_distance quicker than before.
>
> Signed-off-by: Zhen Lei <thunder.leizhen@xxxxxxxxxx>
> ---
> arch/arm64/include/asm/numa.h | 1 -
> arch/arm64/mm/numa.c | 74 +++----------------------------------------
> 2 files changed, 5 insertions(+), 70 deletions(-)

I fail to see the advantages of this patch. Do you have some compelling
performance figures or something?

Will