Re: [libvirt] [PATCH 15/24] hostdev: Remove virHostdevGetActivePCIHostDeviceList()
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