[libvirt] [PATCH 0/6 v3] Support forward mode='hostdev' and interface pools

[Shradha Shah]

Hello All,

May I request a review for this patch series that supports forward mode='hostdev' with interface pools?

Many Thanks,
Regards,
Shradha Shah

On 06/29/2012 01:19 PM, Shradha Shah wrote:
> This patch series supports the forward mode='hostdev'. The functionality
> of this mode is the same as interface type='hostdev' but with the added
> benefit of using interface pools.
> 
> The patch series also contains a patch to support use of interface names
> and PCI device addresses interchangeably in a network xml, and return
> the appropriate one in actualDevice when networkAllocateActualDevice is
> called.
> 
> At the top level managed attribute can be specified with identical results 
> as when it's specified for a hostdev.
> 
> Currently forward mode='hostdev' does not support USB devices.
> 
> Shradha Shah (6):
>   Prerequisite Patch. virDomainDevicePCIAddress and respective
>     functions moved to a new file called conf/device_conf.ch
>   Moved the code to create implicit interface pool from PF to a new
>     function
>   RNG updates, new xml parser/formatter code to support forward
>     mode=hostdev
>   Code to return interface name or pci_addr of the VF in actualDevice
>   Forward Mode Hostdev network driver Implementation
>   Forward Mode 'Hostdev' qemu driver implementation
> 
>  docs/formatnetwork.html.in             |   62 ++++++
>  docs/schemas/network.rng               |   82 ++++++++-
>  include/libvirt/virterror.h            |    1 +
>  src/Makefile.am                        |    7 +-
>  src/conf/device_conf.c                 |  135 +++++++++++++
>  src/conf/device_conf.h                 |   65 +++++++
>  src/conf/domain_conf.c                 |  114 ++----------
>  src/conf/domain_conf.h                 |   25 +---
>  src/conf/network_conf.c                |  126 +++++++++++--
>  src/conf/network_conf.h                |   29 +++-
>  src/libvirt_private.syms               |   10 +-
>  src/network/bridge_driver.c            |  322 +++++++++++++++++++++++++-------
>  src/qemu/qemu_command.c                |   27 ++-
>  src/qemu/qemu_hotplug.c                |    7 +-
>  src/qemu/qemu_monitor.c                |   14 +-
>  src/qemu/qemu_monitor.h                |   17 +-
>  src/qemu/qemu_monitor_json.c           |   14 +-
>  src/qemu/qemu_monitor_json.h           |   14 +-
>  src/qemu/qemu_monitor_text.c           |   16 +-
>  src/qemu/qemu_monitor_text.h           |   14 +-
>  src/util/virnetdev.c                   |   29 ++--
>  src/util/virnetdev.h                   |    4 +-
>  src/xen/xend_internal.c                |    3 +-
>  tests/networkxml2xmlin/hostdev-pf.xml  |   11 +
>  tests/networkxml2xmlin/hostdev.xml     |   10 +
>  tests/networkxml2xmlout/hostdev-pf.xml |    7 +
>  tests/networkxml2xmlout/hostdev.xml    |   10 +
>  tests/networkxml2xmltest.c             |    2 +
>  28 files changed, 890 insertions(+), 287 deletions(-)
>  create mode 100644 src/conf/device_conf.c
>  create mode 100644 src/conf/device_conf.h
>  create mode 100644 tests/networkxml2xmlin/hostdev-pf.xml
>  create mode 100644 tests/networkxml2xmlin/hostdev.xml
>  create mode 100644 tests/networkxml2xmlout/hostdev-pf.xml
>  create mode 100644 tests/networkxml2xmlout/hostdev.xml
>