[libvirt PATCH 0/7] Add boot order to virtiofs

Michal Privoznik mprivozn at redhat.com
Thu Jan 28 19:18:36 UTC 2021


On 1/28/21 4:15 PM, Ján Tomko wrote:
> Sadly, the replies changes for older QEMUs are synthetic.
> Separated for easier review.
> 
> Also available on gitlab:
> git fetch https://gitlab.com/janotomko/libvirt/ virtiofs-bootindex
> https://gitlab.com/janotomko/libvirt/-/tree/virtiofs-bootindex
> 
> And a broken pipeline:
> https://gitlab.com/janotomko/libvirt/-/pipelines/248162273
> 
> Ján Tomko (7):
>    tests: switch vhost-user-fs-hugepages to use boot order
>    conf: add boot order to filesystem
>    qemu: add QEMU_CAPS_VHOST_USER_FS_BOOTINDEX
>    fixup: vhost-user-fs-device properties
>    fixup: renumber
>    Add validation for virtiofs boot order setting
>    qemu: format bootindex for vhost-user-fs
> 
>   docs/schemas/domaincommon.rng                 |   3 +
>   src/conf/domain_conf.c                        |   5 +-
>   src/conf/domain_validate.c                    |  17 ++-
>   src/qemu/qemu_capabilities.c                  |   8 +
>   src/qemu/qemu_capabilities.h                  |   1 +
>   src/qemu/qemu_command.c                       |   3 +
>   src/qemu/qemu_validate.c                      |   6 +
>   .../caps_4.2.0.aarch64.replies                | 131 ++++++++++++----
>   .../caps_4.2.0.s390x.replies                  | 119 ++++++++++++---
>   .../caps_4.2.0.x86_64.replies                 | 131 ++++++++++++----
>   .../caps_5.0.0.aarch64.replies                | 136 +++++++++++++----
>   .../caps_5.0.0.ppc64.replies                  | 124 +++++++++++++---
>   .../caps_5.0.0.riscv64.replies                | 120 ++++++++++++---
>   .../caps_5.0.0.x86_64.replies                 | 136 +++++++++++++----
>   .../caps_5.1.0.x86_64.replies                 | 136 +++++++++++++----
>   .../caps_5.2.0.aarch64.replies                | 136 +++++++++++++----
>   .../caps_5.2.0.ppc64.replies                  | 124 +++++++++++++---
>   .../caps_5.2.0.riscv64.replies                | 120 ++++++++++++---
>   .../caps_5.2.0.s390x.replies                  | 124 +++++++++++++---
>   .../caps_5.2.0.x86_64.replies                 | 136 +++++++++++++----
>   .../caps_6.0.0.x86_64.replies                 | 140 ++++++++++++++----
>   .../caps_6.0.0.x86_64.xml                     |   1 +
>   ...vhost-user-fs-hugepages.x86_64-latest.args |   3 +-
>   .../vhost-user-fs-hugepages.xml               |   3 +-
>   24 files changed, 1534 insertions(+), 329 deletions(-)
> 

Hopefully, you'll squash 4/7 and 5/7 into 3/7 so that we're able to 
compile && test after each commit.

Also, news entry would be nice (it's fine a follow up patch).

Reviewed-by: Michal Privoznik <mprivozn at redhat.com>

Michal




More information about the libvir-list mailing list