[libvirt] [PATCH 15/24] hostdev: Remove virHostdevGetActivePCIHostDeviceList()

Andrea Bolognani abologna at redhat.com
Mon Mar 14 15:21:23 UTC 2016


On Thu, 2016-03-10 at 14:25 -0500, John Ferlan wrote:
> <SIGH>  Should have read patch 18 & 19 first... Looks like I'm getting
> confuzzled by all these lists and multitude of ways names were
> generated. The comparison being done is against the copy that came from
> the activePCIHostdev list which will have the fields I was concerned
> about. So in retrospect...
> 
> ACK

To be fair, patches 18 and 19 were written *exactly* to make it a
tiny bit easier to follow along, especially with patch 21 in mind.

It would probably have been better if the name changes had been
made *before* changing the code too much, as this patch is doing,
but the comments in patch 19 would not really be true if patch 15
had not been applied first, so... Rock and hard place, really :(

I'll reply to your other concern down the thread, consider this
just an aside.

Cheers.

-- 
Andrea Bolognani
Software Engineer - Virtualization Team




More information about the libvir-list mailing list