Comments

It turns out that we'll have to live with attributes which are
inherited at cgroup creation time but not affected by further updates
to the parent afterwards - such attributes are already in wide use
e.g. for cpuset.
So, there's nothing to do for netcls_cgroup for hierarchy support.
Its current behavior - inherit only during creation - is good enough.
Move config inheriting from ->css_alloc() to ->css_online() for
consistency, which doesn't change behavior at all, and remove
.broken_hierarchy marking.
Signed-off-by: Tejun Heo <tj@kernel.org>
---
net/sched/cls_cgroup.c | 20 ++++++++------------
1 file changed, 8 insertions(+), 12 deletions(-)

On 20.11.2012 09:30, Tejun Heo wrote:
> It turns out that we'll have to live with attributes which are> inherited at cgroup creation time but not affected by further updates> to the parent afterwards - such attributes are already in wide use> e.g. for cpuset.>> So, there's nothing to do for netcls_cgroup for hierarchy support.> Its current behavior - inherit only during creation - is good enough.>> Move config inheriting from ->css_alloc() to ->css_online() for> consistency, which doesn't change behavior at all, and remove> .broken_hierarchy marking.>> Signed-off-by: Tejun Heo <tj@kernel.org>
Not introducing the 'is_local' is a good thing.
Tested and Acked-by: Daniel Wagner <daniel.wagner@bmw-carit.de>
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html