Re: [PATCH 03/11] conf: Allow specifying CPU clusters

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Jan 08, 2024 at 20:26:02 +0100, Andrea Bolognani wrote:
> The default number of CPU clusters is 1, and values other than
> that one are currently rejected by all hypervisor drivers.
> 
> Signed-off-by: Andrea Bolognani <abologna@xxxxxxxxxx>
> ---
>  src/bhyve/bhyve_command.c                        |  5 +++++
>  src/conf/cpu_conf.c                              | 16 +++++++++++++++-
>  src/conf/cpu_conf.h                              |  1 +
>  src/conf/domain_conf.c                           |  1 +
>  src/conf/schemas/cputypes.rng                    |  5 +++++
>  src/cpu/cpu.c                                    |  1 +
>  src/libxl/libxl_capabilities.c                   |  1 +
>  src/qemu/qemu_command.c                          |  5 +++++
>  src/vmx/vmx.c                                    |  7 +++++++
>  .../x86_64-host+guest,model486-result.xml        |  2 +-
>  .../x86_64-host+guest,models-result.xml          |  2 +-
>  tests/cputestdata/x86_64-host+guest-result.xml   |  2 +-
>  tests/cputestdata/x86_64-host+guest.xml          |  2 +-
>  .../x86_64-host+host-model-nofallback.xml        |  2 +-
>  ...host-Haswell-noTSX+Haswell,haswell-result.xml |  2 +-
>  ...aswell-noTSX+Haswell-noTSX,haswell-result.xml |  2 +-
>  ...4-host-Haswell-noTSX+Haswell-noTSX-result.xml |  2 +-
>  .../x86_64-host-worse+guest-result.xml           |  2 +-
>  .../ppc64-modern-bulk-result-conf.xml            |  2 +-
>  .../ppc64-modern-bulk-result-live.xml            |  2 +-
>  .../ppc64-modern-individual-result-conf.xml      |  2 +-
>  .../ppc64-modern-individual-result-live.xml      |  2 +-
>  .../x86-modern-bulk-result-conf.xml              |  2 +-
>  .../x86-modern-bulk-result-live.xml              |  2 +-
>  .../x86-modern-individual-add-result-conf.xml    |  2 +-
>  .../x86-modern-individual-add-result-live.xml    |  2 +-
>  ...e-timeout+graphics-spice-timeout-password.xml |  2 +-
>  .../qemuhotplug-graphics-spice-timeout.xml       |  2 +-
>  .../fd-memory-no-numa-topology.xml               |  2 +-
>  .../qemuxml2argvdata/fd-memory-numa-topology.xml |  2 +-
>  .../fd-memory-numa-topology2.xml                 |  2 +-
>  .../fd-memory-numa-topology3.xml                 |  2 +-
>  tests/qemuxml2argvdata/hugepages-nvdimm.xml      |  2 +-
>  .../memfd-memory-default-hugepage.xml            |  2 +-
>  tests/qemuxml2argvdata/memfd-memory-numa.xml     |  2 +-
>  .../memory-hotplug-nvdimm-access.xml             |  2 +-
>  .../memory-hotplug-nvdimm-align.xml              |  2 +-
>  .../memory-hotplug-nvdimm-label.xml              |  2 +-
>  .../memory-hotplug-nvdimm-pmem.xml               |  2 +-
>  .../memory-hotplug-nvdimm-readonly.xml           |  2 +-
>  tests/qemuxml2argvdata/memory-hotplug-nvdimm.xml |  2 +-
>  .../memory-hotplug-virtio-mem.xml                |  2 +-
>  .../memory-hotplug-virtio-pmem.xml               |  2 +-
>  .../cpu-numa-disjoint.x86_64-latest.xml          |  2 +-
>  .../cpu-numa-disordered.x86_64-latest.xml        |  2 +-
>  .../cpu-numa-memshared.x86_64-latest.xml         |  2 +-
>  .../cpu-numa-no-memory-element.x86_64-latest.xml |  2 +-
>  .../cpu-numa1.x86_64-latest.xml                  |  2 +-
>  .../cpu-numa2.x86_64-latest.xml                  |  2 +-
>  .../memory-hotplug-dimm-addr.x86_64-latest.xml   |  2 +-
>  .../memory-hotplug-dimm.x86_64-latest.xml        |  2 +-
>  .../memory-hotplug-multiple.x86_64-latest.xml    |  2 +-
>  ...plug-nvdimm-ppc64-abi-update.ppc64-latest.xml |  2 +-
>  .../memory-hotplug-nvdimm-ppc64.ppc64-latest.xml |  2 +-
>  .../memory-hotplug.x86_64-latest.xml             |  2 +-
>  ...mad-auto-memory-vcpu-cpuset.x86_64-latest.xml |  2 +-
>  ...cpu-no-cpuset-and-placement.x86_64-latest.xml |  2 +-
>  ...numad-auto-vcpu-no-numatune.x86_64-latest.xml |  2 +-
>  ...mad-static-vcpu-no-numatune.x86_64-latest.xml |  2 +-
>  .../pci-expander-bus.x86_64-latest.xml           |  2 +-
>  .../pcie-expander-bus.x86_64-latest.xml          |  2 +-
>  .../pseries-phb-numa-node.ppc64-latest.xml       |  2 +-
>  tests/vmx2xmldata/esx-in-the-wild-10.xml         |  2 +-
>  tests/vmx2xmldata/esx-in-the-wild-8.xml          |  2 +-
>  tests/vmx2xmldata/esx-in-the-wild-9.xml          |  2 +-
>  65 files changed, 97 insertions(+), 57 deletions(-)

[...]

> 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.

Reviewed-by: Peter Krempa <pkrempa@xxxxxxxxxx>
_______________________________________________
Devel mailing list -- devel@xxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxx




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux