[Devel] Re: [PATCH 2/6] Container Freezer: Make refrigerator always available
Rafael J. Wysocki
rjw at sisk.pl
Sat Aug 2 07:34:45 PDT 2008
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 at us.ibm.com> a +AOk-crit :
> > > >
> > > > > --- a/kernel/Makefile
> > > > > +-+-+- b/kernel/Makefile
> > > > > @@ -5,7 +-5,7 @@
> > > > > obj-y = sched.o fork.o exec_domain.o panic.o printk.o +AFw
> > > > > cpu.o exit.o itimer.o time.o softirq.o resource.o +AFw
> > > > > sysctl.o capability.o ptrace.o timer.o user.o +AFw
> > > > > - signal.o sys.o kmod.o workqueue.o pid.o +AFw
> > > > > +- signal.o sys.o kmod.o workqueue.o pid.o freezer.o +AFw
> > > >
> > > > 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 at us.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
-------------- next part --------------
_______________________________________________
Containers mailing list
Containers at lists.linux-foundation.org
https:+AC8ALw-lists.linux-foundation.org+AC8-mailman+AC8-listinfo+AC8-containers
More information about the Devel
mailing list