tty_ldisc: don't use flush_scheduled_work()
authorTejun Heo <tj@kernel.org>
Mon, 24 Jan 2011 16:54:12 +0000 (17:54 +0100)
committerGreg Kroah-Hartman <gregkh@suse.de>
Thu, 3 Feb 2011 22:12:25 +0000 (14:12 -0800)
flush_scheduled_work() is scheduled to be deprecated.  Explicitly sync
flush the used work items instead.  Note that before this change,
flush_scheduled_work() wouldn't have properly flushed tty->buf.work if
it were on timer.

Signed-off-by: Tejun Heo <tj@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/tty/tty_ldisc.c

index 4214d58276f709a8743c03f431cef5b6360c2b46..c42f402db9baddfd8e2648a59e9447b8d427b394 100644 (file)
@@ -534,6 +534,19 @@ static int tty_ldisc_halt(struct tty_struct *tty)
        return cancel_delayed_work_sync(&tty->buf.work);
 }
 
+/**
+ *     tty_ldisc_flush_works   -       flush all works of a tty
+ *     @tty: tty device to flush works for
+ *
+ *     Sync flush all works belonging to @tty.
+ */
+static void tty_ldisc_flush_works(struct tty_struct *tty)
+{
+       flush_work_sync(&tty->hangup_work);
+       flush_work_sync(&tty->SAK_work);
+       flush_delayed_work_sync(&tty->buf.work);
+}
+
 /**
  *     tty_ldisc_wait_idle     -       wait for the ldisc to become idle
  *     @tty: tty to wait for
@@ -653,7 +666,7 @@ int tty_set_ldisc(struct tty_struct *tty, int ldisc)
 
        mutex_unlock(&tty->ldisc_mutex);
 
-       flush_scheduled_work();
+       tty_ldisc_flush_works(tty);
 
        retval = tty_ldisc_wait_idle(tty);
 
@@ -905,7 +918,7 @@ void tty_ldisc_release(struct tty_struct *tty, struct tty_struct *o_tty)
 
        tty_unlock();
        tty_ldisc_halt(tty);
-       flush_scheduled_work();
+       tty_ldisc_flush_works(tty);
        tty_lock();
 
        mutex_lock(&tty->ldisc_mutex);