Messages in this thread | | | Date | Wed, 2 Apr 2008 12:53:47 -0700 | From | "Paul Menage" <> | Subject | Re: [RFC][-mm] Add an owner to the mm_struct (v4) |
| |
On Wed, Apr 2, 2008 at 11:53 AM, Balbir Singh <balbir@linux.vnet.ibm.com> wrote: > > So far I've heard no objections or seen any review suggestions. Paul if you are > OK with this patch, I'll ask Andrew to include it in -mm.
My only thoughts were:
- I think I'd still prefer CONFIG_MM_OWNER to be auto-selected rather than manually configured, but it's not a huge deal either way.
- in theory I think we should goto retry if we get to the end of mm_update_next_owner() without finding any other owner. Otherwise we could miss another user if we race with one process forking a new child and then exiting?
- I was looking through the exit code trying to convince myself that current is still on the tasklist until after it makes this call. If it isn't then we could have trouble finding the new owner. But I can't figure out for sure exactly at what point we come off the tasklist.
- I think we only need the cgroup callback in the event that current->cgroups != new_owner->cgroups. (Hmm, have we already been moved back to the root cgroup by this point? If so, then we'll have no way to know which cgruop to unaccount from).
Paul
> > People waiting on this patch > > 1. Pekka Enberg for revoke* syscalls > 2. Serge Hallyn for swap namespaces > 3. Myself to implement the rlimit controller for cgroups > > > > -- > Warm Regards, > Balbir Singh > Linux Technology Center > IBM, ISTL >
| |