Re: [PATCH v6 00/17] Use more PCIe less legacy PCI

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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...

[...]
>   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?

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.

-- 
Andrea Bolognani / Red Hat / Virtualization

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]