On Mon, Mar 31, 2008 at 10:43 PM, Balbir Singh
<balbir@linux.vnet.ibm.com> wrote:
This should probably be inlined in the header file if it's needed
outside this file.
I think this is stale.
I think (as you mentioned earlier) that we need an RCU critical
section in this function, in order for the tasklist traversal to be
safe.
Maybe also BUG_ON(p != mm->owner) ?
Since mm != NULL, no need to test for c->mm since if it's NULL then c->mm != mm
Here we'll want to call vm_cgroup_update_mm_owner(), to adjust the
accounting. (Or if in future we end up with more than a couple of
subsystems that want notification at this time, we'll want to call
cgroup_update_mm_owner() and have it call any interested subsystems.
Paul
--