lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<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

Powered by Openwall GNU/*/Linux Powered by OpenVZ