[libvirt] [PATCHv2 0/7] Sharing code for domain and network routes
Cedric Bosdonnat
cbosdonnat at suse.com
Fri Jan 16 09:15:46 UTC 2015
On Thu, 2015-01-15 at 11:58 +0100, Michal Privoznik wrote:
> On 15.01.2015 10:25, Cédric Bosdonnat wrote:
> > Hi guys,
> >
> > Here are a few patches to have common route definitions for domains and networks.
> > What has changed since v1:
> > * Split into several patches for backportability as suggested by Laine
> > * Moved the virNetworkRouteDef struct definition in the c file to make sure all
> > routes are created with the virNetworkRouteDefCreate (or Parse) function. This
> > way we can get rid of the redundant checks in the format function.
> > * Moved the prefix computing code into a new virNetworkRouteDefGetPrefix function.
> > * Applied and tweaked John's patches.
> > * Fix the virSocketAddrGetIpPrefix 0.0.0.0 special case as suggested by Laine.
> >
> > Cédric Bosdonnat (5):
> > Fix ipv6 regex in RNG schemas to match '::'
> > Move network route definition to networkcommon.rng
> > Move code related to network routes to networkcommon_conf.[ch]
> > Use the network route definitions for domains
> > virSocketAddrGetIpPrefix 0.0.0.0 special case
> >
> > John Ferlan (2):
> > domain_conf: Resolve Coverity RESOURCE_LEAK
> > domain_conf: Check errors from virSocketAddrFormat
> >
> > docs/formatdomain.html.in | 9 +-
> > docs/schemas/basictypes.rng | 2 +-
> > docs/schemas/domaincommon.rng | 29 +-
> > docs/schemas/network.rng | 20 +-
> > docs/schemas/networkcommon.rng | 22 ++
> > po/POTFILES.in | 1 +
> > src/Makefile.am | 3 +-
> > src/conf/domain_conf.c | 136 ++-----
> > src/conf/domain_conf.h | 14 +-
> > src/conf/network_conf.c | 297 +---------------
> > src/conf/network_conf.h | 22 +-
> > src/conf/networkcommon_conf.c | 414 ++++++++++++++++++++++
> > src/conf/networkcommon_conf.h | 72 ++++
> > src/libvirt_private.syms | 11 +
> > src/lxc/lxc_container.c | 22 +-
> > src/lxc/lxc_native.c | 20 +-
> > src/network/bridge_driver.c | 44 +--
> > src/util/virsocketaddr.c | 6 +
> > tests/lxcconf2xmldata/lxcconf2xml-physnetwork.xml | 4 +-
> > tests/lxcconf2xmldata/lxcconf2xml-simple.xml | 4 +-
> > tests/lxcxml2xmldata/lxc-hostdev.xml | 4 +-
> > tests/lxcxml2xmldata/lxc-idmap.xml | 4 +-
> > 22 files changed, 633 insertions(+), 527 deletions(-)
> > create mode 100644 src/conf/networkcommon_conf.c
> > create mode 100644 src/conf/networkcommon_conf.h
> >
>
> ACK series. Good job!
Pushed with minor changes from Lain's comments.
--
Cedric
More information about the libvir-list
mailing list