| Message ID | 1292086307-19211-28-git-send-email-tj@kernel.org |
|---|---|
| State | Accepted |
| Commit | 75c52a49630a478ffe9c1473441779676817fce6 |
| Headers | show |
On Sat, 2010-12-11 at 17:51 +0100, Tejun Heo wrote: > flush_scheduled_work() is deprecated and scheduled to be removed. > Directly flush cxt->work_{erase|write} on removal instead. > > Signed-off-by: Tejun Heo <tj@kernel.org> > Cc: David Woodhouse <dwmw2@infradead.org> > Cc: linux-mtd@lists.infradead.org > --- > This is part of a series to remove flush_scheduled_work() usage to > prepare for deprecation of flush_scheduled_work(). Patches in this > series are self contained and mostly straight-forward. > > Please feel free to take it into the appropriate tree, or just ack it. > In the latter case, I'll merge the patch through the workqueue tree > during the next merge window. > > If you're seeing this patch for the second time, it's because the > commit hasn't showed up in mainline yet. Please let me know what > should be done. Pushed to l2-mtd-2.6.git, thanks. This means this will be merged to the mtd tree a bit later and will go upstream.
diff --git a/drivers/mtd/mtdoops.c b/drivers/mtd/mtdoops.c index 1ee72f3..8b10273 100644 --- a/drivers/mtd/mtdoops.c +++ b/drivers/mtd/mtdoops.c @@ -396,7 +396,8 @@ static void mtdoops_notify_remove(struct mtd_info *mtd) printk(KERN_WARNING "mtdoops: could not unregister kmsg_dumper\n"); cxt->mtd = NULL; - flush_scheduled_work(); + flush_work_sync(&cxt->work_erase); + flush_work_sync(&cxt->work_write); }
flush_scheduled_work() is deprecated and scheduled to be removed. Directly flush cxt->work_{erase|write} on removal instead. Signed-off-by: Tejun Heo <tj@kernel.org> Cc: David Woodhouse <dwmw2@infradead.org> Cc: linux-mtd@lists.infradead.org --- This is part of a series to remove flush_scheduled_work() usage to prepare for deprecation of flush_scheduled_work(). Patches in this series are self contained and mostly straight-forward. Please feel free to take it into the appropriate tree, or just ack it. In the latter case, I'll merge the patch through the workqueue tree during the next merge window. If you're seeing this patch for the second time, it's because the commit hasn't showed up in mainline yet. Please let me know what should be done. Thank you. drivers/mtd/mtdoops.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-)