public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Jens Axboe <axboe@suse.de>
To: Al Boldi <a1426z@gawab.com>
Cc: linux-kernel@vger.kernel.org
Subject: Re: [PATCHSET] 0/15 IO scheduler improvements
Date: Sat, 15 Jul 2006 13:06:38 +0200	[thread overview]
Message-ID: <20060715110638.GC22724@suse.de> (raw)
In-Reply-To: <200607142253.26372.a1426z@gawab.com>

[-- Attachment #1: Type: text/plain, Size: 1605 bytes --]

On Fri, Jul 14 2006, Al Boldi wrote:
> Jens Axboe wrote:
> > On Thu, Jul 13 2006, Al Boldi wrote:
> > > Jens Axboe wrote:
> > > > This is a continuation of the patches posted yesterday, I continued
> > > > to build on them. The patch series does:
> > > >
> > > > - Move the hash backmerging into the elevator core.
> > > > - Move the rbtree handling into the elevator core.
> > > > - Abstract the FIFO handling into the elevator core.
> > > > - Kill the io scheduler private requests, that require allocation/free
> > > >   for each request passed through the system.
> > > >
> > > > The result is a faster elevator core (and faster IO schedulers), with
> > > > a nice net reduction of kernel text and code as well.
> > >
> > > Thanks!
> > >
> > > Your efforts are much appreciated, as the current situation is a bit
> > > awkward.
> >
> > It's a good step forward, at least.
> >
> > > > If you have time, please give this patch series a test spin just to
> > > > verify that everything still works for you. Thanks!
> > >
> > > Do you have a combo-patch against 2.6.17?
> >
> > Not really, but git let me generate one pretty easily. It has a few
> > select changes outside of the patchset as well, but should be ok. It's
> > not tested though, should work but the rbtree changes needed to be done
> > additionally. If it boots, it should work :-)
> 
> patch applies ok
> compiles ok
> panics on boot at elv_rb_del
> patch -R succeeds with lot's of hunks

So I most likely botched the rbtree conversion, sorry about that. Oh, I
think it's a silly reverted condition, can you try this one?

-- 
Jens Axboe


[-- Attachment #2: iosched-rollup-2.6.17.4-2.gz --]
[-- Type: application/x-gzip, Size: 25948 bytes --]

  reply	other threads:[~2006-07-15 11:04 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-07-13 20:50 [PATCHSET] 0/15 IO scheduler improvements Al Boldi
2006-07-14  7:15 ` Jens Axboe
2006-07-14 19:53   ` Al Boldi
2006-07-15 11:06     ` Jens Axboe [this message]
2006-07-15 12:35       ` Al Boldi
2006-07-15 17:46         ` Jens Axboe
2006-07-15 20:27           ` Al Boldi
2006-07-16 11:26             ` Jens Axboe
  -- strict thread matches above, loose matches on Subject: below --
2006-07-13 12:46 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=20060715110638.GC22724@suse.de \
    --to=axboe@suse.de \
    --cc=a1426z@gawab.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