Commit 7b292b4b authored by Andrew McGregor's avatar Andrew McGregor Committed by Greg Kroah-Hartman

tty: fix "IRQ45: nobody cared"

Unthrottling the TTY during close ends up enabling interrupts
on a device not on the active list, which will never have the
interrupts cleared.  Doctor, it hurts when I do this.

>>> On 6/2/2011 at 01:56 AM, in message <20110601145608.3e586e16@bob.linux.org.uk>, Alan Cox <alan@linux.intel.com> wrote:
> On Wed, 01 Jun 2011 10:34:07 +1200
> "andrew mcgregor" <andrew.mcgregor@alliedtelesis.co.nz> wrote:
> > The LKML message
> > http://kerneltrap.org/mailarchive/linux-kernel/2010/2/25/4541847 from
> > February doesn't seem to have been resolved since.  We struck the
> > issue, and the patch below (against 2.6.32) fixes it.  Should I
> > supply a patch against 3.0.0rc?
>
> I think that would be sensible. I don't actually see how you hit it as
> the IRQ ought to be masked by then but it's certainly wrong for n_tty
> to be calling into check_unthrottle at that point.
>
> So yes please send a patch with a suitable Signed-off-by: line to
> linux-serial and cc GregKH <greg@kroah.com> as well.
>
> Alan
Signed-off-by: default avatarAndrew McGregor <andrew.mcgregor@alliedtelesis.co.nz>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent 8168e9c2
...@@ -185,7 +185,6 @@ static void reset_buffer_flags(struct tty_struct *tty) ...@@ -185,7 +185,6 @@ static void reset_buffer_flags(struct tty_struct *tty)
tty->canon_head = tty->canon_data = tty->erasing = 0; tty->canon_head = tty->canon_data = tty->erasing = 0;
memset(&tty->read_flags, 0, sizeof tty->read_flags); memset(&tty->read_flags, 0, sizeof tty->read_flags);
n_tty_set_room(tty); n_tty_set_room(tty);
check_unthrottle(tty);
} }
/** /**
...@@ -1587,6 +1586,7 @@ static int n_tty_open(struct tty_struct *tty) ...@@ -1587,6 +1586,7 @@ static int n_tty_open(struct tty_struct *tty)
return -ENOMEM; return -ENOMEM;
} }
reset_buffer_flags(tty); reset_buffer_flags(tty);
tty_unthrottle(tty);
tty->column = 0; tty->column = 0;
n_tty_set_termios(tty, NULL); n_tty_set_termios(tty, NULL);
tty->minimum_to_wake = 1; tty->minimum_to_wake = 1;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment