[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] i386: Disable OSPKE on CPU model definitions
From: |
Richard Henderson |
Subject: |
Re: [Qemu-devel] [PATCH] i386: Disable OSPKE on CPU model definitions |
Date: |
Tue, 19 Mar 2019 13:47:44 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.4.0 |
On 3/19/19 1:05 PM, Eduardo Habkost wrote:
> Currently, the Cascadelake-Server, Icelake-Client, and
> Icelake-Server are always generating the following warning:
>
> qemu-system-x86_64: warning: \
> host doesn't support requested feature: CPUID.07H:ECX [bit 4]
>
> This happens because OSPKE was never returned by
> GET_SUPPORTED_CPUID or x86_cpu_get_supported_feature_word().
> OSPKE is a runtime flag automatically set by the KVM module or by
> TCG code, was always cleared by x86_cpu_filter_features(), and
> was not supposed to appear on the CPU model table.
>
> Remove the OSPKE flag from the CPU model table entries, to avoid
> the bogus warning and avoid returning invalid feature data on
> query-cpu-* QMP commands. As OSPKE was always cleared by
> x86_cpu_filter_features(), this won't have any guest-visible
> impact.
>
> Include a test case that should detect the problem if we introduce
> a similar bug again.
>
> Fixes: c7a88b52f62b ("i386: Add new model of Cascadelake-Server")
> Fixes: 8a11c62da914 ("i386: Add new CPU model Icelake-{Server,Client}")
> Cc: Tao Xu <address@hidden>
> Cc: Robert Hoo <address@hidden>
> Signed-off-by: Eduardo Habkost <address@hidden>
> ---
> target/i386/cpu.c | 6 +++---
> tests/acceptance/cpu_queries.py | 33 +++++++++++++++++++++++++++++++++
> 2 files changed, 36 insertions(+), 3 deletions(-)
> create mode 100644 tests/acceptance/cpu_queries.py
Reviewed-by: Richard Henderson <address@hidden>
r~