linux-kernel - Re: [PATCH V2 3/3] init/calibrate.c: no prefix in logging

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]

Message-ID: <20140529144321.GB32669@windriver.com>
Date:	Thu, 29 May 2014 10:43:21 -0400
From:	Paul Gortmaker <paul.gortmaker@...driver.com>
To:	Fabian Frederick <fabf@...net.be>
CC:	linux-kernel <linux-kernel@...r.kernel.org>,
	akpm <akpm@...ux-foundation.org>
Subject: Re: [PATCH V2 3/3] init/calibrate.c: no prefix in logging

[[PATCH V2 3/3] init/calibrate.c: no prefix in logging] On 29/05/2014 (Thu 10:14) Fabian Frederick wrote:

> define pr_fmt without prefix to avoid any default prefix update

Again, it isn't clear to me what you are trying to achieve here.
The printk.h header already has the define which you add here.

A general rule for commits is to cover off three points.

1) Describe what the user visible symptom is, i.e. "printk from
calibrate.c are appearing with verbose prefix blah blah..."

2) Describe the underlying reason for the bug/issue, i.e. "This happens
because macro blah comes from foo.h but only when bar is defined."

3) Describe how your fix works and why it is the right/best fix for the
issue, i.e. "Here we introduce blah blah which will ensure ...."

Also, to make life easier for maintainers, you probably want to add a
0/N lead-in (see --cover-letter) which describes what you are trying to
achieve with the whole series, and when you create a v2 of the series,
you need to list in there for the maintainer how v2 differs from v1.

Finally try not to send a v2 within hours of sending a v1 -- give a
chance for the v1 to get looked at 1st; typically at least one day.

Thanks,
Paul.
--

> 
> Cc: Paul Gortmaker <paul.gortmaker@...driver.com>
> Cc: Andrew Morton <akpm@...ux-foundation.org>
> Signed-off-by: Fabian Frederick <fabf@...net.be>
> ---
>  init/calibrate.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/init/calibrate.c b/init/calibrate.c
> index fb9be44..169e98d 100644
> --- a/init/calibrate.c
> +++ b/init/calibrate.c
> @@ -4,6 +4,8 @@
>   *  Copyright (C) 1991, 1992  Linus Torvalds
>   */
>  
> +#define pr_fmt(fmt) fmt
> +
>  #include <linux/jiffies.h>
>  #include <linux/delay.h>
>  #include <linux/init.h>
> -- 
> 1.9.1
> 
--
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/