diff options
author | Tejun Heo <tj@kernel.org> | 2010-10-28 08:15:26 -0400 |
---|---|---|
committer | Jens Axboe <jaxboe@fusionio.com> | 2010-10-28 08:15:26 -0400 |
commit | 5ad21a337420e82bffb7836ab34027dd7ad624b8 (patch) | |
tree | 64fea23f93077564abc0678262721b9d27b3d8d8 /drivers/block | |
parent | 2027ae1fa914640c287171035cb607a03ad36e95 (diff) |
aoe: don't use flush_scheduled_work()
flush_scheduled_work() is deprecated and scheduled to be removed.
Directly cancel aoedev->work on free instead of depending on
flush_scheduled_works().
Signed-off-by: Tejun Heo <tj@kernel.org>
Cc: "Ed L. Cashin" <ecashin@coraid.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Jens Axboe <jaxboe@fusionio.com>
Diffstat (limited to 'drivers/block')
-rw-r--r-- | drivers/block/aoe/aoedev.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/block/aoe/aoedev.c b/drivers/block/aoe/aoedev.c index 0849280bfc1c..6b5110a47458 100644 --- a/drivers/block/aoe/aoedev.c +++ b/drivers/block/aoe/aoedev.c | |||
@@ -102,6 +102,7 @@ aoedev_freedev(struct aoedev *d) | |||
102 | { | 102 | { |
103 | struct aoetgt **t, **e; | 103 | struct aoetgt **t, **e; |
104 | 104 | ||
105 | cancel_work_sync(&d->work); | ||
105 | if (d->gd) { | 106 | if (d->gd) { |
106 | aoedisk_rm_sysfs(d); | 107 | aoedisk_rm_sysfs(d); |
107 | del_gendisk(d->gd); | 108 | del_gendisk(d->gd); |
@@ -135,7 +136,6 @@ aoedev_flush(const char __user *str, size_t cnt) | |||
135 | all = !strncmp(buf, "all", 3); | 136 | all = !strncmp(buf, "all", 3); |
136 | } | 137 | } |
137 | 138 | ||
138 | flush_scheduled_work(); | ||
139 | spin_lock_irqsave(&devlist_lock, flags); | 139 | spin_lock_irqsave(&devlist_lock, flags); |
140 | dd = &devlist; | 140 | dd = &devlist; |
141 | while ((d = *dd)) { | 141 | while ((d = *dd)) { |
@@ -257,8 +257,6 @@ aoedev_exit(void) | |||
257 | struct aoedev *d; | 257 | struct aoedev *d; |
258 | ulong flags; | 258 | ulong flags; |
259 | 259 | ||
260 | flush_scheduled_work(); | ||
261 | |||
262 | while ((d = devlist)) { | 260 | while ((d = devlist)) { |
263 | devlist = d->next; | 261 | devlist = d->next; |
264 | 262 | ||