Re: [RFC PATCH v2 1/3] PM / Core: suspend_again callback fordevice PM.

From: Pavel Machek
Date: Tue Apr 26 2011 - 16:39:00 EST


Hi!

> > To conclude, I'm not sure about the approach. In particular, I'm not sure
> > if the benefit is worth the effort and the resulting complications (ie. the
> > possibility of having to deal with wakeup signals not requested by user
> > space) seem to be a bit too far reaching.
> >
> > Greg, what do you think?
>
> I agree with you in that I don't think that this type of feature is
> valid at the moment.

Our current "solution" is low level suspend code on Zaurus directly
looking at charger state and doing the "wakeup or not" decision by hand.

> I don't understand why our current situation doesn't work, what are we
> lacking that is needed for these systems that we have not seen
> before?

It works, but it is ugly; and it seems samsung now needs similar
hacks.

> What is the root problem that this is trying to solve?

It is trying to fix machines that need to run periodic kernel tasks
even when user asked them to sleep. Zaurus needs to periodicaly wake
up to be able to charge battery in s2ram state, for example.
Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/