Re: [PATCH 3/4] Add a back-pointer from struct cg_cgroup_link to struct cgroup

From: Paul Menage
Date: Thu Jul 23 2009 - 10:31:46 EST


On Wed, Jul 22, 2009 at 11:44 PM, Li Zefan<lizf@xxxxxxxxxxxxxx> wrote:
>
> I think we need to cgroup_enable_task_cg_lists():
>
>        if (!use_task_css_set_links)
>                cgroup_enable_task_cg_lists();
>
> Otherwise we'll see no tasks when reading this debug file
> if use_task_css_set_links == 0.
>

I think that's OK - the debug file should indicate the state of the
kernel, not alter it. If there are no links, then no links should be
printed.

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/