[Pm-utils] [PATCH 1/8] pm-utils 1.2.3 proposed patches

Michael Biebl mbiebl at gmail.com
Sat Nov 29 11:25:57 PST 2008

2008/11/29 Matthew Garrett <mjg59 at srcf.ucam.org>:
> On Sat, Nov 29, 2008 at 06:17:20PM +0100, Michael Biebl wrote:
>> 2008/11/29 Matthew Garrett <mjg59 at srcf.ucam.org>:
>> > The acpi_sleep parameter is a documented kernel interface and many
>> > people currently rely upon it. It's not reasonable to override it.
>> >
>> Have you followed the discussion, why we added this?
>> If we don't clear the acpi flags, we get all sort of weird and
>> unexpected results.
> So make it conditional on whether the user passed acpi_sleep to the
> kernel. You can check /proc/cmdline.

Would still be confusing as hell, if e.g. a user wants to test quirks and runs
one of the vbe-* quirks, but what he actually gets is a strange mix of
acpi_flags and vbe-* quirks.

Past experience has shown, that users were confused by the old behaviour.

It also bears the risk, that we get wrong suspend quirks from users
and add them to hal-info (unless we don't forget to ask each time if
the users has passed acpi flags via the kernel cmd line).


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

More information about the Pm-utils mailing list