From: Dan Magenheimer <dan.magenheimer@oracle.com>
To: Paul Durrant <paul.durrant@citrix.com>, xen-devel@lists.xensource.com
Subject: RE: Add multi-page shared ring support to xen-blkfront/back
Date: Tue, 12 Jan 2010 15:52:00 -0800 (PST) [thread overview]
Message-ID: <e7a2aa47-d831-4d79-a15a-0cb64d2623bc@default> (raw)
In-Reply-To: <4B4CB6AE.70005@citrix.com>
Haven't read the patch carefully, but a question:
Will blkback fall back safely to single-page allocation
if fragmentation has occurred... and, if so, will this inability
to allocate multi-page be dynamically propogated to blkfronts
(rather than just the largest ring size "supported" by
blkback)?
> -----Original Message-----
> From: Paul Durrant [mailto:paul.durrant@citrix.com]
> Sent: Tuesday, January 12, 2010 10:52 AM
> To: xen-devel@lists.xensource.com
> Subject: Re: [Xen-devel] Add multi-page shared ring support to
> xen-blkfront/back
>
>
> Paul Durrant wrote:
> > These patches add support for multi-page shared rings to
> xen-blkfront/back.
> > Shared rings must always be a power of 2 in size and so the
> size is controlled
> > by the new 'max_ring_page_order' module parameter of
> xen-blkback. This defaults
> > to 0 so operationally there should be no difference with
> these patches in place.
> > The maximum value of 'max_ring_page_order' is currently limited to 2
> > (i.e. 4 pages).
> > xen-blkfront will always use the largest shared ring size
> supported by the back
> > end. To override this and continue to use a single page
> ring, as before, set the
> > 'allow_multi_page_ring' module parameter to 0.
> >
>
> I don't know why git-send-email seems to have sent my patch
> series into
> a black hole. Here they are as attachments this time.
>
> Paul
>
> --
> ===============================
> Paul Durrant, Software Engineer
>
> Citrix Systems (R&D) Ltd.
> First Floor, Building 101
> Cambridge Science Park
> Milton Road
> Cambridge CB4 0FY
> United Kingdom
> ===============================
>
next prev parent reply other threads:[~2010-01-12 23:52 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-01-12 17:44 Add multi-page shared ring support to xen-blkfront/back Paul Durrant
2010-01-12 17:44 ` [PATCH 1/2] Add multi-page ring support to xen-blkback Paul Durrant
2010-01-12 17:44 ` [PATCH 2/2] Add multi-page ring support to xen-blkfront Paul Durrant
2010-01-12 17:51 ` Add multi-page shared ring support to xen-blkfront/back Paul Durrant
2010-01-12 20:58 ` Add multi-page shared ring support toxen-blkfront/back James Harper
2010-01-12 21:22 ` Daniel Stodden
2010-01-13 6:03 ` James Harper
2010-01-12 23:52 ` Dan Magenheimer [this message]
2011-08-03 18:17 ` Add multi-page shared ring support to xen-blkfront/back Konrad Rzeszutek Wilk
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=e7a2aa47-d831-4d79-a15a-0cb64d2623bc@default \
--to=dan.magenheimer@oracle.com \
--cc=paul.durrant@citrix.com \
--cc=xen-devel@lists.xensource.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).