diff options
author | Martin Schwidefsky <schwidefsky@de.ibm.com> | 2015-07-27 12:47:40 +0200 |
---|---|---|
committer | Martin Schwidefsky <schwidefsky@de.ibm.com> | 2015-10-14 14:32:25 +0200 |
commit | 8139b89d7cc5f53cff63a72c8e226385f34a70a2 (patch) | |
tree | e0a4ebe38e30f5d0734ee9fe0caa3e325df6d602 /drivers | |
parent | b96a9e5140d4d6253e499bd18c35a7d085ac46e2 (diff) | |
download | op-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.c | 26 |
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++) |