[lustre-devel] [PATCH v3 19/26] staging: lustre: libcfs: report NUMA node instead of just node
James Simmons
jsimmons at infradead.org
Mon Jun 25 17:54:43 PDT 2018
> On Sun, Jun 24 2018, James Simmons wrote:
>
> > From: Dmitry Eremin <dmitry.eremin at intel.com>
> >
> > Reporting "HW nodes" is too generic. It really is reporting
> > "HW NUMA nodes". Update the debug message.
>
> I'm not happy with this patch description.....
How about:
In the HPC world a node refers to the actual whole computer system
used in a cluster. Reporting just "HW nodes" is not clear so change
the debug report to "HW NUMA nodes" since this report the number
of NUMA nodes in use by Lustre.
> >
> > Signed-off-by: Dmitry Eremin <dmitry.eremin at intel.com>
> > WC-bug-id: https://jira.whamcloud.com/browse/LU-8703
> > Reviewed-on: https://review.whamcloud.com/23306
> > Reviewed-by: James Simmons <uja.ornl at yahoo.com>
> > Reviewed-by: Andreas Dilger <adilger at whamcloud.com>
> > Reviewed-by: Patrick Farrell <paf at cray.com>
> > Reviewed-by: Olaf Weber <olaf.weber at hpe.com>
> > Reviewed-by: Oleg Drokin <green at whamcloud.com>
> > Signed-off-by: James Simmons <jsimmons at infradead.org>
> > ---
> > drivers/staging/lustre/include/linux/libcfs/libcfs_cpu.h | 2 ++
> > drivers/staging/lustre/lnet/libcfs/libcfs_cpu.c | 2 +-
> > drivers/staging/lustre/lnet/lnet/lib-msg.c | 2 ++
> > 3 files changed, 5 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/staging/lustre/include/linux/libcfs/libcfs_cpu.h b/drivers/staging/lustre/include/linux/libcfs/libcfs_cpu.h
> > index 2bb2140..29c5071 100644
> > --- a/drivers/staging/lustre/include/linux/libcfs/libcfs_cpu.h
> > +++ b/drivers/staging/lustre/include/linux/libcfs/libcfs_cpu.h
> > @@ -90,6 +90,8 @@ struct cfs_cpu_partition {
> > unsigned int *cpt_distance;
> > /* spread rotor for NUMA allocator */
> > int cpt_spread_rotor;
> > + /* NUMA node if cpt_nodemask is empty */
> > + int cpt_node;
> > };
>
> It doesn't give any reason why this (unused) field was added.
> So I've removed it.
>
>
> >
> >
> > diff --git a/drivers/staging/lustre/lnet/libcfs/libcfs_cpu.c b/drivers/staging/lustre/lnet/libcfs/libcfs_cpu.c
> > index 18925c7..86afa31 100644
> > --- a/drivers/staging/lustre/lnet/libcfs/libcfs_cpu.c
> > +++ b/drivers/staging/lustre/lnet/libcfs/libcfs_cpu.c
> > @@ -1142,7 +1142,7 @@ int cfs_cpu_init(void)
> >
> > put_online_cpus();
> >
> > - LCONSOLE(0, "HW nodes: %d, HW CPU cores: %d, npartitions: %d\n",
> > + LCONSOLE(0, "HW NUMA nodes: %d, HW CPU cores: %d, npartitions: %d\n",
> > num_online_nodes(), num_online_cpus(),
> > cfs_cpt_number(cfs_cpt_tab));
> > return 0;
>
> It does explain this hunk, which is fine.
>
>
> > diff --git a/drivers/staging/lustre/lnet/lnet/lib-msg.c b/drivers/staging/lustre/lnet/lnet/lib-msg.c
> > index 0091273..27bdefa 100644
> > --- a/drivers/staging/lustre/lnet/lnet/lib-msg.c
> > +++ b/drivers/staging/lustre/lnet/lnet/lib-msg.c
> > @@ -568,6 +568,8 @@
> >
> > /* number of CPUs */
> > container->msc_nfinalizers = cfs_cpt_weight(lnet_cpt_table(), cpt);
> > + if (container->msc_nfinalizers == 0)
> > + container->msc_nfinalizers = 1;
>
> It doesn't justify this at all.
>
> I guess this was meant to be in the previous patch, so I've moved it.
>
> Thanks,
> NeilBrown
>
>
> >
> > container->msc_finalizers = kvzalloc_cpt(container->msc_nfinalizers *
> > sizeof(*container->msc_finalizers),
> > --
> > 1.8.3.1
>
More information about the lustre-devel
mailing list