um: race fix: initialize delayed_work *before* registering IRQ
authorAl Viro <viro@zeniv.linux.org.uk>
Sat, 10 Sep 2011 12:17:04 +0000 (08:17 -0400)
committerRichard Weinberger <richard@nod.at>
Sat, 24 Mar 2012 23:29:54 +0000 (00:29 +0100)
... since chan_interrupt() might schedule it if there's too much
incoming data.  Kill task argument of chan_interrupt(), while
we are at it - it's always &line->task.

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Richard Weinberger <richard@nod.at>
arch/um/drivers/chan.h
arch/um/drivers/chan_kern.c
arch/um/drivers/line.c

index 5078ec701c616386ab859fa81697725b9e5b7b63..d3974a91aa480d9e55bc76757774032b97d395a3 100644 (file)
@@ -27,7 +27,7 @@ struct chan {
        void *data;
 };
 
-extern void chan_interrupt(struct line *line, struct delayed_work *task,
+extern void chan_interrupt(struct line *line,
                           struct tty_struct *tty, int irq);
 extern int parse_chan_pair(char *str, struct line *line, int device,
                           const struct chan_opts *opts, char **error_out);
index 73d7bc018ab2c8b293b3c24ccee9cb1cd2c9295f..4744b5840fa97874f1bb7280778142b316eb5977 100644 (file)
@@ -146,12 +146,22 @@ void chan_enable_winch(struct chan *chan, struct tty_struct *tty)
                register_winch(chan->fd, tty);
 }
 
+static void line_timer_cb(struct work_struct *work)
+{
+       struct line *line = container_of(work, struct line, task.work);
+
+       if (!line->throttled)
+               chan_interrupt(line, line->tty, line->driver->read_irq);
+}
+
 int enable_chan(struct line *line)
 {
        struct list_head *ele;
        struct chan *chan;
        int err;
 
+       INIT_DELAYED_WORK(&line->task, line_timer_cb);
+
        list_for_each(ele, &line->chan_list) {
                chan = list_entry(ele, struct chan, list);
                err = open_one_chan(chan);
@@ -552,8 +562,7 @@ int parse_chan_pair(char *str, struct line *line, int device,
        return 0;
 }
 
-void chan_interrupt(struct line *line, struct delayed_work *task,
-                   struct tty_struct *tty, int irq)
+void chan_interrupt(struct line *line, struct tty_struct *tty, int irq)
 {
        struct chan *chan = line->chan_in;
        int err;
@@ -564,7 +573,7 @@ void chan_interrupt(struct line *line, struct delayed_work *task,
 
        do {
                if (tty && !tty_buffer_request_room(tty, 1)) {
-                       schedule_delayed_work(task, 1);
+                       schedule_delayed_work(&line->task, 1);
                        goto out;
                }
                err = chan->ops->read(chan->fd, &c, chan->data);
index 08f63930da8b0563688fe066c8da365d0e0d1741..9299b8a2d1717cd43bdffa37377f4b459dfcd929 100644 (file)
@@ -21,19 +21,10 @@ static irqreturn_t line_interrupt(int irq, void *data)
        struct line *line = chan->line;
 
        if (line)
-               chan_interrupt(line, &line->task, line->tty, irq);
+               chan_interrupt(line, line->tty, irq);
        return IRQ_HANDLED;
 }
 
-static void line_timer_cb(struct work_struct *work)
-{
-       struct line *line = container_of(work, struct line, task.work);
-
-       if (!line->throttled)
-               chan_interrupt(line, &line->task, line->tty,
-                              line->driver->read_irq);
-}
-
 /*
  * Returns the free space inside the ring buffer of this line.
  *
@@ -327,8 +318,7 @@ void line_unthrottle(struct tty_struct *tty)
        struct line *line = tty->driver_data;
 
        line->throttled = 0;
-       chan_interrupt(line, &line->task, tty,
-                      line->driver->read_irq);
+       chan_interrupt(line, tty, line->driver->read_irq);
 
        /*
         * Maybe there is enough stuff pending that calling the interrupt
@@ -424,8 +414,6 @@ int line_open(struct line *lines, struct tty_struct *tty)
        if (err) /* line_close() will be called by our caller */
                goto out_unlock;
 
-       INIT_DELAYED_WORK(&line->task, line_timer_cb);
-
        if (!line->sigio) {
                chan_enable_winch(line->chan_out, tty);
                line->sigio = 1;