On Wed, Jan 10, 2024 at 01:06:30PM +0100, Peter Krempa wrote:
> On Mon, Jan 08, 2024 at 20:26:02 +0100, Andrea Bolognani wrote:
> > diff --git a/src/conf/cpu_conf.h b/src/conf/cpu_conf.h
> > index 3e4c53675c..2694022fed 100644
> > --- a/src/conf/cpu_conf.h
> > +++ b/src/conf/cpu_conf.h
> > @@ -148,6 +148,7 @@ struct _virCPUDef {
> > unsigned int microcodeVersion;
> > unsigned int sockets;
> > unsigned int dies;
> > + unsigned int clusters;
> > unsigned int cores;
> > unsigned int threads;
> > unsigned int sigFamily;
>
> git grep 'cpu->dies' shows that in src/libxl/libxl_capabilities.c
> libxlCapsInitCPU 'dies' is assigned to '1' but not in this patch. With
> the logic to format the cpu element this could cause problems.
It is though:
> > diff --git a/src/libxl/libxl_capabilities.c b/src/libxl/libxl_capabilities.c
> > index dfb602ca2f..522256777d 100644
> > --- a/src/libxl/libxl_capabilities.c
> > +++ b/src/libxl/libxl_capabilities.c
> > @@ -152,6 +152,7 @@ libxlCapsInitCPU(virCaps *caps, libxl_physinfo
> > *phy_info)
> > cpu->cores = phy_info->cores_per_socket;
> > cpu->threads = phy_info->threads_per_core;
> > cpu->dies = 1;
> > + cpu->clusters = 1;
> > cpu->sockets = phy_info->nr_cpus / (cpu->cores * cpu->threads);
Or is this not what you were talking about?
--
Andrea Bolognani / Red Hat / Virtualization
_______________________________________________
Devel mailing list -- [email protected]
To unsubscribe send an email to [email protected]
