[SCSI] zfcp: Assign scheduled work to driver queue
authorSwen Schillig <swen@vnet.ibm.com>
Tue, 24 Nov 2009 15:54:06 +0000 (16:54 +0100)
committerJames Bottomley <James.Bottomley@suse.de>
Fri, 4 Dec 2009 18:02:08 +0000 (12:02 -0600)
The port_scan work was scheduled to the work_queue provided by the
kernel. This resulted on SMP systems to a likely situation that more
than one scan_work were processed in parallel. This is not required
and openes the possibility of race conditions between the removal of
invalid ports and the enqueue of just scanned ports.  This patch
synchronizes the scan_work tasks by scheduling them to adapter local
work_queue.

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>
drivers/s390/scsi/zfcp_aux.c
drivers/s390/scsi/zfcp_erp.c
drivers/s390/scsi/zfcp_ext.h
drivers/s390/scsi/zfcp_fc.c
drivers/s390/scsi/zfcp_fsf.c
drivers/s390/scsi/zfcp_sysfs.c

index 6b94f8d0609c2501c060f35869a3df4a7b9c58ad..107d3f2b6e949f93fece253ffd7b1cd3abd1c5db 100644 (file)
@@ -522,7 +522,7 @@ struct zfcp_adapter *zfcp_adapter_enqueue(struct ccw_device *ccw_device)
        adapter->ccw_device = ccw_device;
 
        INIT_WORK(&adapter->stat_work, _zfcp_status_read_scheduler);
-       INIT_WORK(&adapter->scan_work, _zfcp_fc_scan_ports_later);
+       INIT_WORK(&adapter->scan_work, zfcp_fc_scan_ports);
 
        if (zfcp_qdio_setup(adapter))
                goto failed;
index 3454c2a3b6b13cf5acd75a4a571faa04a2e20cee..b51a11a82e63bf37783e8287f1b1ba5cb8a04064 100644 (file)
@@ -1197,7 +1197,7 @@ static void zfcp_erp_action_cleanup(struct zfcp_erp_action *act, int result)
        case ZFCP_ERP_ACTION_REOPEN_ADAPTER:
                if (result == ZFCP_ERP_SUCCEEDED) {
                        register_service_level(&adapter->service_level);
-                       schedule_work(&adapter->scan_work);
+                       queue_work(adapter->work_queue, &adapter->scan_work);
                } else
                        unregister_service_level(&adapter->service_level);
                kref_put(&adapter->ref, zfcp_adapter_release);
index 5f205f85e6f96ceee2b006ac6ba603ec4bfe90bb..d372146af38d31657b24e7472aa96ba69ace95c0 100644 (file)
@@ -94,8 +94,7 @@ extern void zfcp_erp_adapter_access_changed(struct zfcp_adapter *, char *,
 extern void zfcp_erp_timeout_handler(unsigned long);
 
 /* zfcp_fc.c */
-extern int zfcp_fc_scan_ports(struct zfcp_adapter *);
-extern void _zfcp_fc_scan_ports_later(struct work_struct *);
+extern void zfcp_fc_scan_ports(struct work_struct *);
 extern void zfcp_fc_incoming_els(struct zfcp_fsf_req *);
 extern void zfcp_fc_port_did_lookup(struct work_struct *);
 extern void zfcp_fc_trigger_did_lookup(struct zfcp_port *);
index 9252b65a13a5e31b124a991402ab63acfe80951e..7d6b3cadfb7345c4ded33236fecfeb6eb9f65f5c 100644 (file)
@@ -184,7 +184,7 @@ static void zfcp_fc_incoming_rscn(struct zfcp_fsf_req *fsf_req)
                range_mask = rscn_range_mask[fcp_rscn_element->addr_format];
                _zfcp_fc_incoming_rscn(fsf_req, range_mask, fcp_rscn_element);
        }
-       schedule_work(&fsf_req->adapter->scan_work);
+       queue_work(fsf_req->adapter->work_queue, &fsf_req->adapter->scan_work);
 }
 
 static void zfcp_fc_incoming_wwpn(struct zfcp_fsf_req *req, u64 wwpn)
@@ -664,10 +664,12 @@ static int zfcp_fc_eval_gpn_ft(struct zfcp_gpn_ft *gpn_ft, int max_entries)
 
 /**
  * zfcp_fc_scan_ports - scan remote ports and attach new ports
- * @adapter: pointer to struct zfcp_adapter
+ * @work: reference to scheduled work
  */
-int zfcp_fc_scan_ports(struct zfcp_adapter *adapter)
+void zfcp_fc_scan_ports(struct work_struct *work)
 {
+       struct zfcp_adapter *adapter = container_of(work, struct zfcp_adapter,
+                                                   scan_work);
        int ret, i;
        struct zfcp_gpn_ft *gpn_ft;
        int chain, max_entries, buf_num, max_bytes;
@@ -679,17 +681,14 @@ int zfcp_fc_scan_ports(struct zfcp_adapter *adapter)
 
        if (fc_host_port_type(adapter->scsi_host) != FC_PORTTYPE_NPORT &&
            fc_host_port_type(adapter->scsi_host) != FC_PORTTYPE_NPIV)
-               return 0;
+               return;
 
-       ret = zfcp_fc_wka_port_get(&adapter->gs->ds);
-       if (ret)
-               return ret;
+       if (zfcp_fc_wka_port_get(&adapter->gs->ds))
+               return;
 
        gpn_ft = zfcp_alloc_sg_env(buf_num);
-       if (!gpn_ft) {
-               ret = -ENOMEM;
+       if (!gpn_ft)
                goto out;
-       }
 
        for (i = 0; i < 3; i++) {
                ret = zfcp_fc_send_gpn_ft(gpn_ft, adapter, max_bytes);
@@ -704,15 +703,9 @@ int zfcp_fc_scan_ports(struct zfcp_adapter *adapter)
        zfcp_free_sg_env(gpn_ft, buf_num);
 out:
        zfcp_fc_wka_port_put(&adapter->gs->ds);
-       return ret;
 }
 
 
-void _zfcp_fc_scan_ports_later(struct work_struct *work)
-{
-       zfcp_fc_scan_ports(container_of(work, struct zfcp_adapter, scan_work));
-}
-
 struct zfcp_els_fc_job {
        struct zfcp_send_els els;
        struct fc_bsg_job *job;
index 5eb96052941a6545837f8fe857e7caa822ba313a..b6f12c826b79c9bd8d8aedc5df085d78f4dacdcd 100644 (file)
@@ -287,7 +287,7 @@ static void zfcp_fsf_status_read_handler(struct zfcp_fsf_req *req)
                        zfcp_erp_adapter_access_changed(adapter, "fssrh_3",
                                                        req);
                if (sr_buf->status_subtype & FSF_STATUS_READ_SUB_INCOMING_ELS)
-                       schedule_work(&adapter->scan_work);
+                       queue_work(adapter->work_queue, &adapter->scan_work);
                break;
        case FSF_STATUS_READ_CFDC_UPDATED:
                zfcp_erp_adapter_access_changed(adapter, "fssrh_4", req);
index 35e920b4fd8acb43ffe738c0bc482a2f7bb59d1c..f539e006683cdf3ef0aff3941bcd82cbcdae2266 100644 (file)
@@ -184,15 +184,16 @@ static ssize_t zfcp_sysfs_port_rescan_store(struct device *dev,
 {
        struct ccw_device *cdev = to_ccwdev(dev);
        struct zfcp_adapter *adapter = zfcp_ccw_adapter_by_cdev(cdev);
-       int ret;
 
        if (!adapter)
                return -ENODEV;
 
-       ret = zfcp_fc_scan_ports(adapter);
+       /* sync the user-space- with the kernel-invocation of scan_work */
+       queue_work(adapter->work_queue, &adapter->scan_work);
+       flush_work(&adapter->scan_work);
        zfcp_ccw_adapter_put(adapter);
 
-       return ret ? ret : (ssize_t) count;
+       return (ssize_t) count;
 }
 static ZFCP_DEV_ATTR(adapter, port_rescan, S_IWUSR, NULL,
                     zfcp_sysfs_port_rescan_store);