[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[RFC 01/52] hw/smbios: Fix smbios_smp_sockets caculation
From: |
Zhao Liu |
Subject: |
[RFC 01/52] hw/smbios: Fix smbios_smp_sockets caculation |
Date: |
Mon, 13 Feb 2023 17:49:44 +0800 |
From: Zhao Liu <zhao1.liu@intel.com>
Here're 2 mistakes:
1. 003f230 (machine: Tweak the order of topology members in struct
CpuTopology) changes the meaning of smp.cores but doesn't fix
original smp.cores uses. And because of the introduction of cluster,
now smp.cores means the number of cores in one cluster. So smp.cores
* smp.threads just means the cpus in a cluster not in a socket.
2. smp.cpus means the number of initial online cpus, not the total
number of cpus. For such topology calculation, smp.max_cpus
should be considered.
Since the number of sockets has already been recorded in smp structure,
use smp.sockets directly.
Cc: Michael S. Tsirkin <mst@redhat.com>
Cc: Igor Mammedov <imammedo@redhat.com>
Cc: Ani Sinha <ani@anisinha.ca>
Fixes: 003f230 (machine: Tweak the order of topology members in struct
CpuTopology)
Signed-off-by: Zhao Liu <zhao1.liu@intel.com>
---
hw/smbios/smbios.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/hw/smbios/smbios.c b/hw/smbios/smbios.c
index 4869566cf5df..993e8d899b9a 100644
--- a/hw/smbios/smbios.c
+++ b/hw/smbios/smbios.c
@@ -1086,8 +1086,7 @@ void smbios_get_tables(MachineState *ms,
smbios_build_type_2_table();
smbios_build_type_3_table();
- smbios_smp_sockets = DIV_ROUND_UP(ms->smp.cpus,
- ms->smp.cores * ms->smp.threads);
+ smbios_smp_sockets = ms->smp.sockets;
assert(smbios_smp_sockets >= 1);
for (i = 0; i < smbios_smp_sockets; i++) {
--
2.34.1
- [RFC 00/52] Introduce hybrid CPU topology, Zhao Liu, 2023/02/13
- [RFC 01/52] hw/smbios: Fix smbios_smp_sockets caculation,
Zhao Liu <=
- [RFC 02/52] hw/smbios: Fix thread count in type4, Zhao Liu, 2023/02/13
- [RFC 03/52] hw/smbios: Fix core count in type4, Zhao Liu, 2023/02/13
- [RFC 04/52] i386/WHPX: Fix error message when fail to set ProcessorCount, Zhao Liu, 2023/02/13
- [RFC 05/52] hw/core/machine: Rename machine-smp.c to machine-topo.c, Zhao Liu, 2023/02/13
- [RFC 06/52] hw/cpu: Introduce hybrid CPU topology, Zhao Liu, 2023/02/13