mbox series

[for,7.2,0/2] s390x/s390-virtio-ccw:

Message ID 20221107161349.1032730-1-clg@kaod.org
Headers show
Series s390x/s390-virtio-ccw: | expand

Message

Cédric Le Goater Nov. 7, 2022, 4:13 p.m. UTC
From: Cédric Le Goater <clg@redhat.com>

Hello,

This series removes the redundant "zpcii-disable" machine property and
and introduces a GlobalProperty compat array to maintain the same
behaviour on older machines.

Thanks,

C.

Cédric Le Goater (2):
  Revert "s390x/s390-virtio-ccw: add zpcii-disable machine property"
  s390x/s390-virtio-ccw: Switch off zPCI enhancements on older machines

 include/hw/s390x/s390-virtio-ccw.h |  1 -
 hw/s390x/s390-pci-kvm.c            |  4 +---
 hw/s390x/s390-virtio-ccw.c         | 29 +++++------------------------
 util/qemu-config.c                 |  4 ----
 qemu-options.hx                    |  8 +-------
 5 files changed, 7 insertions(+), 39 deletions(-)

Comments

Matthew Rosato Nov. 7, 2022, 4:49 p.m. UTC | #1
On 11/7/22 11:13 AM, Cédric Le Goater wrote:
> From: Cédric Le Goater <clg@redhat.com>
> 
> Hello,
> 
> This series removes the redundant "zpcii-disable" machine property and
> and introduces a GlobalProperty compat array to maintain the same
> behaviour on older machines.
> 
> Thanks,
> 
> C.

Thanks Cedric.

For the series:
Reviewed-by: Matthew Rosato <mjrosato@linux.ibm.com>

Also did a quick test on top of
https://gitlab.com/thuth/qemu.git tags/pull-request-2022-11-06

to verify that zPCI interpretation/forwarding is available as expected with machine 7.2 and off by default in 7.1 and older.

> 
> Cédric Le Goater (2):
>   Revert "s390x/s390-virtio-ccw: add zpcii-disable machine property"
>   s390x/s390-virtio-ccw: Switch off zPCI enhancements on older machines
> 
>  include/hw/s390x/s390-virtio-ccw.h |  1 -
>  hw/s390x/s390-pci-kvm.c            |  4 +---
>  hw/s390x/s390-virtio-ccw.c         | 29 +++++------------------------
>  util/qemu-config.c                 |  4 ----
>  qemu-options.hx                    |  8 +-------
>  5 files changed, 7 insertions(+), 39 deletions(-)
>