Re: [Qemu-devel] [Xen-devel] [PATCH v4] igd-passthrough-i440FX: convert to realize()

2016-01-12 Thread Hao, Xudong
> -Original Message-
> From: qemu-devel-bounces+xudong.hao=intel@nongnu.org [mailto:qemu-
> devel-bounces+xudong.hao=intel@nongnu.org] On Behalf Of Gerd
> Hoffmann
> Sent: Tuesday, January 12, 2016 6:25 PM
> To: Hao, Xudong 
> Cc: Lars Kurth ; xen-de...@lists.xensource.com; Stefano
> Stabellini ; Lars Kurth
> ; Michael S. Tsirkin ; qemu-
> de...@nongnu.org; Cao jin ; Stefano Stabellini
> 
> Subject: Re: [Qemu-devel] [Xen-devel] [PATCH v4] igd-passthrough-i440FX:
> convert to realize()
> 
> On Di, 2016-01-12 at 09:50 +, Hao, Xudong wrote:
> > With latest qemu 7b8a354d4716, RHEL7.2 (with default kernel) VM still can't
> boot up with IGD.
> 
> There is another bug, using pci_default_write_config() doesn't fly as this 
> checks
> writes against wmask and the registers in question are not whitelisted ...
> 
> I've just rebased my igd patch series which (among other stuff) fixes that 
> bug:
>   https://www.kraxel.org/cgit/qemu/log/?h=work/igd
>   git://git.kraxel.org/qemu work/igd
> 
> Can you give it a spin?

The issue persist on this branch of tree (commit 1a0d06ce6), the VM kernel log 
attached in another mail while answered the following question.

> 
> Also: what does "can't boot up" mean exactly?  Guest doesn't boot at all?  
> Guest
> boots, but igd/console/Xorg doesn't work?  In case of the
> latter:  Any chance to login via network and get a kernel log?
> 
> thanks,
>   Gerd
> 



Re: [Qemu-devel] [Xen-devel] [PATCH v4] igd-passthrough-i440FX: convert to realize()

2016-01-12 Thread Hao, Xudong
"can't boot up" means guest doesn't boot at all, guest will stop to booting 
after adding vga device, detail log in attachment.

Thanks,
-Xudong


> -Original Message-
> From: qemu-devel-bounces+xudong.hao=intel@nongnu.org [mailto:qemu-
> devel-bounces+xudong.hao=intel@nongnu.org] On Behalf Of Gerd
> Hoffmann
> Sent: Tuesday, January 12, 2016 6:25 PM
> To: Hao, Xudong 
> Cc: Lars Kurth ; xen-de...@lists.xensource.com; Stefano
> Stabellini ; Lars Kurth
> ; Michael S. Tsirkin ; qemu-
> de...@nongnu.org; Cao jin ; Stefano Stabellini
> 
> Subject: Re: [Qemu-devel] [Xen-devel] [PATCH v4] igd-passthrough-i440FX:
> convert to realize()
> 
> On Di, 2016-01-12 at 09:50 +, Hao, Xudong wrote:
> > With latest qemu 7b8a354d4716, RHEL7.2 (with default kernel) VM still can't
> boot up with IGD.
> 
> There is another bug, using pci_default_write_config() doesn't fly as this 
> checks
> writes against wmask and the registers in question are not whitelisted ...
> 
> I've just rebased my igd patch series which (among other stuff) fixes that 
> bug:
>   https://www.kraxel.org/cgit/qemu/log/?h=work/igd
>   git://git.kraxel.org/qemu work/igd
> 
> Can you give it a spin?
> 
> Also: what does "can't boot up" mean exactly?  Guest doesn't boot at all?  
> Guest
> boots, but igd/console/Xorg doesn't work?  In case of the
> latter:  Any chance to login via network and get a kernel log?
> 
> thanks,
>   Gerd
> 



rhel7-default-kernel-boot.log
Description: rhel7-default-kernel-boot.log


Re: [Qemu-devel] [Xen-devel] [PATCH v4] igd-passthrough-i440FX: convert to realize()

2016-01-12 Thread Gerd Hoffmann
On Di, 2016-01-12 at 09:50 +, Hao, Xudong wrote:
> With latest qemu 7b8a354d4716, RHEL7.2 (with default kernel) VM still can't 
> boot up with IGD.

There is another bug, using pci_default_write_config() doesn't fly as
this checks writes against wmask and the registers in question are not
whitelisted ...

I've just rebased my igd patch series which (among other stuff) fixes
that bug:
  https://www.kraxel.org/cgit/qemu/log/?h=work/igd
  git://git.kraxel.org/qemu work/igd

Can you give it a spin?

Also: what does "can't boot up" mean exactly?  Guest doesn't boot at
all?  Guest boots, but igd/console/Xorg doesn't work?  In case of the
latter:  Any chance to login via network and get a kernel log?

thanks,
  Gerd




Re: [Qemu-devel] [Xen-devel] [PATCH v4] igd-passthrough-i440FX: convert to realize()

2016-01-12 Thread Hao, Xudong


> -Original Message-
> From: Stefano Stabellini [mailto:stefano.stabell...@eu.citrix.com]
> Sent: Monday, January 11, 2016 6:46 PM
> To: Hao, Xudong 
> Cc: Stefano Stabellini ; Lars Kurth
> ; Lars Kurth ; Cao jin
> ; xen-de...@lists.xensource.com; Stefano Stabellini
> ; qemu-devel@nongnu.org; Michael S. Tsirkin
> 
> Subject: RE: [Xen-devel] [PATCH v4] igd-passthrough-i440FX: convert to 
> realize()
> 
> 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.
> 
The Linux VM's kernel is 3.18.

> 
> > Will try the two patch together later.
> 
> That would be useful
> 
With the two patch, Linux VM (with 3.18) boot up successfully. RHEL7.2(default 
kernel 3.10) and Windows8.1 VM boot up fail with IGD pass-through.
The result is same w/o these two patches.
 
> 
> > > -Original Message-
> > > From: Stefano Stabellini [mailto:stefano.stabell...@eu.citrix.com]
> > > Sent: Friday, January 8, 2016 7:57 PM
> > > To: Hao, Xudong 
> > > Cc: Stefano Stabellini ; Lars
> > > Kurth ; Lars Kurth
> > > ; Cao jin ;
> > > xen-de...@lists.xensource.com; Stefano Stabellini
> > > ; qemu-devel@nongnu.org; Michael S.
> > > Tsirkin 
> > > 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.stabell...@eu.citrix.com]
> > > > > Sent: Wednesday, January 6, 2016 8:18 PM
> > > > > To: Lars Kurth 
> > > > > Cc: Stefano Stabellini ; Hao,
> > > > > Xudong ; Lars Kurth
> > > > > ; Cao jin ;
> > > > > xen-de...@lists.xensource.com; Stefano Stabellini
> > > > > ; qemu-devel@nongnu.org; Michael
> > > > > S. Tsirkin 
> > > > > 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
> > > > > > > 
> > > > > wrote:
> > > > > > >
> > > > > > > On Mon, 4 Jan 2016, Lars Kurth wrote:
> > > > > > >> On 04/01/2016 14:47, "Stefano Stabellini"
> > > > > > >>  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-de...@lists.xen.org
> > > > > > > http://lists.xen.org/xen-devel
> > > > > >
> > > >
> >



Re: [Qemu-devel] [Xen-devel] [PATCH v4] igd-passthrough-i440FX: convert to realize()

2016-01-12 Thread Hao, Xudong
With latest qemu 7b8a354d4716, RHEL7.2 (with default kernel) VM still can't 
boot up with IGD.

Thanks,
-Xudong


> -Original Message-
> From: Michael S. Tsirkin [mailto:m...@redhat.com]
> Sent: Tuesday, January 12, 2016 4:48 PM
> To: Hao, Xudong 
> Cc: Gerd Hoffmann ; Stefano Stabellini
> ; Lars Kurth ; xen-
> de...@lists.xensource.com; Lars Kurth ; qemu-
> de...@nongnu.org; Cao jin ; Stefano Stabellini
> 
> Subject: Re: [Qemu-devel] [Xen-devel] [PATCH v4] igd-passthrough-i440FX:
> convert to realize()
> 
> OK - it's possible that this patch
>   commit 349a3b1cc9023f67f8fa336cb3c4a8f21a4aaaf3
>   Author: Cao jin 
>   Date:   Sat Jan 2 16:02:20 2016 +0800
> 
>   igd-passthrough: fix use of host_pci_config_read
> 
> is required for older guests.
> This patch just went it - could you test latest master please?
> 
> On Tue, Jan 12, 2016 at 08:41:13AM +, Hao, Xudong wrote:
> > Yes, Linux VM update to a 3.18 kernel.
> > The RHEL7.2 default kernel (should be 3.10) VM don't boot up with IGD pass-
> through, and Windows can't boot up either.
> >
> > Thanks,
> > -Xudong
> >
> >
> > > -Original Message-
> > > From: Gerd Hoffmann [mailto:kra...@redhat.com]
> > > Sent: Monday, January 11, 2016 6:32 PM
> > > To: Hao, Xudong 
> > > Cc: Stefano Stabellini ; Lars
> > > Kurth ; xen-de...@lists.xensource.com;
> > > Michael S. Tsirkin ; Lars Kurth
> > > ; qemu- de...@nongnu.org; Cao jin
> > > ; Stefano Stabellini
> > > 
> > > Subject: Re: [Qemu-devel] [Xen-devel] [PATCH v4] igd-passthrough-i440FX:
> > > convert to realize()
> > >
> > >   Hi,
> > >
> > > > I can boot up Linux VM with IGD pass-through with latest qemu
> > > > (without any additional patch), guest run 3D "nexuiz" and get 180fps.
> > >
> > > That is a pretty recent linux guest I assume?
> > > Tried older kernels too, possibly even the old userspace xorg driver?
> > > Do windows guest work as well?
> > >
> > > cheers,
> > >   Gerd
> >



Re: [Qemu-devel] [Xen-devel] [PATCH v4] igd-passthrough-i440FX: convert to realize()

2016-01-12 Thread Michael S. Tsirkin
OK - it's possible that this patch
commit 349a3b1cc9023f67f8fa336cb3c4a8f21a4aaaf3
Author: Cao jin 
Date:   Sat Jan 2 16:02:20 2016 +0800

igd-passthrough: fix use of host_pci_config_read

is required for older guests.
This patch just went it - could you test latest master please?

On Tue, Jan 12, 2016 at 08:41:13AM +, Hao, Xudong wrote:
> Yes, Linux VM update to a 3.18 kernel.
> The RHEL7.2 default kernel (should be 3.10) VM don't boot up with IGD 
> pass-through, and Windows can't boot up either.
> 
> Thanks,
> -Xudong
> 
> 
> > -Original Message-
> > From: Gerd Hoffmann [mailto:kra...@redhat.com]
> > Sent: Monday, January 11, 2016 6:32 PM
> > To: Hao, Xudong 
> > Cc: Stefano Stabellini ; Lars Kurth
> > ; xen-de...@lists.xensource.com; Michael S. Tsirkin
> > ; Lars Kurth ; qemu-
> > de...@nongnu.org; Cao jin ; Stefano Stabellini
> > 
> > Subject: Re: [Qemu-devel] [Xen-devel] [PATCH v4] igd-passthrough-i440FX:
> > convert to realize()
> > 
> >   Hi,
> > 
> > > I can boot up Linux VM with IGD pass-through with latest qemu (without
> > > any additional patch), guest run 3D "nexuiz" and get 180fps.
> > 
> > That is a pretty recent linux guest I assume?
> > Tried older kernels too, possibly even the old userspace xorg driver?
> > Do windows guest work as well?
> > 
> > cheers,
> >   Gerd
> 



Re: [Qemu-devel] [Xen-devel] [PATCH v4] igd-passthrough-i440FX: convert to realize()

2016-01-12 Thread Hao, Xudong
Yes, Linux VM update to a 3.18 kernel.
The RHEL7.2 default kernel (should be 3.10) VM don't boot up with IGD 
pass-through, and Windows can't boot up either.

Thanks,
-Xudong


> -Original Message-
> From: Gerd Hoffmann [mailto:kra...@redhat.com]
> Sent: Monday, January 11, 2016 6:32 PM
> To: Hao, Xudong 
> Cc: Stefano Stabellini ; Lars Kurth
> ; xen-de...@lists.xensource.com; Michael S. Tsirkin
> ; Lars Kurth ; qemu-
> de...@nongnu.org; Cao jin ; Stefano Stabellini
> 
> Subject: Re: [Qemu-devel] [Xen-devel] [PATCH v4] igd-passthrough-i440FX:
> convert to realize()
> 
>   Hi,
> 
> > I can boot up Linux VM with IGD pass-through with latest qemu (without
> > any additional patch), guest run 3D "nexuiz" and get 180fps.
> 
> That is a pretty recent linux guest I assume?
> Tried older kernels too, possibly even the old userspace xorg driver?
> Do windows guest work as well?
> 
> cheers,
>   Gerd



Re: [Qemu-devel] [Xen-devel] [PATCH v4] igd-passthrough-i440FX: convert to realize()

2016-01-11 Thread Hao, Xudong
I used 6bb9ead762bf749af11ea225fc2a74db1b93c105 yesterday, this version don't 
include commit 349a3b1cc. 

Thanks,
-Xudong


> -Original Message-
> From: Cao jin [mailto:caoj.f...@cn.fujitsu.com]
> Sent: Tuesday, January 12, 2016 10:01 AM
> To: Hao, Xudong ; Stefano Stabellini
> 
> Cc: Lars Kurth ; Lars Kurth ;
> xen-de...@lists.xensource.com; Stefano Stabellini
> ; qemu-devel@nongnu.org; Michael S. Tsirkin
> 
> Subject: Re: [Xen-devel] [PATCH v4] igd-passthrough-i440FX: convert to 
> realize()
> 
> Hi
> 
> On 01/11/2016 05:53 PM, 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.
> >
> 
> Because commit: 349a3b1cc is already in the latest qemu(maybe the day before
> yesterday?), so I guess that is why latest qemu works well with igd-passthru 
> and
> also the same reason for git apply conflict?
> 
> > Will try the two patch together later.
> >
> > Thanks,
> > -Xudong
> >
> >
> >> -Original Message-
> >> From: Stefano Stabellini [mailto:stefano.stabell...@eu.citrix.com]
> >> Sent: Friday, January 8, 2016 7:57 PM
> >> To: Hao, Xudong 
> >> Cc: Stefano Stabellini ; Lars Kurth
> >> ; Lars Kurth ; Cao
> >> jin ; xen-de...@lists.xensource.com;
> >> Stefano Stabellini ;
> >> qemu-devel@nongnu.org; Michael S. Tsirkin 
> >> 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.stabell...@eu.citrix.com]
>  Sent: Wednesday, January 6, 2016 8:18 PM
>  To: Lars Kurth 
>  Cc: Stefano Stabellini ; Hao,
>  Xudong ; Lars Kurth ;
>  Cao jin ; xen-de...@lists.xensource.com;
>  Stefano Stabellini ;
>  qemu-devel@nongnu.org; Michael S. Tsirkin 
>  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
> >> 
>  wrote:
> >>
> >> On Mon, 4 Jan 2016, Lars Kurth wrote:
> >>> On 04/01/2016 14:47, "Stefano Stabellini"
> >>>  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-de...@lists.xen.org
> >> http://lists.xen.org/xen-devel
> >
> >>>
> >
> >
> > .
> >
> 
> --
> Yours Sincerely,
> 
> Cao jin
> 




Re: [Qemu-devel] [Xen-devel] [PATCH v4] igd-passthrough-i440FX: convert to realize()

2016-01-11 Thread Cao jin

Hi

On 01/11/2016 05:53 PM, 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.



Because commit: 349a3b1cc is already in the latest qemu(maybe the day 
before yesterday?), so I guess that is why latest qemu works well with

igd-passthru and also the same reason for git apply conflict?


Will try the two patch together later.

Thanks,
-Xudong



-Original Message-
From: Stefano Stabellini [mailto:stefano.stabell...@eu.citrix.com]
Sent: Friday, January 8, 2016 7:57 PM
To: Hao, Xudong 
Cc: Stefano Stabellini ; Lars Kurth
; Lars Kurth ; Cao jin
; xen-de...@lists.xensource.com; Stefano Stabellini
; qemu-devel@nongnu.org; Michael S. Tsirkin

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.stabell...@eu.citrix.com]
Sent: Wednesday, January 6, 2016 8:18 PM
To: Lars Kurth 
Cc: Stefano Stabellini ; Hao,
Xudong ; Lars Kurth ;
Cao jin ; xen-de...@lists.xensource.com;
Stefano Stabellini ;
qemu-devel@nongnu.org; Michael S. Tsirkin 
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


wrote:


On Mon, 4 Jan 2016, Lars Kurth wrote:

On 04/01/2016 14:47, "Stefano Stabellini"
 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-de...@lists.xen.org
http://lists.xen.org/xen-devel







.



--
Yours Sincerely,

Cao jin





Re: [Qemu-devel] [Xen-devel] [PATCH v4] igd-passthrough-i440FX: convert to realize()

2016-01-11 Thread Michael S. Tsirkin
On Mon, Jan 11, 2016 at 10:46:20AM +, 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.stabell...@eu.citrix.com]
> > > Sent: Friday, January 8, 2016 7:57 PM
> > > To: Hao, Xudong 
> > > Cc: Stefano Stabellini ; Lars Kurth
> > > ; Lars Kurth ; Cao jin
> > > ; xen-de...@lists.xensource.com; Stefano 
> > > Stabellini
> > > ; qemu-devel@nongnu.org; Michael S. Tsirkin
> > > 
> > > 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.stabell...@eu.citrix.com]
> > > > > Sent: Wednesday, January 6, 2016 8:18 PM
> > > > > To: Lars Kurth 
> > > > > Cc: Stefano Stabellini ; Hao,
> > > > > Xudong ; Lars Kurth ;
> > > > > Cao jin ; xen-de...@lists.xensource.com;
> > > > > Stefano Stabellini ;
> > > > > qemu-devel@nongnu.org; Michael S. Tsirkin 
> > > > > 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
> > > > > > > 
> > > > > wrote:
> > > > > > >
> > > > > > > On Mon, 4 Jan 2016, Lars Kurth wrote:
> > > > > > >> On 04/01/2016 14:47, "Stefano Stabellini"
> > > > > > >>  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-de...@lists.xen.org
> > > > > > > http://lists.xen.org/xen-devel
> > > > > >
> > > >
> > 



Re: [Qemu-devel] [Xen-devel] [PATCH v4] igd-passthrough-i440FX: convert to realize()

2016-01-11 Thread Stefano Stabellini
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.


> Will try the two patch together later.

That would be useful


> > -Original Message-
> > From: Stefano Stabellini [mailto:stefano.stabell...@eu.citrix.com]
> > Sent: Friday, January 8, 2016 7:57 PM
> > To: Hao, Xudong 
> > Cc: Stefano Stabellini ; Lars Kurth
> > ; Lars Kurth ; Cao jin
> > ; xen-de...@lists.xensource.com; Stefano 
> > Stabellini
> > ; qemu-devel@nongnu.org; Michael S. Tsirkin
> > 
> > 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.stabell...@eu.citrix.com]
> > > > Sent: Wednesday, January 6, 2016 8:18 PM
> > > > To: Lars Kurth 
> > > > Cc: Stefano Stabellini ; Hao,
> > > > Xudong ; Lars Kurth ;
> > > > Cao jin ; xen-de...@lists.xensource.com;
> > > > Stefano Stabellini ;
> > > > qemu-devel@nongnu.org; Michael S. Tsirkin 
> > > > 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
> > > > > > 
> > > > wrote:
> > > > > >
> > > > > > On Mon, 4 Jan 2016, Lars Kurth wrote:
> > > > > >> On 04/01/2016 14:47, "Stefano Stabellini"
> > > > > >>  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-de...@lists.xen.org
> > > > > > http://lists.xen.org/xen-devel
> > > > >
> > >
> 



Re: [Qemu-devel] [Xen-devel] [PATCH v4] igd-passthrough-i440FX: convert to realize()

2016-01-11 Thread Gerd Hoffmann
  Hi,

> I can boot up Linux VM with IGD pass-through with latest qemu (without
> any additional patch), guest run 3D "nexuiz" and get 180fps. 

That is a pretty recent linux guest I assume?
Tried older kernels too, possibly even the old userspace xorg driver?
Do windows guest work as well?

cheers,
  Gerd




Re: [Qemu-devel] [Xen-devel] [PATCH v4] igd-passthrough-i440FX: convert to realize()

2016-01-11 Thread Hao, Xudong
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. 

Will try the two patch together later.

Thanks,
-Xudong


> -Original Message-
> From: Stefano Stabellini [mailto:stefano.stabell...@eu.citrix.com]
> Sent: Friday, January 8, 2016 7:57 PM
> To: Hao, Xudong 
> Cc: Stefano Stabellini ; Lars Kurth
> ; Lars Kurth ; Cao jin
> ; xen-de...@lists.xensource.com; Stefano Stabellini
> ; qemu-devel@nongnu.org; Michael S. Tsirkin
> 
> 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.stabell...@eu.citrix.com]
> > > Sent: Wednesday, January 6, 2016 8:18 PM
> > > To: Lars Kurth 
> > > Cc: Stefano Stabellini ; Hao,
> > > Xudong ; Lars Kurth ;
> > > Cao jin ; xen-de...@lists.xensource.com;
> > > Stefano Stabellini ;
> > > qemu-devel@nongnu.org; Michael S. Tsirkin 
> > > 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
> > > > > 
> > > wrote:
> > > > >
> > > > > On Mon, 4 Jan 2016, Lars Kurth wrote:
> > > > >> On 04/01/2016 14:47, "Stefano Stabellini"
> > > > >>  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-de...@lists.xen.org
> > > > > http://lists.xen.org/xen-devel
> > > >
> >



Re: [Qemu-devel] [Xen-devel] [PATCH v4] igd-passthrough-i440FX: convert to realize()

2016-01-10 Thread Hao, Xudong
Qemu with the patch can't boot VM with IGD pass-through, I'm checking if it 
works w/o this patch to eliminate the environment influence.

Thanks,
-Xudong


> -Original Message-
> From: Stefano Stabellini [mailto:stefano.stabell...@eu.citrix.com]
> Sent: Friday, January 8, 2016 7:57 PM
> To: Hao, Xudong 
> Cc: Stefano Stabellini ; Lars Kurth
> ; Lars Kurth ; Cao jin
> ; xen-de...@lists.xensource.com; Stefano Stabellini
> ; qemu-devel@nongnu.org; Michael S. Tsirkin
> 
> 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.stabell...@eu.citrix.com]
> > > Sent: Wednesday, January 6, 2016 8:18 PM
> > > To: Lars Kurth 
> > > Cc: Stefano Stabellini ; Hao,
> > > Xudong ; Lars Kurth ;
> > > Cao jin ; xen-de...@lists.xensource.com;
> > > Stefano Stabellini ;
> > > qemu-devel@nongnu.org; Michael S. Tsirkin 
> > > 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
> > > > > 
> > > wrote:
> > > > >
> > > > > On Mon, 4 Jan 2016, Lars Kurth wrote:
> > > > >> On 04/01/2016 14:47, "Stefano Stabellini"
> > > > >>  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-de...@lists.xen.org
> > > > > http://lists.xen.org/xen-devel
> > > >
> >



Re: [Qemu-devel] [Xen-devel] [PATCH v4] igd-passthrough-i440FX: convert to realize()

2016-01-08 Thread Stefano Stabellini
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.stabell...@eu.citrix.com]
> > Sent: Wednesday, January 6, 2016 8:18 PM
> > To: Lars Kurth 
> > Cc: Stefano Stabellini ; Hao, Xudong
> > ; Lars Kurth ; Cao jin
> > ; xen-de...@lists.xensource.com; Stefano 
> > Stabellini
> > ; qemu-devel@nongnu.org; Michael S. Tsirkin
> > 
> > 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 
> > > > 
> > wrote:
> > > >
> > > > On Mon, 4 Jan 2016, Lars Kurth wrote:
> > > >> On 04/01/2016 14:47, "Stefano Stabellini"
> > > >>  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-de...@lists.xen.org
> > > > http://lists.xen.org/xen-devel
> > >
> 



Re: [Qemu-devel] [Xen-devel] [PATCH v4] igd-passthrough-i440FX: convert to realize()

2016-01-07 Thread Hao, Xudong
Sure. I'll test it soon.

Thanks,
-Xudong

> -Original Message-
> From: Stefano Stabellini [mailto:stefano.stabell...@eu.citrix.com]
> Sent: Wednesday, January 6, 2016 8:18 PM
> To: Lars Kurth 
> Cc: Stefano Stabellini ; Hao, Xudong
> ; Lars Kurth ; Cao jin
> ; xen-de...@lists.xensource.com; Stefano Stabellini
> ; qemu-devel@nongnu.org; Michael S. Tsirkin
> 
> 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 
> > > 
> wrote:
> > >
> > > On Mon, 4 Jan 2016, Lars Kurth wrote:
> > >> On 04/01/2016 14:47, "Stefano Stabellini"
> > >>  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-de...@lists.xen.org
> > > http://lists.xen.org/xen-devel
> >



Re: [Qemu-devel] [Xen-devel] [PATCH v4] igd-passthrough-i440FX: convert to realize()

2016-01-06 Thread Lars Kurth
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 
>  wrote:
> 
> On Mon, 4 Jan 2016, Lars Kurth wrote:
>> On 04/01/2016 14:47, "Stefano Stabellini"
>>  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-de...@lists.xen.org
> http://lists.xen.org/xen-devel




Re: [Qemu-devel] [Xen-devel] [PATCH v4] igd-passthrough-i440FX: convert to realize()

2016-01-06 Thread Stefano Stabellini
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 
> >  wrote:
> > 
> > On Mon, 4 Jan 2016, Lars Kurth wrote:
> >> On 04/01/2016 14:47, "Stefano Stabellini"
> >>  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-de...@lists.xen.org
> > http://lists.xen.org/xen-devel
>