Re: [PATCH] cgroup: Convert synchronize_rcu to call_rcu in cgroup_attach_task

From: Paul Menage
Date: Wed Nov 24 2010 - 13:58:54 EST


On Tue, Nov 23, 2010 at 6:06 PM, Li Zefan <lizf@xxxxxxxxxxxxxx> wrote:
>
> If we do this, it's not anymore safe to use get_css_set(), which just
> increments the refcount without checking if it's zero.

I don't believe that it's currently safe to use get_css_set() on a
zero-refcount css_set anyway - a css_set starts with refcount 1 and
can be freed the moment that its refcount reaches 0. You can only use
get_css_set() on a stable reference to an existing css_set, which by
definition has a refcount of at least 1. Were you thinking of any
particular existing uses?

Paul
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/