From: Jens Axboe <jens.axboe@oracle.com>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: linux-kernel@vger.kernel.org
Subject: Re: What's in linux-2.6-block.git for 2.6.24
Date: Fri, 21 Sep 2007 11:24:23 +0200 [thread overview]
Message-ID: <20070921092423.GH2367@kernel.dk> (raw)
In-Reply-To: <20070921021505.99c37589.akpm@linux-foundation.org>
On Fri, Sep 21 2007, Andrew Morton wrote:
> > SG chaining bits:
> > - This is the bulk of the patchset. It consists of three major
> > components:
> >
> > - sglist-core, which add helpers for iterating sg lists and
> > switches the block layer and SCSI to use those. Should not
> > have any functional changes.
> > - sglist-drivers, which converts drivers to use the sg list
> > helpers. Again, should not contain functional changes.
> > - sglist-arch, which adds support to most architectures and
> > actually enables sg chaining.
> >
> > The goal of sg chaining is to allow support for very large sgtables,
> > without requiring that they be allocated from one contigious piece of
> > memory.
>
> Presumably sg chaining means more overhead on the IO submission paths? If
> so, has this been quantified?
Depends on how you look at it. For sizes that are small enough to not
use sg chaining (like we do now), there are no changes. Just cleanups to
drivers to use sg_next() and for_each_sg() and so on. Well there is one
snag and that is sg_last(), since that needs to iterate the list. But
that should not be used in performance critical sections. And we can get
rid of that completely as well should we want to, if we define a
per-arch chain limit so that sg_last() can just index the last segment
even if ARCH_HAS_SG_CHAIN is set but nents <= ARCH_SG_CHAIN_SIZE (or
whatever that define would be).
For actually using the sg chaining, there's some overhead of course. Say
we support 256 entries without chaining, or 1mb with 4kb pages. A
request with 1000 entried would require 4 trips to the allocator to
allocate the chainable lists and 4 trips when freeing that list again.
We don't loop the sg list on setup of freeing, just jump to the correct
locations.
So even for chaining, the cost isn't that big. It enables us to support
much larger IO commands and potentially speed up some devices quite a
lot, so CPU cost is less of a concern. And for small sglists, there
isn't a noticable overhead.
--
Jens Axboe
next prev parent reply other threads:[~2007-09-21 9:23 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-09-21 8:57 What's in linux-2.6-block.git for 2.6.24 Jens Axboe
2007-09-21 9:15 ` Andrew Morton
2007-09-21 9:24 ` Jens Axboe [this message]
2007-09-21 9:36 ` Jens Axboe
2007-09-23 13:19 ` Torsten Kaiser
2007-09-23 13:55 ` FUJITA Tomonori
2007-09-23 15:31 ` Torsten Kaiser
2007-09-24 18:48 ` Torsten Kaiser
2007-09-25 5:52 ` Torsten Kaiser
2007-09-23 14:11 ` Alan Cox
2007-09-23 15:40 ` Torsten Kaiser
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=20070921092423.GH2367@kernel.dk \
--to=jens.axboe@oracle.com \
--cc=akpm@linux-foundation.org \
--cc=linux-kernel@vger.kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox