lkml.org 
[lkml]   [2012]   [Oct]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 21/21] TTY: move tty buffers to tty_port
    Hi guys,

    On 10/18/2012 04:26 PM, Jiri Slaby wrote:
    > So this is it. The big step why we did all the work over the past
    > kernel releases. Now everything is prepared, so nothing protects us
    > from doing that big step.
    >
    > | | \ \ nnnn/^l | |
    > | | \ / / | |
    > | '-,.__ => \/ ,-` => | '-,.__
    > | O __.´´) ( .` | O __.´´)
    > ~~~ ~~ `` ~~~ ~~
    > The buffers are now in the tty_port structure and we can start
    > teaching the buffer helpers (insert char/string, flip etc.) to use
    > tty_port instead of tty_struct all around.
    >
    > Signed-off-by: Jiri Slaby <jslaby@suse.cz>
    > ---

    Fuzzing with trinity inside a KVM tools (lkvm) guest with -next kernel
    uncovered the following warning:

    [ 1339.448706] ------------[ cut here ]------------
    [ 1339.451224] WARNING: at drivers/tty/tty_buffer.c:476 flush_to_ldisc+0x60/0x200()
    [ 1339.454272] tty is NULLPid: 7147, comm: kworker/4:0 Tainted: G W 3.7.0-rc2-next-20121025-sasha-00001-g673f98e-dirty #75
    [ 1339.458693] Call Trace:
    [ 1339.459410] [<ffffffff81bb1ea0>] ? flush_to_ldisc+0x60/0x200
    [ 1339.461289] [<ffffffff81109b86>] warn_slowpath_common+0x86/0xb0
    [ 1339.462992] [<ffffffff81109c11>] warn_slowpath_fmt+0x41/0x50
    [ 1339.464772] [<ffffffff81bb1ea0>] flush_to_ldisc+0x60/0x200
    [ 1339.467076] [<ffffffff8112d5a9>] process_one_work+0x3b9/0x770
    [ 1339.469501] [<ffffffff8112d458>] ? process_one_work+0x268/0x770
    [ 1339.472053] [<ffffffff8112dcc1>] ? worker_thread+0x51/0x3f0
    [ 1339.473831] [<ffffffff81bb1e40>] ? __tty_buffer_request_room+0x180/0x180
    [ 1339.475834] [<ffffffff8112df2a>] worker_thread+0x2ba/0x3f0
    [ 1339.478027] [<ffffffff8112dc70>] ? rescuer_thread+0x2d0/0x2d0
    [ 1339.480431] [<ffffffff81138c33>] kthread+0xe3/0xf0
    [ 1339.482383] [<ffffffff8117d7be>] ? put_lock_stats.isra.16+0xe/0x40
    [ 1339.484171] [<ffffffff81138b50>] ? insert_kthread_work+0x90/0x90
    [ 1339.485886] [<ffffffff83aedebc>] ret_from_fork+0x7c/0xb0
    [ 1339.487943] [<ffffffff81138b50>] ? insert_kthread_work+0x90/0x90
    [ 1339.490435] ---[ end trace e01a8b0af77894c4 ]---

    I'm guessing it happens because we never cancel the scheduled work when we
    free the buffer, so the scheduled work may run even after we freed the buffer.

    Besides the warning itself, I think that 'tty is NULL' would need a newline
    after it. Greg, should I send a patch for that?


    Thanks,
    Sasha
    --
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2012-10-26 05:21    [W:2.957 / U:0.020 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site