[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 2/2] target/i386: add clflushopt to "Skylake-
From: |
Eduardo Habkost |
Subject: |
Re: [Qemu-devel] [PATCH v3 2/2] target/i386: add clflushopt to "Skylake-Server" cpu model |
Date: |
Tue, 19 Dec 2017 14:35:20 -0200 |
User-agent: |
Mutt/1.9.1 (2017-09-22) |
On Tue, Dec 19, 2017 at 11:37:30AM +0800, Haozhong Zhang wrote:
> CPUID_7_0_EBX_CLFLUSHOPT is missed in current "Skylake-Server" cpu
> model. Add it to "Skylake-Server" cpu model on pc-i440fx-2.12 and
> pc-q35-2.12. Keep it disabled in "Skylake-Server" cpu model on older
> machine types.
>
> Signed-off-by: Haozhong Zhang <address@hidden>
Reviewed-by: Eduardo Habkost <address@hidden>
Michael, do you want to merge it through your tree as it needs
the pc-2.12 patch?
> ---
> include/hw/i386/pc.h | 5 +++++
> target/i386/cpu.c | 2 +-
> 2 files changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/include/hw/i386/pc.h b/include/hw/i386/pc.h
> index 83d825e896..bb49165fe0 100644
> --- a/include/hw/i386/pc.h
> +++ b/include/hw/i386/pc.h
> @@ -323,6 +323,11 @@ bool e820_get_entry(int, uint32_t, uint64_t *, uint64_t
> *);
>
> #define PC_COMPAT_2_11 \
> HW_COMPAT_2_11 \
> + {\
> + .driver = "Skylake-Server" "-" TYPE_X86_CPU,\
> + .property = "clflushopt",\
> + .value = "off",\
> + },
>
> #define PC_COMPAT_2_10 \
> HW_COMPAT_2_10 \
> diff --git a/target/i386/cpu.c b/target/i386/cpu.c
> index 82603e3130..54574fa232 100644
> --- a/target/i386/cpu.c
> +++ b/target/i386/cpu.c
> @@ -1376,7 +1376,7 @@ static X86CPUDefinition builtin_x86_defs[] = {
> CPUID_7_0_EBX_SMAP | CPUID_7_0_EBX_MPX | CPUID_7_0_EBX_CLWB |
> CPUID_7_0_EBX_AVX512F | CPUID_7_0_EBX_AVX512DQ |
> CPUID_7_0_EBX_AVX512BW | CPUID_7_0_EBX_AVX512CD |
> - CPUID_7_0_EBX_AVX512VL,
> + CPUID_7_0_EBX_AVX512VL | CPUID_7_0_EBX_CLFLUSHOPT,
> /* Missing: XSAVES (not supported by some Linux versions,
> * including v4.1 to v4.12).
> * KVM doesn't yet expose any XSAVES state save component,
> --
> 2.14.1
>
>
--
Eduardo