[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