Re: [PATCH 2/6] Container Freezer: Make refrigerator always available

Rafael J. Wysocki Fri, 01 Aug 2008 15:52:22 -0700

On Friday, 1 of August 2008, Matt Helsley wrote:
> 
> On Fri, 2008-08-01 at 16:27 +0200, Thomas Petazzoni wrote:
> > Hi,
> > 
> > Le Thu, 31 Jul 2008 22:07:01 -0700,
> > Matt Helsley <[EMAIL PROTECTED]> a écrit :
> > 
> > > --- a/kernel/Makefile
> > > +++ b/kernel/Makefile
> > > @@ -5,7 +5,7 @@
> > >  obj-y     = sched.o fork.o exec_domain.o panic.o printk.o \
> > >       cpu.o exit.o itimer.o time.o softirq.o resource.o \
> > >       sysctl.o capability.o ptrace.o timer.o user.o \
> > > -     signal.o sys.o kmod.o workqueue.o pid.o \
> > > +     signal.o sys.o kmod.o workqueue.o pid.o freezer.o \
> > 
> > I have the impression that the code in kernel/power/process.c was
> > compiled only if CONFIG_PM_SLEEP was set. Now that the code has been
> > moved to kernel/freezer.c, it is unconditionnaly compiled in every
> > kernel. Is that correct ?
> >
> > If so, is it possible to put this new feature under some
> > CONFIG_SOMETHING option, for people who care about the kernel size ?
> 
>       How about making it depend on a combination of CONFIG variables?
> Here's an RFC PATCH. Completely untested.
> 
> Signed-off-by: Matt Helsley <[EMAIL PROTECTED]>


Can you please also make the contents of include/linux/freezer.h depend on
CONFIG_FREEZER instead of CONFIG_PM_SLEEP?

Also, I'm not really sure if kernel/power/Kconfig is the right place to define
CONFIG_FREEZER.

Perhaps we should even move freezer.c from kernel/power to kernel
and define CONFIG_FREEZER in Kconfig in there.  Andrew, what do you think?

> ---
>  kernel/Makefile      |    3 ++-
>  kernel/power/Kconfig |    3 +++
>  2 files changed, 5 insertions(+), 1 deletion(-)
> 
> Index: linux-2.6.27-rc1-mm1/kernel/Makefile
> ===================================================================
> --- linux-2.6.27-rc1-mm1.orig/kernel/Makefile
> +++ linux-2.6.27-rc1-mm1/kernel/Makefile
> @@ -5,7 +5,7 @@
>  obj-y     = sched.o fork.o exec_domain.o panic.o printk.o \
>           cpu.o exit.o itimer.o time.o softirq.o resource.o \
>           sysctl.o capability.o ptrace.o timer.o user.o \
> -         signal.o sys.o kmod.o workqueue.o pid.o freezer.o \
> +         signal.o sys.o kmod.o workqueue.o pid.o \
>           rcupdate.o extable.o params.o posix-timers.o \
>           kthread.o wait.o kfifo.o sys_ni.o posix-cpu-timers.o mutex.o \
>           hrtimer.o rwsem.o nsproxy.o srcu.o semaphore.o \
> @@ -24,6 +24,7 @@ CFLAGS_REMOVE_sched_clock.o = -pg
>  CFLAGS_REMOVE_sched.o = -mno-spe -pg
>  endif
>  
> +obj-$(CONFIG_FREEZER) += freezer.o
>  obj-$(CONFIG_PROFILING) += profile.o
>  obj-$(CONFIG_SYSCTL_SYSCALL_CHECK) += sysctl_check.o
>  obj-$(CONFIG_STACKTRACE) += stacktrace.o
> 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 || CGROUP_FREEZER
> +
>  config SUSPEND
>       bool "Suspend to RAM and standby"
>       depends on PM && ARCH_SUSPEND_POSSIBLE
> 
> 
> 
> 
> 


¹þÿ°þÿ¦þÿþÿþÿþÿ`âÿ #: #' # [EMAIL PROTECTED]@Íþÿ1Åþÿºþÿ
°þÿ«þÿ¢þÿþÿþÿþÿZàÿ #A #/ # 
#ÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿþþýýýûâÔÍÏÿüüúÿÎÇÊÿüûúÿüûúÿüûúÿüûúÿÒËÎÿóîîÿýýüÿÔÍÐÿÕÎÑÿõññÿùôôÿôìíÿõîïÿõíîÿôììÿíãäÿ©ÿ%rÿ-oÿKÿ7§Þÿ=·ôÿD¿üÿéþÿ4Åþÿ!¼þÿ´þÿ¬þÿ£þÿþÿþÿþÿþÿWâÿ
 #D #2 # 
#ÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿüüúvÓÌÎÿñììÿüûúÿÏÇÊÿüûúÿüûúÿüûúÿüûúÿÓÌÏÿóîîÿÔÍÐÿôïðÿüûúÿöððÿòéêÿôìíÿôììÿòèèÿôììÿ¾«ÿp}ÿ!qÿ6sÿ9~ÿ\³ÿ:ºùÿ?¿þÿåþÿ%¿þÿ¶þÿ¬þÿ¤þÿþÿþÿþÿþÿzþÿSâÿ
 #D #3 # 
#ÿÿÿÿÿÿÿÿÿÿÿÿüüúÓÌÎùÖÏÑÿûúøÿÏÇÊÿûúøÿûúùÿûúøÿûúùÿûûùÿÓÌÏÿòíîÿýýûÿú÷öÿôìíÿñçèÿóêëÿòèéÿðååÿñèçÿáÔÕÿvxÿmÿsÿ7vÿFvÿGÿgÉÿ;»þÿ{ßþÿ¸þÿ¬þÿ¥þÿþÿþÿþÿþÿ{þÿrþÿPàÿ
 #D #3 # 
#ÿÿÿÿÿÿÿÿÿÿÿÿûúøÔÍÏÿðêêÿúù÷ÿÏÇÊÿúø÷ÿúø÷ÿûúøÿûúøÿûúøÿÕÍÐÿòííÿöñðÿñçèÿñæçÿñçèÿñææÿîââÿðååÿñèçÿÿorÿmÿu¤ÿa§ÿDxÿZÿP®ÿkÎÿsÛþÿ®þÿ¦þÿþÿþÿþÿþÿ}þÿtþÿlþÿMàÿ
 #D #3 # 
#ÿÿÿÿÿÿÿÿÿûûø.ÔÍÏýÖÏÑÿúøöÿúøöÿúøöÿúøöÿúøöÿúø÷ÿúù÷ÿúù÷ÿÓÉÌÿéÝßÿïãäÿðåæÿðåæÿîââÿíààÿïããÿðæåÿàÓÓÿvxÿiÿoÿwªÿ¿ÿr¦ÄÿUÿf³ÿT¶ÿtâÿpâÿlâÿhâÿdâÿ`àÿZâÿVâÿRàÿMàÿ?Åÿ
 #D #3 # 
#ÿÿÿÿÿÿÿÿÿúùö½×ÐÒÿïêéÿúøöÿù÷õÿúøöÿúøöÿúøöÿúøöÿ÷òñÿòééÿÒÂÆÿå×ÙÿïãäÿîáâÿëÜÜÿìÞÞÿíßßÿîââÿïäãÿ¯ÿmoÿjÿp£ÿz±ÿÅÿÝÿ¼âÿa³ÿmÂÿÚÿÝÿéÿòÿúÿþÿþÿmÚÿLzw
 #J #H #> #. # 
#ÿÿÿÿÿÿù÷õS×ÐÒÿúùöÿøöóÿù÷ôÿù÷ôÿù÷ôÿù÷õÿöðïÿñççÿíàáÿíàáÿÎ½ÁÿàÐÒÿêÚÚÿêÚÚÿêÜÛÿëÝÜÿíàßÿíáàÿÝÏÏÿuxÿhÿjÿr©ÿ¹ÿËÿàÿ
 §ôÿÉóÿkÅÿ~ÚÿâÿìÿòÿüÿþÿþÿiÚÿmÛÿOzt #? #3 #$ # 
#ÿÿÿ÷õñ÷õò×º°´ÿîçæÿøöóÿøõóÿöðïÿòêéÿïããÿëÜÝÿéÙÚÿèØÙÿè××ÿº¥©ÿÕÂÄÿéÙÙÿêÛÛÿêÜÛÿëÝÜÿìßÞÿíâàÿ®ÿjnÿfÿk£ÿ
     u°ÿ
¿ÿÏÿáÿ ©ôÿ+²ýÿÎüÿxÚÿzßÿéÿõÿüÿþÿþÿþÿzìÿjÜÿW®l #& # #
 
#ÿÿÿöôð¶©«üÁ·¹ÿ¾°³ÿ»ª®ÿº¦ªÿ²ÿ²ÿ®ÿ«ÿ«ÿ¤ÿ®ÿ¹¥©ÿÖÄÆÿéÚÙÿêÛÚÿëÝÜÿëÞÝÿìàÞÿÛÌÌÿtxÿhÿhÿn©ÿx¸ÿ
ÅÿÒÿâÿ!«ôÿ/·ýÿ8¹þÿ¾ýþÿ¸øþÿ°õþÿ©òþÿ îþÿéþÿåþÿâþÿÝþÿxÙþÿhÊþÿ #' # 
# 
#ÿÿÿ*Ñµÿ¤ÿ ÿ¥ÿ¦ÿ§ÿº ¢ÿº ¢ÿÁ©«ÿÃ¬®ÿË·¸ÿÖÅÆÿ×ÇÇÿéÚÙÿêÛÚÿéÛÚÿêÞÜÿêÞÜÿÿg]|[EMAIL
 PROTECTED] #0 # # #ÿÿÿ [EMAIL PROTECTED]
°þÿ«þÿ¢þÿþÿþÿþÿZàÿ #< #* # #ÿÿÿ 
#Q;0dq\SæÓ¼¼ÿàÎÎÿßÍÍÿßÍÍÿßÍÍÿÞÍÌÿßÎÎÿ¶¤§ÿÕÄÄÿÞÎÍÿßÏÎÿçØ×ÿèÚØÿèÚØÿèÚØÿéÜÙÿ©ÿimÏ0ÿk¯ÿm²ÿt¾ÿ}ÈÿÌÿ4ÿQ¯ÿ!©ôÿ4¸ýÿGÀþÿéþÿ4Åþÿ!¼þÿ´þÿ¬þÿ£þÿþÿþÿþÿþÿWâÿ
 #C #1 # [EMAIL 
PROTECTED]&V2ÿm·ÿp»ÿxÄÿ|ËÿÍÿ;ÿ5ÿZÄÿ8¸ýÿPÂþÿåþÿ%¿þÿ¶þÿ¬þÿ¤þÿþÿþÿþÿþÿzþÿSâÿ
 #D #3 # 
#ÿÿÿÿÿÿw_YêÔ¿¾ÿàÏÎÿàÑÏÿáÑÐÿàÑÏÿáÒÐÿµ£¥ÿÓÄÃÿäÕÓÿäÖÓÿäÖÓÿãÖÓÿãÖÓÿãÖÓÿ¦ÿloÒ"J3ÿp¿ÿqÂÿwÈÿ}ËÿÍÿ;ÿ=ÿ=«ÿeÐÿTÇþÿ{ßþÿ¸þÿ¬þÿ¥þÿþÿþÿþÿþÿ{þÿrþÿPàÿ
 #D #3 # #ÿÿÿÿÿÿ

T?5¿ ýåÖÔÿßÐÎÿßÐÎÿßÐÎÿßÐÎÿ²¡£ÿÑÂÁÿãÕÒÿãÕÒÿãÕÒÿáÔÐÿàÓÏÿÑÂÀÿrrù,%'V"[EMAIL
 PROTECTED] #D #3 # #ÿÿÿÿÿÿÿÿÿ


s[UéÒ¿½ÿÝÎÌÿÜÍÊÿÝÎÌÿÛÍÊÿ± ¡ÿÍ¾½ÿàÒÎÿáÔÐÿÜÎËÿÞÐÍÿßÒÎÿ¥ÿkmÖOB3¥ÿtÏÿuÍÿwËÿzÌÿ~ÍÿÓÿf¬ÚÿL¼ÿYÔÿaßÿzñÿrñÿtöÿ}ýÿþÿþÿpëÿ'çù
 #K #I #G #> #. # #ÿÿÿÿÿÿÿÿÿ
R=4¦ýãÕÑÿÛÍÊÿÚËÉÿÚÌÉÿ®ÿË¼ºÿÞÑÍÿÜÍÊÿÜÎÊÿÛÍÉÿÊ»¹ÿpoû!kW1;Sa
4þcÒþuÐÿuÌÿuÌÿ|ÍÿÓÿâÿr½óÿ[Öÿbàÿiëÿpõÿvøÿ}þÿþÿþÿoëÿlÙþ #I 
#C #; #0 #" # #ÿÿÿÿÿÿÿÿÿÿÿÿ


pZRåÎº¸ÿÙËÈÿ×ÈÅÿÖÇÄÿ¬ÿÈ¹·ÿÙËÇÿÚËÇÿØÉÅÿØÉÅÿ¡ÿhiÝj]K-9S[6ý^ÌþsÌÿwÌÿ|ÍÿÓÿâÿ-«ôÿ~Ìýÿbæÿlïÿröÿxüÿ{þÿþÿþÿþÿþÿqÙüQÏÏ
 #- # # #    #ÿÿÿÿÿÿÿÿÿÿÿÿ

M:1~ýÊ»¹ÿÔÅÂÿÔÅÂÿ©ÿÅ¶´ÿØÉÅÿÔÅÁÿÓÅÀÿÄ´±ÿnmüvi[
I?%/CV7ý
ZÇþwÌÿ}ÍÿÓÿâÿ6±ôÿM¿ýÿÓþÿmôÿr÷ÿwþÿ~þÿþÿþÿþÿþÿþÿqÙü #* # 
# # #ÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿ


jLDåÉ¶³ÿÒÃ¿ÿÐÁ¾ÿ¦ÿÁ²¯ÿÒÄ¿ÿÑÂ¾ÿÏÁ¼ÿÿefßmc

T    
C:<"1U9ýWÅÿ}ÍÿÓÿâÿ:³ôÿ[ÇýÿvÑþÿ¾ýþÿ¸øþÿ°õþÿ©òþÿ îþÿéþÿåþÿâþÿÝþÿxÙþÿhÊþÿ
 #! # # 
#ÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿP71gb]þÃ³±ÿÌ¼¸ÿ¢ÿ¼®«ÿÏÀ»ÿÌ½¸ÿ¼¬¨ÿljývi\

J<56=!O<[EMAIL PROTECTED] #0 # # 
#ÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿ
fHAÌÂ®«ÿÈ¸´ÿ ÿºª§ÿÈ¸³ÿÈ¹³ÿ~ÿceßmd

T    C8336="[EMAIL PROTECTED]@Íþÿ1Åþÿºþÿ
°þÿ«þÿ¢þÿþÿþÿþÿZàÿ #< #) # #ÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿ      
8-':pQJþºª§ÿÿ´¤¡ÿÃ´®ÿ´£ÿ{WSþwi\

J<421125
;#H£ù^ÞÿC¶ùÿlÍýÿßþÿéþÿ4Åþÿ!¼þÿ´þÿ¬þÿ£þÿþÿþÿþÿþÿWâÿ #B #0 # 
#ÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿ
_D=¤¸¤ ÿÿ°ÿ¿®©ÿywÿ{XUømd

U    C82/+&!

%*R´öcçÿaÇþÿàþÿåþÿ%¿þÿ¶þÿ¬þÿ¤þÿþÿþÿþÿþÿzþÿSâÿ #D #3 # 
#ÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿ
lNFôÿ«ÿ«ÿyUQþ!{i]
J;/(#

       ".X·óbéÿÕþÿ{ßþÿ¸þÿ¬þÿ¥þÿþÿþÿþÿþÿ{þÿrþÿPàÿ #D 
#3 # #ÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿ

W>7~vuÿ¥ÿwXUÿ{XUømcR
        <+  # # # 
#1]¸ñaæÿpÙþÿ®þÿ¦þÿþÿþÿþÿþÿ}þÿtþÿlþÿMàÿ #D #3 # 
#ÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿ

hJCÝ¤ÿwTOþ#xeU>
' # # # # # # 
#<j¶ätâÿpâÿlâÿhâÿdâÿ`àÿZâÿVâÿRàÿMàÿHàÿ #D #3 # 
#ÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿ

I4/W~]Xÿllð]T>% # # # # # # # # 
# # # #) #7 #@ #D #D #D #D #D #D #D #D #= #. # 
#ÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿ
lUOµ7-.PE;%ÿÿÿ # # # # # # # # # 
# # # # #$ #. #2 #3 #3 #3 #3 #3 #3 #3 #. #" # 
#ÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿ#"#$ +!&! # 
#ÿÿÿÿÿÿ # # # # # # # # # # # # #   # # # 
# # # # # # # # # # 
#ÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿ #! # # # 
#ÿÿÿÿÿÿÿÿÿ # # # # # # # # # # # # # # # # 
# # # # # # # # # # #ÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿÿ

Re: [PATCH 2/6] Container Freezer: Make refrigerator always available

Reply via email to