From: NeilBrown <neilb@suse.de>
To: Alasdair G Kergon <agk@redhat.com>
Cc: Mike Snitzer <snitzer@redhat.com>,
Ming Lei <ming.lei@canonical.com>,
lkml <linux-kernel@vger.kernel.org>,
dm-devel@redhat.com, Kent, Christoph Hellwig <hch@lst.de>,
Lars Ellenberg <drbd-dev@lists.linbit.com>,
Philip Kelleher <pjk1939@linux.vnet.ibm.com>,
Joshua Morris <josh.h.morris@us.ibm.com>,
Christoph Hellwig <hch@infradead.org>,
Overstreet <kent.overstreet@gmail.com>,
Nitin Gupta <ngupta@vflare.org>, Drokin <oleg.drokin@intel.com>,
Oleg, Viro <viro@zeniv.linux.org.uk>,
Dave Chinner <dchinner@redhat.com>, Ming Lin <mlin@kernel.org>,
Jens Axboe <axboe@kernel.dk>,
Andreas Dilger <andreas.dilger@intel.com>,
Geoff Levand <geoff@infradead.org>, Jiri Kosina <jkosina@suse.cz>,
Al, Jim Paris <jim@jtan.com>, Minchan Kim <minchan@kernel.org>,
Dongsu Park <dpark@posteo.net>,
drbd-user@lists.linbit.com
Subject: Re: [Drbd-dev] [PATCH v4 01/11] block: make generic_make_request handle arbitrarily sized bios
Date: Wed, 27 May 2015 09:06:40 +1000 [thread overview]
Message-ID: <20150527090640.44c346e2@notabene.brown> (raw)
In-Reply-To: <20150526153414.GA31182@agk-dp.fab.redhat.com>
[-- Attachment #1: Type: text/plain, Size: 1299 bytes --]
On Tue, 26 May 2015 16:34:14 +0100 Alasdair G Kergon <agk@redhat.com> wrote:
> On Tue, May 26, 2015 at 08:02:08AM -0700, Ming Lin wrote:
> > Now bio_add_page() can build lager bios.
> > And blk_queue_split() can split the bios in ->make_request() if needed.
>
> But why not try to make the bio the right size in the first place so you
> don't have to incur the performance impact of splitting?
Because we don't know what the "right" size is. And the "right" size can
change when array reconfiguration happens.
Splitting has to happen somewhere, if only in bio_addpage where it decides to
create a new bio rather than add another page to the current one. So moving
the split to a different level of the stack shouldn't necessarily change the
performance profile.
Obviously testing is important to confirm that.
NeilBrown
>
> What performance testing have you yet done to demonstrate the *actual* impact
> of this patchset in situations where merge_bvec_fn is currently a net benefit?
>
> Alasdair
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 811 bytes --]
next prev parent reply other threads:[~2015-05-26 23:06 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1432318723-18829-1-git-send-email-mlin@kernel.org>
2015-05-22 18:18 ` [Drbd-dev] [PATCH v4 01/11] block: make generic_make_request handle arbitrarily sized bios Ming Lin
2015-05-25 5:46 ` NeilBrown
2015-05-26 14:36 ` Mike Snitzer
2015-05-26 15:02 ` Ming Lin
2015-05-26 15:34 ` Alasdair G Kergon
2015-05-26 23:06 ` NeilBrown [this message]
2015-05-27 0:40 ` [Drbd-dev] [dm-devel] " Alasdair G Kergon
2015-05-27 8:20 ` Christoph Hellwig
2015-05-26 16:04 ` [Drbd-dev] " Mike Snitzer
2015-05-26 17:17 ` Ming Lin
2015-05-27 23:42 ` Ming Lin
2015-05-28 0:36 ` Alasdair G Kergon
2015-05-28 5:54 ` Ming Lin
2015-05-29 7:05 ` Ming Lin
2015-05-29 15:15 ` Mike Snitzer
2015-06-01 6:02 ` Ming Lin
2015-06-02 20:59 ` Ming Lin
2015-06-04 21:06 ` Mike Snitzer
2015-06-04 22:21 ` Ming Lin
2015-06-05 0:06 ` Mike Snitzer
2015-06-05 5:21 ` Ming Lin
2015-06-09 6:09 ` Ming Lin
2015-06-10 21:20 ` Ming Lin
2015-06-10 21:46 ` Mike Snitzer
2015-06-10 22:06 ` Ming Lin
2015-06-12 5:49 ` Ming Lin
2015-06-18 5:27 ` Ming Lin
2015-05-22 18:18 ` [Drbd-dev] [PATCH v4 08/11] block: kill merge_bvec_fn() completely Ming Lin
2015-05-25 5:49 ` NeilBrown
2015-05-25 14:04 ` Christoph Hellwig
2015-05-25 15:02 ` Ilya Dryomov
2015-05-25 15:08 ` Christoph Hellwig
2015-05-25 15:19 ` Ilya Dryomov
2015-05-25 15:35 ` Alex Elder
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=20150527090640.44c346e2@notabene.brown \
--to=neilb@suse.de \
--cc=agk@redhat.com \
--cc=andreas.dilger@intel.com \
--cc=axboe@kernel.dk \
--cc=dchinner@redhat.com \
--cc=dm-devel@redhat.com \
--cc=dpark@posteo.net \
--cc=drbd-dev@lists.linbit.com \
--cc=drbd-user@lists.linbit.com \
--cc=geoff@infradead.org \
--cc=hch@infradead.org \
--cc=hch@lst.de \
--cc=jim@jtan.com \
--cc=jkosina@suse.cz \
--cc=josh.h.morris@us.ibm.com \
--cc=kent.overstreet@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=minchan@kernel.org \
--cc=ming.lei@canonical.com \
--cc=mlin@kernel.org \
--cc=ngupta@vflare.org \
--cc=oleg.drokin@intel.com \
--cc=pjk1939@linux.vnet.ibm.com \
--cc=snitzer@redhat.com \
--cc=viro@zeniv.linux.org.uk \
/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