public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Jens Axboe <jens.axboe@oracle.com>
To: "Ozan Ça??layan" <ozan@pardus.org.tr>
Cc: "Miller, Mike (OS Dev)" <Mike.Miller@hp.com>,
	linux-kernel <linux-kernel@vger.kernel.org>,
	"scameron@beardog.cce.hp.com" <scameron@beardog.cce.hp.com>
Subject: Re: CCISS performance drop in buffered disk reads in newer kernels
Date: Mon, 7 Dec 2009 19:39:46 +0100	[thread overview]
Message-ID: <20091207183946.GE8742@kernel.dk> (raw)
In-Reply-To: <4B1D47C5.7030804@pardus.org.tr>

On Mon, Dec 07 2009, Ozan Ça??layan wrote:
> Miller, Mike (OS Dev) wrote:
> > Ozan,
> > I'm aware of the performance drop. Please see: http://bugzilla.kernel.org/show_bug.cgi?id=13127. I removed the huge read ahead value of 1024 that we used because users were complaining about small writes being starved. That was back around the 2.6.25 timeframe. Since that timeframe there have no changes in the main i/o path. I'll get back on this as time allows.
> >
> > Meanwhile, you can tweak some of the block layer tunables as such.
> >
> >     echo 64 > /sys/block/cciss\!c0d1/queue/read_ahead_kb
> > OR
> >     blockdev --setra 128 /dev/cciss/c0d1
> >
> > These are just example values. There is also max_hw_sectors_kb and max_sectors_kb that be adjusted.
> >   
> 
> Hi,
> 
> Actually the "#define READ_AHEAD 1024" was removed on March 2008 which
> was included in the 2.6.25.y tree so 2.6.25.20 has 128kB read_ahead
> value too.
> 
> *But* setting read_ahead to 2048 increases buffered disk read average
> from 60~MB/s to 190~MB/s hence the kernel compile time drops to 2 minutes.
> 
> So maybe the regression/change is in another place?
> 
> The server is just a compile-farm so it's triggered by hand, compiles
> distribution's packages and stays idle until the next compilation queue.
> Is it safe/OK to use that 2048kB read_ahead value for such workload?

Yes, it's definitely safe.

> (max_hw_sectors_kb is 512 on my 2.6.25.20 setup and 1024 on 2.6.30.9 but
> it seems that it's read-only)

The *_hw_* values are the driver exported hardware limits, so they are
always read-only.

-- 
Jens Axboe


  reply	other threads:[~2009-12-07 18:39 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-12-07 10:45 CCISS performance drop in buffered disk reads in newer kernels Ozan Çağlayan
2009-12-07 16:32 ` Miller, Mike (OS Dev)
2009-12-07 18:21   ` Ozan Çağlayan
2009-12-07 18:39     ` Jens Axboe [this message]
2009-12-07 19:30       ` Miller, Mike (OS Dev)
2009-12-07 19:32         ` Jens Axboe

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=20091207183946.GE8742@kernel.dk \
    --to=jens.axboe@oracle.com \
    --cc=Mike.Miller@hp.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=ozan@pardus.org.tr \
    --cc=scameron@beardog.cce.hp.com \
    /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