Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752245AbbHQASV (ORCPT ); Sun, 16 Aug 2015 20:18:21 -0400 Received: from TYO202.gate.nec.co.jp ([210.143.35.52]:40962 "EHLO tyo202.gate.nec.co.jp" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751440AbbHQAST convert rfc822-to-8bit (ORCPT ); Sun, 16 Aug 2015 20:18:19 -0400 From: Taichi Kageyama To: "gregkh@linuxfoundation.org" CC: "tglx@linutronix.de" , "peter@hurleysoftware.com" , "linux-serial@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "jslaby@suse.cz" , "prarit@redhat.com" , Naoya Horiguchi , "jiang.liu@linux.intel.com" Subject: Re: [PATCH v3] serial: 8250: Fix autoconfig_irq() to avoid race conditions Thread-Topic: [PATCH v3] serial: 8250: Fix autoconfig_irq() to avoid race conditions Thread-Index: AQHQ0wGw5XO1sCOvyUqQSyujR/JPsp4LoL6AgAMmDAA= Date: Mon, 17 Aug 2015 00:15:50 +0000 Message-ID: <55D127A8.9080607@cp.jp.nec.com> References: <1439165707-2082-1-git-send-email-t-kageyama@cp.jp.nec.com> <20150815001039.GA4885@kroah.com> In-Reply-To: <20150815001039.GA4885@kroah.com> Accept-Language: en-US, ja-JP Content-Language: ja-JP X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.34.108.73] Content-Type: text/plain; charset="iso-2022-jp" Content-ID: Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3137 Lines: 79 Hi Greg, On 2015/08/15 9:10, gregkh@linuxfoundation.org wrote: > On Mon, Aug 10, 2015 at 12:15:40AM +0000, Taichi Kageyama wrote: >> The following race conditions can happen when a serial port is used >> as console. >> >> Case1: CPU_B is used to detect an interrupt from a serial port, >> but it can have interrupts disabled during the waiting time. >> Case2: CPU_B clears UART_IER just after CPU_A sets UART_IER and then >> a serial port may not make an interrupt. >> Case3: CPU_A sets UART_IER just after CPU_B clears UART_IER. >> This is an unexpected behavior for serial8250_console_write(). >> >> CPU_A [autoconfig_irq] | CPU_B [serial8250_console_write] >> ----------------------------|--------------------------------------- >> | >> probe_irq_on() | spin_lock_irqsave(&port->lock,) >> serial_outp(,UART_IER,0x0f) | serial_out(,UART_IER,0) >> udelay(20); | uart_console_write() >> probe_irq_off() | >> | spin_unlock_irqrestore(&port->lock,) >> >> Case1 and 2 can make autoconfig_irq() failed. >> In these cases, the console doesn't work in interrupt mode and >> "input overrun" (which can make operation mistakes) can happen >> on some systems. Especially in the Case1, It is known that the >> problem happens with high rate every boot once it occurs >> because the boot sequence is always almost same. >> >> port mutex makes sure that the autoconfig operation is exclusive of >> any other concurrent HW access except by the console operation. >> console lock is required in autoconfig_irq(). >> >> Signed-off-by: Taichi Kageyama >> Cc: Naoya Horiguchi >> Reviewed-by: Peter Hurley >> --- >> drivers/tty/serial/8250/8250_core.c | 6 ++++++ >> 1 file changed, 6 insertions(+) >> >> diff --git v4.2-rc4.org/drivers/tty/serial/8250/8250_core.c v4.2-rc4.work/drivers/tty/serial/8250/8250_core.c >> index 37fff12..ed1e23e 100644 >> --- v4.2-rc4.org/drivers/tty/serial/8250/8250_core.c >> +++ v4.2-rc4.work/drivers/tty/serial/8250/8250_core.c >> @@ -1303,6 +1303,9 @@ static void autoconfig_irq(struct uart_8250_port *up) >> inb_p(ICP); >> } >> >> + if (uart_console(port)) >> + console_lock(); >> + >> /* forget possible initially masked and pending IRQ */ >> probe_irq_off(probe_irq_on()); >> save_mcr = serial_in(up, UART_MCR); >> @@ -1334,6 +1337,9 @@ static void autoconfig_irq(struct uart_8250_port *up) >> if (port->flags & UPF_FOURPORT) >> outb_p(save_ICP, ICP); >> >> + if (uart_console(port)) >> + console_unlock(); >> + >> port->irq = (irq > 0) ? irq : 0; >> } > > This doesn't apply to my tty-next tree at all. Can you please rebase it > and resend? Oh, I didn't know the file name was changed in tty-next. I'll rebase it. Thanks, Taichi -- 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/