severity 488144 wishlist
tags 488144 wontfix
thanks

Jon Dowland wrote:
Package: pm-utils
Version: 1.1.2.3-1
Severity: normal

Hello,

With the move to 2.6.26, suspend stopped working for me.
This turned out to be a bug in s2ram, rather than pm-utils,
but this took me a lot longer to figure out due to the
"auto" mode implemented in
10-sleep-module-auto-detection.patch.

I think it would be a good idea to drop this: I think
pm-utils may handle suspend better than s2ram these days,
and more and more quirk-handling is being moved into the
kernel.

Having the behaviour of pm-utils change depending on
whether uswsusp is installed or not makes behaviour very
unpredictable and bugs harder to triage.

Also, it's quite a significant behavioural change from what
people may expect if they are familiar with pm-utils
upstream (or in another distro).

At the very least, if you insist on keeping the patch,
submit it upstream for inclusion or for their opinion (I
suspect they would disagree with it's inclusion).

Remember, that I'm also part of the upstream team of pm-utils ;-)
The simple reason, why I didn't commit the patch upstream is, that the way it is implemented is a bit "hackish".


Along similar lines, I think you should move s2ram from
Recommends: to Suggests:. with s2ram present, pm-utils is
little more than a quirk-handling wrapper around a program
that implements quirk-handling all by itself, and most
people installing pm-utils are going to end up with
pm-utils as long as you Recommend it.

I think the future of suspend on most Linux desktops is
going to be one with out uswsusp and one with all the quirk
handling in the kernel.

You have to make a distinction between s2ram (suspend-to-ram) and s2disk (suspend-to-disk) here. While it maybe true, that s2ram will become simpler (e.g. the whitelist is supposed to be moved out of s2ram), it is basically an advanced vbetools. On the other hand, there is s2disk. I believe s2disk is clearly the way forward (as it allows for things like encryption, compression, integration into splash systems, like splashy or usplash). Also, upstream of the (u)swsusp code in the kernel is a strong proponent of uswsusp.
Short answer is, that I won't change the current behaviour.

If someone installs uswusp, it can be safely expected, that uswsusp should be used, that's why this patch was added (it's the same assumption as in Debian system services, like apache, are automatically activated and started on boot upon installation)
The simplest way actually to not use uswsusp, is to uninstall it.
May I ask, why you have uswsusp installed if you don't want to use it?


Cheers,
Michael

--
Why is it that all of the instruments seeking intelligent life in the
universe are pointed away from Earth?

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to