From: James Bottomley <James.Bottomley@steeleye.com>
To: Jeremy Higdon <jeremy@sgi.com>
Cc: mpm@waste.org, SCSI Mailing List <linux-scsi@vger.kernel.org>
Subject: Re: [RFC] Make the SCSI mempool allocations variable
Date: 10 Mar 2004 10:29:36 -0500 [thread overview]
Message-ID: <1078932577.10834.44.camel@mulgrave> (raw)
In-Reply-To: <20040310100740.GA619810@sgi.com>
On Wed, 2004-03-10 at 05:07, Jeremy Higdon wrote:
> I first tried direct I/O to the block device and found that the max
> I/O size increased to 4MB from 2MB. I then built an XFS filesystem and
> tried direct I/O to it, and found the same results.
Well, I'm glad someone saw a benefit.
> Believe it or not, there are some h/w RAIDs in which you can get a
> significant performance benefit by increasing I/O size to 4MB from
> 2MB. I don't have one immediately available to post MB/s numbers,
> but you can trust me ('cause I can trust those who told me) :-)
>
> So I heartily approve of this change.
>
> Are you thinking of making SCSI_MAX_PHYS_SEGMENTS a config variable?
> Or would you increase it to 256 by default?
Depending on the arch size of struct scatterlist, the 256 enty sglist is
usually around 6k (on 64 bits). That leads the slab to use order 3
allocations and with a default mempool size of 32, that's eight order 3
allocations that get eaten.
Since my tests showed this pool never to be used on ext3, I certainly
don't think it should be the default if all it does is waste memory for
the majority.
The best course of action is probably to put the patch in (since it has
its current default at 128, it will change nothing) so that people who
wish to play with increasing the allocation can.
To address the question of how you set this, I'd like more information
about which filesystems and workloads find this useful, perhaps with a
view to tying it to an indirect config option (CONFIG_ENTERPRISE or
something).
James
next prev parent reply other threads:[~2004-03-10 15:29 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-03-09 19:02 [RFC] Make the SCSI mempool allocations variable James Bottomley
2004-03-10 6:21 ` Jeremy Higdon
2004-03-10 10:07 ` Jeremy Higdon
2004-03-10 15:29 ` James Bottomley [this message]
2004-03-11 6:38 ` Jeremy Higdon
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=1078932577.10834.44.camel@mulgrave \
--to=james.bottomley@steeleye.com \
--cc=jeremy@sgi.com \
--cc=linux-scsi@vger.kernel.org \
--cc=mpm@waste.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