vfio IGD assignment (was Re: [PATCH v6 0/8] Series short description)

Alex Williamson
2016-05-17T20:42:14Z

On Tue, 17 May 2016 14:19:19 -0600
Alex Williamson <alex.williamson@redhat.com> wrote:

> The following series implements...

Gag, I blame a bum vpn connection and botch vim recovery ;)

Here's what I meant to list:

This is the first non-RFC posting, so if you wish to review and
provide comments, now is the time to do so. What's new in v6:

* The fw_cfg entry for BDSM reservation has changed per Gerd's
feedback. This will require a new SeaBIOS to make the matching
change (incompatible with previous postings).
* Added a docs entry for IGD assignment covering both the usage
of UPT vs legacy mode as well as the ABI using fw_cfg.
* Fix some uninitialized pointer issues if we bail out of legacy
mode setup.
* Leak igd_opregion if we happen to be hot-unplugged. I don't
see a better option given the fw_cfg dependency, we really don't
want to support unplug of IGD at all.

I believe we're no longer blocked by SeaBIOS being willing to
consume the new fw_cfg entries, so barring feedback otherwise, my
plan will be to send a pull request including these changes (after
an appropriate review period), ask Kevin to apply the matching
SeaBIOS change, then see if Gerd will help me pull that support
into the default QEMU SeaBIOS image.

There is a plethora of comments and documentation in this series,
so I will forego further description of what this enables. Jump to
the last patch to read about it or hit patch 6 for the very well
documented (IMO) meat of IGD support. Thanks!

Alex

> ---
>
> Alex Williamson (8):
> vfio: Enable sparse mmap capability
> vfio: Create device specific region info helper
> vfio/pci: Fix return of vfio_populate_vga()
> vfio/pci: Consolidate VGA setup
> vfio/pci: Setup BAR quirks after capabilities probing
> vfio/pci: Intel graphics legacy mode assignment
> vfio/pci: Add a separate option for IGD OpRegion support
> vfio/pci: Add IGD documentation
>
>
> docs/igd-assign.txt | 133 ++++++++
> hw/vfio/common.c | 103 ++++++-
> hw/vfio/pci-quirks.c | 643 +++++++++++++++++++++++++++++++++++++++++
> hw/vfio/pci.c | 151 ++++++----
> hw/vfio/pci.h | 8 +
> include/hw/vfio/vfio-common.h | 2
> trace-events | 11 +
> 7 files changed, 989 insertions(+), 62 deletions(-)
> create mode 100644 docs/igd-assign.txt