Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1257160

Re: [PATCH 2/3] megaraid_sas: Convert printk to printk_<level>

Show all headers | View raw

On Tue, 2015-10-27 at 16:26 +0800, Weidong Wang wrote:
> Reduce object size a little by using pr_<level>
> calls instead of printk(KERN_<LEVEL>.

pr_debug does not behave the same as printk(KERN_DEBUG

pr_debug is only active when DEBUG is #defined or dynamic_debug
is enabled.

printk(KERN_DEBUG is always emitted as long as the debug level
is enabled for the console.

At a minimum, your commit message should show you know that.

> diff --git a/drivers/scsi/megaraid/megaraid_sas_base.c b/drivers/scsi/megaraid/megaraid_sas_base.c
[]
> @@ -5889,7 +5889,7 @@ static int megasas_mgmt_fasync(int fd, struct file *filep, int mode)
>  		return 0;
>  	}
>  
> -	printk(KERN_DEBUG "megasas: fasync_helper failed [%d]\n", rc);
> +	pr_debug("megasas: fasync_helper failed [%d]\n", rc);

[]

> @@ -6233,7 +6233,7 @@ static int megasas_mgmt_ioctl_aen(struct file *file, unsigned long arg)
>  	u32 wait_time = MEGASAS_RESET_WAIT_TIME;
>  
>  	if (file->private_data != file) {
> -		printk(KERN_DEBUG "megasas: fasync_helper was not "
> +		pr_debug("megasas: fasync_helper was not "
>  		       "called first\n");

Please also coalesce format strings where possible.


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

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 0/3] megaraid_sas: copule of fixes Weidong Wang <wangweidong1@huawei.com> - 2015-10-27 09:30 +0100
  [PATCH 3/3] megaraid_sas: return -ENOMEM when create DMA pool for cmd frames failed Weidong Wang <wangweidong1@huawei.com> - 2015-10-27 09:30 +0100
    Re: [PATCH 3/3] megaraid_sas: return -ENOMEM when create DMA pool  for cmd frames failed Johannes Thumshirn <jthumshirn@suse.de> - 2015-10-27 11:20 +0100
  [PATCH 2/3] megaraid_sas: Convert printk to printk_<level> Weidong Wang <wangweidong1@huawei.com> - 2015-10-27 09:30 +0100
    Re: [PATCH 2/3] megaraid_sas: Convert printk to printk_<level> Johannes Thumshirn <jthumshirn@suse.de> - 2015-10-27 11:30 +0100
    Re: [PATCH 2/3] megaraid_sas: Convert printk to printk_<level> Joe Perches <joe@perches.com> - 2015-10-27 20:40 +0100
  [PATCH 1/3] megaraid_sas: Convert dev_printk to dev_<level> Weidong Wang <wangweidong1@huawei.com> - 2015-10-27 09:30 +0100
    Re: [PATCH 1/3] megaraid_sas: Convert dev_printk to dev_<level> Johannes Thumshirn <jthumshirn@suse.de> - 2015-10-27 11:30 +0100
      RE: [PATCH 1/3] megaraid_sas: Convert dev_printk to dev_<level> Kashyap Desai <kashyap.desai@avagotech.com> - 2015-10-27 11:40 +0100
        Re: [PATCH 1/3] megaraid_sas: Convert dev_printk to dev_<level> Weidong Wang <wangweidong1@huawei.com> - 2015-10-28 03:40 +0100
    Re: [PATCH 1/3] megaraid_sas: Convert dev_printk to dev_<level> Joe Perches <joe@perches.com> - 2015-10-27 20:40 +0100
      Re: [PATCH 1/3] megaraid_sas: Convert dev_printk to dev_<level> Weidong Wang <wangweidong1@huawei.com> - 2015-10-28 03:10 +0100

csiph-web