Hi Stefano,
> -----Original Message-----
> From: Stefano Stabellini <sstabell...@kernel.org>
> Sent: 2021年9月24日 10:10
> To: Wei Chen <wei.c...@arm.com>
> Cc: xen-devel@lists.xenproject.org; sstabell...@kernel.org; jul...@xen.org;
> Bertrand Marquis <bertrand.marq...@arm.com>; jbeul...@suse.com;
> andrew.coop...@citrix.com; roger....@citrix.com; w...@xen.org
> Subject: Re: [PATCH 25/37] xen/arm: implement bad_srat for Arm NUMA
> initialization
>
> On Thu, 23 Sep 2021, Wei Chen wrote:
> > NUMA initialization will parse information from firmware provided
> > static resource affinity table (ACPI SRAT or DTB). bad_srat if a
> > function that will be used when initialization code encounters
> > some unexcepted errors.
> >
> > In this patch, we introduce Arm version bad_srat for NUMA common
> > initialization code to invoke it.
> >
> > Signed-off-by: Wei Chen <wei.c...@arm.com>
> > ---
> > xen/arch/arm/numa.c | 7 +++++++
> > 1 file changed, 7 insertions(+)
> >
> > diff --git a/xen/arch/arm/numa.c b/xen/arch/arm/numa.c
> > index 3755b01ef4..5209d3de4d 100644
> > --- a/xen/arch/arm/numa.c
> > +++ b/xen/arch/arm/numa.c
> > @@ -18,6 +18,7 @@
> > *
> > */
> > #include <xen/init.h>
> > +#include <xen/nodemask.h>
> > #include <xen/numa.h>
> >
> > static uint8_t __read_mostly
> > @@ -25,6 +26,12 @@ node_distance_map[MAX_NUMNODES][MAX_NUMNODES] = {
> > { 0 }
> > };
> >
> > +__init void bad_srat(void)
> > +{
> > + printk(KERN_ERR "NUMA: Firmware SRAT table not used.\n");
> > + fw_numa = -1;
> > +}
>
> I realize that the series keeps the "srat" terminology everywhere on DT
> too. I wonder if it is worth replacing srat with something like
> "numa_distance" everywhere as appropriate. I am adding the x86
> maintainers for an opinion.
>
> If you guys prefer to keep srat (if nothing else, it is concise), I am
> also OK with keeping srat although it is not technically accurate.
I have placed some comments in patch#23 for srat.
I wouldn't like to replace srat by numa_distance. because srat not only
contains distance, and it also includes some affinity information for CPU
or other devices.
