lkml.org 
[lkml]   [2014]   [May]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 6/6] cgroup, memcg: implement css->id and convert css_from_id() to use it
On Mon, Apr 28, 2014 at 11:33:16AM +0800, Li Zefan wrote:
> On 2014/4/25 5:02, Tejun Heo wrote:
> > Until now, cgroup->id has been used to identify all the associated
> > csses and css_from_id() takes cgroup ID and returns the matching css
> > by looking up the cgroup and then dereferencing the css associated
> > with it; however, now that the lifetimes of cgroup and css are
> > separate, this is incorrect and breaks on the unified hierarchy when a
> > controller is disabled and enabled back again before the previous
> > instance is released.
> >
> > This patch adds css->id which is a subsystem-unique ID and converts
> > css_from_id() to look up by the new css->id instead. memcg is the
> > only user of css_from_id() and also converted to use css->id instead.
> >
>
> netprio_cgroup also needs to be updated.

Why? Wouldn't it be more natural to tie that to the associated
cgroup's ID rather than the specific css instance. It's different for
memcg as it involves css refcnts but netprio_cgroup, AFAICS, only uses
it for cgroup identification anyway.

Thanks.

--
tejun


\
 
 \ /
  Last update: 2014-05-01 18:41    [W:2.857 / U:0.168 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site