Re: [PATCH -tip 26/32] sched: Add a second-level tag for nested CGroup usecase

2020-12-04 Thread Tejun Heo
Hello, On Thu, Dec 03, 2020 at 04:51:42PM -0800, Josh Don wrote: > > So me the color thing reads like an end-run around the cgroup hierarchy. > > Restructuring the cgroup resource hierarchy to incorporate the trust > domains is not necessarily trivial (as is the case for us). I agree > though

Re: [PATCH -tip 26/32] sched: Add a second-level tag for nested CGroup usecase

2020-12-03 Thread Josh Don
On Wed, Dec 2, 2020 at 12:02 AM Peter Zijlstra wrote: > > On Tue, Dec 01, 2020 at 10:18:00PM -0800, Josh Don wrote: > > Hey Peter, > > > > On Wed, Nov 25, 2020 at 5:43 AM Peter Zijlstra wrote: > > > > > > Why can't the above work by setting 'tag' (that's a terrible name, why > > > does that

Re: [PATCH -tip 26/32] sched: Add a second-level tag for nested CGroup usecase

2020-12-02 Thread Tejun Heo
Hello, On Wed, Dec 02, 2020 at 09:02:11AM +0100, Peter Zijlstra wrote: > > the user might only want subsets of {B, C, D, E} to share. For > > instance, the user might only want {B,C} and {D, E} to share. One way > > to solve this would be to allow the user to write the group cookie > >

Re: [PATCH -tip 26/32] sched: Add a second-level tag for nested CGroup usecase

2020-12-02 Thread Peter Zijlstra
On Tue, Dec 01, 2020 at 10:18:00PM -0800, Josh Don wrote: > Hey Peter, > > On Wed, Nov 25, 2020 at 5:43 AM Peter Zijlstra wrote: > > > > Why can't the above work by setting 'tag' (that's a terrible name, why > > does that still live) in CDE? Have the most specific tag live. Same with > > that

Re: [PATCH -tip 26/32] sched: Add a second-level tag for nested CGroup usecase

2020-12-01 Thread Josh Don
Hey Peter, On Wed, Nov 25, 2020 at 5:43 AM Peter Zijlstra wrote: > > Why can't the above work by setting 'tag' (that's a terrible name, why > does that still live) in CDE? Have the most specific tag live. Same with > that thread stuff. The motivation is to allow an unprivileged user the ability

Re: [PATCH -tip 26/32] sched: Add a second-level tag for nested CGroup usecase

2020-12-01 Thread Joel Fernandes
Hi Peter, On Wed, Nov 25, 2020 at 02:42:37PM +0100, Peter Zijlstra wrote: > On Tue, Nov 17, 2020 at 06:19:56PM -0500, Joel Fernandes (Google) wrote: > > From: Josh Don > > > > Google has a usecase where the first level tag to tag a CGroup is not > > sufficient. So, a patch is carried for years

Re: [PATCH -tip 26/32] sched: Add a second-level tag for nested CGroup usecase

2020-11-30 Thread Balbir Singh
On Wed, Nov 25, 2020 at 02:42:37PM +0100, Peter Zijlstra wrote: > On Tue, Nov 17, 2020 at 06:19:56PM -0500, Joel Fernandes (Google) wrote: > > From: Josh Don > > > > Google has a usecase where the first level tag to tag a CGroup is not > > sufficient. So, a patch is carried for years where a

Re: [PATCH -tip 26/32] sched: Add a second-level tag for nested CGroup usecase

2020-11-25 Thread Peter Zijlstra
On Tue, Nov 17, 2020 at 06:19:56PM -0500, Joel Fernandes (Google) wrote: > From: Josh Don > > Google has a usecase where the first level tag to tag a CGroup is not > sufficient. So, a patch is carried for years where a second tag is added which > is writeable by unprivileged users. > > Google

[PATCH -tip 26/32] sched: Add a second-level tag for nested CGroup usecase

2020-11-17 Thread Joel Fernandes (Google)
From: Josh Don Google has a usecase where the first level tag to tag a CGroup is not sufficient. So, a patch is carried for years where a second tag is added which is writeable by unprivileged users. Google uses DAC controls to make the 'tag' possible to set only by root while the second-level