All of lore.kernel.org
 help / color / mirror / Atom feed
From: Neil Schemenauer <nas@python.ca>
To: Con Kolivas <kernel@kolivas.org>
Cc: linux-kernel@vger.kernel.org, akpm@digeo.com
Subject: Re: [PATCH][CFT] new IO scheduler for 2.4.20
Date: Fri, 30 May 2003 17:52:47 -0700	[thread overview]
Message-ID: <20030531005247.GA646@glacier.arctrix.com> (raw)
In-Reply-To: <200305310940.41780.kernel@kolivas.org>

Con Kolivas wrote:
> How does this compare to akpm's read-latency2 patch that he posed some
> time ago? That seems to make a massive difference but was knocked back
> for style or approach.

It looks like they do fairly similar things.  Andrew's patch puts
unmergable read requests at a fixed distance from the front of the
queue.  My patch lets unmerged reads skip max((reads - writes), 0)
requests.  That's probably more fair when lots of reads and writes are
in queue.

Andrew's idea of always allowing a merge is probably a good idea and
could be adopted.

My patch uses a fixed deadline for requests (similar to Jen's deadline
scheduler).  I'm not sure if that's an advantage or not.  Note that the
deadline of writes are ignored when inserting a read.

I didn't change the size of the request queue.  I can't find where that
gets set in 2.4.20. :-(

Sorry for the hand-waving.  I didn't know about Andrew's patch and I
obviously didn't do enough testing yet.

  Neil

  reply	other threads:[~2003-05-31  0:36 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-05-30 22:09 [PATCH][CFT] new IO scheduler for 2.4.20 Neil Schemenauer
2003-05-30 23:40 ` Con Kolivas
2003-05-31  0:52   ` Neil Schemenauer [this message]
2003-05-30 17:58     ` Robert Love
  -- strict thread matches above, loose matches on Subject: below --
2003-06-02 17:21 Andreas Dilger
2003-04-17 17:28 Neil Schemenauer
2003-04-17 20:41 ` Andrew Morton
2003-04-20 18:26   ` Neil Schemenauer
2003-04-20 22:06     ` Marc-Christian Petersen
2003-04-21  1:46       ` Neil Schemenauer
2003-04-21 11:33 ` Andrea Arcangeli

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=20030531005247.GA646@glacier.arctrix.com \
    --to=nas@python.ca \
    --cc=akpm@digeo.com \
    --cc=kernel@kolivas.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.