diff options
author | Tejun Heo <tj@kernel.org> | 2010-12-14 10:21:17 -0500 |
---|---|---|
committer | Tejun Heo <tj@kernel.org> | 2010-12-15 04:56:11 -0500 |
commit | afe2c511fb2d75f1515081ff1be15bd79cfe722d (patch) | |
tree | 28aa74e9e0c654a95bf3306101e10ac1d16919d1 /drivers | |
parent | 2d64672ed38721b7a3815009d79bfb90a1f34a17 (diff) |
workqueue: convert cancel_rearming_delayed_work[queue]() users to cancel_delayed_work_sync()
cancel_rearming_delayed_work[queue]() has been superceded by
cancel_delayed_work_sync() quite some time ago. Convert all the
in-kernel users. The conversions are completely equivalent and
trivial.
Signed-off-by: Tejun Heo <tj@kernel.org>
Acked-by: "David S. Miller" <davem@davemloft.net>
Acked-by: Greg Kroah-Hartman <gregkh@suse.de>
Acked-by: Evgeniy Polyakov <zbr@ioremap.net>
Cc: Jeff Garzik <jgarzik@pobox.com>
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Cc: Mauro Carvalho Chehab <mchehab@infradead.org>
Cc: netdev@vger.kernel.org
Cc: Anton Vorontsov <cbou@mail.ru>
Cc: David Woodhouse <dwmw2@infradead.org>
Cc: "J. Bruce Fields" <bfields@fieldses.org>
Cc: Neil Brown <neilb@suse.de>
Cc: Alex Elder <aelder@sgi.com>
Cc: xfs-masters@oss.sgi.com
Cc: Christoph Lameter <cl@linux-foundation.org>
Cc: Pekka Enberg <penberg@cs.helsinki.fi>
Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: netfilter-devel@vger.kernel.org
Cc: Trond Myklebust <Trond.Myklebust@netapp.com>
Cc: linux-nfs@vger.kernel.org
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/ata/libata-core.c | 2 | ||||
-rw-r--r-- | drivers/ata/libata-sff.c | 2 | ||||
-rw-r--r-- | drivers/macintosh/rack-meter.c | 4 | ||||
-rw-r--r-- | drivers/media/dvb/dvb-usb/dvb-usb-remote.c | 2 | ||||
-rw-r--r-- | drivers/media/video/em28xx/em28xx-input.c | 2 | ||||
-rw-r--r-- | drivers/net/chelsio/my3126.c | 2 | ||||
-rw-r--r-- | drivers/net/ibm_newemac/core.c | 4 | ||||
-rw-r--r-- | drivers/net/wireless/zd1211rw/zd_mac.c | 3 | ||||
-rw-r--r-- | drivers/power/ds2760_battery.c | 6 | ||||
-rw-r--r-- | drivers/power/intel_mid_battery.c | 6 | ||||
-rw-r--r-- | drivers/staging/pohmelfs/inode.c | 4 | ||||
-rw-r--r-- | drivers/usb/atm/cxacru.c | 2 | ||||
-rw-r--r-- | drivers/video/fb_defio.c | 2 | ||||
-rw-r--r-- | drivers/video/omap/lcd_mipid.c | 2 |
14 files changed, 19 insertions, 24 deletions
diff --git a/drivers/ata/libata-core.c b/drivers/ata/libata-core.c index 7f77c67d267c..6669b44044fb 100644 --- a/drivers/ata/libata-core.c +++ b/drivers/ata/libata-core.c | |||
@@ -6122,7 +6122,7 @@ static void ata_port_detach(struct ata_port *ap) | |||
6122 | /* it better be dead now */ | 6122 | /* it better be dead now */ |
6123 | WARN_ON(!(ap->pflags & ATA_PFLAG_UNLOADED)); | 6123 | WARN_ON(!(ap->pflags & ATA_PFLAG_UNLOADED)); |
6124 | 6124 | ||
6125 | cancel_rearming_delayed_work(&ap->hotplug_task); | 6125 | cancel_delayed_work_sync(&ap->hotplug_task); |
6126 | 6126 | ||
6127 | skip_eh: | 6127 | skip_eh: |
6128 | if (ap->pmp_link) { | 6128 | if (ap->pmp_link) { |
diff --git a/drivers/ata/libata-sff.c b/drivers/ata/libata-sff.c index d05387d1e14b..8660a70f164a 100644 --- a/drivers/ata/libata-sff.c +++ b/drivers/ata/libata-sff.c | |||
@@ -1320,7 +1320,7 @@ void ata_sff_flush_pio_task(struct ata_port *ap) | |||
1320 | { | 1320 | { |
1321 | DPRINTK("ENTER\n"); | 1321 | DPRINTK("ENTER\n"); |
1322 | 1322 | ||
1323 | cancel_rearming_delayed_work(&ap->sff_pio_task); | 1323 | cancel_delayed_work_sync(&ap->sff_pio_task); |
1324 | ap->hsm_task_state = HSM_ST_IDLE; | 1324 | ap->hsm_task_state = HSM_ST_IDLE; |
1325 | 1325 | ||
1326 | if (ata_msg_ctl(ap)) | 1326 | if (ata_msg_ctl(ap)) |
diff --git a/drivers/macintosh/rack-meter.c b/drivers/macintosh/rack-meter.c index 53cce3a5da23..39f660b2a60d 100644 --- a/drivers/macintosh/rack-meter.c +++ b/drivers/macintosh/rack-meter.c | |||
@@ -285,8 +285,8 @@ static void __devinit rackmeter_init_cpu_sniffer(struct rackmeter *rm) | |||
285 | 285 | ||
286 | static void __devexit rackmeter_stop_cpu_sniffer(struct rackmeter *rm) | 286 | static void __devexit rackmeter_stop_cpu_sniffer(struct rackmeter *rm) |
287 | { | 287 | { |
288 | cancel_rearming_delayed_work(&rm->cpu[0].sniffer); | 288 | cancel_delayed_work_sync(&rm->cpu[0].sniffer); |
289 | cancel_rearming_delayed_work(&rm->cpu[1].sniffer); | 289 | cancel_delayed_work_sync(&rm->cpu[1].sniffer); |
290 | } | 290 | } |
291 | 291 | ||
292 | static int __devinit rackmeter_setup(struct rackmeter *rm) | 292 | static int __devinit rackmeter_setup(struct rackmeter *rm) |
diff --git a/drivers/media/dvb/dvb-usb/dvb-usb-remote.c b/drivers/media/dvb/dvb-usb/dvb-usb-remote.c index b579fed3ab3f..0831469af69f 100644 --- a/drivers/media/dvb/dvb-usb/dvb-usb-remote.c +++ b/drivers/media/dvb/dvb-usb/dvb-usb-remote.c | |||
@@ -298,7 +298,7 @@ int dvb_usb_remote_init(struct dvb_usb_device *d) | |||
298 | int dvb_usb_remote_exit(struct dvb_usb_device *d) | 298 | int dvb_usb_remote_exit(struct dvb_usb_device *d) |
299 | { | 299 | { |
300 | if (d->state & DVB_USB_STATE_REMOTE) { | 300 | if (d->state & DVB_USB_STATE_REMOTE) { |
301 | cancel_rearming_delayed_work(&d->rc_query_work); | 301 | cancel_delayed_work_sync(&d->rc_query_work); |
302 | flush_scheduled_work(); | 302 | flush_scheduled_work(); |
303 | if (d->props.rc.mode == DVB_RC_LEGACY) | 303 | if (d->props.rc.mode == DVB_RC_LEGACY) |
304 | input_unregister_device(d->rc_input_dev); | 304 | input_unregister_device(d->rc_input_dev); |
diff --git a/drivers/media/video/em28xx/em28xx-input.c b/drivers/media/video/em28xx/em28xx-input.c index 6759cd5570dd..99403c720e3a 100644 --- a/drivers/media/video/em28xx/em28xx-input.c +++ b/drivers/media/video/em28xx/em28xx-input.c | |||
@@ -557,7 +557,7 @@ void em28xx_deregister_snapshot_button(struct em28xx *dev) | |||
557 | { | 557 | { |
558 | if (dev->sbutton_input_dev != NULL) { | 558 | if (dev->sbutton_input_dev != NULL) { |
559 | em28xx_info("Deregistering snapshot button\n"); | 559 | em28xx_info("Deregistering snapshot button\n"); |
560 | cancel_rearming_delayed_work(&dev->sbutton_query_work); | 560 | cancel_delayed_work_sync(&dev->sbutton_query_work); |
561 | input_unregister_device(dev->sbutton_input_dev); | 561 | input_unregister_device(dev->sbutton_input_dev); |
562 | dev->sbutton_input_dev = NULL; | 562 | dev->sbutton_input_dev = NULL; |
563 | } | 563 | } |
diff --git a/drivers/net/chelsio/my3126.c b/drivers/net/chelsio/my3126.c index 4c6028512d10..a683fd3bb624 100644 --- a/drivers/net/chelsio/my3126.c +++ b/drivers/net/chelsio/my3126.c | |||
@@ -22,7 +22,7 @@ static int my3126_interrupt_enable(struct cphy *cphy) | |||
22 | 22 | ||
23 | static int my3126_interrupt_disable(struct cphy *cphy) | 23 | static int my3126_interrupt_disable(struct cphy *cphy) |
24 | { | 24 | { |
25 | cancel_rearming_delayed_work(&cphy->phy_update); | 25 | cancel_delayed_work_sync(&cphy->phy_update); |
26 | return 0; | 26 | return 0; |
27 | } | 27 | } |
28 | 28 | ||
diff --git a/drivers/net/ibm_newemac/core.c b/drivers/net/ibm_newemac/core.c index 06bb9b799458..e209efaa01b9 100644 --- a/drivers/net/ibm_newemac/core.c +++ b/drivers/net/ibm_newemac/core.c | |||
@@ -1279,7 +1279,7 @@ static void emac_force_link_update(struct emac_instance *dev) | |||
1279 | netif_carrier_off(dev->ndev); | 1279 | netif_carrier_off(dev->ndev); |
1280 | smp_rmb(); | 1280 | smp_rmb(); |
1281 | if (dev->link_polling) { | 1281 | if (dev->link_polling) { |
1282 | cancel_rearming_delayed_work(&dev->link_work); | 1282 | cancel_delayed_work_sync(&dev->link_work); |
1283 | if (dev->link_polling) | 1283 | if (dev->link_polling) |
1284 | schedule_delayed_work(&dev->link_work, PHY_POLL_LINK_OFF); | 1284 | schedule_delayed_work(&dev->link_work, PHY_POLL_LINK_OFF); |
1285 | } | 1285 | } |
@@ -1294,7 +1294,7 @@ static int emac_close(struct net_device *ndev) | |||
1294 | 1294 | ||
1295 | if (dev->phy.address >= 0) { | 1295 | if (dev->phy.address >= 0) { |
1296 | dev->link_polling = 0; | 1296 | dev->link_polling = 0; |
1297 | cancel_rearming_delayed_work(&dev->link_work); | 1297 | cancel_delayed_work_sync(&dev->link_work); |
1298 | } | 1298 | } |
1299 | mutex_lock(&dev->link_lock); | 1299 | mutex_lock(&dev->link_lock); |
1300 | emac_netif_stop(dev); | 1300 | emac_netif_stop(dev); |
diff --git a/drivers/net/wireless/zd1211rw/zd_mac.c b/drivers/net/wireless/zd1211rw/zd_mac.c index 43307bd42a69..6107304cb94c 100644 --- a/drivers/net/wireless/zd1211rw/zd_mac.c +++ b/drivers/net/wireless/zd1211rw/zd_mac.c | |||
@@ -1207,7 +1207,6 @@ static void housekeeping_enable(struct zd_mac *mac) | |||
1207 | static void housekeeping_disable(struct zd_mac *mac) | 1207 | static void housekeeping_disable(struct zd_mac *mac) |
1208 | { | 1208 | { |
1209 | dev_dbg_f(zd_mac_dev(mac), "\n"); | 1209 | dev_dbg_f(zd_mac_dev(mac), "\n"); |
1210 | cancel_rearming_delayed_workqueue(zd_workqueue, | 1210 | cancel_delayed_work_sync(&mac->housekeeping.link_led_work); |
1211 | &mac->housekeeping.link_led_work); | ||
1212 | zd_chip_control_leds(&mac->chip, ZD_LED_OFF); | 1211 | zd_chip_control_leds(&mac->chip, ZD_LED_OFF); |
1213 | } | 1212 | } |
diff --git a/drivers/power/ds2760_battery.c b/drivers/power/ds2760_battery.c index b3c01c16a164..e7f89785beef 100644 --- a/drivers/power/ds2760_battery.c +++ b/drivers/power/ds2760_battery.c | |||
@@ -580,10 +580,8 @@ static int ds2760_battery_remove(struct platform_device *pdev) | |||
580 | { | 580 | { |
581 | struct ds2760_device_info *di = platform_get_drvdata(pdev); | 581 | struct ds2760_device_info *di = platform_get_drvdata(pdev); |
582 | 582 | ||
583 | cancel_rearming_delayed_workqueue(di->monitor_wqueue, | 583 | cancel_delayed_work_sync(&di->monitor_work); |
584 | &di->monitor_work); | 584 | cancel_delayed_work_sync(&di->set_charged_work); |
585 | cancel_rearming_delayed_workqueue(di->monitor_wqueue, | ||
586 | &di->set_charged_work); | ||
587 | destroy_workqueue(di->monitor_wqueue); | 585 | destroy_workqueue(di->monitor_wqueue); |
588 | power_supply_unregister(&di->bat); | 586 | power_supply_unregister(&di->bat); |
589 | kfree(di); | 587 | kfree(di); |
diff --git a/drivers/power/intel_mid_battery.c b/drivers/power/intel_mid_battery.c index 2a10cd361181..36cf402c0677 100644 --- a/drivers/power/intel_mid_battery.c +++ b/drivers/power/intel_mid_battery.c | |||
@@ -730,8 +730,7 @@ static __devinit int probe(int irq, struct device *dev) | |||
730 | power_reg_failed_1: | 730 | power_reg_failed_1: |
731 | power_supply_unregister(&pbi->batt); | 731 | power_supply_unregister(&pbi->batt); |
732 | power_reg_failed: | 732 | power_reg_failed: |
733 | cancel_rearming_delayed_workqueue(pbi->monitor_wqueue, | 733 | cancel_delayed_work_sync(&pbi->monitor_battery); |
734 | &pbi->monitor_battery); | ||
735 | requestirq_failed: | 734 | requestirq_failed: |
736 | destroy_workqueue(pbi->monitor_wqueue); | 735 | destroy_workqueue(pbi->monitor_wqueue); |
737 | wqueue_failed: | 736 | wqueue_failed: |
@@ -760,8 +759,7 @@ static int __devexit platform_pmic_battery_remove(struct platform_device *pdev) | |||
760 | struct pmic_power_module_info *pbi = dev_get_drvdata(&pdev->dev); | 759 | struct pmic_power_module_info *pbi = dev_get_drvdata(&pdev->dev); |
761 | 760 | ||
762 | free_irq(pbi->irq, pbi); | 761 | free_irq(pbi->irq, pbi); |
763 | cancel_rearming_delayed_workqueue(pbi->monitor_wqueue, | 762 | cancel_delayed_work_sync(&pbi->monitor_battery); |
764 | &pbi->monitor_battery); | ||
765 | destroy_workqueue(pbi->monitor_wqueue); | 763 | destroy_workqueue(pbi->monitor_wqueue); |
766 | 764 | ||
767 | power_supply_unregister(&pbi->usb); | 765 | power_supply_unregister(&pbi->usb); |
diff --git a/drivers/staging/pohmelfs/inode.c b/drivers/staging/pohmelfs/inode.c index 61685ccceda8..d4a1f204b9d5 100644 --- a/drivers/staging/pohmelfs/inode.c +++ b/drivers/staging/pohmelfs/inode.c | |||
@@ -1318,8 +1318,8 @@ static void pohmelfs_put_super(struct super_block *sb) | |||
1318 | } | 1318 | } |
1319 | 1319 | ||
1320 | psb->trans_scan_timeout = psb->drop_scan_timeout = 0; | 1320 | psb->trans_scan_timeout = psb->drop_scan_timeout = 0; |
1321 | cancel_rearming_delayed_work(&psb->dwork); | 1321 | cancel_delayed_work_sync(&psb->dwork); |
1322 | cancel_rearming_delayed_work(&psb->drop_dwork); | 1322 | cancel_delayed_work_sync(&psb->drop_dwork); |
1323 | flush_scheduled_work(); | 1323 | flush_scheduled_work(); |
1324 | 1324 | ||
1325 | dprintk("%s: stopped workqueues.\n", __func__); | 1325 | dprintk("%s: stopped workqueues.\n", __func__); |
diff --git a/drivers/usb/atm/cxacru.c b/drivers/usb/atm/cxacru.c index f383cb42b1d7..a845f8b8382f 100644 --- a/drivers/usb/atm/cxacru.c +++ b/drivers/usb/atm/cxacru.c | |||
@@ -1247,7 +1247,7 @@ static void cxacru_unbind(struct usbatm_data *usbatm_instance, | |||
1247 | mutex_unlock(&instance->poll_state_serialize); | 1247 | mutex_unlock(&instance->poll_state_serialize); |
1248 | 1248 | ||
1249 | if (is_polling) | 1249 | if (is_polling) |
1250 | cancel_rearming_delayed_work(&instance->poll_work); | 1250 | cancel_delayed_work_sync(&instance->poll_work); |
1251 | 1251 | ||
1252 | usb_kill_urb(instance->snd_urb); | 1252 | usb_kill_urb(instance->snd_urb); |
1253 | usb_kill_urb(instance->rcv_urb); | 1253 | usb_kill_urb(instance->rcv_urb); |
diff --git a/drivers/video/fb_defio.c b/drivers/video/fb_defio.c index 6b93ef93cb12..804000183c5e 100644 --- a/drivers/video/fb_defio.c +++ b/drivers/video/fb_defio.c | |||
@@ -75,7 +75,7 @@ int fb_deferred_io_fsync(struct file *file, int datasync) | |||
75 | return 0; | 75 | return 0; |
76 | 76 | ||
77 | /* Kill off the delayed work */ | 77 | /* Kill off the delayed work */ |
78 | cancel_rearming_delayed_work(&info->deferred_work); | 78 | cancel_delayed_work_sync(&info->deferred_work); |
79 | 79 | ||
80 | /* Run it immediately */ | 80 | /* Run it immediately */ |
81 | return schedule_delayed_work(&info->deferred_work, 0); | 81 | return schedule_delayed_work(&info->deferred_work, 0); |
diff --git a/drivers/video/omap/lcd_mipid.c b/drivers/video/omap/lcd_mipid.c index 64dcc7439c99..90e3bdd1b7ab 100644 --- a/drivers/video/omap/lcd_mipid.c +++ b/drivers/video/omap/lcd_mipid.c | |||
@@ -396,7 +396,7 @@ static void mipid_esd_start_check(struct mipid_device *md) | |||
396 | static void mipid_esd_stop_check(struct mipid_device *md) | 396 | static void mipid_esd_stop_check(struct mipid_device *md) |
397 | { | 397 | { |
398 | if (md->esd_check != NULL) | 398 | if (md->esd_check != NULL) |
399 | cancel_rearming_delayed_workqueue(md->esd_wq, &md->esd_work); | 399 | cancel_delayed_work_sync(&md->esd_work); |
400 | } | 400 | } |
401 | 401 | ||
402 | static void mipid_esd_work(struct work_struct *work) | 402 | static void mipid_esd_work(struct work_struct *work) |