[Qemu-devel] [PATCH v3 0/9] s390x: zPCI detangling

Cornelia Huck posted 9 patches 6 years, 8 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20170725153330.14966-1-cohuck@redhat.com
Test FreeBSD passed
Test checkpatch passed
Test s390x passed
There is a newer version of this series
default-configs/s390x-softmmu.mak |  2 +-
hw/pci/pci-stub.c                 | 12 ++++++
hw/s390x/Makefile.objs            |  3 +-
hw/s390x/s390-pci-bus.c           |  9 ++++-
hw/s390x/s390-pci-bus.h           |  5 ++-
hw/s390x/s390-pci-stub.c          | 78 +++++++++++++++++++++++++++++++++++++++
hw/s390x/s390-virtio-ccw.c        | 12 +++---
hw/s390x/sclp.c                   | 19 ++++++++--
target/s390x/ioinst.c             | 16 ++++++++
target/s390x/kvm.c                | 61 +++++++++++++++++++++++-------
10 files changed, 188 insertions(+), 29 deletions(-)
create mode 100644 hw/s390x/s390-pci-stub.c
[Qemu-devel] [PATCH v3 0/9] s390x: zPCI detangling
Posted by Cornelia Huck 6 years, 8 months ago
Next version, now without RFC. The goals are still the same:
- Being able to disable PCI support in a build completely.
- Properly fencing off PCI if the relevant facility bit is not provided.

Changes v2->v3:
- Only enable the zpci bit conditionally, keep the aen bit always on.
  Use a method that actually works...
- Some reordering in the msi route patch to keep the change small.
- Collected some r-bs.

Branch is still git://github.com/cohuck/qemu no-zpci-cpumodel

Likely 2.11 material, even though the 'fence off if facility bit is not
set' part could count as bugfixes. Thoughts?

Cornelia Huck (9):
  kvm: remove hard dependency on pci
  s390x/pci: add stubs
  s390x: chsc nt2 events are pci-only
  s390x/pci: do not advertise pci on non-pci builds
  s390x/ccw: create s390 phb conditionally
  s390x/sclp: properly guard pci-specific functions
  s390x/pci: fence off instructions for non-pci
  s390x/kvm: msi route fixup for non-pci
  s390x: refine pci dependencies

 default-configs/s390x-softmmu.mak |  2 +-
 hw/pci/pci-stub.c                 | 12 ++++++
 hw/s390x/Makefile.objs            |  3 +-
 hw/s390x/s390-pci-bus.c           |  9 ++++-
 hw/s390x/s390-pci-bus.h           |  5 ++-
 hw/s390x/s390-pci-stub.c          | 78 +++++++++++++++++++++++++++++++++++++++
 hw/s390x/s390-virtio-ccw.c        | 12 +++---
 hw/s390x/sclp.c                   | 19 ++++++++--
 target/s390x/ioinst.c             | 16 ++++++++
 target/s390x/kvm.c                | 61 +++++++++++++++++++++++-------
 10 files changed, 188 insertions(+), 29 deletions(-)
 create mode 100644 hw/s390x/s390-pci-stub.c

-- 
2.13.3


Re: [Qemu-devel] [PATCH v3 0/9] s390x: zPCI detangling
Posted by Christian Borntraeger 6 years, 8 months ago

On 07/25/2017 05:33 PM, Cornelia Huck wrote:
> Next version, now without RFC. The goals are still the same:
> - Being able to disable PCI support in a build completely.
> - Properly fencing off PCI if the relevant facility bit is not provided.
> 
> Changes v2->v3:
> - Only enable the zpci bit conditionally, keep the aen bit always on.
>   Use a method that actually works...
> - Some reordering in the msi route patch to keep the change small.
> - Collected some r-bs.
> 
> Branch is still git://github.com/cohuck/qemu no-zpci-cpumodel
> 
> Likely 2.11 material, even though the 'fence off if facility bit is not
> set' part could count as bugfixes. Thoughts?

In general, this series looks pretty good already, but I would prefer to
have it in 2.11. The PCI code now seems to work ok for passthrough devices
and this change would certainly require some retesting. 
I think it would stretch things too far to consider the fencing a 
must have bug fix.

> 
> Cornelia Huck (9):
>   kvm: remove hard dependency on pci
>   s390x/pci: add stubs
>   s390x: chsc nt2 events are pci-only
>   s390x/pci: do not advertise pci on non-pci builds
>   s390x/ccw: create s390 phb conditionally
>   s390x/sclp: properly guard pci-specific functions
>   s390x/pci: fence off instructions for non-pci
>   s390x/kvm: msi route fixup for non-pci
>   s390x: refine pci dependencies
> 
>  default-configs/s390x-softmmu.mak |  2 +-
>  hw/pci/pci-stub.c                 | 12 ++++++
>  hw/s390x/Makefile.objs            |  3 +-
>  hw/s390x/s390-pci-bus.c           |  9 ++++-
>  hw/s390x/s390-pci-bus.h           |  5 ++-
>  hw/s390x/s390-pci-stub.c          | 78 +++++++++++++++++++++++++++++++++++++++
>  hw/s390x/s390-virtio-ccw.c        | 12 +++---
>  hw/s390x/sclp.c                   | 19 ++++++++--
>  target/s390x/ioinst.c             | 16 ++++++++
>  target/s390x/kvm.c                | 61 +++++++++++++++++++++++-------
>  10 files changed, 188 insertions(+), 29 deletions(-)
>  create mode 100644 hw/s390x/s390-pci-stub.c
>