Re: [PATCH v3 0/8] s390x: Add support for virtio-scsi-pci boot device

[Eric Farman <farman@linux.ibm.com>]

On 8/31/26 3:49 PM, jrossi@linux.ibm.com wrote:
> From: Jared Rossi <jrossi@linux.ibm.com>
> 
> Add support for booting from SCSI devices with a virtio-scsi-pci controller.
> 
> Some minor clean-up is done with naming in order to clarify if the function
> or value used with CCW, PCI, or generic IPL and to check if a PCI function
> is already enabled.
> 
> Changes v2 -> v3:
>      - Rebase on recently added Secure IPL features added to BIOS
>      - Define PCI IPLB length using offset and sizeof rather than magic
>      - Check for virtio setup failures during scsi setup
>      - Fix commit messages
>      - Update the title of patch 3 to indicate multiple lengths change
> 
> Jared Rossi (8):
>    pc-bios/s390-ccw: Check if a PCI function is already enabled before
>      trying to enable it
>    hw/s390x/ipl: Fix incorrect PCI IPL block lengths
>    pc-bios/s390-ccw: Use bus specific IPL_TYPE directly for scsi IPL
>    s390x/ipl: Add PCI and bus fields to IplBlockQemuScsi
>    pc-bios/s390-ccw: Abstract virtio_run() for generic use
>    pc-bios/s390-ccw: Add support for virtio-scsi-pci IPL
>    s390x: Find scsi-pci boot device and build IPLB
>    tests/qtest: Add s390x PCI SCSI fallback test to cdrom-test.c
> 
>   hw/s390x/ipl.c                   | 40 ++++++++++++++++++++++++++++++++
>   hw/s390x/ipl.h                   |  2 ++
>   include/hw/s390x/ipl/qipl.h      | 15 ++++++++----
>   pc-bios/s390-ccw/clp.c           |  4 ++++
>   pc-bios/s390-ccw/main.c          | 19 +++++++++++----
>   pc-bios/s390-ccw/virtio-blkdev.c |  1 -
>   pc-bios/s390-ccw/virtio-ccw.c    | 17 --------------
>   pc-bios/s390-ccw/virtio-ccw.h    |  1 -
>   pc-bios/s390-ccw/virtio-pci.c    | 31 ++++++++++++++++++++++++-
>   pc-bios/s390-ccw/virtio-scsi.c   | 18 +++++++++++++-
>   pc-bios/s390-ccw/virtio.c        | 21 +++++++++--------
>   tests/qtest/cdrom-test.c         |  8 +++++++
>   12 files changed, 137 insertions(+), 40 deletions(-)
> 

Thanks! Applied for 11.2