Messages in this thread | | | From | "Rafael J. Wysocki" <> | Subject | Re: [PATCH 2/5] Container Freezer: Make refrigerator always available | Date | Tue, 12 Aug 2008 22:49:26 +0200 |
| |
On Tuesday, 12 of August 2008, Matt Helsley wrote: > Now that the TIF_FREEZE flag is available in all architectures, > extract the refrigerator() and freeze_task() from kernel/power/process.c > and make it available to all. > > The refrigerator() can now be used in a control group subsystem > implementing a control group freezer. > > Signed-off-by: Cedric Le Goater <clg@fr.ibm.com> > Signed-off-by: Matt Helsley <matthltc@us.ibm.com> > Acked-by: Serge E. Hallyn <serue@us.ibm.com> > Tested-by: Matt Helsley <matthltc@us.ibm.com>
Your Signed-off-by implies your Tested-by (at least it should ;-)).
> --- [--snip--] > Index: linux-2.6.27-rc1-mm1/kernel/power/Kconfig > =================================================================== > --- linux-2.6.27-rc1-mm1.orig/kernel/power/Kconfig > +++ linux-2.6.27-rc1-mm1/kernel/power/Kconfig > @@ -85,6 +85,9 @@ config PM_SLEEP > depends on SUSPEND || HIBERNATION || XEN_SAVE_RESTORE > default y > > +config FREEZER > + def_bool PM_SLEEP > +
I'd still prefer this to go into a Kconfig in the parent directory (ie. where freezer.c and the Makefile building it are located). Otherwise it's guaranteed to confuse someone.
> config SUSPEND > bool "Suspend to RAM and standby" > depends on PM && ARCH_SUSPEND_POSSIBLE >
| |