From: Brad Campbell <brad@wasp.net.au>
To: Pavel Machek <pavel@ucw.cz>
Cc: Nigel Cunningham <ncunningham@linuxmail.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
suspend2-devel@lists.suspend2.net
Subject: Re: [Suspend2-devel] Re: Suspend2 - Request for review & inclusion in -mm
Date: Wed, 28 Jun 2006 10:00:47 +0400 [thread overview]
Message-ID: <44A21B0F.20304@wasp.net.au> (raw)
In-Reply-To: <20060627190323.GA28863@elf.ucw.cz>
Pavel Machek wrote:
> On Tue 2006-06-27 19:22:37, Brad Campbell wrote:
>> Pavel Machek wrote:
>>>> Some of the advantages of suspend2 over swsusp and uswsusp are:
>>>>
>>>> - Speed (Asynchronous I/O and readahead for synchronous I/O)
>>> uswsusp should be able to match suspend2's speed. It can do async I/O,
>>> etc...
>> ARGH!
>>
>> And the next version of windows will have all the wonderful features that
>> MacOSX has now so best not upgrade to Mac as you can just wait for the next
>> version of windows.
>>
>> suspend2 has it *now*. It works, it's stable.
>
> uswsusp also has it *now*, in case you missed it. I just do not do
> benchmark runs all the time, and don't know how fast suspend2
> is. uswsusp already uses normal I/O ... and that is asynchronous.
Perhaps I was a little hasty then snipping the rest of your reply to Nigel.
You make a single point here regarding Speed, and you *may* be right. However you conveniently
ignore all the other neat features of suspend2 that actually make it usable by stating that they
"would/could/should" be available/doable in uswsusp. It's starting to sound like vapourware.
When I installed ubuntu 6.06 on my shiny new laptop, I pressed the hibernate button. The screen went
black, the hard disk light locked on and it just sat there. I thought to myself "oh dear, it's
locked up" so I pulled the battery out and restarted the machine. (Ubuntu uses the in-kernel
swsusp). It turns out the machine was actually hibernating. Who would have known? I told me nothing
and behaved *exactly* like a machine hard-locked. So on this one box, the in-kernel suspend actually
works, for certain definitions of works.
On resume there is a lovely swap storm as all my apps are swapped back in. If for some reason the
machine decides not to suspend or has a problem while doing so, I don't know about it. It just sits
there until the battery goes flat. No progress/error reports.
And of course on my other laptop it just does weird things. I could probably help debug it if I had
the time or inclination, but seriously.. I simply add
deb http://dagobah.ucc.asn.au/ubuntu-suspend2 dapper/
.. to my /etc/apt/sources list and type apt-get install suspend2 and all of a sudden it works. (Most
of my machines actually run self-patched/compiled kernels, but the installation is just as easy)
Not only works, but it gives me progress information. It actually *tells* me what it's doing.. (and
what might have gone wrong, if something does). Fancy that! And if I've hit hibernate and think "Oh
dear, I needed to add that appointment to my calendar", I can just tap the "esc" key and it will
abort the hibernate and put everything back where it was.
Not to mention all my apps popping back exactly the way they were, with no loss in responsiveness
while they swap back in as soon as the machine becomes live.
I know I might be one of these strange breed of people that actually like these features, but as
much as I love hacking, I'm sick of having to beat my machines upside the head to figure out what is
actually going on or even make them work. Suspend2 just gives it to me out of the box, and in
combination with the hibernate script set it works 1st time, every time.
Yes, suspend2 is more complex than what is in the kernel.. but whadda ya know.. it works. Perhaps
that extra complexity is there for a reason..
What I'd like to see really, rather than obstinate outright rejection, is people to actually look at
Nigel's code and give valid technical commentary on what needs to be changed, and why it needs to be
changed. Rather than "We can do this out of tree, so we'll not accept this code". You might be able
to do it out of tree and make it work, but the number of people using suspend2 is a pretty good
indicator that the current in-kernel code is sub-optimal.
People want a stable, reliable hibernate, and they want it *now*. Not in the next release, or when
someone feels like hacking on it. A number of those same people use the external suspend2 patches,
while the rest of the population simply pine for something that works.
I know I sound like a broken record, but this has already been done to death so many times while I
stood on the sidelines and watched. I really felt the need to throw my worthless .2c into the ring.
Let's get something that actually works into the tree.. hell we had swsusp and pmdisk in there
"competing" for a while, and I've seen discussion about a couple of ieee802.11 stacks. Why not give
it a try.
Brad
--
"Human beings, who are almost unique in having the ability
to learn from the experience of others, are also remarkable
for their apparent disinclination to do so." -- Douglas Adams
next prev parent reply other threads:[~2006-06-28 6:01 UTC|newest]
Thread overview: 135+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-06-26 15:47 Suspend2 - Request for review & inclusion in -mm Nigel Cunningham
2006-06-27 13:33 ` Pavel Machek
2006-06-27 15:22 ` [Suspend2-devel] " Brad Campbell
2006-06-27 15:41 ` Andreas Mohr
2006-06-27 16:01 ` Avuton Olrich
2006-06-27 22:23 ` Pavel Machek
2006-06-27 22:22 ` swsusp / suspend2 reliability (was Re: [Suspend2-devel] Re: Suspend2 - Request for review & inclusion in -mm) Pavel Machek
2006-06-27 22:38 ` Sebastian Kügler
2006-06-27 22:51 ` Pavel Machek
2006-06-27 23:18 ` Sebastian Kügler
2006-06-28 19:53 ` Pavel Machek
2006-06-28 22:19 ` Sebastian Kügler
2006-06-28 22:24 ` Pavel Machek
2006-06-28 22:37 ` Sebastian Kügler
2006-06-28 22:46 ` Pavel Machek
2006-06-28 23:06 ` Sebastian Kügler
2006-06-28 22:52 ` Rafael J. Wysocki
2006-06-28 23:09 ` Sebastian Kügler
2006-06-28 8:56 ` Andreas Jellinghaus
2006-06-28 19:58 ` Pavel Machek
2006-07-06 19:15 ` swsusp / suspend2 reliability Jan Rychter
2006-07-07 13:50 ` Pavel Machek
2006-07-07 14:05 ` [Suspend2-devel] " Rohan Dhruva
2006-07-07 18:21 ` David Fox
2006-07-07 21:42 ` Pavel Machek
2006-07-07 15:03 ` dirk husemann
2006-07-07 23:19 ` Pavel Machek
2006-07-07 18:03 ` Olivier Galibert
2006-07-07 23:18 ` Pavel Machek
2006-07-07 15:19 ` Avuton Olrich
2006-07-07 16:09 ` grundig
2006-07-07 17:44 ` Olivier Galibert
2006-07-07 21:39 ` Pavel Machek
2006-07-07 21:56 ` Olivier Galibert
2006-07-07 23:25 ` Pavel Machek
2006-07-07 23:33 ` [Suspend2-devel] " Nigel Cunningham
2006-07-08 0:04 ` Pavel Machek
2006-07-08 0:28 ` uswsusp history lesson [was Re: [Suspend2-devel] Re: swsusp / suspend2 reliability] Pavel Machek
2006-07-08 3:42 ` Nigel Cunningham
2006-07-08 10:38 ` Rafael J. Wysocki
2006-07-08 11:13 ` Bojan Smojver
2006-07-08 18:34 ` Rafael J. Wysocki
2006-07-08 22:35 ` Bojan Smojver
2006-07-08 11:31 ` Nigel Cunningham
2006-07-08 11:42 ` Bojan Smojver
2006-07-08 12:52 ` Pavel Machek
2006-07-08 13:26 ` Nigel Cunningham
2006-07-08 21:04 ` Pavel Machek
2006-07-08 22:25 ` Nigel Cunningham
2006-07-08 18:52 ` Rafael J. Wysocki
2006-07-08 21:10 ` Pavel Machek
2006-07-08 22:28 ` Nigel Cunningham
2006-07-08 23:54 ` Pavel Machek
2006-07-09 0:02 ` Nigel Cunningham
2006-07-09 0:09 ` Pavel Machek
2006-07-09 10:03 ` Rafael J. Wysocki
2006-07-11 12:45 ` Nigel Cunningham
2006-07-11 21:54 ` Rafael J. Wysocki
2006-07-11 22:01 ` Nigel Cunningham
2006-07-11 22:34 ` Rafael J. Wysocki
2006-07-11 23:00 ` Nigel Cunningham
2006-07-12 10:09 ` Rafael J. Wysocki
2006-07-12 10:16 ` Nigel Cunningham
2006-07-08 11:22 ` Pavel Machek
2006-07-08 4:33 ` Avuton Olrich
2006-07-08 11:12 ` Pavel Machek
2006-07-08 11:21 ` Nigel Cunningham
2006-07-08 4:58 ` Bojan Smojver
2006-07-08 9:11 ` uswsusp history lesson Jan Rychter
2006-07-08 10:14 ` [Suspend2-devel] " Bojan Smojver
2006-07-08 10:41 ` Arjan van de Ven
2006-07-08 11:11 ` Bojan Smojver
2006-07-08 11:13 ` Pavel Machek
2006-07-08 11:16 ` Bojan Smojver
2006-07-08 11:20 ` Nigel Cunningham
2006-07-08 13:19 ` Arjan van de Ven
2006-07-08 22:32 ` Bojan Smojver
2006-07-08 16:43 ` Olivier Galibert
2006-07-08 16:47 ` Arjan van de Ven
2006-07-08 17:01 ` Alon Bar-Lev
2006-07-08 19:36 ` grundig
2006-07-08 17:49 ` Olivier Galibert
2006-07-08 18:03 ` Arjan van de Ven
2006-07-08 21:46 ` Alan Cox
2006-07-09 0:19 ` Olivier Galibert
2006-07-08 17:39 ` Alan Cox
2006-07-08 23:57 ` Pavel Machek
2006-07-09 0:03 ` Nigel Cunningham
[not found] ` <ce9ef0d90607080942w685a6b60q7611278856c78ac0@mail.gmail.com>
2006-07-08 16:50 ` Arjan van de Ven
2006-07-08 19:25 ` Rafael J. Wysocki
2006-07-08 19:39 ` Arjan van de Ven
2006-07-08 20:22 ` Pavel Machek
2006-07-10 9:11 ` dirk husemann
2006-07-10 9:18 ` Arjan van de Ven
2006-07-10 10:02 ` Pavel Machek
2006-07-10 21:49 ` Nigel Cunningham
2006-07-10 23:22 ` Pavel Machek
2006-07-10 23:37 ` Nigel Cunningham
2006-07-10 12:45 ` Thomas Tuttle
2006-07-10 13:05 ` Arjan van de Ven
[not found] ` <ce9ef0d90607081248n1f2fc79fw199b493f3ca6313@mail.gmail.com>
2006-07-08 19:58 ` Rafael J. Wysocki
2006-07-08 20:13 ` Alon Bar-Lev
2006-07-08 20:23 ` Rafael J. Wysocki
2006-07-08 22:20 ` Nigel Cunningham
2006-07-08 23:46 ` Bojan Smojver
2006-07-08 23:53 ` Pavel Machek
2006-07-09 0:18 ` Bojan Smojver
2006-07-09 0:32 ` Pavel Machek
2006-07-09 1:05 ` Bojan Smojver
2006-07-09 13:51 ` Rafael J. Wysocki
2006-07-09 21:06 ` Nigel Cunningham
2006-07-09 21:36 ` Rafael J. Wysocki
2006-07-09 21:46 ` Nigel Cunningham
2006-07-09 22:30 ` Rafael J. Wysocki
2006-07-10 3:57 ` Jason Lunz
2006-07-10 6:20 ` Nigel Cunningham
2006-07-11 14:47 ` Jason Lunz
2006-07-11 20:13 ` Bojan Smojver
2006-07-10 0:28 ` Bojan Smojver
2006-07-09 12:15 ` Matthew Garrett
2006-07-09 21:04 ` Nigel Cunningham
2006-07-10 9:10 ` dirk husemann
2006-07-08 0:28 ` [Suspend2-devel] Re: swsusp / suspend2 reliability Bojan Smojver
2006-07-07 19:27 ` Hua Zhong
2006-07-07 21:10 ` Alon Bar-Lev
2006-07-07 23:48 ` Christian Trefzer
2006-06-27 16:50 ` [Suspend2-devel] Re: Suspend2 - Request for review & inclusion in -mm dirk husemann
2006-06-27 19:03 ` Pavel Machek
2006-06-27 19:19 ` Dave Jones
2006-06-27 21:47 ` Pavel Machek
2006-06-28 6:00 ` Brad Campbell [this message]
2006-06-28 20:03 ` Pavel Machek
2006-06-28 6:09 ` Markus Gaugusch
[not found] ` <200606271940.23934.jaroslav@aster.pl>
[not found] ` <1e1a7e1b0606280228y6c4a0d19p12f8112d216d1aba@mail.gmail.com>
2006-06-28 11:31 ` [Suspend2-devel] " Tim Dijkstra
2006-06-27 23:37 ` Nigel Cunningham
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=44A21B0F.20304@wasp.net.au \
--to=brad@wasp.net.au \
--cc=linux-kernel@vger.kernel.org \
--cc=ncunningham@linuxmail.org \
--cc=pavel@ucw.cz \
--cc=suspend2-devel@lists.suspend2.net \
/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