summaryrefslogtreecommitdiffstats
path: root/drivers
diff options
context:
space:
mode:
authorMartin Schwidefsky <schwidefsky@de.ibm.com>2015-07-27 12:47:40 +0200
committerMartin Schwidefsky <schwidefsky@de.ibm.com>2015-10-14 14:32:25 +0200
commit8139b89d7cc5f53cff63a72c8e226385f34a70a2 (patch)
treee0a4ebe38e30f5d0734ee9fe0caa3e325df6d602 /drivers
parentb96a9e5140d4d6253e499bd18c35a7d085ac46e2 (diff)
downloadop-kernel-dev-8139b89d7cc5f53cff63a72c8e226385f34a70a2.zip
op-kernel-dev-8139b89d7cc5f53cff63a72c8e226385f34a70a2.tar.gz
s390/zcrypt: use system work queue for ap_scan_bus
There is a system work queue system_long_wq for long running work. Use this work queue for the AP bus scan loop. Reviewd-by: Ingo Tuchscherer <ingo.tuchscherer@linux.vnet.ibm.com> Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/s390/crypto/ap_bus.c26
1 files changed, 8 insertions, 18 deletions
diff --git a/drivers/s390/crypto/ap_bus.c b/drivers/s390/crypto/ap_bus.c
index 1d61c86..9cb3dfb 100644
--- a/drivers/s390/crypto/ap_bus.c
+++ b/drivers/s390/crypto/ap_bus.c
@@ -76,13 +76,12 @@ static DEFINE_SPINLOCK(ap_device_list_lock);
static LIST_HEAD(ap_device_list);
/*
- * Workqueue & timer for bus rescan.
+ * Workqueue timer for bus rescan.
*/
-static struct workqueue_struct *ap_work_queue;
static struct timer_list ap_config_timer;
static int ap_config_time = AP_CONFIG_TIME;
static void ap_scan_bus(struct work_struct *);
-static DECLARE_WORK(ap_config_work, ap_scan_bus);
+static DECLARE_WORK(ap_scan_work, ap_scan_bus);
/*
* Tasklet & timer for AP request polling and interrupts
@@ -1247,7 +1246,7 @@ static void ap_bus_suspend(void)
* Disable scanning for devices, thus we do not want to scan
* for them after removing.
*/
- flush_workqueue(ap_work_queue);
+ flush_work(&ap_scan_work);
tasklet_disable(&ap_tasklet);
}
@@ -1280,7 +1279,7 @@ static void ap_bus_resume(void)
if (ap_airq_flag)
xchg(ap_airq.lsi_ptr, 0);
tasklet_enable(&ap_tasklet);
- queue_work(ap_work_queue, &ap_config_work);
+ queue_work(system_long_wq, &ap_scan_work);
}
static int ap_power_event(struct notifier_block *this, unsigned long event,
@@ -1406,8 +1405,8 @@ void ap_bus_force_rescan(void)
return;
/* processing a asynchronous bus rescan */
del_timer(&ap_config_timer);
- queue_work(ap_work_queue, &ap_config_work);
- flush_work(&ap_config_work);
+ queue_work(system_long_wq, &ap_scan_work);
+ flush_work(&ap_scan_work);
}
EXPORT_SYMBOL(ap_bus_force_rescan);
@@ -1696,7 +1695,7 @@ static void ap_config_timeout(unsigned long ptr)
{
if (ap_suspend_flag)
return;
- queue_work(ap_work_queue, &ap_config_work);
+ queue_work(system_long_wq, &ap_scan_work);
}
static void ap_reset_domain(void)
@@ -1784,12 +1783,6 @@ int __init ap_module_init(void)
if (rc)
goto out_bus;
- ap_work_queue = create_singlethread_workqueue("kapwork");
- if (!ap_work_queue) {
- rc = -ENOMEM;
- goto out_root;
- }
-
/* Setup the AP bus rescan timer. */
setup_timer(&ap_config_timer, ap_config_timeout, 0);
@@ -1814,7 +1807,7 @@ int __init ap_module_init(void)
if (rc)
goto out_pm;
- queue_work(ap_work_queue, &ap_config_work);
+ queue_work(system_long_wq, &ap_scan_work);
return 0;
@@ -1822,8 +1815,6 @@ out_pm:
ap_poll_thread_stop();
out_work:
hrtimer_cancel(&ap_poll_timer);
- destroy_workqueue(ap_work_queue);
-out_root:
root_device_unregister(ap_root_device);
out_bus:
while (i--)
@@ -1850,7 +1841,6 @@ void ap_module_exit(void)
ap_poll_thread_stop();
del_timer_sync(&ap_config_timer);
hrtimer_cancel(&ap_poll_timer);
- destroy_workqueue(ap_work_queue);
tasklet_kill(&ap_tasklet);
bus_for_each_dev(&ap_bus_type, NULL, NULL, __ap_devices_unregister);
for (i = 0; ap_bus_attrs[i]; i++)
OpenPOWER on IntegriCloud