aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/s390/scsi/zfcp_fsf.c
diff options
context:
space:
mode:
authorSwen Schillig <swen@vnet.ibm.com>2010-07-16 09:37:38 -0400
committerJames Bottomley <James.Bottomley@suse.de>2010-07-28 10:48:50 -0400
commit706eca49a044a1ea89352dcc4b96ffc1631b2cb5 (patch)
treef36692c277a38c051536e06272ade7ed759ab4cc /drivers/s390/scsi/zfcp_fsf.c
parent01b047599ade30051bf6c14fbe64181d1fec3dfa (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.c10
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:
2407void zfcp_fsf_reqid_check(struct zfcp_qdio *qdio, int sbal_idx) 2407void 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))