On Thu, Sep 26, 2024 at 3:55 PM Ivan Shapovalov <intelfx@xxxxxxxxxxxx> wrote: > > Improve the inheritance behavior of the `memory.zswap.writeback` cgroup > attribute introduced during the 6.11 cycle. Specifically, in 6.11 we > walk the parent cgroups until we find a _disabled_ writeback, which does > not allow the user to selectively enable zswap writeback while having it > disabled by default. Is there an actual need for this? This is a theoretical use case I thought of (and raised), but I don't think anybody actually wants this...? Besides, most people who want this can just: 1. Enable zswap writeback on root cgroup (and all non-leaf cgroups). 2. Disable zswap writeback on leaf cgroups on creation by default. 3. Selectively enable zswap writeback for the leaf cgroups. All of this is quite doable in userspace. It's not even _that_ racy - just do this before adding tasks etc. to the cgroup?