From: Alex Williamson <alex.williamson@redhat.com>
To: Gerd Hoffmann <kraxel@redhat.com>
Cc: allen.m.kay@intel.com, qemu-devel@nongnu.org, kvm@vger.kernel.org
Subject: Re: [Qemu-devel] vfio IGD assignment (was Re: [PATCH v6 0/8] Series short description)
Date: Fri, 20 May 2016 09:08:57 -0600 [thread overview]
Message-ID: <20160520090857.22174bd9@t450s.home> (raw)
In-Reply-To: <1463746759.3526.88.camel@redhat.com>
On Fri, 20 May 2016 14:19:19 +0200
Gerd Hoffmann <kraxel@redhat.com> wrote:
> On Mi, 2016-05-18 at 12:45 -0600, Alex Williamson wrote:
> > On Wed, 18 May 2016 16:24:49 +0200
> > Gerd Hoffmann <kraxel@redhat.com> wrote:
> >
> > > Hi,
> > >
> > > > 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.
> > >
> > > Sounds good. Patches look sane to me too.
> > >
> > > Will try to set aside some time to test them later this week (and also
> > > push a branch with updated seabios etc for others to try).
> >
> > Thanks Gerd!
>
> Works flawlessly.
>
> Linux works fine with the igd assigned as primary (pc machine type and
> igd using slot 2).
>
> Linux doesn't work in UPT mode for me. That is more a guest driver
> issue though. The i915 driver of older kernels Oopses. The i915 driver
> of recent kernels (4.4.5+) loads fine on q35, but fails to drive my
> monitor, no matter whenever I enable the opregion or not. For pc an
> additional kernel patch is needed (which hopefully lands upstream soon)
> to reach feature parity (as-in: no Oops on load) with q35.
Yes, I've never had success with UPT for Linux guests either.
> For windows UPT mode is the only thing supported by the drivers. Needs
> the opregion enabled to show something on the connected display. Works
> fine on both pc and q35, in parallel with a emulated display, and you
> can configure things as you want (mirror, expand desktop, use only one
> of the displays).
UPT is probably the only thing Intel cares to support on Windows, yes,
but did you have any issues or try legacy mode on Windows? I find that
Windows 7/8/10 all work well with legacy mode, including the basic
drivers in the shrink-wrap images. Please let me know if you find
otherwise.
>
> Reviewed-by: Gerd Hoffmann <kraxel@redhat.com>
> Tested-by: Gerd Hoffmann <kraxel@redhat.com>
Awesome, thanks!
Alex
next prev parent reply other threads:[~2016-05-20 15:09 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-05-17 20:19 [Qemu-devel] [PATCH v6 0/8] Series short description Alex Williamson
2016-05-17 20:19 ` [Qemu-devel] [PATCH v6 1/8] vfio: Enable sparse mmap capability Alex Williamson
2016-05-17 20:19 ` [Qemu-devel] [PATCH v6 2/8] vfio: Create device specific region info helper Alex Williamson
2016-05-17 20:19 ` [Qemu-devel] [PATCH v6 3/8] vfio/pci: Fix return of vfio_populate_vga() Alex Williamson
2016-05-17 20:19 ` [Qemu-devel] [PATCH v6 4/8] vfio/pci: Consolidate VGA setup Alex Williamson
2016-05-17 20:19 ` [Qemu-devel] [PATCH v6 5/8] vfio/pci: Setup BAR quirks after capabilities probing Alex Williamson
2016-05-17 20:19 ` [Qemu-devel] [PATCH v6 6/8] vfio/pci: Intel graphics legacy mode assignment Alex Williamson
2016-05-18 14:21 ` Gerd Hoffmann
2016-05-18 16:50 ` Alex Williamson
2016-05-17 20:20 ` [Qemu-devel] [PATCH v6 7/8] vfio/pci: Add a separate option for IGD OpRegion support Alex Williamson
2016-05-17 20:20 ` [Qemu-devel] [PATCH v6 8/8] vfio/pci: Add IGD documentation Alex Williamson
2016-05-18 2:23 ` Eric Blake
2016-05-18 18:35 ` Alex Williamson
2016-05-18 19:28 ` Eric Blake
2016-05-17 20:42 ` [Qemu-devel] vfio IGD assignment (was Re: [PATCH v6 0/8] Series short description) Alex Williamson
2016-05-18 14:24 ` Gerd Hoffmann
2016-05-18 18:45 ` Alex Williamson
2016-05-19 9:00 ` Gerd Hoffmann
2016-05-20 12:19 ` Gerd Hoffmann
2016-05-20 15:08 ` Alex Williamson [this message]
2016-05-23 13:34 ` Gerd Hoffmann
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20160520090857.22174bd9@t450s.home \
--to=alex.williamson@redhat.com \
--cc=allen.m.kay@intel.com \
--cc=kraxel@redhat.com \
--cc=kvm@vger.kernel.org \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).