xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
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
> ===============================
>

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