sr: reduce debug noise in sr_do_ioctl
authorChristoph Hellwig <hch@lst.de>
Sun, 18 Jan 2015 15:11:31 +0000 (16:11 +0100)
committerChristoph Hellwig <hch@lst.de>
Tue, 20 Jan 2015 18:43:24 +0000 (19:43 +0100)
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>
drivers/scsi/sr_ioctl.c

index e8deb9ccf20d8c29f29c3e8aef13a5e7634c480b..03054c0e7689bd0587e17c5a56d22212739d8c04 100644 (file)
@@ -188,7 +188,6 @@ int sr_do_ioctl(Scsi_CD *cd, struct packet_command *cgc)
        struct scsi_sense_hdr sshdr;
        int result, err = 0, retries = 0;
        struct request_sense *sense = cgc->sense;
-       char logbuf[SCSI_LOG_BUFSIZE];
 
        SDev = cd->device;
 
@@ -246,9 +245,6 @@ int sr_do_ioctl(Scsi_CD *cd, struct packet_command *cgc)
                                sr_printk(KERN_INFO, cd,
                                          "CDROM not ready.  Make sure there "
                                          "is a disc in the drive.\n");
-#ifdef DEBUG
-                       scsi_print_sense_hdr(cd->device, cd->cdi.name, &sshdr);
-#endif
                        err = -ENOMEDIUM;
                        break;
                case ILLEGAL_REQUEST:
@@ -257,22 +253,8 @@ int sr_do_ioctl(Scsi_CD *cd, struct packet_command *cgc)
                            sshdr.ascq == 0x00)
                                /* sense: Invalid command operation code */
                                err = -EDRIVE_CANT_DO_THIS;
-#ifdef DEBUG
-                       __scsi_format_command(logbuf, sizeof(logbuf),
-                                             cgc->cmd, CDROM_PACKET_SIZE);
-                       sr_printk(KERN_INFO, cd,
-                                 "CDROM (ioctl) invalid command: %s\n",
-                                 logbuf);
-                       scsi_print_sense_hdr(cd->device, cd->cdi.name, &sshdr);
-#endif
                        break;
                default:
-                       __scsi_format_command(logbuf, sizeof(logbuf),
-                                             cgc->cmd, CDROM_PACKET_SIZE);
-                       sr_printk(KERN_ERR, cd,
-                                 "CDROM (ioctl) error, command: %s\n",
-                                 logbuf);
-                       scsi_print_sense_hdr(cd->device, cd->cdi.name, &sshdr);
                        err = -EIO;
                }
        }