Re: [PATCH] Improve cgroup printks

[Andrew Morton <akpm@linux-foundation.org>]

On Sat, 10 Nov 2007 12:11:56 +0100 Diego Calleja <diegocg@gmail.com> wrote:

> When I boot with the 'quiet' parameter, I see on the screen:
> 
> [    0.000000] Initializing cgroup subsys cpuset
> [    0.000000] Initializing cgroup subsys cpu
> [   39.036026] Initializing cgroup subsys cpuacct
> [   39.036080] Initializing cgroup subsys debug
> [   39.036118] Initializing cgroup subsys ns
> 
> This patch lowers the priority of those messages, adds a "cgroup: " prefix
> to another couple of printks and kills the useless reference to the source file.
> 
> 
> Signed-off-by: Diego Calleja <diegocg@gmail.com>
> 
> --- 2.6/kernel/cgroup.c.old	2007-11-10 11:35:51.000000000 +0100
> +++ 2.6/kernel/cgroup.c	2007-11-10 11:56:46.000000000 +0100
> @@ -1,6 +1,4 @@
>  /*
> - *  kernel/cgroup.c
> - *
>   *  Generic process-grouping system.
>   *
>   *  Based originally on the cpuset system, extracted by Paul Menage
> @@ -2200,7 +2198,7 @@ static void cgroup_init_subsys(struct cg
>  {
>  	struct cgroup_subsys_state *css;
>  	struct list_head *l;
> -	printk(KERN_ERR "Initializing cgroup subsys %s\n", ss->name);
> +	printk("Initializing cgroup subsys %s\n", ss->name);

But this one now has no facility level at all.

I'll switch it to KERN_INFO, OK?