[libvirt PATCH 0/6] implement vhost-user-blk support

Pavel Hrdina posted 6 patches 3 years, 1 month ago
Test syntax-check failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1612277922.git.phrdina@redhat.com
There is a newer version of this series
docs/formatdomain.rst                         |  32 ++-
docs/schemas/domaincommon.rng                 |  19 ++
src/conf/domain_conf.c                        |  85 ++++++++
src/conf/domain_validate.c                    | 187 ++++++++++++++++++
src/conf/storage_source_conf.c                |   3 +
src/conf/storage_source_conf.h                |   4 +
src/libxl/xen_xl.c                            |   1 +
src/qemu/qemu_alias.c                         |   6 +
src/qemu/qemu_alias.h                         |   2 +
src/qemu/qemu_block.c                         |  20 ++
src/qemu/qemu_block.h                         |   5 +
src/qemu/qemu_capabilities.c                  |   4 +
src/qemu/qemu_capabilities.h                  |   3 +
src/qemu/qemu_command.c                       | 100 ++++++++--
src/qemu/qemu_command.h                       |   8 +
src/qemu/qemu_domain.c                        |   5 +
src/qemu/qemu_hotplug.c                       |   5 +-
src/qemu/qemu_migration.c                     |   2 +
src/qemu/qemu_snapshot.c                      |   4 +
src/qemu/qemu_validate.c                      | 112 ++++++-----
src/storage_file/storage_source.c             |   1 +
.../caps_2.12.0.aarch64.xml                   |   1 +
.../caps_2.12.0.ppc64.xml                     |   1 +
.../caps_2.12.0.s390x.xml                     |   1 +
.../caps_2.12.0.x86_64.xml                    |   1 +
.../qemucapabilitiesdata/caps_3.0.0.ppc64.xml |   1 +
.../caps_3.0.0.riscv32.xml                    |   1 +
.../caps_3.0.0.riscv64.xml                    |   1 +
.../qemucapabilitiesdata/caps_3.0.0.s390x.xml |   1 +
.../caps_3.0.0.x86_64.xml                     |   1 +
.../qemucapabilitiesdata/caps_3.1.0.ppc64.xml |   1 +
.../caps_3.1.0.x86_64.xml                     |   1 +
.../caps_4.0.0.aarch64.xml                    |   1 +
.../qemucapabilitiesdata/caps_4.0.0.ppc64.xml |   1 +
.../caps_4.0.0.riscv32.xml                    |   1 +
.../caps_4.0.0.riscv64.xml                    |   1 +
.../qemucapabilitiesdata/caps_4.0.0.s390x.xml |   1 +
.../caps_4.0.0.x86_64.xml                     |   1 +
.../caps_4.1.0.x86_64.xml                     |   1 +
.../caps_4.2.0.aarch64.xml                    |   1 +
.../qemucapabilitiesdata/caps_4.2.0.ppc64.xml |   1 +
.../qemucapabilitiesdata/caps_4.2.0.s390x.xml |   1 +
.../caps_4.2.0.x86_64.xml                     |   1 +
.../caps_5.0.0.aarch64.xml                    |   1 +
.../qemucapabilitiesdata/caps_5.0.0.ppc64.xml |   1 +
.../caps_5.0.0.riscv64.xml                    |   1 +
.../caps_5.0.0.x86_64.xml                     |   1 +
.../caps_5.1.0.x86_64.xml                     |   1 +
.../caps_5.2.0.aarch64.xml                    |   1 +
.../qemucapabilitiesdata/caps_5.2.0.ppc64.xml |   1 +
.../caps_5.2.0.riscv64.xml                    |   1 +
.../qemucapabilitiesdata/caps_5.2.0.s390x.xml |   1 +
.../caps_5.2.0.x86_64.xml                     |   1 +
.../caps_6.0.0.x86_64.xml                     |   1 +
.../disk-vhostuser.x86_64-latest.args         |  41 ++++
tests/qemuxml2argvdata/disk-vhostuser.xml     |  29 +++
tests/qemuxml2argvtest.c                      |   1 +
.../disk-vhostuser.x86_64-latest.xml          |  48 +++++
tests/qemuxml2xmltest.c                       |   1 +
59 files changed, 699 insertions(+), 62 deletions(-)
create mode 100644 tests/qemuxml2argvdata/disk-vhostuser.x86_64-latest.args
create mode 100644 tests/qemuxml2argvdata/disk-vhostuser.xml
create mode 100644 tests/qemuxml2xmloutdata/disk-vhostuser.x86_64-latest.xml
[libvirt PATCH 0/6] implement vhost-user-blk support
Posted by Pavel Hrdina 3 years, 1 month ago
Pavel Hrdina (6):
  qemu_alias: introduce qemuDomainGetVhostUserAlias helper
  qemu_validate: move and refactor
    qemuValidateDomainDefVirtioFSSharedMemory
  docs: introduces new vhostuser disk type
  conf: implement support for vhostuser disk
  qemu_capabilities: introduce vhost-user-blk capability
  qemu: implement vhost-user-blk support

 docs/formatdomain.rst                         |  32 ++-
 docs/schemas/domaincommon.rng                 |  19 ++
 src/conf/domain_conf.c                        |  85 ++++++++
 src/conf/domain_validate.c                    | 187 ++++++++++++++++++
 src/conf/storage_source_conf.c                |   3 +
 src/conf/storage_source_conf.h                |   4 +
 src/libxl/xen_xl.c                            |   1 +
 src/qemu/qemu_alias.c                         |   6 +
 src/qemu/qemu_alias.h                         |   2 +
 src/qemu/qemu_block.c                         |  20 ++
 src/qemu/qemu_block.h                         |   5 +
 src/qemu/qemu_capabilities.c                  |   4 +
 src/qemu/qemu_capabilities.h                  |   3 +
 src/qemu/qemu_command.c                       | 100 ++++++++--
 src/qemu/qemu_command.h                       |   8 +
 src/qemu/qemu_domain.c                        |   5 +
 src/qemu/qemu_hotplug.c                       |   5 +-
 src/qemu/qemu_migration.c                     |   2 +
 src/qemu/qemu_snapshot.c                      |   4 +
 src/qemu/qemu_validate.c                      | 112 ++++++-----
 src/storage_file/storage_source.c             |   1 +
 .../caps_2.12.0.aarch64.xml                   |   1 +
 .../caps_2.12.0.ppc64.xml                     |   1 +
 .../caps_2.12.0.s390x.xml                     |   1 +
 .../caps_2.12.0.x86_64.xml                    |   1 +
 .../qemucapabilitiesdata/caps_3.0.0.ppc64.xml |   1 +
 .../caps_3.0.0.riscv32.xml                    |   1 +
 .../caps_3.0.0.riscv64.xml                    |   1 +
 .../qemucapabilitiesdata/caps_3.0.0.s390x.xml |   1 +
 .../caps_3.0.0.x86_64.xml                     |   1 +
 .../qemucapabilitiesdata/caps_3.1.0.ppc64.xml |   1 +
 .../caps_3.1.0.x86_64.xml                     |   1 +
 .../caps_4.0.0.aarch64.xml                    |   1 +
 .../qemucapabilitiesdata/caps_4.0.0.ppc64.xml |   1 +
 .../caps_4.0.0.riscv32.xml                    |   1 +
 .../caps_4.0.0.riscv64.xml                    |   1 +
 .../qemucapabilitiesdata/caps_4.0.0.s390x.xml |   1 +
 .../caps_4.0.0.x86_64.xml                     |   1 +
 .../caps_4.1.0.x86_64.xml                     |   1 +
 .../caps_4.2.0.aarch64.xml                    |   1 +
 .../qemucapabilitiesdata/caps_4.2.0.ppc64.xml |   1 +
 .../qemucapabilitiesdata/caps_4.2.0.s390x.xml |   1 +
 .../caps_4.2.0.x86_64.xml                     |   1 +
 .../caps_5.0.0.aarch64.xml                    |   1 +
 .../qemucapabilitiesdata/caps_5.0.0.ppc64.xml |   1 +
 .../caps_5.0.0.riscv64.xml                    |   1 +
 .../caps_5.0.0.x86_64.xml                     |   1 +
 .../caps_5.1.0.x86_64.xml                     |   1 +
 .../caps_5.2.0.aarch64.xml                    |   1 +
 .../qemucapabilitiesdata/caps_5.2.0.ppc64.xml |   1 +
 .../caps_5.2.0.riscv64.xml                    |   1 +
 .../qemucapabilitiesdata/caps_5.2.0.s390x.xml |   1 +
 .../caps_5.2.0.x86_64.xml                     |   1 +
 .../caps_6.0.0.x86_64.xml                     |   1 +
 .../disk-vhostuser.x86_64-latest.args         |  41 ++++
 tests/qemuxml2argvdata/disk-vhostuser.xml     |  29 +++
 tests/qemuxml2argvtest.c                      |   1 +
 .../disk-vhostuser.x86_64-latest.xml          |  48 +++++
 tests/qemuxml2xmltest.c                       |   1 +
 59 files changed, 699 insertions(+), 62 deletions(-)
 create mode 100644 tests/qemuxml2argvdata/disk-vhostuser.x86_64-latest.args
 create mode 100644 tests/qemuxml2argvdata/disk-vhostuser.xml
 create mode 100644 tests/qemuxml2xmloutdata/disk-vhostuser.x86_64-latest.xml

-- 
2.29.2

Re: [libvirt PATCH 0/6] implement vhost-user-blk support
Posted by Ján Tomko 3 years, 1 month ago
On a Tuesday in 2021, Pavel Hrdina wrote:
>Pavel Hrdina (6):
>  qemu_alias: introduce qemuDomainGetVhostUserAlias helper
>  qemu_validate: move and refactor
>    qemuValidateDomainDefVirtioFSSharedMemory

>  docs: introduces new vhostuser disk type
>  conf: implement support for vhostuser disk

Typically the schema changes are introduced with the parser,
since the addition of the XML file also runs it through virschematest.

>  qemu_capabilities: introduce vhost-user-blk capability
>  qemu: implement vhost-user-blk support
>

Reviewed-by: Ján Tomko <jtomko@redhat.com>

Jano
Re: [libvirt PATCH 0/6] implement vhost-user-blk support
Posted by Peter Krempa 3 years, 1 month ago
On Tue, Feb 02, 2021 at 16:04:06 +0100, Pavel Hrdina wrote:
> Pavel Hrdina (6):
>   qemu_alias: introduce qemuDomainGetVhostUserAlias helper
>   qemu_validate: move and refactor
>     qemuValidateDomainDefVirtioFSSharedMemory
>   docs: introduces new vhostuser disk type
>   conf: implement support for vhostuser disk
>   qemu_capabilities: introduce vhost-user-blk capability

For the above commits you can use:

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

after applying my feedback.

>   qemu: implement vhost-user-blk support

This one has a bit too much to add though.