Re: [PATCH 6/7] cgroup: remove duplicate code in find_css_set()

2008-02-19 Thread Paul Menage
On Feb 17, 2008 9:49 PM, Li Zefan <[EMAIL PROTECTED]> wrote:
> The list head res->tasks gets initialized twice in find_css_set().
>
> Signed-off-by: Li Zefan <[EMAIL PROTECTED]>

Acked-by: Paul Menage <[EMAIL PROTECTED]>

> ---
>  kernel/cgroup.c |1 -
>  1 files changed, 0 insertions(+), 1 deletions(-)
>
> diff --git a/kernel/cgroup.c b/kernel/cgroup.c
> index e8c8e58..71cf961 100644
> --- a/kernel/cgroup.c
> +++ b/kernel/cgroup.c
> @@ -473,7 +473,6 @@ static struct css_set *find_css_set(
> /* Link this cgroup group into the list */
> list_add(>list, _css_set.list);
> css_set_count++;
> -   INIT_LIST_HEAD(>tasks);
> write_unlock(_set_lock);
>
> return res;
> --
> 1.5.4.rc3
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 6/7] cgroup: remove duplicate code in find_css_set()

2008-02-19 Thread Paul Menage
On Feb 17, 2008 9:49 PM, Li Zefan [EMAIL PROTECTED] wrote:
 The list head res-tasks gets initialized twice in find_css_set().

 Signed-off-by: Li Zefan [EMAIL PROTECTED]

Acked-by: Paul Menage [EMAIL PROTECTED]

 ---
  kernel/cgroup.c |1 -
  1 files changed, 0 insertions(+), 1 deletions(-)

 diff --git a/kernel/cgroup.c b/kernel/cgroup.c
 index e8c8e58..71cf961 100644
 --- a/kernel/cgroup.c
 +++ b/kernel/cgroup.c
 @@ -473,7 +473,6 @@ static struct css_set *find_css_set(
 /* Link this cgroup group into the list */
 list_add(res-list, init_css_set.list);
 css_set_count++;
 -   INIT_LIST_HEAD(res-tasks);
 write_unlock(css_set_lock);

 return res;
 --
 1.5.4.rc3


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


[PATCH 6/7] cgroup: remove duplicate code in find_css_set()

2008-02-17 Thread Li Zefan
The list head res->tasks gets initialized twice in find_css_set().

Signed-off-by: Li Zefan <[EMAIL PROTECTED]>
---
 kernel/cgroup.c |1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/kernel/cgroup.c b/kernel/cgroup.c
index e8c8e58..71cf961 100644
--- a/kernel/cgroup.c
+++ b/kernel/cgroup.c
@@ -473,7 +473,6 @@ static struct css_set *find_css_set(
/* Link this cgroup group into the list */
list_add(>list, _css_set.list);
css_set_count++;
-   INIT_LIST_HEAD(>tasks);
write_unlock(_set_lock);
 
return res;
-- 
1.5.4.rc3

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


[PATCH 6/7] cgroup: remove duplicate code in find_css_set()

2008-02-17 Thread Li Zefan
The list head res-tasks gets initialized twice in find_css_set().

Signed-off-by: Li Zefan [EMAIL PROTECTED]
---
 kernel/cgroup.c |1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/kernel/cgroup.c b/kernel/cgroup.c
index e8c8e58..71cf961 100644
--- a/kernel/cgroup.c
+++ b/kernel/cgroup.c
@@ -473,7 +473,6 @@ static struct css_set *find_css_set(
/* Link this cgroup group into the list */
list_add(res-list, init_css_set.list);
css_set_count++;
-   INIT_LIST_HEAD(res-tasks);
write_unlock(css_set_lock);
 
return res;
-- 
1.5.4.rc3

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