On Tue, 2007-03-20 at 10:31 +0100, Pavel Machek wrote: > Is the pm_disk_mode still bitmask? If yes, say so. > > ...no, it does not appear so. No, I should have explained. Since the prepare/enter/finish callbacks aren't told what the chosen pm_disk_mode is, there is no point in allowing multiple since they can't differentiate. Changing that is something that could be done, but doesn't seem necessary since ACPI is the only user. > Please don't do this. We want to keep the "use platform if available" > behaviour. [Changing platform->shutdown is really _big_ change, > independend from any cleanups, and it needs to go separate at the very > least. It will break some machines.] As far as I can tell "use platform if available" means "use platform when ACPI pm_ops are present" since that's the only pm_ops that has platform. And notice that I change the default when pm_ops are registered. If, of course, my assumption about ACPI here is wrong, then some other pm_ops implementations will need to be changed to have .pm_disk_mode = PM_DISK_PLATFORM. However, using PM_DISK_PLATFORM by default w/o pm_ops support is *wrong* as I explained previously since it leads to user-interface inconsistencies, the user can switch *away* from platform but *not back* to platform. johannes
Attachment:
signature.asc
Description: This is a digitally signed message part
_______________________________________________ linux-pm mailing list linux-pm@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linux-foundation.org/mailman/listinfo/linux-pm