From: "Roger Pau Monné" <roger.pau@citrix.com>
To: Wei Liu <wei.liu2@citrix.com>
Cc: George Dunlap <George.Dunlap@eu.citrix.com>,
xen-devel <xen-devel@lists.xenproject.org>,
Fabio Fantoni <fabio.fantoni@m2r.biz>,
Ian Jackson <Ian.Jackson@eu.citrix.com>,
Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Subject: Re: [PATCH] libxl: prefer qdisk over blktap when choosing disk backend
Date: Wed, 28 Aug 2013 17:04:07 +0200 [thread overview]
Message-ID: <521E1167.40404@citrix.com> (raw)
In-Reply-To: <20130828134113.GI15729@zion.uk.xensource.com>
On 28/08/13 15:41, Wei Liu wrote:
> On Wed, Aug 28, 2013 at 02:35:56PM +0100, Wei Liu wrote:
>> On Wed, Aug 28, 2013 at 03:16:23PM +0200, Fabio Fantoni wrote:
>>> Il 28/08/2013 15:04, Ian Jackson ha scritto:
>>>> Fabio Fantoni writes ("Re: [Xen-devel] [PATCH] libxl: prefer qdisk over blktap when choosing disk backend"):
>>>>> I think is good prefer qdisk also for significant performance increase
>>>>> in comparison with blktap2.
>>>> Thanks, that's useful information. That, and what George said, have
>>>> convinced me this is the right change.
>>>>
>>>> Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>
>>>
>>> What about qemu traditional with this patch?
>>> I haven't tested qdisk with qemu traditional but unfortunately
>>> qemu-trad. is still widely used and therefore you have to be sure
>>> that does not cause problems.
>>
>> Ah, now I get your question.
>>
>>> I asked about because devices parts seem the same with both qemu but
>>> I not sure about it.
>>
>> We've already switched to qemu-upstream in 4.3. And this patch is not
>> backport material so old system would just work fine IMHO.
>>
>> The only risk of breakage is: users have device_model_version set to
>> qemu-trad and run blktap kernel with Xen pre-4.3, then upgrade to Xen
>> post-4.3 (with this patch). I've tested that, and qemu-trad runs fine
>> for me -- at least it boots and dd works well.
>>
>
> I didn't have a VHD image (format commonly supported by blktap and qemu)
> at hand so the test was not complete. But qemu-trad and qemu-xen are
> both maintained so even if it breaks we are able to fix them -- which is
> main point of this patch, to let users have better supported backend.
I might be completely wrong, but wasn't there a problem when using
blktap VHD images with Qemu (ie VHD images created with blktap weren't
copatible with VHD Qemu implementation)?
next prev parent reply other threads:[~2013-08-28 15:04 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-08-27 14:22 [PATCH] libxl: prefer qdisk over blktap when choosing disk backend Wei Liu
2013-08-28 9:15 ` Wei Liu
2013-08-28 10:35 ` Ian Jackson
2013-08-28 10:51 ` Wei Liu
2013-08-28 10:59 ` George Dunlap
2013-08-28 11:53 ` Fabio Fantoni
2013-08-28 12:46 ` Wei Liu
2013-08-28 13:04 ` Ian Jackson
2013-08-28 13:16 ` Fabio Fantoni
2013-08-28 13:35 ` Wei Liu
2013-08-28 13:41 ` Wei Liu
2013-08-28 15:04 ` Roger Pau Monné [this message]
2013-08-28 15:22 ` Wei Liu
2013-09-03 16:32 ` Ian Campbell
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=521E1167.40404@citrix.com \
--to=roger.pau@citrix.com \
--cc=George.Dunlap@eu.citrix.com \
--cc=Ian.Jackson@eu.citrix.com \
--cc=fabio.fantoni@m2r.biz \
--cc=stefano.stabellini@eu.citrix.com \
--cc=wei.liu2@citrix.com \
--cc=xen-devel@lists.xenproject.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.