[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <200808020053.18638.rjw@sisk.pl>
Date: Sat, 2 Aug 2008 00:53:17 +0200
From: "Rafael J. Wysocki" <rjw@...k.pl>
To: Matt Helsley <matthltc@...ibm.com>,
Andrew Morton <akpm@...ux-foundation.org>
Cc: Thomas Petazzoni <thomas.petazzoni@...e-electrons.com>,
Paul Menage <menage@...gle.com>,
Li Zefan <lizf@...fujitsu.com>,
Linux-Kernel <linux-kernel@...r.kernel.org>,
Linux Containers <containers@...ts.linux-foundation.org>,
linux-pm@...ts.linux-foundation.org,
Cedric Le Goater <clg@...ibm.com>,
"Serge E. Hallyn" <serue@...ibm.com>,
Michael Opdenacker <michael@...e-electrons.com>,
linux-embedded@...r.kernel.org
Subject: Re: [PATCH 2/6] Container Freezer: Make refrigerator always available
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 <matthltc@...ibm.com> 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 <matthltc@...ibm.com>
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
>
>
>
>
>
��������������������������`�� �#: �#' �#� �#����������������������������X��������������������������������������������������������������������������������n}���F��0���8���@���E�������@���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��Lz�w �#J �#H �#> �#. �#� �#������������S���������������������������������������������������������������������������������ux��h���j���r�������������� ��������k�������������������������������i���m��Oz�t �#? �#3 �#$ �#� �#���������������������������������������������������������������������������������������������jn���f���k�� u������������� �����������x���z���������������������������z���j��W��l �#& �#� �#
�#������������������������������������������������������������������������������������������tx��h���h���n���x��������������!���/���8���������������������������������������x���h��� �#' �#� �#� �#��������*��������������������������������������������������������������������������������g]|��i���h���q���{��������������$���4���@�������P���?���-����������������������������`�� �#0 �#� �#� �#����� �#�q�S��������������������������������������������������������������������������uv��1���i���j���r���}�������Q������$���2���F�������@���1�������
������������������������Z�� �#< �#* �#� �#����� �#�Q;0dq�S����������������������������������������������������������������������im��0���k���m���t���}�������4���Q��!���4���G�������4���!��������������������������������W�� �#C �#1 �#� �#�������������V@5������������������������������������������������������������������tu�$&V�2���m���p���x���|�������;���5���Z��8���P�������%��������������������������������z���S�� �#D �#3 �#� �#�����������������w_Y��������������������������������������������������������������lo���"J�3���p���q���w���}�������;���=���=���e��T���{��������������������������������{���r���P�� �#D �#3 �#� �#����������
������T?5����������������������������������������������������������rr�,%'V��"E�5���q���r���w���z������Q����=���L���V���l���x���s���o���k���f���b���[���Z���V���R���@�� �#D �#3 �#� �#������������������s[U������������������������������������������������������km����O���B�3���t���u���w���z����������f����L���Y���a���z���r���t���}�����������p��'��� �#K �#I �#G �#> �#. �#� �#��������������������R=4��������������������������������������������������po�!��k���W1;Sa4���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����v���i���[�
�I���?%/CV�7��
Z���w���}����������6���M��������m���r���w���������������������������q�� �#* �#� �#� �#� �#��������������������������jLD��������������������������������������ef����m���c�T�� C���:���<�"1U�9���W���}����������:���[���v���������������������������������������x���h��� �#! �#� �#� �#�����������������������������P71g�b]������������������������������lj����v���i����
�J���<���5���6���=��!O�<���V����������@���b�����������P���?���-����������������������������`�� �#0 �#� �#� �#��������������������������������fHA������������������������������ce����m���d�T�� C���8���3���3���6���=��"N�@���Y������H���l�����������@���1�������
������������������������Z�� �#< �#) �#� �#�������������������������������� 8-':pQJ���������������������{WS����w���i����
�J���<���4���2���1���1���2���5���;#H���^��C���l�����������4���!��������������������������������W�� �#B �#0 �#� �#������������������������������
������_D=������������������yw�{XU����m���d�U�� C���8���2���/������&���!���������%*R���c��a�����������%��������������������������������z���S�� �#D �#3 �#� �#����������������������������������������lNF�������������yUQ�!��{���i���]�
�J���;���/���(���#���������������
�� ������"�.X���b������{��������������������������������{���r���P�� �#D �#3 �#� �#����������������������������������
�����W>7��vu�����wXU�{XU����m���c���R
�<������ ������������������������ �#� �#� �#� �#�1]���a��p����������������������������}���t���l���M�� �#D �#3 �#� �#��������������������������������������
�����hJC�����wTO�#��x���e���U���>�
�'������������������������ �#� �#� �#� �#� �#� �#� �#�<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-.P���E���;���%���������������� �#� �#� �#� �#� �#� �#� �#� �#� �#� �#� �#� �#� �#� �#$ �#. �#2 �#3 �#3 �#3 �#3 �#3 �#3 �#3 �#. �#" �#� �#�������������������������������������������������#"#����$�� ��!&��!� �#� �#��������� �#� �#� �#� �#� �#� �#� �#� �#� �#� �#� �#� �#� �# �#� �#� �#� �#� �#� �#� �#� �#� �#� �#� �#� �#� �#� �#��������������������������������������������� �#�������!� �#� �#� �#� �#������������� �#� �#� �#� �#� �#� �#� �#� �#� �#� �#� �#� �#� �#� �#� �#� �#� �#� �#� �#� �#� �#� �#� �#� �#� �#� �#� �#�����������������������������������
Powered by blists - more mailing lists