Re: [PATCH 00/12] move validations from domain_conf.c to

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

 





On 12/14/20 7:18 AM, Michal Privoznik wrote:
On 12/11/20 7:05 PM, Daniel Henrique Barboza wrote:
Hi,

This is a continuation of what we started in [1]. The changes
are all trivial, but done in parts to make the review less
annoying.

domain_conf.c is still a huge beast, but here we're putting it on
a diet, losing ~1k lines.


[1] https://www.redhat.com/archives/libvir-list/2020-December/msg00509.html

Daniel Henrique Barboza (12):
   domain_conf: move duplicate check functions to domain_validate.c
   domain_conf.c: rename virDomainDeviceInfoIterateInternal()
   domain_conf: move address validation functions to domain_validate.c
   domain_conf: move all virDomainDefValidateInternal() helpers to
     domain_validate.c
   domain_conf.c: move virDomainDeviceDefValidate() to domain_validate.c
   domain_validate.c: make local functions static
   domain_conf: move net device validation to domain_validate.c
   domain_conf: move all DeviceDefValidateInternal() helpers to
     domain_validate
   domain_conf.c: move virDomainDeviceDefValidate() to domain_validate.c
   domain_validate.c: make virDomainDeviceDefValidateInternal() helpers
     static
   domain_validate.c: put IOMMU validation into a new function
   domain_validate.c: use VIR_ERR_CONFIG_UNSUPPORTED in validate
     functions

  src/conf/domain_conf.c     | 1040 ++----------------------------------
  src/conf/domain_conf.h     |   58 +-
  src/conf/domain_validate.c |  966 ++++++++++++++++++++++++++++++++-
  src/conf/domain_validate.h |   31 +-
  src/libvirt_private.syms   |    9 +-
  src/libxl/libxl_domain.c   |    1 +
  src/libxl/libxl_driver.c   |    1 +
  src/lxc/lxc_driver.c       |    1 +
  src/lxc/lxc_process.c      |    1 +
  src/qemu/qemu_domain.c     |    1 +
  src/qemu/qemu_driver.c     |    1 +
  src/qemu/qemu_process.c    |    1 +
  12 files changed, 1066 insertions(+), 1045 deletions(-)


Reviewed-by: Michal Privoznik <mprivozn@xxxxxxxxxx>

Nice cleanup.


Thanks! Pushed to master.


DHB







Michal





[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