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