On Tue, May 13, 2025 at 11:16:52PM -0400, Xiaoyao Li wrote:
> Date: Tue, 13 May 2025 23:16:52 -0400
> From: Xiaoyao Li <xiaoyao...@intel.com>
> Subject: [PATCH 2/2] i386/hvf: Make CPUID_HT supported
> X-Mailer: git-send-email 2.43.0
>
> Since Commit c6bd2dd63420 ("i386/cpu: Set up CPUID_HT in
> x86_cpu_expand_features() instead of cpu_x86_cpuid()"), CPUID_HT will be
> set in env->features[] in x86_cpu_expand_features() when vcpus >= 2.
>
> Later in x86_cpu_filter_features() it will check against the HVF
> supported bits. It will trigger the warning like
>
> qemu-system-x86_64: warning: host doesn't support requested feature:
> CPUID.01H:EDX.ht [bit 28]
>
> Add CPUID_HT to HVF supported CPUID bits to fix it.
>
> Signed-off-by: Xiaoyao Li <xiaoyao...@intel.com>
> ---
> Note, the issue is totally by my analysis (which should be the same as
> the TCG warnings) because I don't have HVF environment to verify it.
>
> If would be helpful if anyone can help reproduce it and test the patch
> in HVF environment.
Ah, I found someone has reported a bug on HVF. In his log, there's the
ht warning:
https://gitlab.com/qemu-project/qemu/-/issues/2938
But I don't know if this fix can help on his bug...let's wait for his
feedback.
> ---
> target/i386/hvf/x86_cpuid.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Anyway, this fix is fine for me, so,
Reviewed-by: Zhao Liu <zhao1....@intel.com>
