[SCSI] iscsi: fix wrong order of opcode and itt in iscsi_handle_reject prompt
authorVaughan Cao <vaughan.cao@oracle.com>
Thu, 9 Jan 2014 02:21:34 +0000 (10:21 +0800)
committerJames Bottomley <JBottomley@Parallels.com>
Sat, 15 Mar 2014 17:19:00 +0000 (10:19 -0700)
This patch makes reject messages show right value for opcode and itt, which
is converse previously.

Signed-off-by: Vaughan Cao <vaughan.cao@oracle.com>
Acked-by: Mike Christie <michaelc@cs.wisc.edu>
Signed-off-by: James Bottomley <JBottomley@Parallels.com>
drivers/scsi/libiscsi.c

index 4046241..cd330c0 100644 (file)
@@ -1013,13 +1013,13 @@ static int iscsi_handle_reject(struct iscsi_conn *conn, struct iscsi_hdr *hdr,
                iscsi_conn_printk(KERN_ERR, conn,
                                  "pdu (op 0x%x itt 0x%x) rejected "
                                  "due to DataDigest error.\n",
-                                 rejected_pdu.itt, opcode);
+                                 opcode, rejected_pdu.itt);
                break;
        case ISCSI_REASON_IMM_CMD_REJECT:
                iscsi_conn_printk(KERN_ERR, conn,
                                  "pdu (op 0x%x itt 0x%x) rejected. Too many "
                                  "immediate commands.\n",
-                                 rejected_pdu.itt, opcode);
+                                 opcode, rejected_pdu.itt);
                /*
                 * We only send one TMF at a time so if the target could not
                 * handle it, then it should get fixed (RFC mandates that
@@ -1059,8 +1059,8 @@ static int iscsi_handle_reject(struct iscsi_conn *conn, struct iscsi_hdr *hdr,
        default:
                iscsi_conn_printk(KERN_ERR, conn,
                                  "pdu (op 0x%x itt 0x%x) rejected. Reason "
-                                 "code 0x%x\n", rejected_pdu.itt,
-                                 rejected_pdu.opcode, reject->reason);
+                                 "code 0x%x\n", rejected_pdu.opcode,
+                                 rejected_pdu.itt, reject->reason);
                break;
        }
        return rc;