RE: [PATCH] usertools: enhance logic to display NUMA

["Varghese, Vipin" <Vipin.Varghese@amd.com>]

[AMD Official Use Only - General]

> -----Original Message-----
> From: Thomas Monjalon <thomas@monjalon.net>
> Sent: Monday, July 17, 2023 8:37 PM
> To: Stephen Hemminger <stephen@networkplumber.org>; Varghese, Vipin
> <Vipin.Varghese@amd.com>
> Cc: david.marchand@redhat.com; Tummala, Sivaprasad
> <Sivaprasad.Tummala@amd.com>; dev@dpdk.org
> Subject: Re: [PATCH] usertools: enhance logic to display NUMA
>
> Caution: This message originated from an External Source. Use proper
> caution when opening attachments, clicking links, or responding.
>
>
> 14/07/2023 11:14, Varghese, Vipin:
> > From: Thomas Monjalon <thomas@monjalon.net>
> > > 11/07/2023 17:42, Stephen Hemminger:
> > > > On Sat, 26 Mar 2022 00:32:07 -0700 Vipin Varghese
> > > > <vipin.varghese@amd.com> wrote:
> > > >

<<Snipped>>

> > > >
> > > > Git complains because you added new blank line at end of file.
> > > >
> > > > One wording suggestion would be to use the term "Node" instead of
> > > "Numa" in the table.
> > > > And fix heading alignment. The new headings don't look right.
> > > >
> > > > For the case with only single socket, single node, some of the
> > > > headings could be dropped as well.
> > >
> > > I don't understand why we continue working on this script.
> > > I thought we agreed it should be removed in favor of lstopo.
> > >
> >
> > Sorry Thomas, I did not follow your ` I don't understand why we
> > continue working on this script. I thought we agreed it should be
> > removed in favor of lstopo.`
> >
> > From last email from my end `we should promote and document the
> changes provided the existing tool is phased out and use lstopo`.
> >
> > Note:
> > 1. This is with assumption that both Linux and Windows `lstopo` is modified
> and handles `ACPI L3 SRAT NUMA` and `Node per Socket NUMA`.
> > 2. I have not seen a depreciation notice for cpu_layout.py too.
>
> My take is that we should stop working on this script.
> It is better to contribute to lstopo. Did you try?
>
> We probably won't deprecate cpu_layout.py easily.
> The first step would be to add a message when running the script,
> recommending to use lstopo.
> Vipin would you like to write such a patch?

Sure, we can do this. Based on the discussion with Ferruh, we as AMD find a need to update documentation for ` Platform Specific Guides`, which will help to capture various combinations of `NPS` and `L3 for Compute Tiles` using lstopo and other tools as required.

>