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 for Android: free password hash cracker in your pocket
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:	Sat, 2 Aug 2008 16:34:45 +0200
From:	"Rafael J. Wysocki" <rjw@...k.pl>
To:	Matt Helsley <matthltc@...ibm.com>
Cc:	Andrew Morton <akpm@...ux-foundation.org>,
	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 Saturday, 2 of August 2008, Matt Helsley wrote:
> 
> On Sat, 2008-08-02 at 00:53 +0200, Rafael J. Wysocki wrote:
> > 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?
> 
> Good point -- I'll add that to this patch and repost.
> 
> > Also, I'm not really sure if kernel/power/Kconfig is the right place to define
> > CONFIG_FREEZER.
> 
> 	There's no nice place to put it since we're dealing with CONFIG_
> variables in two different Kconfig files. I put it in
> kernel/power/Kconfig because I wasn't certain Kbuild would do the right
> thing if I referenced PM_SLEEP from init/Kconfig.
> 
> > Perhaps we should even move freezer.c from kernel/power to kernel
> 
> It's already there.

Yes, sorry.

> Perhaps you meant something else (kernel/power/process.c?)?

Well, I'll have to actually apply the patches and look at the modified code.

> > and define CONFIG_FREEZER in Kconfig in there.  Andrew, what do you think?
> 
> The Kconfig files in kernel/ are Kconfig.hz and Kconfig.preemt which
> don't seem appropriate. I suppose we could add another (perhaps
> Kconfig.cgroup).

Either that, or Kconfig.freezer maybe?  After all, it will also be used for
PM_SLEEP, at least for some time.

> Thanks for the review!

You're welcome.

Thanks,
Rafael
�7mN������n������7mN����p4o�������������0�g�����A�������p����������M���������������������7mN���� 5o��������������7mN������n������7mN����`5o��������������5o�������������p����������M���������������������7mN�����5o��������������7mN������n������7mN����P6o�������������p6o�����a�������p����������M���������������������7mN�����6o��������������7mN������n������7mN����@7o�������������`7o�������������p����������M���������������������7mN�����7o��������������7mN������n������7mN����08o�������������P8o�������������p����������M���������������������7mN����p8o��������������7mN������n������7mN���� 9o�������������@9o�������������p����������M���������������������7mN����`9o��������������7mN������n������7mN�����:o�������������0:o�������������p����������M���������������������7mN����P:o��������������7mN������n������7mN�����;o������������� ;o�����1�������p����������M���������������������7mN����@;o��������������7mN������n������7mN�����;o��������������<o�������������p����������M���������������������7mN����0<o��������������7mN������n������7mN�����<o��������������=o�����Q�������p����������M���������������������7mN���� =o��������������7mN������n������7mN�����=o��������������=o�������������p����������M���������������������7mN�����>o��������������7mN������n������7mN�����>o��������������>o�����q�������p����������M���������������������7mN�����?o��������������7mN������n������7mN�����?o��������������?o�������������p����������M���������������������7mN�����?o��������������7mN������n������7mN�����@o��������������@o�������������p����������M���������������������7mN�����@o��������������7mN������n������7mN�����Ao��������������Ao�����!�������p����������M���������������������7mN�����Ao��������������7mN������n������7mN�����Bo��������������Bo�������������p����������M���������������������7mN�����Bo��������������7mN������n������7mN����pCo��������������Co�����A�������p����������M���������������������7mN�����Co��������������7mN������n������7mN����`Do��������������Do������������p����������M���������������������7mN�����Do��������������7mN������n������7mN����PEo�������������pEo�����a������p����������M���������������������7mN�����Eo��������������7mN������n������7mN����@Fo�������������`Fo�������������p����������M���������������������7mN�����Fo��������������7mN������n������7mN����0Go�������������PGo�������������p����������M���������������������7mN����pGo��������������7mN������n������7mN���� Ho�������������@Ho�������������p����������M���������������������7mN����`Ho��������������7mN������n������7mN�����Io�������������0Io�������������p����������M���������������������7mN�����4o��������������7mN������n������7mN�����Io���������������g�����1�������p����������M���������������������7mN���� Jo��������������7mN������n������7mN�����Jo��������������Jo������
������p����������M���������������������7mN������o��������������7mN������n������7mN�����Ko��������������Ko�����Q
������p����������M���������������������7mN�����Lo��������������7mN������n������7mN�����Lo��������������Lo������	������p����������M���������������������7mN�����Lo��������������7mN������n������7mN�����Mo��������������Mo�����q	������p����������M���������������������7mN����pNo��������������7mN����@!o������7mN�����Mo����������������������	������p����������M���������������������7mN����@Oo��������������7mN����@!o������7mN�����Oo��������������Oo�������������p����������M���������������������7mN�����Oo��������������7mN����@!o������7mN����pPo��������������Po�����!�������p����������M���������������������7mN�����Po��������������7mN����@!o������7mN����`Qo��������������Qo�������������p����������M���������������������7mN�����Qo��������������7mN����@!o������7mN����PRo�������������pRo�����A�������p����������M���������������������7mN�����Ro��������������7mN����@!o������7mN����@So�������������`So�������������p����������M���������������������7mN�����So��������������7mN����@!o������7mN����0To�������������PTo�����a�������p����������M���������������������7mN����pTo��������������7mN����@!o������7mN���� Uo�������������@Uo�������������p����������M���������������������7mN����`Uo��������������7mN����@!o������7mN�����Vo�������������0Vo�������������p����������M���������������������7mN����PVo��������������7mN����@!o������7mN�����Wo������������� Wo�������������p����������M���������������������7mN����@Wo��������������7mN����@!o������7mN�����Wo��������������Xo�������������p����������M���������������������7mN����0Xo��������������7mN����@!o������7mN�����Xo��������������Yo�����1�������p����������M���������������������7mN���� Yo��������������7mN����@!o������7mN�����Yo��������������Yo�������������p����������M���������������������7mN�����No��������������7mN����@!o������7mN�����Zo��������������[o�����Q�������p����������M����`����������������7mN�����Zo��������������7mN����@!o������7mN�����[o��������������[o�������������p����������M���������������������7mN������o��������������7mN����@!o������7mN������o���������������o�����q�������p����������M���� ����������������7mN������o��������������7mN����@!o������7mN�����]o��������������]o�������������p����������M����@����������������7mN�����]o��������������7mN����@!o������7mN�����^o��������������^o�������������p����������M����`����������������7mN�����^o��������������7mN����@!o������7mN�����_o�����������

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ