Re: [RFC][PATCH 1/2] memcg: Ensure every task that uses an mm is in the same memory cgroup
From: Tejun Heo
Date: Fri Jun 01 2018 - 15:17:00 EST
Hello,
On Fri, Jun 01, 2018 at 01:11:59PM -0500, Eric W. Biederman wrote:
> Widening the definition of a process sounds good. The memory control
> group code would still need a way to forbid these in cgroup v1 mode,
> when someone uses the task file.
Yeap, you're right. We'll need memcg's can_attach rejecting for v1.
> Using widening instead of denying should reduce the risk of introducing
> a regression.
>
> The only reason I support the crazy case in my earlier patch is so that
> we can have this discussion and in case we do cause a regression with
> this change the previous algorithmic cleanup won't have to be reverted
> as well.
Yeah, sure thing.
Thanks a lot.
--
tejun