From: Will <whansard@sbcglobal.net>
To: Jon Smirl <jonsmirl@gmail.com>
Cc: linux-kernel@vger.kernel.org
Subject: Re: disk speed regression kernel 2.6.29 and after
Date: Wed, 23 Sep 2009 22:06:54 -0500 [thread overview]
Message-ID: <20090923220654.71cd5548.whansard@sbcglobal.net> (raw)
In-Reply-To: <9e4733910909230523v47d1fb3fn2575ed5d8e86069c@mail.gmail.com>
> On Wed, Sep 23, 2009 at 8:08 AM, will <whansard@sbcglobal.net> wrote:
> > I had been using an old 2.6.22 kernel on my machine, and I often backup one
> > partition of my main hard drive to a partition on a second hard drive. The
> > main hard drive is sata 640 gigs, and the second is a pata 320 gig. copying
> > this partition from one drive to the other with dd takes about 3 minutes and
> > 30 seconds. When I installed kernel 2.6.30, and 2.6.31, the time took 9
> > minutes and 20 seconds. I decided to go to the trouble of compiling all the
> > kernels between, and kernels 2.6.22-2.6.28 all do the operation in about 3
>
> > the drives
> > with the kernels 2.6.29 and after.
> > Please cc me with any followups or anything. thanks
295f00042aaf6b553b5f37348f89bab463d4a469 is first bad commit
commit 295f00042aaf6b553b5f37348f89bab463d4a469
Author: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Date: Fri Jan 2 16:12:48 2009 +0100
ide: don't execute the next queued command from the hard-IRQ context (v2)
* Tell the block layer that we are not done handling requests by using
blk_plug_device() in ide_do_request() (request handling function)
and ide_timer_expiry() (timeout handler) if the queue is not empty.
* Remove optimization which directly calls ide_do_request() for the next
queued command from the ide_intr() (IRQ handler) and ide_timer_expiry().
* Remove no longer needed IRQ masking from ide_do_request() - in case of
IDE ports needing serialization disable_irq_nosync()/enable_irq() was
used for the (possibly shared) IRQ of the other IDE port.
* Put the misplaced comment in the right place in ide_do_request().
* Drop no longer needed 'int masked_irq' argument from ide_do_request().
* Merge ide_do_request() into do_ide_request().
* Remove no longer needed IDE_NO_IRQ define.
While at it:
* Don't use HWGROUP() macro in do_ide_request().
* Use __func__ in ide_intr().
This patch reduces IRQ hadling latency for IDE and improves the system-wide
handling of shared IRQs (which should result in more timeout resistant and
stable IDE systems). It also makes it possible to do some further changes
later (i.e. replace some busy-waiting delays with sleeping equivalents).
v2:
Changes per review from Elias Oltmanns:
- fix wrong goto statement in 'if (startstop == ide_stopped)' block
- use spin_unlock_irq()
- don't use obsolete HWIF() macro
Cc: Elias Oltmanns <eo@nebensachen.de>
Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
:040000 040000 fff6eb1cea019a90a494005e0916ed2387018c5d 7a6e4e5282416cffc86db5cc50da21672476312d M drivers
:040000 040000 1e3015fb52fe682ce22fb98f952bb0bcdf1bc3c3 21a9860a6dc3a07f2e1944f1b4aaa16a8720d7ab M include
next prev parent reply other threads:[~2009-09-24 3:07 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-09-23 12:08 will
2009-09-23 12:23 ` Jon Smirl
2009-09-24 3:06 ` Will [this message]
2009-09-24 9:10 ` Arjan van de Ven
2009-09-24 3:07 ` Robert Hancock
2009-09-24 4:35 ` Will
2009-09-24 7:44 ` odie
2009-09-24 9:01 ` Will
2009-09-24 10:41 ` Will
2009-09-24 12:26 ` Bartlomiej Zolnierkiewicz
2009-09-24 16:26 ` Will
2009-09-24 17:34 ` Bartlomiej Zolnierkiewicz
2009-09-24 18:11 ` Frans Pop
2009-09-24 19:12 ` Bartlomiej Zolnierkiewicz
2009-09-24 21:50 ` David Miller
2009-09-24 21:34 Andrew Randrianasulu
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=20090923220654.71cd5548.whansard@sbcglobal.net \
--to=whansard@sbcglobal.net \
--cc=jonsmirl@gmail.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®