Re: [RFC][-mm] Add an owner to the mm_struct (v4)

["Paul Menage" <menage@google.com>]

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
>