On 01/20, Christian Brauner wrote:

On Mon, Jan 20, 2020 at 03:00:30PM +0100, Oleg Nesterov wrote:

quoted

This is probably the only patch in series I can understand ;)

To me it looks like a good cleanup regardless, but

On 01/17, Christian Brauner wrote:

quoted

The patch just passes in the parent task_struct

For what? "parent" is always "current", no?

Yes. What exactly are you hinting at? :) Would you prefer that the
commit message speaks of "current" instead of "parent"?

I meant, I don't understand why did you add the new "parent" arg,
cgroup_xxx_fork() can simply use "current" ?

Oleg.