[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <200808021634.46459.rjw@sisk.pl>
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