From: "Chen, Tiejun" <tiejun.chen@intel.com>
To: Wei Liu <wei.liu2@citrix.com>
Cc: ian.campbell@citrix.com, "mst@redhat.com" <mst@redhat.com>,
Ian Jackson <Ian.Jackson@eu.citrix.com>,
qemu-devel@nongnu.org, xen-devel@lists.xen.org,
Gerd Hoffmann <kraxel@redhat.com>
Subject: Re: [Qemu-devel] [RFC][PATCH 1/1] libxl: add one machine property to support IGD GFX passthrough
Date: Mon, 02 Feb 2015 08:43:08 +0800 [thread overview]
Message-ID: <54CEC81C.2090604@intel.com> (raw)
In-Reply-To: <20150130122613.GB6334@zion.uk.xensource.com>
On 2015/1/30 20:26, Wei Liu wrote:
> On Fri, Jan 30, 2015 at 08:56:48AM +0800, Chen, Tiejun wrote:
> [...]
>>>>>
>>>>> Just remember to handle old option in libxl if your old option is already
>>>>> released by some older version of QEMUs.
>>>>
>>>> I just drop that old option, -gfx_passthru, if we're under qemu upstream
>>>> circumstance, like this,
>>>>
>>>
>>> The question is, is there any version of qemu upstream that has
>>> been released that has the old option (-gfx-passthru)?
>>
>> No. Just now we're starting to support IGD passthrough in qemu upstream.
>>
>
> Right, as of QEMU 2.2.0 there's no support of IGD passthrough in QMEU
> upstream.
>
>>>
>>> This gives us a situation that we need to support both the old
>>> (-gfx-passthru) and new (-igd-passthru) options. Presumably we (libxl)
>>> would need to fork a qemu process to determine which option it has and
>>> pass the right one.
>>>
>>> Or you can try to keep both old and new option at the same time but
>>
>> Yeah, actually I also have considered to keep both two options at the same
>> time. Its really friendly to any qemu version.
>>
>>> deprecate the old one. Then in a few qemu release cycles later (or
>>
>> This should be like 'accel=kvm' versus 'enable-kvm' in qemu upstream.
>> They're coexisted now but just the former is a modern option.
>>
>>> probably one year or two?) you can finally remove the old one. The point
>>> is that to give downstream (in this case, Xen) time to cope with the
>>> change.
>>
>> Here I'm fine to this way.
>>
>> So Gerd,
>>
>
> So you don't actually need to ask Gerd this question because there is no
> old option to keep in qemu upstream.
>
> Libxl (or any sensible toolstack) will just do the right thing to either
> pass -igd-passthru (or whatever you guys agree upon) to qemu upstream or
> pass -gfx-passthru to qemu traditional. :-)
>
Okay let me try do this.
Thanks
Tiejun
next prev parent reply other threads:[~2015-02-02 0:43 UTC|newest]
Thread overview: 39+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-01-21 7:19 [Qemu-devel] [RFC][PATCH 1/1] libxl: add one machine property to support IGD GFX passthrough Tiejun Chen
2015-01-21 7:26 ` Chen, Tiejun
2015-01-21 7:26 ` Chen, Tiejun
2015-01-21 11:37 ` Ian Jackson
2015-01-21 11:56 ` Ian Campbell
2015-01-22 0:55 ` Chen, Tiejun
2015-01-22 0:55 ` Chen, Tiejun
2015-01-21 11:56 ` Ian Campbell
2015-01-21 13:48 ` Gerd Hoffmann
2015-01-21 13:48 ` [Qemu-devel] " Gerd Hoffmann
2015-01-22 0:51 ` Chen, Tiejun
2015-01-23 0:43 ` Chen, Tiejun
2015-01-23 0:43 ` Chen, Tiejun
2015-01-26 0:44 ` Chen, Tiejun
2015-01-27 14:40 ` Ian Jackson
2015-01-27 14:40 ` Ian Jackson
2015-01-28 0:42 ` Chen, Tiejun
2015-01-28 11:12 ` Wei Liu
2015-01-28 11:12 ` Wei Liu
2015-01-29 0:41 ` Chen, Tiejun
2015-01-29 0:41 ` Chen, Tiejun
2015-01-29 10:50 ` Wei Liu
2015-01-29 10:50 ` Wei Liu
2015-01-30 0:56 ` Chen, Tiejun
2015-01-30 12:26 ` Wei Liu
2015-01-30 12:26 ` Wei Liu
2015-01-31 7:07 ` Xu, Quan
2015-01-31 7:07 ` [Qemu-devel] [Xen-devel] " Xu, Quan
2015-01-31 14:33 ` Wei Liu
2015-01-31 15:21 ` Xu, Quan
2015-01-31 15:21 ` [Qemu-devel] " Xu, Quan
2015-01-31 14:33 ` Wei Liu
2015-02-02 0:43 ` Chen, Tiejun [this message]
2015-02-02 0:43 ` Chen, Tiejun
2015-01-30 0:56 ` Chen, Tiejun
2015-01-28 0:42 ` Chen, Tiejun
2015-01-26 0:44 ` Chen, Tiejun
2015-01-22 0:51 ` Chen, Tiejun
2015-01-21 11:37 ` Ian Jackson
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=54CEC81C.2090604@intel.com \
--to=tiejun.chen@intel.com \
--cc=Ian.Jackson@eu.citrix.com \
--cc=ian.campbell@citrix.com \
--cc=kraxel@redhat.com \
--cc=mst@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=wei.liu2@citrix.com \
--cc=xen-devel@lists.xen.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 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.