[Devel] Re: [PATCH v21 015/100] cgroup freezer: Update stale locking comments

Matt Helsley matthltc at us.ibm.com
Thu May 6 15:34:57 PDT 2010


On Thu, May 06, 2010 at 01:31:17PM -0700, Matt Helsley wrote:
> On Thu, May 06, 2010 at 09:40:19PM +0200, Rafael J. Wysocki wrote:
> > On Saturday 01 May 2010, Oren Laadan wrote:
> > > From: Matt Helsley <matthltc at us.ibm.com>
> > > 
> > > Update stale comments regarding locking order and add a little more detail
> > > so it's easier to follow the locking between the cgroup freezer and the
> > > power management freezer code.
> > 
> > I guess these three patches are for me.
> > 
> > Do you want me to handle them?
> > 
> > Rafael
> 
> Hi Rafael,
> 
> 	I think you can take 15 as it's an update to the comments.
> 
> 16 and 17 are specific to checkpoint/restart. There are no other in-kernel
> uses, that we've identified, for them. So I was under the impression that it's
> inappropriate to add them without also adding the checkpoint/restart patches
> that use them.

Of course, assuming I'm wrong about the dependency on checkpoint/restart, I
have no objection to your taking them.

Thanks again!

Cheers,
	-Matt Helsley
_______________________________________________
Containers mailing list
Containers at lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/containers




More information about the Devel mailing list