Skip to content
This repository was archived by the owner on May 12, 2021. It is now read-only.

Commit 4c5527f

Browse files
authored
Merge pull request #1606 from devimc/topic/virtcontainers/fixCpuTopology
virtcontainers: fix invalid CPU topology
2 parents a7e2bbd + fa5de87 commit 4c5527f

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

virtcontainers/qemu_arch_base.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -257,7 +257,7 @@ func (q *qemuArchBase) bridges(number uint32) []types.PCIBridge {
257257
func (q *qemuArchBase) cpuTopology(vcpus, maxvcpus uint32) govmmQemu.SMP {
258258
smp := govmmQemu.SMP{
259259
CPUs: vcpus,
260-
Sockets: vcpus,
260+
Sockets: maxvcpus,
261261
Cores: defaultCores,
262262
Threads: defaultThreads,
263263
MaxCPUs: maxvcpus,

virtcontainers/qemu_arch_base_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -165,7 +165,7 @@ func TestQemuArchBaseCPUTopology(t *testing.T) {
165165

166166
expectedSMP := govmmQemu.SMP{
167167
CPUs: vcpus,
168-
Sockets: vcpus,
168+
Sockets: defaultMaxQemuVCPUs,
169169
Cores: defaultCores,
170170
Threads: defaultThreads,
171171
MaxCPUs: defaultMaxQemuVCPUs,

0 commit comments

Comments
 (0)