Re: [libvirt] [PATCH 22/24] hostdev: Streamline device ownership tracking

2016-03-19 Thread John Ferlan
[...] >>> -/* Step 5: Now mark all the devices as active */ >>> +/* Step 5: Move devices from the inactive list to the active list */ >>> for (i = 0; i < virPCIDeviceListCount(pcidevs); i++) { >>> virPCIDevicePtr pci = virPCIDeviceListGet(pcidevs, i); >>> +

Re: [libvirt] [PATCH 22/24] hostdev: Streamline device ownership tracking

2016-03-18 Thread Andrea Bolognani
On Wed, 2016-03-16 at 15:33 -0400, John Ferlan wrote: > While writing the tomb for patch 20, I had one question/issue... You > could have already addressed in your upcoming patches, but I'll ask > anyway before I see them > > Essentially combining step5/step6 will *steal* from the

Re: [libvirt] [PATCH 22/24] hostdev: Streamline device ownership tracking

2016-03-15 Thread Andrea Bolognani
On Sun, 2016-03-13 at 10:06 -0400, John Ferlan wrote: >  > On 03/07/2016 12:24 PM, Andrea Bolognani wrote: > >  > > After this patch, ownership of virPCIDevice instances is very easy > > to keep track of: for each host PCI device, the only instance that > > actually matters is the one inside one

Re: [libvirt] [PATCH 22/24] hostdev: Streamline device ownership tracking

2016-03-13 Thread John Ferlan
On 03/07/2016 12:24 PM, Andrea Bolognani wrote: > After this patch, ownership of virPCIDevice instances is very easy > to keep track of: for each host PCI device, the only instance that > actually matters is the one inside one of the bookkeeping list. > > Whenever some operation needs to be

[libvirt] [PATCH 22/24] hostdev: Streamline device ownership tracking

2016-03-07 Thread Andrea Bolognani
After this patch, ownership of virPCIDevice instances is very easy to keep track of: for each host PCI device, the only instance that actually matters is the one inside one of the bookkeeping list. Whenever some operation needs to be performed on a PCI device, the actual device is looked up