Re: [PATCH 2/2] tools: vm: slabinfo: Add numa information for objects

From: Tobin C. Harding
Date: Wed Feb 19 2020 - 15:27:02 EST


On Tue, Feb 18, 2020 at 04:24:54PM +0000, Christopher Lameter wrote:
> On Mon, 17 Feb 2020, Tobin C. Harding wrote:
>
> > Add a field to the slabinfo struct for the NUMA information and
> > output it during a NUMA report as is done for `slabs` and `partial`.
>
> How will this look? Note that there are boxes now with potentially huge
> NUMA nodes (AMD Rome can already do 32 with an optimal BIOS layout for
> minimal latency).
>
> Maybe make it optional with a --numa switch or so?


Cool, can do. Thanks for the review.

Tobin