All of lore.kernel.org
 help / color / mirror / Atom feed
From: Cao jin <caoj.fnst@cn.fujitsu.com>
To: "Hao, Xudong" <xudong.hao@intel.com>,
	Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Cc: Lars Kurth <lars.kurth@citrix.com>,
	"xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>,
	"Michael S. Tsirkin" <mst@redhat.com>,
	Lars Kurth <lars.kurth.xen@gmail.com>,
	"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>,
	Stefano Stabellini <Stefano.Stabellini@citrix.com>
Subject: Re: [Qemu-devel] [Xen-devel] [PATCH v4] igd-passthrough-i440FX: convert to realize()
Date: Tue, 12 Jan 2016 10:01:21 +0800	[thread overview]
Message-ID: <56945E71.3060807@cn.fujitsu.com> (raw)
In-Reply-To: <B8376D2DEA074F45BA033984477C453E02A2E752@SHSMSX103.ccr.corp.intel.com>

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

-- 
Yours Sincerely,

Cao jin

WARNING: multiple messages have this Message-ID (diff)
From: Cao jin <caoj.fnst@cn.fujitsu.com>
To: "Hao, Xudong" <xudong.hao@intel.com>,
	Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Cc: Lars Kurth <lars.kurth@citrix.com>,
	"xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>,
	"Michael S. Tsirkin" <mst@redhat.com>,
	Lars Kurth <lars.kurth.xen@gmail.com>,
	"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>,
	Stefano Stabellini <Stefano.Stabellini@citrix.com>
Subject: Re: [Xen-devel] [PATCH v4] igd-passthrough-i440FX: convert to realize()
Date: Tue, 12 Jan 2016 10:01:21 +0800	[thread overview]
Message-ID: <56945E71.3060807@cn.fujitsu.com> (raw)
In-Reply-To: <B8376D2DEA074F45BA033984477C453E02A2E752@SHSMSX103.ccr.corp.intel.com>

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

-- 
Yours Sincerely,

Cao jin

  parent reply	other threads:[~2016-01-12  1:58 UTC|newest]

Thread overview: 51+ 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 14:47       ` Stefano Stabellini
2016-01-04 15:33       ` [Qemu-devel] " Lars Kurth
2016-01-04 15:33         ` Lars Kurth
2016-01-04 15:41         ` [Qemu-devel] " Stefano Stabellini
2016-01-04 15:41           ` Stefano Stabellini
2016-01-04 19:49           ` Konrad Rzeszutek Wilk
2016-01-06 10:21           ` [Qemu-devel] [Xen-devel] " Lars Kurth
2016-01-06 10:21             ` Lars Kurth
2016-01-06 12:18             ` [Qemu-devel] [Xen-devel] " Stefano Stabellini
2016-01-06 12:18               ` Stefano Stabellini
2016-01-07  1:32               ` [Qemu-devel] " Hao, Xudong
2016-01-07  1:32                 ` Hao, Xudong
2016-01-08 11:57                 ` [Qemu-devel] [Xen-devel] " Stefano Stabellini
2016-01-08 11:57                   ` Stefano Stabellini
2016-01-11  1:46                   ` [Qemu-devel] " Hao, Xudong
2016-01-11  1:46                     ` Hao, Xudong
2016-01-11  9:53                   ` [Qemu-devel] " Hao, Xudong
2016-01-11  9:53                     ` Hao, Xudong
2016-01-11 10:32                     ` [Qemu-devel] " Gerd Hoffmann
2016-01-11 10:32                       ` Gerd Hoffmann
2016-01-12  8:41                       ` [Qemu-devel] " Hao, Xudong
2016-01-12  8:41                         ` Hao, Xudong
2016-01-12  8:47                         ` [Qemu-devel] " Michael S. Tsirkin
2016-01-12  8:47                           ` Michael S. Tsirkin
2016-01-12  9:50                           ` [Qemu-devel] " Hao, Xudong
2016-01-12  9:50                             ` Hao, Xudong
2016-01-12 10:24                             ` [Qemu-devel] " Gerd Hoffmann
2016-01-12 10:24                               ` Gerd Hoffmann
2016-01-13  1:55                               ` [Qemu-devel] " Hao, Xudong
2016-01-13  1:55                                 ` Hao, Xudong
2016-01-13  7:37                               ` [Qemu-devel] " Hao, Xudong
2016-01-13  7:37                                 ` Hao, Xudong
2016-01-11 10:46                     ` [Qemu-devel] " Stefano Stabellini
2016-01-11 10:46                       ` Stefano Stabellini
2016-01-11 11:01                       ` [Qemu-devel] " Michael S. Tsirkin
2016-01-11 11:01                         ` Michael S. Tsirkin
2016-01-12  9:54                       ` [Qemu-devel] " Hao, Xudong
2016-01-12  9:54                         ` Hao, Xudong
2016-01-12  2:01                     ` Cao jin [this message]
2016-01-12  2:01                       ` Cao jin
2016-01-12  2:24                       ` [Qemu-devel] " Hao, Xudong
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=56945E71.3060807@cn.fujitsu.com \
    --to=caoj.fnst@cn.fujitsu.com \
    --cc=Stefano.Stabellini@citrix.com \
    --cc=lars.kurth.xen@gmail.com \
    --cc=lars.kurth@citrix.com \
    --cc=mst@redhat.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.