Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752568AbbKJXQE (ORCPT ); Tue, 10 Nov 2015 18:16:04 -0500 Received: from mail-io0-f176.google.com ([209.85.223.176]:32885 "EHLO mail-io0-f176.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752726AbbKJXP7 (ORCPT ); Tue, 10 Nov 2015 18:15:59 -0500 Subject: Re: 4.3 serial driver crashes with console shortly after boot To: Andi Kleen References: <20151110002939.GG3533@two.firstfloor.org> <56414DE9.20707@hurleysoftware.com> <20151110223957.GI3533@two.firstfloor.org> <20151110224340.GA30452@two.firstfloor.org> Cc: gregkh@linuxfoundation.org, linux-serial@vger.kernel.org, linux-kernel@vger.kernel.org From: Peter Hurley Message-ID: <56427AAC.3000409@hurleysoftware.com> Date: Tue, 10 Nov 2015 18:15:56 -0500 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 MIME-Version: 1.0 In-Reply-To: <20151110224340.GA30452@two.firstfloor.org> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2943 Lines: 95 On 11/10/2015 05:43 PM, Andi Kleen wrote: > On Tue, Nov 10, 2015 at 11:39:57PM +0100, Andi Kleen wrote: >>> I've just tried to reproduce this without success on my current >>> tree which has some additional patches I just posted this am. They weren't >>> intended to fix crashes but they directly impact the area of concern. Could >>> you try these three? >>> >>> [PATCH v2 2/4] n_tty: Ignore all read data when closing >>> [PATCH v2 3/4] tty: Abstract and encapsulate tty->closing behavior >>> [PATCH v2 4/4] tty: Remove drivers' extra tty_ldisc_flush() >>> >> Applying the three patches fixes the crash. >> I haven't tried to figure out which one did the trick. > > Actually I was wrong sorry. It still crashes, but now it doesn't > hang the system anymore. Argghh. Can you run the patch below and send me full dmesg (privately if you prefer)? Regards, Peter Hurley --- >% --- Subject: [DEBUG PATCH] tty: Turn on core debugging Signed-off-by: Peter Hurley --- drivers/tty/pty.c | 2 +- drivers/tty/tty_io.c | 2 +- drivers/tty/tty_ioctl.c | 2 +- drivers/tty/tty_ldisc.c | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/tty/pty.c b/drivers/tty/pty.c index 8ba5792..21219a3 100644 --- a/drivers/tty/pty.c +++ b/drivers/tty/pty.c @@ -25,7 +25,7 @@ #include #include -#undef TTY_DEBUG_HANGUP +#define TTY_DEBUG_HANGUP #ifdef TTY_DEBUG_HANGUP # define tty_debug_hangup(tty, f, args...) tty_debug(tty, f, ##args) #else diff --git a/drivers/tty/tty_io.c b/drivers/tty/tty_io.c index 2f8c21e..f3bd522 100644 --- a/drivers/tty/tty_io.c +++ b/drivers/tty/tty_io.c @@ -105,7 +105,7 @@ #include #include -#undef TTY_DEBUG_HANGUP +#define TTY_DEBUG_HANGUP #ifdef TTY_DEBUG_HANGUP # define tty_debug_hangup(tty, f, args...) tty_debug(tty, f, ##args) #else diff --git a/drivers/tty/tty_ioctl.c b/drivers/tty/tty_ioctl.c index b8c5c12..7970c94 100644 --- a/drivers/tty/tty_ioctl.c +++ b/drivers/tty/tty_ioctl.c @@ -24,7 +24,7 @@ #include #include -#undef TTY_DEBUG_WAIT_UNTIL_SENT +#define TTY_DEBUG_WAIT_UNTIL_SENT #ifdef TTY_DEBUG_WAIT_UNTIL_SENT # define tty_debug_wait_until_sent(tty, f, args...) tty_debug(tty, f, ##args) diff --git a/drivers/tty/tty_ldisc.c b/drivers/tty/tty_ldisc.c index ab0f559..760373a 100644 --- a/drivers/tty/tty_ldisc.c +++ b/drivers/tty/tty_ldisc.c @@ -19,7 +19,7 @@ #include #include -#undef LDISC_DEBUG_HANGUP +#define LDISC_DEBUG_HANGUP #ifdef LDISC_DEBUG_HANGUP #define tty_ldisc_debug(tty, f, args...) tty_debug(tty, f, ##args) -- 2.6.3 -- 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/