diff options
author | Swen Schillig <swen@vnet.ibm.com> | 2010-07-16 09:37:38 -0400 |
---|---|---|
committer | James Bottomley <James.Bottomley@suse.de> | 2010-07-28 10:48:50 -0400 |
commit | 706eca49a044a1ea89352dcc4b96ffc1631b2cb5 (patch) | |
tree | f36692c277a38c051536e06272ade7ed759ab4cc /drivers/s390/scsi/zfcp_fsf.c | |
parent | 01b047599ade30051bf6c14fbe64181d1fec3dfa (diff) |
[SCSI] zfcp: Cleanup QDIO attachment and improve processing.
Some definitions and structures in the zfcp QDIO processing are
improved by the removal of not required variables and processing steps.
I addition the naming of some variables is changed to make their purpose
more clear.
Signed-off-by: Swen Schillig <swen@vnet.ibm.com>
Signed-off-by: Christof Schmitt <christof.schmitt@de.ibm.com>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
Diffstat (limited to 'drivers/s390/scsi/zfcp_fsf.c')
-rw-r--r-- | drivers/s390/scsi/zfcp_fsf.c | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/drivers/s390/scsi/zfcp_fsf.c b/drivers/s390/scsi/zfcp_fsf.c index 5f502c9cb067..0710c59b80ae 100644 --- a/drivers/s390/scsi/zfcp_fsf.c +++ b/drivers/s390/scsi/zfcp_fsf.c | |||
@@ -732,7 +732,7 @@ static int zfcp_fsf_req_send(struct zfcp_fsf_req *req) | |||
732 | 732 | ||
733 | zfcp_reqlist_add(adapter->req_list, req); | 733 | zfcp_reqlist_add(adapter->req_list, req); |
734 | 734 | ||
735 | req->qdio_req.qdio_outb_usage = atomic_read(&qdio->req_q.count); | 735 | req->qdio_req.qdio_outb_usage = atomic_read(&qdio->req_q_free); |
736 | req->issued = get_clock(); | 736 | req->issued = get_clock(); |
737 | if (zfcp_qdio_send(qdio, &req->qdio_req)) { | 737 | if (zfcp_qdio_send(qdio, &req->qdio_req)) { |
738 | del_timer(&req->timer); | 738 | del_timer(&req->timer); |
@@ -2025,7 +2025,7 @@ static void zfcp_fsf_req_trace(struct zfcp_fsf_req *req, struct scsi_cmnd *scsi) | |||
2025 | blktrc.magic = ZFCP_BLK_DRV_DATA_MAGIC; | 2025 | blktrc.magic = ZFCP_BLK_DRV_DATA_MAGIC; |
2026 | if (req->status & ZFCP_STATUS_FSFREQ_ERROR) | 2026 | if (req->status & ZFCP_STATUS_FSFREQ_ERROR) |
2027 | blktrc.flags |= ZFCP_BLK_REQ_ERROR; | 2027 | blktrc.flags |= ZFCP_BLK_REQ_ERROR; |
2028 | blktrc.inb_usage = req->qdio_req.qdio_inb_usage; | 2028 | blktrc.inb_usage = 0; |
2029 | blktrc.outb_usage = req->qdio_req.qdio_outb_usage; | 2029 | blktrc.outb_usage = req->qdio_req.qdio_outb_usage; |
2030 | 2030 | ||
2031 | if (req->adapter->adapter_features & FSF_FEATURE_MEASUREMENT_DATA && | 2031 | if (req->adapter->adapter_features & FSF_FEATURE_MEASUREMENT_DATA && |
@@ -2207,7 +2207,7 @@ int zfcp_fsf_send_fcp_command_task(struct zfcp_unit *unit, | |||
2207 | return -EBUSY; | 2207 | return -EBUSY; |
2208 | 2208 | ||
2209 | spin_lock(&qdio->req_q_lock); | 2209 | spin_lock(&qdio->req_q_lock); |
2210 | if (atomic_read(&qdio->req_q.count) <= 0) { | 2210 | if (atomic_read(&qdio->req_q_free) <= 0) { |
2211 | atomic_inc(&qdio->req_q_full); | 2211 | atomic_inc(&qdio->req_q_full); |
2212 | goto out; | 2212 | goto out; |
2213 | } | 2213 | } |
@@ -2407,7 +2407,7 @@ out: | |||
2407 | void zfcp_fsf_reqid_check(struct zfcp_qdio *qdio, int sbal_idx) | 2407 | void zfcp_fsf_reqid_check(struct zfcp_qdio *qdio, int sbal_idx) |
2408 | { | 2408 | { |
2409 | struct zfcp_adapter *adapter = qdio->adapter; | 2409 | struct zfcp_adapter *adapter = qdio->adapter; |
2410 | struct qdio_buffer *sbal = qdio->resp_q.sbal[sbal_idx]; | 2410 | struct qdio_buffer *sbal = qdio->res_q[sbal_idx]; |
2411 | struct qdio_buffer_element *sbale; | 2411 | struct qdio_buffer_element *sbale; |
2412 | struct zfcp_fsf_req *fsf_req; | 2412 | struct zfcp_fsf_req *fsf_req; |
2413 | unsigned long req_id; | 2413 | unsigned long req_id; |
@@ -2428,8 +2428,6 @@ void zfcp_fsf_reqid_check(struct zfcp_qdio *qdio, int sbal_idx) | |||
2428 | req_id, dev_name(&adapter->ccw_device->dev)); | 2428 | req_id, dev_name(&adapter->ccw_device->dev)); |
2429 | 2429 | ||
2430 | fsf_req->qdio_req.sbal_response = sbal_idx; | 2430 | fsf_req->qdio_req.sbal_response = sbal_idx; |
2431 | fsf_req->qdio_req.qdio_inb_usage = | ||
2432 | atomic_read(&qdio->resp_q.count); | ||
2433 | zfcp_fsf_req_complete(fsf_req); | 2431 | zfcp_fsf_req_complete(fsf_req); |
2434 | 2432 | ||
2435 | if (likely(sbale->flags & SBAL_FLAGS_LAST_ENTRY)) | 2433 | if (likely(sbale->flags & SBAL_FLAGS_LAST_ENTRY)) |