[PATCH v2 0/4] Full boot order support on s390x

Boris Fiuczynski posted 4 patches 1 week, 5 days ago
NEWS.rst                                      |    16 +
src/qemu/qemu_capabilities.c                  |     8 +
src/qemu/qemu_capabilities.h                  |     1 +
src/qemu/qemu_command.c                       |    36 +-
src/qemu/qemu_command.h                       |     6 +-
src/qemu/qemu_hotplug.c                       |     6 +-
tests/domaincapsdata/qemu_9.2.0.s390x.xml     |   311 +
.../caps_5.2.0_s390x.replies                  |   321 +-
.../caps_6.0.0_s390x.replies                  |   321 +-
.../caps_8.1.0_s390x.replies                  |   321 +-
.../caps_8.2.0_s390x.replies                  |   321 +-
.../caps_9.1.0_s390x.replies                  |   321 +-
.../caps_9.2.0_s390x.replies                  | 36741 ++++++++++++++++
.../qemucapabilitiesdata/caps_9.2.0_s390x.xml |  3752 ++
.../machine-loadparm-hostdev.s390x-9.1.0.args |    33 +
.../machine-loadparm-hostdev.s390x-9.1.0.xml  |    33 +
...machine-loadparm-hostdev.s390x-latest.args |     4 +-
...-multiple-disks-nets-s390.s390x-9.1.0.args |    40 +
...m-multiple-disks-nets-s390.s390x-9.1.0.xml |    51 +
...multiple-disks-nets-s390.s390x-latest.args |     8 +-
...machine-loadparm-net-s390.s390x-9.1.0.args |    34 +
.../machine-loadparm-net-s390.s390x-9.1.0.xml |    32 +
...achine-loadparm-net-s390.s390x-latest.args |     4 +-
.../machine-loadparm-s390.s390x-9.1.0.args    |    34 +
.../machine-loadparm-s390.s390x-9.1.0.xml     |    33 +
.../machine-loadparm-s390.s390x-latest.args   |     4 +-
tests/qemuxmlconftest.c                       |     4 +
27 files changed, 42676 insertions(+), 120 deletions(-)
create mode 100644 tests/domaincapsdata/qemu_9.2.0.s390x.xml
create mode 100644 tests/qemucapabilitiesdata/caps_9.2.0_s390x.replies
create mode 100644 tests/qemucapabilitiesdata/caps_9.2.0_s390x.xml
create mode 100644 tests/qemuxmlconfdata/machine-loadparm-hostdev.s390x-9.1.0.args
create mode 100644 tests/qemuxmlconfdata/machine-loadparm-hostdev.s390x-9.1.0.xml
create mode 100644 tests/qemuxmlconfdata/machine-loadparm-multiple-disks-nets-s390.s390x-9.1.0.args
create mode 100644 tests/qemuxmlconfdata/machine-loadparm-multiple-disks-nets-s390.s390x-9.1.0.xml
create mode 100644 tests/qemuxmlconfdata/machine-loadparm-net-s390.s390x-9.1.0.args
create mode 100644 tests/qemuxmlconfdata/machine-loadparm-net-s390.s390x-9.1.0.xml
create mode 100644 tests/qemuxmlconfdata/machine-loadparm-s390.s390x-9.1.0.args
create mode 100644 tests/qemuxmlconfdata/machine-loadparm-s390.s390x-9.1.0.xml
[PATCH v2 0/4] Full boot order support on s390x
Posted by Boris Fiuczynski 1 week, 5 days ago
This series adds on s390x full boot order support which has been
introduced recently in QEMU with the PR
https://lore.kernel.org/qemu-devel/20241023131710.906748-1-thuth@redhat.com/

Changes in v2:
 - fixed up patch 2 with Peter's feedback
 - added RBs given
 - added NEWS update

The replies and xml files are removed from patch 2 in this series and
are available in https://gitlab.com/fiuczy/libvirt/-/commits/fullbootorder

Boris Fiuczynski (4):
  qemu: capabilities: Add QEMU_CAPS_VIRTIO_CCW_DEVICE_LOADPARM
  tests: add capabilities for QEMU 9.2.0 on s390x
  qemu: command: add multi boot device support on s390x
  NEWS: qemu: add multi boot device support on s390x

 NEWS.rst                                      |    16 +
 src/qemu/qemu_capabilities.c                  |     8 +
 src/qemu/qemu_capabilities.h                  |     1 +
 src/qemu/qemu_command.c                       |    36 +-
 src/qemu/qemu_command.h                       |     6 +-
 src/qemu/qemu_hotplug.c                       |     6 +-
 tests/domaincapsdata/qemu_9.2.0.s390x.xml     |   311 +
 .../caps_5.2.0_s390x.replies                  |   321 +-
 .../caps_6.0.0_s390x.replies                  |   321 +-
 .../caps_8.1.0_s390x.replies                  |   321 +-
 .../caps_8.2.0_s390x.replies                  |   321 +-
 .../caps_9.1.0_s390x.replies                  |   321 +-
 .../caps_9.2.0_s390x.replies                  | 36741 ++++++++++++++++
 .../qemucapabilitiesdata/caps_9.2.0_s390x.xml |  3752 ++
 .../machine-loadparm-hostdev.s390x-9.1.0.args |    33 +
 .../machine-loadparm-hostdev.s390x-9.1.0.xml  |    33 +
 ...machine-loadparm-hostdev.s390x-latest.args |     4 +-
 ...-multiple-disks-nets-s390.s390x-9.1.0.args |    40 +
 ...m-multiple-disks-nets-s390.s390x-9.1.0.xml |    51 +
 ...multiple-disks-nets-s390.s390x-latest.args |     8 +-
 ...machine-loadparm-net-s390.s390x-9.1.0.args |    34 +
 .../machine-loadparm-net-s390.s390x-9.1.0.xml |    32 +
 ...achine-loadparm-net-s390.s390x-latest.args |     4 +-
 .../machine-loadparm-s390.s390x-9.1.0.args    |    34 +
 .../machine-loadparm-s390.s390x-9.1.0.xml     |    33 +
 .../machine-loadparm-s390.s390x-latest.args   |     4 +-
 tests/qemuxmlconftest.c                       |     4 +
 27 files changed, 42676 insertions(+), 120 deletions(-)
 create mode 100644 tests/domaincapsdata/qemu_9.2.0.s390x.xml
 create mode 100644 tests/qemucapabilitiesdata/caps_9.2.0_s390x.replies
 create mode 100644 tests/qemucapabilitiesdata/caps_9.2.0_s390x.xml
 create mode 100644 tests/qemuxmlconfdata/machine-loadparm-hostdev.s390x-9.1.0.args
 create mode 100644 tests/qemuxmlconfdata/machine-loadparm-hostdev.s390x-9.1.0.xml
 create mode 100644 tests/qemuxmlconfdata/machine-loadparm-multiple-disks-nets-s390.s390x-9.1.0.args
 create mode 100644 tests/qemuxmlconfdata/machine-loadparm-multiple-disks-nets-s390.s390x-9.1.0.xml
 create mode 100644 tests/qemuxmlconfdata/machine-loadparm-net-s390.s390x-9.1.0.args
 create mode 100644 tests/qemuxmlconfdata/machine-loadparm-net-s390.s390x-9.1.0.xml
 create mode 100644 tests/qemuxmlconfdata/machine-loadparm-s390.s390x-9.1.0.args
 create mode 100644 tests/qemuxmlconfdata/machine-loadparm-s390.s390x-9.1.0.xml

-- 
2.45.0
Re: [PATCH v2 0/4] Full boot order support on s390x
Posted by Peter Krempa 1 week, 3 days ago
On Fri, Nov 08, 2024 at 13:16:33 +0100, Boris Fiuczynski wrote:
> This series adds on s390x full boot order support which has been
> introduced recently in QEMU with the PR
> https://lore.kernel.org/qemu-devel/20241023131710.906748-1-thuth@redhat.com/
> 
> Changes in v2:
>  - fixed up patch 2 with Peter's feedback
>  - added RBs given
>  - added NEWS update
> 
> The replies and xml files are removed from patch 2 in this series and
> are available in https://gitlab.com/fiuczy/libvirt/-/commits/fullbootorder

Preferrably if you post truncated patches add the pointer to the series
in the 'git fetch' format instead:

 git fetch https://gitlab.com/fiuczy/libvirt fullbootorder


> 
> Boris Fiuczynski (4):
>   qemu: capabilities: Add QEMU_CAPS_VIRTIO_CCW_DEVICE_LOADPARM
>   tests: add capabilities for QEMU 9.2.0 on s390x
>   qemu: command: add multi boot device support on s390x
>   NEWS: qemu: add multi boot device support on s390x

Reviewed-by: Peter Krempa <pkrempa@redhat.com>

and I'll push them shortly.
Re: [PATCH v2 0/4] Full boot order support on s390x
Posted by Andrea Bolognani 1 week, 2 days ago
On Mon, Nov 11, 2024 at 10:14:46AM +0100, Peter Krempa wrote:
> On Fri, Nov 08, 2024 at 13:16:33 +0100, Boris Fiuczynski wrote:
> > Boris Fiuczynski (4):
> >   qemu: capabilities: Add QEMU_CAPS_VIRTIO_CCW_DEVICE_LOADPARM
> >   tests: add capabilities for QEMU 9.2.0 on s390x
> >   qemu: command: add multi boot device support on s390x
> >   NEWS: qemu: add multi boot device support on s390x
>
> Reviewed-by: Peter Krempa <pkrempa@redhat.com>
>
> and I'll push them shortly.

If it's not too late, please make sure that you indent the NEWS entry
correctly before pushing. Thanks!

-- 
Andrea Bolognani / Red Hat / Virtualization
Re: [PATCH v2 0/4] Full boot order support on s390x
Posted by Peter Krempa 1 week, 2 days ago
On Mon, Nov 11, 2024 at 05:57:13 -0500, Andrea Bolognani wrote:
> On Mon, Nov 11, 2024 at 10:14:46AM +0100, Peter Krempa wrote:
> > On Fri, Nov 08, 2024 at 13:16:33 +0100, Boris Fiuczynski wrote:
> > > Boris Fiuczynski (4):
> > >   qemu: capabilities: Add QEMU_CAPS_VIRTIO_CCW_DEVICE_LOADPARM
> > >   tests: add capabilities for QEMU 9.2.0 on s390x
> > >   qemu: command: add multi boot device support on s390x
> > >   NEWS: qemu: add multi boot device support on s390x
> >
> > Reviewed-by: Peter Krempa <pkrempa@redhat.com>
> >
> > and I'll push them shortly.
> 
> If it's not too late, please make sure that you indent the NEWS entry
> correctly before pushing. Thanks!

Fixed && pushed. I did notice this mail before pushing :)
Re: [PATCH v2 0/4] Full boot order support on s390x
Posted by Boris Fiuczynski 1 week, 2 days ago
On 11/11/24 12:23, Peter Krempa wrote:
> On Mon, Nov 11, 2024 at 05:57:13 -0500, Andrea Bolognani wrote:
>> On Mon, Nov 11, 2024 at 10:14:46AM +0100, Peter Krempa wrote:
>>> On Fri, Nov 08, 2024 at 13:16:33 +0100, Boris Fiuczynski wrote:
>>>> Boris Fiuczynski (4):
>>>>    qemu: capabilities: Add QEMU_CAPS_VIRTIO_CCW_DEVICE_LOADPARM
>>>>    tests: add capabilities for QEMU 9.2.0 on s390x
>>>>    qemu: command: add multi boot device support on s390x
>>>>    NEWS: qemu: add multi boot device support on s390x
>>>
>>> Reviewed-by: Peter Krempa <pkrempa@redhat.com>
>>>
>>> and I'll push them shortly.
>>
>> If it's not too late, please make sure that you indent the NEWS entry
>> correctly before pushing. Thanks!
> 
> Fixed && pushed. I did notice this mail before pushing :)

Thanks for catching and fixing it.

-- 
Mit freundlichen Grüßen/Kind regards
    Boris Fiuczynski

IBM Deutschland Research & Development GmbH
Vorsitzender des Aufsichtsrats: Wolfgang Wendt
Geschäftsführung: David Faller
Sitz der Gesellschaft: Böblingen
Registergericht: Amtsgericht Stuttgart, HRB 243294
Re: [PATCH v2 0/4] Full boot order support on s390x
Posted by Boris Fiuczynski 1 week, 3 days ago
On 11/11/24 10:14, Peter Krempa wrote:
> On Fri, Nov 08, 2024 at 13:16:33 +0100, Boris Fiuczynski wrote:
>> This series adds on s390x full boot order support which has been
>> introduced recently in QEMU with the PR
>> https://lore.kernel.org/qemu-devel/20241023131710.906748-1-thuth@redhat.com/
>>
>> Changes in v2:
>>   - fixed up patch 2 with Peter's feedback
>>   - added RBs given
>>   - added NEWS update
>>
>> The replies and xml files are removed from patch 2 in this series and
>> are available in https://gitlab.com/fiuczy/libvirt/-/commits/fullbootorder
> 
> Preferrably if you post truncated patches add the pointer to the series
> in the 'git fetch' format instead:
> 
>   git fetch https://gitlab.com/fiuczy/libvirt fullbootorder
> 

I will try to remember that for the update to GA 9.2.0.

> 
>>
>> Boris Fiuczynski (4):
>>    qemu: capabilities: Add QEMU_CAPS_VIRTIO_CCW_DEVICE_LOADPARM
>>    tests: add capabilities for QEMU 9.2.0 on s390x
>>    qemu: command: add multi boot device support on s390x
>>    NEWS: qemu: add multi boot device support on s390x
> 
> Reviewed-by: Peter Krempa <pkrempa@redhat.com>
> 
> and I'll push them shortly.

Thanks, Peter


-- 
Mit freundlichen Grüßen/Kind regards
    Boris Fiuczynski

IBM Deutschland Research & Development GmbH
Vorsitzender des Aufsichtsrats: Wolfgang Wendt
Geschäftsführung: David Faller
Sitz der Gesellschaft: Böblingen
Registergericht: Amtsgericht Stuttgart, HRB 243294