From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
To: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Cc: Ronghui Duan <ronghui.duan@intel.com>,
Ian Jackson <Ian.Jackson@eu.citrix.com>,
Jan Beulich <JBeulich@suse.com>,
"xen-devel@lists.xen.org" <xen-devel@lists.xen.org>
Subject: Re: [RFC v1 0/5] VBD: enlarge max segment per request in blkfront
Date: Thu, 13 Sep 2012 09:23:37 -0400 [thread overview]
Message-ID: <20120913132335.GD16635@localhost.localdomain> (raw)
In-Reply-To: <alpine.DEB.2.02.1209131203270.15568@kaball.uk.xensource.com>
On Thu, Sep 13, 2012 at 12:05:35PM +0100, Stefano Stabellini wrote:
> On Thu, 13 Sep 2012, Jan Beulich wrote:
> > >>> On 13.09.12 at 04:28, "Duan, Ronghui" <ronghui.duan@intel.com> wrote:
> > >> And with your patch got:
> > >> read : io=4096.0MB, bw=92606KB/s, iops=23151 , runt= 45292msec
> > >>
> > >> without:
> > >> read : io=4096.0MB, bw=145187KB/s, iops=36296 , runt= 28889msec
> > >>
> > > What type of backend file you are using? In order to remove the influence of
> > > cache in Dom0, I use a physical partition as backend.
> >
> > But you certainly shouldn't be proposing features getting used
> > unconditionally or by default that benefit one class of backing
> > devices and severely penalize others.
>
> Right.
> I am wondering.. Considering that the in-kernel blkback is mainly used
> with physical partitions, is it possible that your patches cause a
> regression with unmodified backends that don't support the new protocol,
> like QEMU for example?
Well for right now I am just using the most simple configuration to
eliminate any extra variables (stacking of components). So my
"testing" has been just on phy:/dev/sda,xvda,w with the sda being
a Corsair SSD.
next prev parent reply other threads:[~2012-09-13 13:23 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-08-16 10:22 [RFC v1 0/5] VBD: enlarge max segment per request in blkfront Duan, Ronghui
2012-08-16 11:14 ` Jan Beulich
2012-08-17 1:12 ` Duan, Ronghui
2012-08-16 13:34 ` Konrad Rzeszutek Wilk
2012-08-16 13:55 ` Konrad Rzeszutek Wilk
2012-08-17 1:26 ` Duan, Ronghui
2012-08-16 14:18 ` Jan Beulich
2012-09-07 17:49 ` Konrad Rzeszutek Wilk
2012-09-13 2:28 ` Duan, Ronghui
2012-09-13 7:32 ` Jan Beulich
2012-09-13 11:05 ` Stefano Stabellini
2012-09-13 13:23 ` Konrad Rzeszutek Wilk [this message]
2012-09-13 14:05 ` Duan, Ronghui
2012-09-17 6:33 ` Duan, Ronghui
2012-09-17 14:37 ` Konrad Rzeszutek Wilk
2012-09-19 21:11 ` Konrad Rzeszutek Wilk
2012-09-13 13:21 ` 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=20120913132335.GD16635@localhost.localdomain \
--to=konrad.wilk@oracle.com \
--cc=Ian.Jackson@eu.citrix.com \
--cc=JBeulich@suse.com \
--cc=ronghui.duan@intel.com \
--cc=stefano.stabellini@eu.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.