Re: [PATCH] scsi_error: Remove scsi_block_when_processing_errors: message
Laurence, Patch completely mangled by your mailer, please use git send-email for submissions. I applied the change by hand. Thanks! -- Martin K. Petersen Oracle Linux Engineering
Re: [PATCH] scsi_error: Remove scsi_block_when_processing_errors: message
- Original Message - From: "Laurence Oberman" To: "linux-scsi" Cc: "Hannes Reinecke" , "Ewan Milne" , "Bart Van Assche" , "James Bottomley" Sent: Tuesday, October 16, 2018 4:39:16 PM Subject: [PATCH] scsi_error: Remove scsi_block_when_processing_errors: message This message floods the log when enabling mask 0x7 for /proc/sys/dev/scsi/logging_level kernel: scsi_block_when_processing_errors: rtn: 1 Its not needed and makes tracing just scsi_eh* messages way too verbose so get rid of it Signed-off-by: Laurence Oberman --- drivers/scsi/scsi_error.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/scsi/scsi_error.c b/drivers/scsi/scsi_error.c index b7a8fdf..c736d61 100644 --- a/drivers/scsi/scsi_error.c +++ b/drivers/scsi/scsi_error.c @@ -338,9 +338,6 @@ int scsi_block_when_processing_errors(struct scsi_device *sdev) online = scsi_device_online(sdev); - SCSI_LOG_ERROR_RECOVERY(5, sdev_printk(KERN_INFO, sdev, - "%s: rtn: %d\n", __func__, online)); - return online; } EXPORT_SYMBOL(scsi_block_when_processing_errors); -- 1.8.3.1 Yes, agreed. Reviewed-by: Ewan D. Milne
Re: [PATCH] scsi_error: Remove scsi_block_when_processing_errors: message
On Tue, 16 Oct 2018, 4:39pm, Laurence Oberman wrote: > External Email > > This message floods the log when enabling mask 0x7 for > /proc/sys/dev/scsi/logging_level > kernel: scsi_block_when_processing_errors: rtn: 1 > Its not needed and makes tracing just scsi_eh* messages way too verbose > so get rid of it > > Signed-off-by: Laurence Oberman > > --- > drivers/scsi/scsi_error.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/drivers/scsi/scsi_error.c b/drivers/scsi/scsi_error.c > index b7a8fdf..c736d61 100644 > --- a/drivers/scsi/scsi_error.c > +++ b/drivers/scsi/scsi_error.c > @@ -338,9 +338,6 @@ int scsi_block_when_processing_errors(struct > scsi_device *sdev) > > online = scsi_device_online(sdev); > > - SCSI_LOG_ERROR_RECOVERY(5, sdev_printk(KERN_INFO, sdev, > - "%s: rtn: %d\n", __func__, online)); > - > return online; > } > EXPORT_SYMBOL(scsi_block_when_processing_errors); > -- > 1.8.3.1 > Hit this a few times myself. Thanks for taking it out. Reviewed-by: Chad Dupuis
Re: [PATCH] scsi_error: Remove scsi_block_when_processing_errors: message
On 10/16/18 10:39 PM, Laurence Oberman wrote: This message floods the log when enabling mask 0x7 for /proc/sys/dev/scsi/logging_level kernel: scsi_block_when_processing_errors: rtn: 1 Its not needed and makes tracing just scsi_eh* messages way too verbose so get rid of it Signed-off-by: Laurence Oberman --- drivers/scsi/scsi_error.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/scsi/scsi_error.c b/drivers/scsi/scsi_error.c index b7a8fdf..c736d61 100644 --- a/drivers/scsi/scsi_error.c +++ b/drivers/scsi/scsi_error.c @@ -338,9 +338,6 @@ int scsi_block_when_processing_errors(struct scsi_device *sdev) online = scsi_device_online(sdev); - SCSI_LOG_ERROR_RECOVERY(5, sdev_printk(KERN_INFO, sdev, - "%s: rtn: %d\n", __func__, online)); - return online; } EXPORT_SYMBOL(scsi_block_when_processing_errors); -- 1.8.3.1 Indeed. Reviewed-by: Hannes Reinecke Cheers, Hannes -- Dr. Hannes ReineckeTeamlead Storage & Networking h...@suse.de +49 911 74053 688 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: F. Imendörffer, J. Smithard, J. Guild, D. Upmanyu, G. Norton HRB 21284 (AG Nürnberg)