[libvirt] [PATCHv6 1/5] Add a hostdev PCI backend type

[Laine Stump]

On 11/22/2013 11:08 PM, Jim Fehlig wrote:
> Chunyan Liu wrote:
>> Add VIR_DOMAIN_HOSTDEV_PCI_BACKEND_XEN. For legacy xen, it will use "pciback" as
>> stub driver.
>>   
> Sorry for the long delay in helping review these patches. I finally have
> some time to work on libvirt :).
>
> With the pending release, this series will have to wait for the next
> cycle, but hopefully we can get it in early for prolonged testing. It
> plugs a big hole in the libxl dirver, so thanks for your work and
> perseverance :).
>
> I've applied your patches to latest git master and testing looks good so
> far, after fixing a small issue in 2/5. See the individual patches for
> further comments.

The part that has made me nervous about this series from the beginning
is that it is moving the implementation of some pretty hairy code from
qemu-specific to a general library while that code has itself been
getting fairly frequently bugfix tweaks. Fortunately we now have several
more unit tests on this code, but before pushing we still should do a
bit of extra due diligence to make sure that no recent bugfixes to the
qemu-specific version of the code are missing in the new versions. (BTW,
for this reason I think it is a very good idea to switch qemu and lxc
over to this new library immediately, rather than waiting as was done in
an earlier version of the patch series.)