diff options
author | Daniel P. Berrangé <berrange@redhat.com> | 2021-10-29 12:21:06 +0100 |
---|---|---|
committer | Cole Robinson <crobinso@redhat.com> | 2022-01-20 14:16:38 -0500 |
commit | 9bb86f51861dbe312676250482e5df7a82a9b6bd (patch) | |
tree | 4bc0f1629c5724f40cb497a2a7704c5dc91f3450 | |
parent | c1491ae1423f5c9b734344c226ca1340347c7ab1 (diff) | |
download | virt-manager-9bb86f51861dbe312676250482e5df7a82a9b6bd.tar.gz |
virtinst: move method for calculating vCPU count from topology
Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
-rw-r--r-- | virtinst/domain/cpu.py | 14 |
1 files changed, 10 insertions, 4 deletions
diff --git a/virtinst/domain/cpu.py b/virtinst/domain/cpu.py index 222d6191..06858a3f 100644 --- a/virtinst/domain/cpu.py +++ b/virtinst/domain/cpu.py @@ -49,6 +49,15 @@ class _CPUTopology(XMLBuilder): return + def total_vcpus(self): + """ + Determine the CPU count represented by topology + """ + return ((self.sockets or 1) * + (self.dies or 1) * + (self.cores or 1) * + (self.threads or 1)) + # Note: CPU cache is weird. The documentation implies that multiples instances # can be declared, one for each cache level one wishes to define. However, @@ -367,10 +376,7 @@ class DomainCpu(XMLBuilder): Determine the CPU count represented by topology, or 1 if no topology is set """ - return ((self.topology.sockets or 1) * - (self.topology.dies or 1) * - (self.topology.cores or 1) * - (self.topology.threads or 1)) + return self.topology.total_vcpus() def has_topology(self): """ |