Quoting Matt Helsley (matthltc@xxxxxxxxxx): > On Wed, Jun 03, 2009 at 08:12:19PM -0400, Oren Laadan wrote: > > From 3c24531980764a71705492c2dfc2cc99366784f3 Mon Sep 17 00:00:00 2001 > > From: Oren Laadan <orenl@xxxxxxxxxxxxxxx> > > Date: Wed, 3 Jun 2009 19:31:21 -0400 > > Subject: [PATCH] c/r: use CHECKPOINTING state for hierarchy's cgroup freezer > > > > Set state of freezer cgroup of checkpointed task hierarchy to > > "CHECKPOINTING" during a checkpoint, to ensure that tasks cannot > > be thawed while at it. > > > > get_container() grabs a reference to the root task's freezer cgroup. > > Then in may_checkpoint_task() each it verifies that all tasks belong > > to the same freezer group. > > Ugh, I really don't like grabbing this reference. It may be necessary > though -- I need to think about it more. This was my first thought too. The rest of Oren's algorithm seems sounds, but this just seems like an unneeded step. If we prevent tasks from being moved out of the freezer container, and from being thawed before checkpoint is done, then why do we need to get the container? -serge _______________________________________________ Containers mailing list Containers@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linux-foundation.org/mailman/listinfo/containers
