From: Bob Liu <bob.liu@oracle.com>
To: Julien Grall <julien.grall@citrix.com>
Cc: xen-devel@lists.xenproject.org, ian.campbell@citrix.com,
stefano.stabellini@eu.citrix.com, linux-kernel@vger.kernel.org,
"David Vrabel" <david.vrabel@citrix.com>,
"Boris Ostrovsky" <boris.ostrovsky@oracle.com>,
"Roger Pau Monné" <roger.pau@citrix.com>
Subject: Re: [Xen-devel] [PATCH 0/2] block/xen-blkfront: Support non-indirect with 64KB page granularity
Date: Mon, 14 Sep 2015 08:37:06 +0800 [thread overview]
Message-ID: <55F616B2.2070703@oracle.com> (raw)
In-Reply-To: <55F5B6BD.4030904@citrix.com>
On 09/14/2015 01:47 AM, Julien Grall wrote:
>
>
> On 13/09/2015 13:44, Bob Liu wrote:
>> I may misunderstood here.
>> But I think same changes are also required even if backend supports indirect grant when frontend is using 64KB page granularity.
>> Else
>> 1) How to set up the grant map for requests in domU?
>> The minimum segment buffer size in a request is PAGE_SIZE(64KB) while grant is 4KB based.
>>
>> 2) Codes like below in blkback.c may not work correctly?
>> if ((req->u.rw.seg[i].last_sect >= (PAGE_SIZE >> 9)) ||
>>
>> Because PAGE_SIZE in backend is 4KB, while the written value by domU is 64KB based.
>
> As mention in my cover letter, this patch is not self-sufficient to support 64KB guest. It's a follow-up of the 64KB page granularity support I sent on the ML (the new version was sent earlier this week [1]).
>
> One of the patch [2] is taking care of breaking down the I/O request in multiple 4KB segment that will be used in the ring request. You may want to give a look to this patch before looking to this series.
>
Oh, sorry! I'll have a look at those patches.
Thanks,
-Bob
next prev parent reply other threads:[~2015-09-14 0:37 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-09-11 19:31 [PATCH 0/2] block/xen-blkfront: Support non-indirect with 64KB page granularity Julien Grall
2015-09-11 19:31 ` [PATCH 1/2] block/xen-blkfront: Introduce blkif_ring_get_request Julien Grall
2015-10-05 15:22 ` Roger Pau Monné
2015-10-05 15:22 ` Roger Pau Monné
2015-09-11 19:31 ` Julien Grall
2015-09-11 19:32 ` [PATCH 2/2] block/xen-blkfront: Handle non-indirect grant with 64KB pages Julien Grall
2015-09-11 19:32 ` Julien Grall
2015-10-05 16:01 ` Roger Pau Monné
2015-10-05 16:01 ` Roger Pau Monné
2015-10-05 17:05 ` Julien Grall
2015-10-05 17:05 ` [Xen-devel] " Julien Grall
2015-10-06 9:39 ` Roger Pau Monné
2015-10-06 9:58 ` Julien Grall
2015-10-06 9:58 ` [Xen-devel] " Julien Grall
2015-10-06 10:06 ` Roger Pau Monné
2015-10-12 18:00 ` Julien Grall
2015-10-19 11:16 ` Roger Pau Monné
2015-10-19 11:16 ` [Xen-devel] " Roger Pau Monné
2015-10-19 11:36 ` Julien Grall
2015-10-19 11:36 ` Julien Grall
2015-10-12 18:00 ` Julien Grall
2015-10-06 10:06 ` Roger Pau Monné
2015-10-06 9:39 ` Roger Pau Monné
2015-09-12 9:46 ` [Xen-devel] [PATCH 0/2] block/xen-blkfront: Support non-indirect with 64KB page granularity Bob Liu
2015-09-13 12:06 ` Julien Grall
2015-09-13 12:06 ` [Xen-devel] " Julien Grall
2015-09-13 12:44 ` Bob Liu
2015-09-13 12:44 ` [Xen-devel] " Bob Liu
2015-09-13 17:47 ` Julien Grall
2015-09-14 0:37 ` Bob Liu [this message]
2015-09-14 0:37 ` Bob Liu
2015-09-13 17:47 ` Julien Grall
2015-09-12 9:46 ` Bob Liu
2015-10-02 9:32 ` Julien Grall
2015-10-02 9:32 ` [Xen-devel] " Julien Grall
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=55F616B2.2070703@oracle.com \
--to=bob.liu@oracle.com \
--cc=boris.ostrovsky@oracle.com \
--cc=david.vrabel@citrix.com \
--cc=ian.campbell@citrix.com \
--cc=julien.grall@citrix.com \
--cc=linux-kernel@vger.kernel.org \
--cc=roger.pau@citrix.com \
--cc=stefano.stabellini@eu.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.