diff options
author | Christof Schmitt <christof.schmitt@de.ibm.com> | 2009-03-02 07:08:58 -0500 |
---|---|---|
committer | James Bottomley <James.Bottomley@HansenPartnership.com> | 2009-03-12 13:58:19 -0400 |
commit | 52bfb558d2803590f86360ec3af1750897a9c010 (patch) | |
tree | 5836ada15e9b01f4d8b1519fbab81b99512d96c4 /drivers | |
parent | 49f0f01c9966639f8fd7ce784a412e22057d9f2a (diff) |
[SCSI] zfcp: Only increment req_id for successfully issued requests
Only increment the req_id for successfully issued requests. This
avoids some confusion when debugging issued fsf requests.
Acked-by: Swen Schillig <swen@vnet.ibm.com>
Signed-off-by: Christof Schmitt <christof.schmitt@de.ibm.com>
Signed-off-by: James Bottomley <James.Bottomley@HansenPartnership.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/s390/scsi/zfcp_fsf.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/s390/scsi/zfcp_fsf.c b/drivers/s390/scsi/zfcp_fsf.c index cce698114b0b..14d7413b9e86 100644 --- a/drivers/s390/scsi/zfcp_fsf.c +++ b/drivers/s390/scsi/zfcp_fsf.c | |||
@@ -735,7 +735,7 @@ static struct zfcp_fsf_req *zfcp_fsf_req_create(struct zfcp_adapter *adapter, | |||
735 | 735 | ||
736 | req->adapter = adapter; | 736 | req->adapter = adapter; |
737 | req->fsf_command = fsf_cmd; | 737 | req->fsf_command = fsf_cmd; |
738 | req->req_id = adapter->req_no++; | 738 | req->req_id = adapter->req_no; |
739 | req->sbal_number = 1; | 739 | req->sbal_number = 1; |
740 | req->sbal_first = req_q->first; | 740 | req->sbal_first = req_q->first; |
741 | req->sbal_last = req_q->first; | 741 | req->sbal_last = req_q->first; |
@@ -798,6 +798,7 @@ static int zfcp_fsf_req_send(struct zfcp_fsf_req *req) | |||
798 | /* Don't increase for unsolicited status */ | 798 | /* Don't increase for unsolicited status */ |
799 | if (req->qtcb) | 799 | if (req->qtcb) |
800 | adapter->fsf_req_seq_no++; | 800 | adapter->fsf_req_seq_no++; |
801 | adapter->req_no++; | ||
801 | 802 | ||
802 | return 0; | 803 | return 0; |
803 | } | 804 | } |