qemu: Don't use updated qemuCaps in validation code

[PATCH 0/4] qemu: Don't use updated qemuCaps in validation code

Peter Krempa posted 4 patches 3 years, 4 months ago

Download series mbox

Test syntax-check failed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1607435748.git.pkrempa@redhat.com

src/bhyve/bhyve_domain.c |   3 +-
src/conf/domain_conf.c   |  21 +++---
src/conf/domain_conf.h   |   9 ++-
src/qemu/qemu_process.c  |   2 +-
src/qemu/qemu_validate.c | 138 ++++++++++++++++++++-------------------
src/qemu/qemu_validate.h |   7 +-
src/vz/vz_driver.c       |   6 +-
7 files changed, 103 insertions(+), 83 deletions(-)

Expand all Fold all

[PATCH 0/4] qemu: Don't use updated qemuCaps in validation code

Posted by Peter Krempa 3 years, 4 months ago

See patch 4/4 for the justification.

CI run:
https://gitlab.com/pipo.sk/libvirt/-/pipelines/226815271

since the series is modifying stuff I don't compile usually.

Peter Krempa (4):
  virDomainDefValidate: Add per-run 'opaque' data
  qemu: validate: Don't check that qemuCaps is non-NULL
  qemuValidateDomainDeviceDefFS: Fix block indentation
  qemu: validate: Prefer existing qemuCaps

 src/bhyve/bhyve_domain.c |   3 +-
 src/conf/domain_conf.c   |  21 +++---
 src/conf/domain_conf.h   |   9 ++-
 src/qemu/qemu_process.c  |   2 +-
 src/qemu/qemu_validate.c | 138 ++++++++++++++++++++-------------------
 src/qemu/qemu_validate.h |   7 +-
 src/vz/vz_driver.c       |   6 +-
 7 files changed, 103 insertions(+), 83 deletions(-)

-- 
2.28.0

Re: [PATCH 0/4] qemu: Don't use updated qemuCaps in validation code

Posted by Ján Tomko 3 years, 4 months ago

On a Tuesday in 2020, Peter Krempa wrote:
>See patch 4/4 for the justification.
>
>CI run:
>https://gitlab.com/pipo.sk/libvirt/-/pipelines/226815271
>
>since the series is modifying stuff I don't compile usually.
>
>Peter Krempa (4):
>  virDomainDefValidate: Add per-run 'opaque' data
>  qemu: validate: Don't check that qemuCaps is non-NULL
>  qemuValidateDomainDeviceDefFS: Fix block indentation
>  qemu: validate: Prefer existing qemuCaps
>
> src/bhyve/bhyve_domain.c |   3 +-
> src/conf/domain_conf.c   |  21 +++---
> src/conf/domain_conf.h   |   9 ++-
> src/qemu/qemu_process.c  |   2 +-
> src/qemu/qemu_validate.c | 138 ++++++++++++++++++++-------------------
> src/qemu/qemu_validate.h |   7 +-
> src/vz/vz_driver.c       |   6 +-
> 7 files changed, 103 insertions(+), 83 deletions(-)
>

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

Jano