mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Patrick Mansfield <patmans@us.ibm.com>
To: Andrew Morton <akpm@digeo.com>
Cc: linux-kernel@vger.kernel.org, Mike Anderson <andmike@us.ibm.com>
Subject: Re: 2.5.62-mm2 slow file system writes across multiple disks
Date: Thu, 27 Feb 2003 17:57:00 -0800	[thread overview]
Message-ID: <20030227175700.A26302@beaverton.ibm.com> (raw)
In-Reply-To: <20030226004454.2bfd8deb.akpm@digeo.com>; from akpm@digeo.com on Wed, Feb 26, 2003 at 12:44:54AM -0800

On Wed, Feb 26, 2003 at 12:44:54AM -0800, Andrew Morton wrote:

> Does the other qlogic driver exhibit the same thing?

OK I finally tried out the qlogic driver on the same 10 drives, actually
with scsi-misc-2.5 (2.5.63).

The qlogic is OK performance wise - as Mike pointed out, it sets a lower
queue depth; and even though it sets can_queue higher than the feral, the
qlogic driver software queues where in the same case the feral would give
us a host busy (i.e. queuecomand returns 1).

andmike> Well the qlogic provided driver should exhibit slightly different
andmike> behavior as its per device queue depth is 16 and the request ring count
andmike> is 128.
andmike> 
andmike> The feral driver is currently running a per device queue of 63 and a
andmike> request ring size of 64. (if I am reading the driver correctly?). When
andmike> the request count is exceeded I believe it should return a 1 to the call
andmike> of queuecommand. Can you tell if scsi_queue_insert is being called.

As Mike implies, the feral driver is setting can_queue too high, so in
addition to large queue depth affects, I am also hitting scsi host busy
code paths - yes, it is calling scsi_queue_insert. The host busy code is
not meant to be hit so often, and likely leads to lower performance.

So the feral driver needs lower can_queue (and/or queueing changes) and
lower queue_depth limits.

> Does writeout to a single disk exhibit the same thing?

No, single disk IO performance is OK (with queue depth/TCQ 63 and
can_queue 744), so the too high can_queue with host busy's is probably
hurting performance than the high queue_depth.

> > The larger queue depths can be nice for disk arrays with lots of cache and
> > (more) random IO patterns.
> 
> So says the scsi lore ;)  Have you observed this yourself?  Have you
> any numbers handy?

No and no :(

I'm not sure if the disk arrays we have available have enough memory to
show such affects (I assume standard disk caches are not large enough to
have much of an affect).

-- Patrick Mansfield

      parent reply	other threads:[~2003-02-28  1:57 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-02-24 20:03 Patrick Mansfield
2003-02-24 21:53 ` Andrew Morton
2003-02-25  1:47   ` Patrick Mansfield
2003-02-25  4:43     ` Andrew Morton
2003-02-25 19:24       ` Patrick Mansfield
2003-02-26  8:44         ` Andrew Morton
2003-02-26 12:17           ` Helge Hafting
2003-02-26 17:18           ` Mike Anderson
2003-02-28  1:57           ` Patrick Mansfield [this message]

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=20030227175700.A26302@beaverton.ibm.com \
    --to=patmans@us.ibm.com \
    --cc=akpm@digeo.com \
    --cc=andmike@us.ibm.com \
    --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

all inboxes | Powered by JetHome®