linux-kernel - Re: [PATCH 2/2] kernel/cpuset.c: convert printk to pr

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Tue, 6 May 2014 09:22:42 +0800
From:	Li Zefan <lizefan@...wei.com>
To:	Fabian Frederick <fabf@...net.be>
CC:	linux-kernel <linux-kernel@...r.kernel.org>,
	akpm <akpm@...ux-foundation.org>, Tejun Heo <tj@...nel.org>
Subject: Re: [PATCH 2/2] kernel/cpuset.c: convert printk to pr_foo()

On 2014/5/6 1:49, Fabian Frederick wrote:
> Cc: Li Zefan <lizefan@...wei.com>
> Cc: Andrew Morton <akpm@...ux-foundation.org>
> Signed-off-by: Fabian Frederick <fabf@...net.be>

Acked-by: Li Zefan <lizefan@...wei.com>

> ---
>  kernel/cpuset.c | 11 ++++-------
>  1 file changed, 4 insertions(+), 7 deletions(-)
> 
> diff --git a/kernel/cpuset.c b/kernel/cpuset.c
> index 1d8c047..7c0e8da 100644
> --- a/kernel/cpuset.c
> +++ b/kernel/cpuset.c
> @@ -696,11 +696,8 @@ restart:
>  		if (nslot == ndoms) {
>  			static int warnings = 10;
>  			if (warnings) {
> -				printk(KERN_WARNING
> -				 "rebuild_sched_domains confused:"
> -				  " nslot %d, ndoms %d, csn %d, i %d,"
> -				  " apn %d\n",
> -				  nslot, ndoms, csn, i, apn);
> +				pr_warn("rebuild_sched_domains confused: nslot %d, ndoms %d, csn %d, i %d, apn %d\n",
> +					nslot, ndoms, csn, i, apn);
>  				warnings--;
>  			}
>  			continue;
> @@ -2018,7 +2015,7 @@ static void remove_tasks_in_empty_cpuset(struct cpuset *cs)
>  		parent = parent_cs(parent);
>  
>  	if (cgroup_transfer_tasks(parent->css.cgroup, cs->css.cgroup)) {
> -		printk(KERN_ERR "cpuset: failed to transfer tasks out of empty cpuset ");
> +		pr_err("cpuset: failed to transfer tasks out of empty cpuset ");
>  		pr_cont_cgroup_name(cs->css.cgroup);
>  		pr_cont("\n");
>  	}
> @@ -2555,7 +2552,7 @@ void cpuset_print_task_mems_allowed(struct task_struct *tsk)
>  	cgrp = task_cs(tsk)->css.cgroup;
>  	nodelist_scnprintf(cpuset_nodelist, CPUSET_NODELIST_LEN,
>  			   tsk->mems_allowed);
> -	printk(KERN_INFO "%s cpuset=", tsk->comm);
> +	pr_info("%s cpuset=", tsk->comm);
>  	pr_cont_cgroup_name(cgrp);
>  	pr_cont(" mems_allowed=%s\n", cpuset_nodelist);
>  
> 

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