diff options
author | Andreas Herrmann <aherrman@de.ibm.com> | 2006-09-18 16:29:20 -0400 |
---|---|---|
committer | James Bottomley <jejb@mulgrave.il.steeleye.com> | 2006-09-23 19:00:56 -0400 |
commit | 4eff4a36516d72e4f6ede901141214a7e05607e7 (patch) | |
tree | b6a775ee5acbfdc91a349fc69580031e17276768 /drivers/s390/scsi/zfcp_dbf.c | |
parent | dd52e0eaf891cd85bf2ca057c15ed6bfd76db4e6 (diff) |
[SCSI] zfcp: fix: use correct req_id in eh_abort_handler
zfcp's eh_abort_handler used the wrong request ID to
identify the request to be aborted. The bug was introduced
with commit fea9d6c7bcd8ff1d60ff74f27ba483b3820b18a3
for improved management of request IDs. The bug is
fixed with this patch.
Signed-off-by: Andreas Herrmann <aherrman@de.ibm.com>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
Diffstat (limited to 'drivers/s390/scsi/zfcp_dbf.c')
-rw-r--r-- | drivers/s390/scsi/zfcp_dbf.c | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/drivers/s390/scsi/zfcp_dbf.c b/drivers/s390/scsi/zfcp_dbf.c index c033145d0f19..0aa3b1ac76af 100644 --- a/drivers/s390/scsi/zfcp_dbf.c +++ b/drivers/s390/scsi/zfcp_dbf.c | |||
@@ -707,7 +707,7 @@ _zfcp_scsi_dbf_event_common(const char *tag, const char *tag2, int level, | |||
707 | struct zfcp_adapter *adapter, | 707 | struct zfcp_adapter *adapter, |
708 | struct scsi_cmnd *scsi_cmnd, | 708 | struct scsi_cmnd *scsi_cmnd, |
709 | struct zfcp_fsf_req *fsf_req, | 709 | struct zfcp_fsf_req *fsf_req, |
710 | struct zfcp_fsf_req *old_fsf_req) | 710 | unsigned long old_req_id) |
711 | { | 711 | { |
712 | struct zfcp_scsi_dbf_record *rec = &adapter->scsi_dbf_buf; | 712 | struct zfcp_scsi_dbf_record *rec = &adapter->scsi_dbf_buf; |
713 | struct zfcp_dbf_dump *dump = (struct zfcp_dbf_dump *)rec; | 713 | struct zfcp_dbf_dump *dump = (struct zfcp_dbf_dump *)rec; |
@@ -768,8 +768,7 @@ _zfcp_scsi_dbf_event_common(const char *tag, const char *tag2, int level, | |||
768 | rec->fsf_seqno = fsf_req->seq_no; | 768 | rec->fsf_seqno = fsf_req->seq_no; |
769 | rec->fsf_issued = fsf_req->issued; | 769 | rec->fsf_issued = fsf_req->issued; |
770 | } | 770 | } |
771 | rec->type.old_fsf_reqid = | 771 | rec->type.old_fsf_reqid = old_req_id; |
772 | (unsigned long) old_fsf_req; | ||
773 | } else { | 772 | } else { |
774 | strncpy(dump->tag, "dump", ZFCP_DBF_TAG_SIZE); | 773 | strncpy(dump->tag, "dump", ZFCP_DBF_TAG_SIZE); |
775 | dump->total_size = buflen; | 774 | dump->total_size = buflen; |
@@ -794,17 +793,17 @@ zfcp_scsi_dbf_event_result(const char *tag, int level, | |||
794 | struct zfcp_fsf_req *fsf_req) | 793 | struct zfcp_fsf_req *fsf_req) |
795 | { | 794 | { |
796 | _zfcp_scsi_dbf_event_common("rslt", tag, level, | 795 | _zfcp_scsi_dbf_event_common("rslt", tag, level, |
797 | adapter, scsi_cmnd, fsf_req, NULL); | 796 | adapter, scsi_cmnd, fsf_req, 0); |
798 | } | 797 | } |
799 | 798 | ||
800 | inline void | 799 | inline void |
801 | zfcp_scsi_dbf_event_abort(const char *tag, struct zfcp_adapter *adapter, | 800 | zfcp_scsi_dbf_event_abort(const char *tag, struct zfcp_adapter *adapter, |
802 | struct scsi_cmnd *scsi_cmnd, | 801 | struct scsi_cmnd *scsi_cmnd, |
803 | struct zfcp_fsf_req *new_fsf_req, | 802 | struct zfcp_fsf_req *new_fsf_req, |
804 | struct zfcp_fsf_req *old_fsf_req) | 803 | unsigned long old_req_id) |
805 | { | 804 | { |
806 | _zfcp_scsi_dbf_event_common("abrt", tag, 1, | 805 | _zfcp_scsi_dbf_event_common("abrt", tag, 1, |
807 | adapter, scsi_cmnd, new_fsf_req, old_fsf_req); | 806 | adapter, scsi_cmnd, new_fsf_req, old_req_id); |
808 | } | 807 | } |
809 | 808 | ||
810 | inline void | 809 | inline void |
@@ -814,7 +813,7 @@ zfcp_scsi_dbf_event_devreset(const char *tag, u8 flag, struct zfcp_unit *unit, | |||
814 | struct zfcp_adapter *adapter = unit->port->adapter; | 813 | struct zfcp_adapter *adapter = unit->port->adapter; |
815 | 814 | ||
816 | _zfcp_scsi_dbf_event_common(flag == FCP_TARGET_RESET ? "trst" : "lrst", | 815 | _zfcp_scsi_dbf_event_common(flag == FCP_TARGET_RESET ? "trst" : "lrst", |
817 | tag, 1, adapter, scsi_cmnd, NULL, NULL); | 816 | tag, 1, adapter, scsi_cmnd, NULL, 0); |
818 | } | 817 | } |
819 | 818 | ||
820 | static int | 819 | static int |