i386/cpu: fixup number of addressable IDs for processor cores in the physical package
authorChuang Xu <xuchuangxclwt@bytedance.com>
Tue, 11 Jun 2024 03:23:14 +0000 (11:23 +0800)
committerPaolo Bonzini <pbonzini@redhat.com>
Tue, 11 Jun 2024 12:25:22 +0000 (14:25 +0200)
commit903916f0a017fe4b7789f1c6c6982333a5a71876
tree305d85ae790d642a295b9721e772e7f1da2382b9
parentb4912afa5f9fc87afd82950965941e31f3b2b06c
i386/cpu: fixup number of addressable IDs for processor cores in the physical package

When QEMU is started with:
-cpu host,host-cache-info=on,l3-cache=off \
-smp 2,sockets=1,dies=1,cores=1,threads=2
Guest can't acquire maximum number of addressable IDs for processor cores in
the physical package from CPUID[04H].

When creating a CPU topology of 1 core per package, host-cache-info only
uses the Host's addressable core IDs field (CPUID.04H.EAX[bits 31-26]),
resulting in a conflict (on the multicore Host) between the Guest core
topology information in this field and the Guest's actual cores number.

Fix it by removing the unnecessary condition to cover 1 core per package
case. This is safe because cores_per_pkg will not be 0 and will be at
least 1.

Fixes: d7caf13b5fcf ("x86: cpu: fixup number of addressable IDs for logical processors sharing cache")
Signed-off-by: Guixiong Wei <weiguixiong@bytedance.com>
Signed-off-by: Yipeng Yin <yinyipeng@bytedance.com>
Signed-off-by: Chuang Xu <xuchuangxclwt@bytedance.com>
Reviewed-by: Zhao Liu <zhao1.liu@intel.com>
Message-ID: <20240611032314.64076-1-xuchuangxclwt@bytedance.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
target/i386/cpu.c