[PATCH] cgroups: update documentation about css_set hash table

[PATCH] cgroups: update documentation about css_set hash table

[Li Zefan]

The css_set hash table was introduced in 2.6.26, so update the documentation
accordingly.

Signed-off-by: Li Zefan <lizf@cn.fujitsu.com>
---
 Documentation/cgroups/cgroups.txt |    6 ++----
 1 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/Documentation/cgroups/cgroups.txt b/Documentation/cgroups/cgroups.txt
index d9e5d6f..93feb84 100644
--- a/Documentation/cgroups/cgroups.txt
+++ b/Documentation/cgroups/cgroups.txt
@@ -252,10 +252,8 @@ cgroup file system directories.
 When a task is moved from one cgroup to another, it gets a new
 css_set pointer - if there's an already existing css_set with the
 desired collection of cgroups then that group is reused, else a new
-css_set is allocated. Note that the current implementation uses a
-linear search to locate an appropriate existing css_set, so isn't
-very efficient. A future version will use a hash table for better
-performance.
+css_set is allocated. The appropriate existing css_set is located by
+looking into a hash table.
 
 To allow access from a cgroup to the css_sets (and hence tasks)
 that comprise it, a set of cg_cgroup_link objects form a lattice;
-- 
1.5.4.rc3

Re: [PATCH] cgroups: update documentation about css_set hash table

[Paul Menage]

On Tue, Feb 17, 2009 at 5:31 PM, Li Zefan <lizf@cn.fujitsu.com> wrote:
> The css_set hash table was introduced in 2.6.26, so update the documentation
> accordingly.
>
> Signed-off-by: Li Zefan <lizf@cn.fujitsu.com>

Acked-by: Paul Menage <menage@google.com>

> ---
>  Documentation/cgroups/cgroups.txt |    6 ++----
>  1 files changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/Documentation/cgroups/cgroups.txt b/Documentation/cgroups/cgroups.txt
> index d9e5d6f..93feb84 100644
> --- a/Documentation/cgroups/cgroups.txt
> +++ b/Documentation/cgroups/cgroups.txt
> @@ -252,10 +252,8 @@ cgroup file system directories.
>  When a task is moved from one cgroup to another, it gets a new
>  css_set pointer - if there's an already existing css_set with the
>  desired collection of cgroups then that group is reused, else a new
> -css_set is allocated. Note that the current implementation uses a
> -linear search to locate an appropriate existing css_set, so isn't
> -very efficient. A future version will use a hash table for better
> -performance.
> +css_set is allocated. The appropriate existing css_set is located by
> +looking into a hash table.
>
>  To allow access from a cgroup to the css_sets (and hence tasks)
>  that comprise it, a set of cg_cgroup_link objects form a lattice;
> --
> 1.5.4.rc3
>