[SCSI] iscsi: fix wrong order of opcode and itt in iscsi_handle_reject prompt

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>

diff --git a/drivers/scsi/libiscsi.c b/drivers/scsi/libiscsi.c
index 40462415291e8e79316951417e06715d34314a48..cd330c0e0fd57e08386ca79edd07cbbb6297e52e 100644 (file)
--- a/drivers/scsi/libiscsi.c
+++ b/drivers/scsi/libiscsi.c
@@ -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;