[SCSI] qla4xxx: Removed residual from overrun debug print
authorMike Christie <michaelc@cs.wisc.edu>
Thu, 20 Aug 2009 20:11:00 +0000 (15:11 -0500)
committerJames Bottomley <James.Bottomley@suse.de>
Sat, 5 Sep 2009 14:42:44 +0000 (09:42 -0500)
The residual variable is only valid for udnerrun so do
not print it out for the overrun case.

Signed-off-by: Karen Higgins <karen.higgins@qlogic.com>
[Mike Christie: Fix coding style issues in patch]
Signed-off-by: Mike Christie <michaelc@cs.wisc.edu>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
drivers/scsi/qla4xxx/ql4_isr.c

index 8025ee16588e5a9f7104cf82fc80d4531fe01b70..c196d55eae3939c71b8cae29afaa335665dc1aea 100644 (file)
@@ -227,11 +227,11 @@ static void qla4xxx_status_entry(struct scsi_qla_host *ha,
        case SCS_DATA_UNDERRUN:
        case SCS_DATA_OVERRUN:
                if ((sts_entry->iscsiFlags & ISCSI_FLAG_RESIDUAL_OVER) ||
-                       (sts_entry->completionStatus == SCS_DATA_OVERRUN)) {
-                       DEBUG2(printk("scsi%ld:%d:%d:%d: %s: " "Data overrun, "
-                                     "residual = 0x%x\n", ha->host_no,
+                    (sts_entry->completionStatus == SCS_DATA_OVERRUN)) {
+                       DEBUG2(printk("scsi%ld:%d:%d:%d: %s: " "Data overrun\n",
+                                     ha->host_no,
                                      cmd->device->channel, cmd->device->id,
-                                     cmd->device->lun, __func__, residual));
+                                     cmd->device->lun, __func__));
 
                        cmd->result = DID_ERROR << 16;
                        break;