qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Michael S. Tsirkin" <mst@redhat.com>
To: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Cc: Lars Kurth <lars.kurth@citrix.com>,
	"xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>,
	Lars Kurth <lars.kurth.xen@gmail.com>,
	"Hao, Xudong" <xudong.hao@intel.com>,
	"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>,
	Cao jin <caoj.fnst@cn.fujitsu.com>,
	Stefano Stabellini <Stefano.Stabellini@citrix.com>
Subject: Re: [Qemu-devel] [Xen-devel] [PATCH v4] igd-passthrough-i440FX: convert to realize()
Date: Mon, 11 Jan 2016 13:01:43 +0200	[thread overview]
Message-ID: <20160111130044-mutt-send-email-mst@redhat.com> (raw)
In-Reply-To: <alpine.DEB.2.02.1601111044540.13564@kaball.uk.xensource.com>

On Mon, Jan 11, 2016 at 10:46:20AM +0000, Stefano Stabellini wrote:
> On Mon, 11 Jan 2016, Hao, Xudong wrote:
> > Stefano, 
> > 
> > Patch http://marc.info/?l=qemu-devel&m=145137863501079 don't works for qemu at all, some conflict when git apply. 
> > Patch http://marc.info/?l=qemu-devel&m=145137863501079 is based on patch http://marc.info/?l=qemu-devel&m=145172165010604, right?
> > 
> > I can boot up Linux VM with IGD pass-through with latest qemu (without any additional patch), guest run 3D "nexuiz" and get 180fps. 
> 
> Very interesting, thanks for testing.

Could windows VM be tested too please?

It might be that the host pci read hacks are only needed
for the benefit of the windows guests.

> 
> > Will try the two patch together later.
> 
> That would be useful
> 
> 
> > > -----Original Message-----
> > > From: Stefano Stabellini [mailto:stefano.stabellini@eu.citrix.com]
> > > Sent: Friday, January 8, 2016 7:57 PM
> > > To: Hao, Xudong <xudong.hao@intel.com>
> > > Cc: Stefano Stabellini <stefano.stabellini@eu.citrix.com>; Lars Kurth
> > > <lars.kurth.xen@gmail.com>; Lars Kurth <lars.kurth@citrix.com>; Cao jin
> > > <caoj.fnst@cn.fujitsu.com>; xen-devel@lists.xensource.com; Stefano Stabellini
> > > <Stefano.Stabellini@citrix.com>; qemu-devel@nongnu.org; Michael S. Tsirkin
> > > <mst@redhat.com>
> > > Subject: RE: [Xen-devel] [PATCH v4] igd-passthrough-i440FX: convert to realize()
> > > 
> > > Since you are at it, could you please let me know how well igd passthrough
> > > works without this bugfix:
> > > 
> > > http://marc.info/?l=qemu-devel&m=145172165010604
> > > 
> > > which is about to land in QEMU.  I guess it doesn't work at all?
> > > 
> > > I am asking because I would like to know the level of support we need to provide
> > > to igd passthrough with the latest QEMU release (2.5).
> > > 
> > > 
> > > On Thu, 7 Jan 2016, Hao, Xudong wrote:
> > > > Sure. I'll test it soon.
> > > >
> > > > Thanks,
> > > > -Xudong
> > > >
> > > > > -----Original Message-----
> > > > > From: Stefano Stabellini [mailto:stefano.stabellini@eu.citrix.com]
> > > > > Sent: Wednesday, January 6, 2016 8:18 PM
> > > > > To: Lars Kurth <lars.kurth.xen@gmail.com>
> > > > > Cc: Stefano Stabellini <stefano.stabellini@eu.citrix.com>; Hao,
> > > > > Xudong <xudong.hao@intel.com>; Lars Kurth <lars.kurth@citrix.com>;
> > > > > Cao jin <caoj.fnst@cn.fujitsu.com>; xen-devel@lists.xensource.com;
> > > > > Stefano Stabellini <Stefano.Stabellini@citrix.com>;
> > > > > qemu-devel@nongnu.org; Michael S. Tsirkin <mst@redhat.com>
> > > > > Subject: Re: [Xen-devel] [PATCH v4] igd-passthrough-i440FX: convert
> > > > > to realize()
> > > > >
> > > > > Hello Xudong,
> > > > >
> > > > > please test this patch:
> > > > >
> > > > > http://marc.info/?l=qemu-devel&m=145137863501079
> > > > >
> > > > > with an intel graphic card assigned to a Xen guest. If everything
> > > > > still works as expected, please reply with your Tested-by.
> > > > >
> > > > > Thanks,
> > > > >
> > > > > Stefano
> > > > >
> > > > > On Wed, 6 Jan 2016, Lars Kurth wrote:
> > > > > > Hi folks,
> > > > > > let me introduce you to Xudong from Intel, who is willing to help out.
> > > > > > Best Regards
> > > > > > Lars
> > > > > >
> > > > > > > On 4 Jan 2016, at 15:41, Stefano Stabellini
> > > > > > > <stefano.stabellini@eu.citrix.com>
> > > > > wrote:
> > > > > > >
> > > > > > > On Mon, 4 Jan 2016, Lars Kurth wrote:
> > > > > > >> On 04/01/2016 14:47, "Stefano Stabellini"
> > > > > > >> <stefano.stabellini@eu.citrix.com> wrote:
> > > > > > >>
> > > > > > >>> Unfortunately I don't have a setup to test this either. Maybe
> > > > > > >>> Lars can find out who should be involved on the Intel side on this.
> > > > > > >>
> > > > > > >> I can certainly help to this and get back to you. What exactly
> > > > > > >> are we asking Intel to do?
> > > > > > >> It is not clear to me from this email thread
> > > > > > >
> > > > > > > Tiejun Chen, the author of the Intel graphic card passthrough
> > > > > > > patches for QEMU, seems to have left the company. It would be
> > > > > > > nice if somebody else tested this patch with an intel graphic
> > > > > > > card assigned to a guest VM.
> > > > > > >
> > > > > > > _______________________________________________
> > > > > > > Xen-devel mailing list
> > > > > > > Xen-devel@lists.xen.org
> > > > > > > http://lists.xen.org/xen-devel
> > > > > >
> > > >
> > 

  reply	other threads:[~2016-01-11 11:01 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-12-29  8:46 [Qemu-devel] [PATCH v4] igd-passthrough-i440FX: convert to realize() Cao jin
2015-12-29  9:25 ` Cao jin
2015-12-29 11:25 ` Michael S. Tsirkin
2015-12-29 13:27   ` Cao jin
2016-01-04 14:47     ` Stefano Stabellini
2016-01-04 15:33       ` Lars Kurth
2016-01-04 15:41         ` Stefano Stabellini
2016-01-06 10:21           ` [Qemu-devel] [Xen-devel] " Lars Kurth
2016-01-06 12:18             ` Stefano Stabellini
2016-01-07  1:32               ` Hao, Xudong
2016-01-08 11:57                 ` Stefano Stabellini
2016-01-11  1:46                   ` Hao, Xudong
2016-01-11  9:53                   ` Hao, Xudong
2016-01-11 10:32                     ` Gerd Hoffmann
2016-01-12  8:41                       ` Hao, Xudong
2016-01-12  8:47                         ` Michael S. Tsirkin
2016-01-12  9:50                           ` Hao, Xudong
2016-01-12 10:24                             ` Gerd Hoffmann
2016-01-13  1:55                               ` Hao, Xudong
2016-01-13  7:37                               ` Hao, Xudong
2016-01-11 10:46                     ` Stefano Stabellini
2016-01-11 11:01                       ` Michael S. Tsirkin [this message]
2016-01-12  9:54                       ` Hao, Xudong
2016-01-12  2:01                     ` Cao jin
2016-01-12  2:24                       ` Hao, Xudong
2016-01-07 13:28   ` [Qemu-devel] " Cao jin
2016-01-07 15:01     ` Stefano Stabellini
2016-01-07 16:21       ` Gerd Hoffmann
2016-01-07 17:42         ` Stefano Stabellini

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=20160111130044-mutt-send-email-mst@redhat.com \
    --to=mst@redhat.com \
    --cc=Stefano.Stabellini@citrix.com \
    --cc=caoj.fnst@cn.fujitsu.com \
    --cc=lars.kurth.xen@gmail.com \
    --cc=lars.kurth@citrix.com \
    --cc=qemu-devel@nongnu.org \
    --cc=stefano.stabellini@eu.citrix.com \
    --cc=xen-devel@lists.xensource.com \
    --cc=xudong.hao@intel.com \
    /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).