From: "Rafael J. Wysocki" <rjw@sisk.pl>
To: Tejun Heo <htejun@gmail.com>
Cc: Maciej Rutecki <maciej.rutecki@gmail.com>,
Michael Sedkowski <sedmich@gmail.com>,
trenn@suse.de, Robert Hancock <hancockr@shaw.ca>,
Henrique de Moraes Holschuh <hmh@hmh.eng.br>,
linux-kernel@vger.kernel.org, linux-ide@vger.kernel.org,
linux-acpi@vger.kernel.org
Subject: Re: Disk spin down issue on shut down/suspend to disk
Date: Wed, 8 Aug 2007 14:24:47 +0200 [thread overview]
Message-ID: <200708081424.48382.rjw@sisk.pl> (raw)
In-Reply-To: <46B930ED.6040607@gmail.com>
On Wednesday, 8 August 2007 04:56, Tejun Heo wrote:
> Rafael J. Wysocki wrote:
> > Well, on my box (nx6325) with the appended (experimental) patch applied
> > on top of 2.6.23-rc1 with the patchset from
> > http://www.sisk.pl/kernel/hibernation_and_suspend/2.6.23-rc2/patches/ , the
> > double spin down doesn't occur during hibernation and the system is shut down
> > notceably faster.
>
> Can you please explain what the patch does?
It replaces the shutting down of devices done by kernel_shutdown_prepare()
with the suspending of the (just like before entering S3, but the target state
is now S4) and the shutting down of sysdevs with suspending them.
It also disables the nonboot CPUs before entering the sleep state (S4), which
generally always is a good idea.
In short, it makes hibernation_platform_enter() execute the enter-a-sleep-state
sequence instead of the mixed shutdown-with-entering-S4 thing.
> So, I take it that entering S4 also spins down the disk.
Yes.
> Does the patch remove the ACPI spin down or libata one?
I _think_ that it removes the ACPI one, although I'll need to do some more
testing to confirm that.
Greetings,
Rafael
--
"Premature optimization is the root of all evil." - Donald Knuth
next prev parent reply other threads:[~2007-08-08 12:15 UTC|newest]
Thread overview: 69+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <fa.bsWpbo79BZgHoaIGhEMfTEBxs54@ifi.uio.no>
[not found] ` <fa.mg6p0itaOQUo/t8deCaYpyqj6Nk@ifi.uio.no>
[not found] ` <fa.RxWYL8z8KktgL8LGZ/cTB2nI1Yc@ifi.uio.no>
[not found] ` <fa.AyyCtZ2yLjC5noHPHtmzn4ffhhE@ifi.uio.no>
[not found] ` <fa.SuAjism+m0z0haqCRVjkc2S978o@ifi.uio.no>
[not found] ` <fa.QgWcLG2hfRb92g4blN61qYfRjlQ@ifi.uio.no>
2007-08-06 23:31 ` Disk spin down issue on shut down/suspend to disk Robert Hancock
2007-08-07 6:41 ` Tejun Heo
2007-08-07 14:28 ` Robert Hancock
2007-08-07 14:37 ` Tejun Heo
2007-08-07 14:45 ` Thomas Renninger
2007-08-07 19:23 ` Michael Sedkowski
2007-08-07 20:09 ` Maciej Rutecki
2007-08-07 20:33 ` Rafael J. Wysocki
2007-08-07 20:41 ` Rafael J. Wysocki
2007-08-07 21:28 ` Maciej Rutecki
2007-08-07 22:30 ` Rafael J. Wysocki
2007-08-08 9:29 ` Michael Sedkowski
2007-08-08 2:56 ` Tejun Heo
2007-08-08 12:24 ` Rafael J. Wysocki [this message]
2007-08-09 21:46 ` Rafael J. Wysocki
2007-08-08 2:54 ` Tejun Heo
2007-08-08 14:08 ` Mark Lord
2007-08-08 14:14 ` Tejun Heo
2007-08-08 14:24 ` Mark Lord
2007-08-08 14:30 ` Tejun Heo
2007-08-08 15:21 ` Mark Lord
2007-08-08 15:35 ` Mark Lord
2007-08-08 16:20 ` Tejun Heo
2007-08-08 17:04 ` Mark Lord
2007-08-09 15:06 ` Pavel Machek
2007-08-09 18:47 ` Rafael J. Wysocki
[not found] ` <fa.Nj/igOoDE6GHsgyWFEo/9WCtE9c@ifi.uio.no>
[not found] ` <fa.usmOl3OVQVx3VExwvKiQI5iAX5Q@ifi.uio.no>
[not found] ` <fa.A9df+SR4FMdI7qZmsXY0xFyurbU@ifi.uio.no>
2007-08-08 0:23 ` Robert Hancock
2007-08-08 12:27 ` Rafael J. Wysocki
[not found] ` <fa.3VfWxxgY+5IvJGIam0al/rUn/Q4@ifi.uio.no>
[not found] ` <fa.WVQYJNYB2uuhgQFoGpK6eCfWazo@ifi.uio.no>
[not found] ` <fa.5IFtia9VWM6ePZjGzb+WVMoHQCY@ifi.uio.no>
[not found] ` <fa.M8tTjYMrntIRJ5aXHPFPjCNEk6k@ifi.uio.no>
[not found] ` <fa.IDurvYCLZ5ArGLsuNcLC3l1CSqQ@ifi.uio.no>
2007-08-11 2:21 ` Robert Hancock
2007-08-08 18:58 Al Boldi
[not found] <fa.kWUtPRLMmU/fKL6dyOOVKsSM0fg@ifi.uio.no>
[not found] ` <fa.EbjjD+/NapKvm3jfMLnkxumomC0@ifi.uio.no>
[not found] ` <fa.dm4O1sGL4RetYq2szbwwJT3cf2c@ifi.uio.no>
[not found] ` <fa.7tYfpSdE8/lmfK5ulSN+/6TRSbw@ifi.uio.no>
[not found] ` <fa.R1sVaL6mATofNd18aEZgwlczb8M@ifi.uio.no>
[not found] ` <fa.Z030SpvZYAHux0f8UtlnJwatsvY@ifi.uio.no>
2007-08-08 0:14 ` Robert Hancock
2007-08-08 4:06 ` Henrique de Moraes Holschuh
2007-08-08 4:26 ` Robert Hancock
-- strict thread matches above, loose matches on Subject: below --
2007-08-06 12:38 Michael Sedkowski
[not found] <838f88840708051428p3d65ff42w627af4bc38f57de3@mail.gmail.com>
2007-08-05 21:33 ` Michał sed
[not found] ` <6bffcb0e0709080832o30fb21b5vcd09baa8387efcc0@mail.gmail.com>
2007-09-08 21:29 ` Rafael J. Wysocki
2007-09-09 9:20 ` Maciek Rutecki
2007-09-09 11:35 ` Rafael J. Wysocki
2007-08-06 5:22 ` Tejun Heo
2007-08-06 14:23 ` Henrique de Moraes Holschuh
2007-08-06 14:50 ` Michael Sedkowski
2007-08-06 15:01 ` Henrique de Moraes Holschuh
2007-08-06 15:27 ` Michael Sedkowski
2007-08-06 15:32 ` Rafael J. Wysocki
2007-08-06 17:03 ` Tejun Heo
2007-08-06 18:21 ` Michael Sedkowski
2007-08-06 18:43 ` Tejun Heo
2007-08-06 19:40 ` Rafael J. Wysocki
2007-08-06 19:43 ` Michael Sedkowski
2007-08-06 22:18 ` Michael Sedkowski
2007-08-07 6:56 ` Tejun Heo
2007-08-07 9:18 ` Michael Sedkowski
2007-08-07 9:31 ` Tejun Heo
2007-08-07 12:56 ` Henrique de Moraes Holschuh
2007-08-07 9:30 ` Michael Sedkowski
2007-08-06 23:45 ` Henrique de Moraes Holschuh
2007-08-07 6:53 ` Tejun Heo
2007-08-07 12:51 ` Henrique de Moraes Holschuh
2007-08-07 13:04 ` Thomas Renninger
2007-08-07 13:35 ` Tejun Heo
2007-08-07 13:09 ` Tejun Heo
2007-08-07 13:16 ` Henrique de Moraes Holschuh
2007-08-07 13:32 ` Tejun Heo
2007-08-07 13:45 ` Thomas Renninger
2007-08-07 14:01 ` Tejun Heo
2007-08-09 15:16 ` Pavel Machek
2007-08-09 15:56 ` Thomas Renninger
2007-08-06 19:25 ` Rafael J. Wysocki
2007-08-05 21:37 Michał sed
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=200708081424.48382.rjw@sisk.pl \
--to=rjw@sisk.pl \
--cc=hancockr@shaw.ca \
--cc=hmh@hmh.eng.br \
--cc=htejun@gmail.com \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-ide@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=maciej.rutecki@gmail.com \
--cc=sedmich@gmail.com \
--cc=trenn@suse.de \
/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;
as well as URLs for NNTP newsgroup(s).