![]() | ||||||||||
Messages in this thread |
On Tue, Apr 22, 2008 at 8:04 PM, Li Zefan <lizf@cn.fujitsu.com> wrote: > The function does not modify anything (except the temporary > css template), so it's sufficient to hold read lock. > > Signed-off-by: Li Zefan <lizf@cn.fujitsu.com> Acked-by: Paul Menage <menage@google.com> Thanks. > --- > kernel/cgroup.c | 4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/kernel/cgroup.c b/kernel/cgroup.c > index 2727f92..e9eb5da 100644 > --- a/kernel/cgroup.c > +++ b/kernel/cgroup.c > @@ -406,11 +406,11 @@ static struct css_set *find_css_set( > > /* First see if we already have a cgroup group that matches > * the desired set */ > - write_lock(&css_set_lock); > + read_lock(&css_set_lock); > res = find_existing_css_set(oldcg, cgrp, template); > if (res) > get_css_set(res); > - write_unlock(&css_set_lock); > + read_unlock(&css_set_lock); > > if (res) > return res; > -- 1.5.4.rc3 > > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/ | |||||||||
| Last update: 2008-04-23 08:51 [from the cache] ©2003-2008 | ||||||||||