On 11/14/2016 11:20 AM, Andrea Bolognani wrote:
On Mon, 2016-11-07 at 14:50 -0500, Laine Stump wrote:
> This is a rebase of v5 (, with the addition of patches 16 & 17, which
> detect the type of VFIO assigned devices and assign them to either an
> Express or a legacy PCI slot as appropriate,
I'm afraid you're going to have to rebase again, hopefully
it won't be too much work.
I think that, seeing as all patches...
[...]
> [RFC] qemu: if pci-bridge is in PCIe config w/o dmi-to-pci-bridge, add
> it
... except for this one, that we've decided to drop...
[...]
I made a separate patch to fix the test cases this affects. It will have
to be pushed before 12-15 can be pushed. I just posted it as a reply to
my last message on that sub-thread.
> qemu: propagate virQEMUDriver object to
> qemuDomainDeviceCalculatePCIConnectFlags
> qemu: assign vfio devices to PCIe addresses when appropriate
... and these two, that AIUI will have to be tweaked
anyway, have been ACKed with only trivial changes needed,
the bulk of the series (1-15) can be pushed now.
I would not hold up the entire series because of the last
two patches as they're fairly self-contained, but up to you.
Do you want me to look in detail at their current form (they
looked okay from a quick look), or would you rather implement
Alex's suggestions first and then make a separate mini-series
out of them?
You can skip them while I try to implement Alex's suggestion. They
should be independent of the other two series that are pending.
It would also be awesome if you could try and update
docs/news.html.in with a couple of sentences describing,
in broad strokes, what this series changes for users,
following the ideas outlined in last month's "Toward a
better NEWS file" thread. You can post it as patch 18/17
and we can iterate over it, the whole thing is still very
much a work in progress at this point anyway.
Yeah, I'll give that a look in a bit.
Thanks!