[Devel] Re: [PATCH 2/2] cpuset: use cgroup_path() in proc_cpuset_show()

Paul Menage menage at google.com
Fri Nov 21 10:56:58 PST 2008


On Fri, Nov 21, 2008 at 12:49 AM, Lai Jiangshan <laijs at cn.fujitsu.com> wrote:
>
> use cgroup_path() instead of
> task_subsys_state(tsk, cpuset_subsys_id)->cgroup

You mean:

use task_cgroup() instead of task_subsys_state()->cgroup

>
> Signed-off-by: Lai Jiangshan <laijs at cn.fujitsu.com>
> ---
> diff --git a/kernel/cpuset.c b/kernel/cpuset.c
> index 3e00526..e0e087e 100644
> --- a/kernel/cpuset.c
> +++ b/kernel/cpuset.c
> @@ -2387,7 +2386,7 @@ static int proc_cpuset_show(struct seq_file *m, void *unused_v)
>        struct pid *pid;
>        struct task_struct *tsk;
>        char *buf;
> -       struct cgroup_subsys_state *css;
> +       struct cgroup *cgp;
>        int retval;
>
>        retval = -ENOMEM;
> @@ -2403,8 +2402,8 @@ static int proc_cpuset_show(struct seq_file *m, void *unused_v)
>
>        retval = -EINVAL;
>        cgroup_lock();
> -       css = task_subsys_state(tsk, cpuset_subsys_id);
> -       retval = cgroup_path(css->cgroup, buf, PAGE_SIZE);
> +       cgp = task_cgroup(tsk, cpuset_subsys_id);
> +       retval = cgroup_path(cgp, buf, PAGE_SIZE);
>        if (retval < 0)
>                goto out_unlock;
>        seq_puts(m, buf);
>
>
>
_______________________________________________
Containers mailing list
Containers at lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/containers




More information about the Devel mailing list