aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorChristoph Hellwig <hch@lst.de>2015-01-18 10:11:31 -0500
committerChristoph Hellwig <hch@lst.de>2015-01-20 13:43:24 -0500
commitc7e09574d620246347a5fb87118e624d69c27f41 (patch)
tree2c62189993968e2d62d3f809173616b28694310c
parent63c6ea173471e926e9739288a5aefac3cb827e15 (diff)
sr: reduce debug noise in sr_do_ioctl
The midlayer scsi logging already logs the command and sense code if the logging level is high enough, no need to duplicate that in the sr driver. Signed-off-by: Christoph Hellwig <hch@lst.de> Reviewed-by: James Bottomley <JBottomley@Parallels.com> Reviewed-by: Ewan D. Milne <emilne@redhat.com> Reviewed-by: Hannes Reinecke <hare@suse.de>
-rw-r--r--drivers/scsi/sr_ioctl.c18
1 files changed, 0 insertions, 18 deletions
diff --git a/drivers/scsi/sr_ioctl.c b/drivers/scsi/sr_ioctl.c
index e8deb9ccf20d..03054c0e7689 100644
--- a/drivers/scsi/sr_ioctl.c
+++ b/drivers/scsi/sr_ioctl.c
@@ -188,7 +188,6 @@ int sr_do_ioctl(Scsi_CD *cd, struct packet_command *cgc)
188 struct scsi_sense_hdr sshdr; 188 struct scsi_sense_hdr sshdr;
189 int result, err = 0, retries = 0; 189 int result, err = 0, retries = 0;
190 struct request_sense *sense = cgc->sense; 190 struct request_sense *sense = cgc->sense;
191 char logbuf[SCSI_LOG_BUFSIZE];
192 191
193 SDev = cd->device; 192 SDev = cd->device;
194 193
@@ -246,9 +245,6 @@ int sr_do_ioctl(Scsi_CD *cd, struct packet_command *cgc)
246 sr_printk(KERN_INFO, cd, 245 sr_printk(KERN_INFO, cd,
247 "CDROM not ready. Make sure there " 246 "CDROM not ready. Make sure there "
248 "is a disc in the drive.\n"); 247 "is a disc in the drive.\n");
249#ifdef DEBUG
250 scsi_print_sense_hdr(cd->device, cd->cdi.name, &sshdr);
251#endif
252 err = -ENOMEDIUM; 248 err = -ENOMEDIUM;
253 break; 249 break;
254 case ILLEGAL_REQUEST: 250 case ILLEGAL_REQUEST:
@@ -257,22 +253,8 @@ int sr_do_ioctl(Scsi_CD *cd, struct packet_command *cgc)
257 sshdr.ascq == 0x00) 253 sshdr.ascq == 0x00)
258 /* sense: Invalid command operation code */ 254 /* sense: Invalid command operation code */
259 err = -EDRIVE_CANT_DO_THIS; 255 err = -EDRIVE_CANT_DO_THIS;
260#ifdef DEBUG
261 __scsi_format_command(logbuf, sizeof(logbuf),
262 cgc->cmd, CDROM_PACKET_SIZE);
263 sr_printk(KERN_INFO, cd,
264 "CDROM (ioctl) invalid command: %s\n",
265 logbuf);
266 scsi_print_sense_hdr(cd->device, cd->cdi.name, &sshdr);
267#endif
268 break; 256 break;
269 default: 257 default:
270 __scsi_format_command(logbuf, sizeof(logbuf),
271 cgc->cmd, CDROM_PACKET_SIZE);
272 sr_printk(KERN_ERR, cd,
273 "CDROM (ioctl) error, command: %s\n",
274 logbuf);
275 scsi_print_sense_hdr(cd->device, cd->cdi.name, &sshdr);
276 err = -EIO; 258 err = -EIO;
277 } 259 }
278 } 260 }