[libvirt] [PATCH 0/3] qemu: Parse CPU stepping from query-cpu-model-expansion

Jiri Denemark posted 3 patches 6 years, 6 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1507640876.git.jdenemar@redhat.com
src/cpu/cpu_x86.c                                  |  16 +-
src/cpu/cpu_x86.h                                  |   3 +-
src/qemu/qemu_capabilities.c                       |   5 +-
tests/cputest.c                                    |   2 +-
.../x86_64-cpuid-Xeon-E7-8890-disabled.xml         |   6 +
.../x86_64-cpuid-Xeon-E7-8890-enabled.xml          |   9 +
.../cputestdata/x86_64-cpuid-Xeon-E7-8890-json.xml |  14 +
tests/cputestdata/x86_64-cpuid-Xeon-E7-8890.json   | 521 +++++++++++++++++++++
tests/cputestdata/x86_64-cpuid-Xeon-E7-8890.xml    |   6 +-
9 files changed, 569 insertions(+), 13 deletions(-)
create mode 100644 tests/cputestdata/x86_64-cpuid-Xeon-E7-8890-disabled.xml
create mode 100644 tests/cputestdata/x86_64-cpuid-Xeon-E7-8890-enabled.xml
create mode 100644 tests/cputestdata/x86_64-cpuid-Xeon-E7-8890-json.xml
create mode 100644 tests/cputestdata/x86_64-cpuid-Xeon-E7-8890.json
[libvirt] [PATCH 0/3] qemu: Parse CPU stepping from query-cpu-model-expansion
Posted by Jiri Denemark 6 years, 6 months ago
Even though only family and model are used for matching CPUID data with
CPU models from cpu_map.xml, stepping is used by x86DataFilterTSX which
is supposed to disable TSX on CPU models with broken TSX support.

This series applies on top of "Fix host-model if the chosen CPU model
has more features in QEMU compared to our cpu_map.xml" series I sent a
week ago.

Jiri Denemark (3):
  qemu: Parse CPU stepping from query-cpu-model-expansion
  cputest: Update Xeon-E7-8890 data
  cputest: Add query-cpu-definitions reply for Xeon-E7-8890

 src/cpu/cpu_x86.c                                  |  16 +-
 src/cpu/cpu_x86.h                                  |   3 +-
 src/qemu/qemu_capabilities.c                       |   5 +-
 tests/cputest.c                                    |   2 +-
 .../x86_64-cpuid-Xeon-E7-8890-disabled.xml         |   6 +
 .../x86_64-cpuid-Xeon-E7-8890-enabled.xml          |   9 +
 .../cputestdata/x86_64-cpuid-Xeon-E7-8890-json.xml |  14 +
 tests/cputestdata/x86_64-cpuid-Xeon-E7-8890.json   | 521 +++++++++++++++++++++
 tests/cputestdata/x86_64-cpuid-Xeon-E7-8890.xml    |   6 +-
 9 files changed, 569 insertions(+), 13 deletions(-)
 create mode 100644 tests/cputestdata/x86_64-cpuid-Xeon-E7-8890-disabled.xml
 create mode 100644 tests/cputestdata/x86_64-cpuid-Xeon-E7-8890-enabled.xml
 create mode 100644 tests/cputestdata/x86_64-cpuid-Xeon-E7-8890-json.xml
 create mode 100644 tests/cputestdata/x86_64-cpuid-Xeon-E7-8890.json

-- 
2.14.2

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH 0/3] qemu: Parse CPU stepping from query-cpu-model-expansion
Posted by John Ferlan 6 years, 6 months ago

On 10/10/2017 09:14 AM, Jiri Denemark wrote:
> Even though only family and model are used for matching CPUID data with
> CPU models from cpu_map.xml, stepping is used by x86DataFilterTSX which
> is supposed to disable TSX on CPU models with broken TSX support.
> 
> This series applies on top of "Fix host-model if the chosen CPU model
> has more features in QEMU compared to our cpu_map.xml" series I sent a
> week ago.
> 
> Jiri Denemark (3):
>   qemu: Parse CPU stepping from query-cpu-model-expansion
>   cputest: Update Xeon-E7-8890 data
>   cputest: Add query-cpu-definitions reply for Xeon-E7-8890
> 
>  src/cpu/cpu_x86.c                                  |  16 +-
>  src/cpu/cpu_x86.h                                  |   3 +-
>  src/qemu/qemu_capabilities.c                       |   5 +-
>  tests/cputest.c                                    |   2 +-
>  .../x86_64-cpuid-Xeon-E7-8890-disabled.xml         |   6 +
>  .../x86_64-cpuid-Xeon-E7-8890-enabled.xml          |   9 +
>  .../cputestdata/x86_64-cpuid-Xeon-E7-8890-json.xml |  14 +
>  tests/cputestdata/x86_64-cpuid-Xeon-E7-8890.json   | 521 +++++++++++++++++++++
>  tests/cputestdata/x86_64-cpuid-Xeon-E7-8890.xml    |   6 +-
>  9 files changed, 569 insertions(+), 13 deletions(-)
>  create mode 100644 tests/cputestdata/x86_64-cpuid-Xeon-E7-8890-disabled.xml
>  create mode 100644 tests/cputestdata/x86_64-cpuid-Xeon-E7-8890-enabled.xml
>  create mode 100644 tests/cputestdata/x86_64-cpuid-Xeon-E7-8890-json.xml
>  create mode 100644 tests/cputestdata/x86_64-cpuid-Xeon-E7-8890.json
> 

Reviewed-by: John Ferlan <jferlan@redhat.com>
(series)

John

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH 0/3] qemu: Parse CPU stepping from query-cpu-model-expansion
Posted by Jiri Denemark 6 years, 6 months ago
On Tue, Oct 17, 2017 at 16:23:49 -0400, John Ferlan wrote:
> 
> 
> On 10/10/2017 09:14 AM, Jiri Denemark wrote:
> > Even though only family and model are used for matching CPUID data with
> > CPU models from cpu_map.xml, stepping is used by x86DataFilterTSX which
> > is supposed to disable TSX on CPU models with broken TSX support.
> > 
> > This series applies on top of "Fix host-model if the chosen CPU model
> > has more features in QEMU compared to our cpu_map.xml" series I sent a
> > week ago.
> > 
> > Jiri Denemark (3):
> >   qemu: Parse CPU stepping from query-cpu-model-expansion
> >   cputest: Update Xeon-E7-8890 data
> >   cputest: Add query-cpu-definitions reply for Xeon-E7-8890
> > 
> >  src/cpu/cpu_x86.c                                  |  16 +-
> >  src/cpu/cpu_x86.h                                  |   3 +-
> >  src/qemu/qemu_capabilities.c                       |   5 +-
> >  tests/cputest.c                                    |   2 +-
> >  .../x86_64-cpuid-Xeon-E7-8890-disabled.xml         |   6 +
> >  .../x86_64-cpuid-Xeon-E7-8890-enabled.xml          |   9 +
> >  .../cputestdata/x86_64-cpuid-Xeon-E7-8890-json.xml |  14 +
> >  tests/cputestdata/x86_64-cpuid-Xeon-E7-8890.json   | 521 +++++++++++++++++++++
> >  tests/cputestdata/x86_64-cpuid-Xeon-E7-8890.xml    |   6 +-
> >  9 files changed, 569 insertions(+), 13 deletions(-)
> >  create mode 100644 tests/cputestdata/x86_64-cpuid-Xeon-E7-8890-disabled.xml
> >  create mode 100644 tests/cputestdata/x86_64-cpuid-Xeon-E7-8890-enabled.xml
> >  create mode 100644 tests/cputestdata/x86_64-cpuid-Xeon-E7-8890-json.xml
> >  create mode 100644 tests/cputestdata/x86_64-cpuid-Xeon-E7-8890.json
> > 
> 
> Reviewed-by: John Ferlan <jferlan@redhat.com>
> (series)

Pushed, thanks.

Jirka

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list