xen-devel.lists.xenproject.org archive mirror
 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: [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  2:01 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1451378763-25398-1-git-send-email-caoj.fnst@cn.fujitsu.com>
     [not found] ` <20151229130853-mutt-send-email-mst@redhat.com>
     [not found]   ` <56828A5E.8060202@cn.fujitsu.com>
2016-01-04 14:47     ` [PATCH v4] igd-passthrough-i440FX: convert to realize() Stefano Stabellini
2016-01-04 15:33       ` Lars Kurth
2016-01-04 15:41         ` Stefano Stabellini
2016-01-04 19:49           ` Konrad Rzeszutek Wilk
2016-01-06 10:21           ` Lars Kurth
2016-01-06 12:18             ` [Xen-devel] " Stefano Stabellini
2016-01-07  1:32               ` Hao, Xudong
2016-01-08 11:57                 ` [Xen-devel] " 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
2016-01-12  9:54                       ` Hao, Xudong
2016-01-12  2:01                     ` Cao jin [this message]
2016-01-12  2:24                       ` Hao, Xudong

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