• P
    n_tty: Fully initialize ldisc before restarting buffer work · b66f4fa5
    Peter Hurley 提交于
    Buffer work may already be pending when the n_tty ldisc is re-opened,
    eg., when setting the ldisc (via TIOCSETD ioctl) and when hanging up
    the tty. Since n_tty_set_room() may restart buffer work, first ensure
    the ldisc is completely initialized.
    
    Factor n_tty_set_room() out of reset_buffer_flags() (only 2 callers)
    and reorganize n_tty_open() to set termios last; buffer work will
    be restarted there if necessary, after the char_map is properly
    initialized.
    
    Fixes this WARNING:
    
    [  549.561769] ------------[ cut here ]------------
    [  549.598755] WARNING: at drivers/tty/n_tty.c:160 n_tty_set_room+0xff/0x130()
    [  549.604058] scheduling buffer work for halted ldisc
    [  549.607741] Pid: 9417, comm: trinity-child28 Tainted: G      D W 3.7.0-next-20121217-sasha-00023-g8689ef9 #219
    [  549.652580] Call Trace:
    [  549.662754]  [<ffffffff81c432cf>] ? n_tty_set_room+0xff/0x130
    [  549.665458]  [<ffffffff8110cae7>] warn_slowpath_common+0x87/0xb0
    [  549.668257]  [<ffffffff8110cb71>] warn_slowpath_fmt+0x41/0x50
    [  549.671007]  [<ffffffff81c432cf>] n_tty_set_room+0xff/0x130
    [  549.673268]  [<ffffffff81c44597>] reset_buffer_flags+0x137/0x150
    [  549.675607]  [<ffffffff81c45b71>] n_tty_open+0x131/0x1c0
    [  549.677699]  [<ffffffff81c47824>] tty_ldisc_open.isra.5+0x54/0x70
    [  549.680147]  [<ffffffff81c482bf>] tty_ldisc_hangup+0x11f/0x1e0
    [  549.682409]  [<ffffffff81c3fa17>] __tty_hangup+0x137/0x440
    [  549.684634]  [<ffffffff81c3fd49>] tty_vhangup+0x9/0x10
    [  549.686443]  [<ffffffff81c4a42c>] pty_close+0x14c/0x160
    [  549.688446]  [<ffffffff81c41225>] tty_release+0xd5/0x490
    [  549.690460]  [<ffffffff8127d8a2>] __fput+0x122/0x250
    [  549.692577]  [<ffffffff8127d9d9>] ____fput+0x9/0x10
    [  549.694534]  [<ffffffff811348c2>] task_work_run+0xb2/0xf0
    [  549.696349]  [<ffffffff81113c6d>] do_exit+0x36d/0x580
    [  549.698286]  [<ffffffff8107d964>] ? syscall_trace_enter+0x24/0x2e0
    [  549.702729]  [<ffffffff81113f4a>] do_group_exit+0x8a/0xc0
    [  549.706775]  [<ffffffff81113f92>] sys_exit_group+0x12/0x20
    [  549.711088]  [<ffffffff83cfab18>] tracesys+0xe1/0xe6
    [  549.728001] ---[ end trace 73eb41728f11f87e ]---
    Reported-by: NSasha Levin <levinsasha928@gmail.com>
    Signed-off-by: NPeter Hurley <peter@hurleysoftware.com>
    Signed-off-by: NGreg Kroah-Hartman <gregkh@linuxfoundation.org>
    b66f4fa5
n_tty.c 55.0 KB