[PATCH v3 00/12] Move qemu command line controller checks to qemuDomainDeviceDefValidateController* checks

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



v2:
https://www.redhat.com/archives/libvir-list/2017-December/msg00173.html

Changes since v2:

 * Push a few patches from v1 that were ACK'd

 * Make the Skip helper private to qemu_command.c and just copied
   the various skip checks from the new helper into the specific
   controller validate helper to make it painfully obvious what
   was being skipped rather than having common code handle that
   and possibly obsfuscate those details from the casual reader.



John Ferlan (11):
  qemu: Introduce qemuBuildSkipController
  qemu: Introduce qemuDomainDeviceDefValidateController
  qemu: Move CCW S390 Address check to controller def validate
  qemu: Introduce qemuDomainDeviceDefValidateControllerSCSI
  qemu: Introduce qemuDomainDeviceDefValidateControllerPCI
  qemu: Use virDomainPCIControllerOpts in qemuBuildControllerDevStr
  qemu: Move PCI command modelName check to controller def validate
  qemu: Move PCI command modelName TypeToString to controller def
    validate
  qemu: Move PCI more command checks to controller def validate
  qemu: Complete PCI command checks to controller def validate
  qemu: Introduce qemuDomainDeviceDefValidateControllerSATA

Lin Ma (1):
  qemu: Introduce qemuDomainDeviceDefValidateControllerIDE

 src/qemu/qemu_command.c | 497 ++++++++----------------------------------------
 src/qemu/qemu_domain.c  | 469 ++++++++++++++++++++++++++++++++++++++++++++-
 tests/qemumemlocktest.c |  14 ++
 tests/qemuxml2xmltest.c |   5 +-
 4 files changed, 564 insertions(+), 421 deletions(-)

-- 
2.13.6

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]
  Powered by Linux