diff options
author | Heiko Carstens <heiko.carstens@de.ibm.com> | 2007-05-08 05:17:54 -0400 |
---|---|---|
committer | James Bottomley <jejb@mulgrave.il.steeleye.com> | 2007-05-08 12:56:50 -0400 |
commit | ca2d02c2f9ea476062ae181eec60b8bcd97857d6 (patch) | |
tree | dffe33856db601a173a117263b9ee4799d555ac6 /drivers/s390/scsi/zfcp_fsf.c | |
parent | 5f852be9e11d62223ea063f6ceed4f9677f54051 (diff) |
[SCSI] zfcp: rework request ID management.
Simplify request ID management and make sure that frequently used
functions are inlined. Also fix a memory leak in zfcp_adapter_enqueue()
which only gets hit in error handling.
Signed-off-by: Heiko Carstens <heiko.carstens@de.ibm.com>
Signed-off-by: Swen Schillig <swen@vnet.ibm.com>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
Diffstat (limited to 'drivers/s390/scsi/zfcp_fsf.c')
-rw-r--r-- | drivers/s390/scsi/zfcp_fsf.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/s390/scsi/zfcp_fsf.c b/drivers/s390/scsi/zfcp_fsf.c index 07094c3dc341..083308b1d3e2 100644 --- a/drivers/s390/scsi/zfcp_fsf.c +++ b/drivers/s390/scsi/zfcp_fsf.c | |||
@@ -4787,7 +4787,7 @@ static int zfcp_fsf_req_send(struct zfcp_fsf_req *fsf_req) | |||
4787 | retval = -EIO; | 4787 | retval = -EIO; |
4788 | del_timer(&fsf_req->timer); | 4788 | del_timer(&fsf_req->timer); |
4789 | spin_lock(&adapter->req_list_lock); | 4789 | spin_lock(&adapter->req_list_lock); |
4790 | zfcp_reqlist_remove(adapter, fsf_req->req_id); | 4790 | zfcp_reqlist_remove(adapter, fsf_req); |
4791 | spin_unlock(&adapter->req_list_lock); | 4791 | spin_unlock(&adapter->req_list_lock); |
4792 | /* undo changes in request queue made for this request */ | 4792 | /* undo changes in request queue made for this request */ |
4793 | zfcp_qdio_zero_sbals(req_queue->buffer, | 4793 | zfcp_qdio_zero_sbals(req_queue->buffer, |