Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753281AbbHOAKm (ORCPT ); Fri, 14 Aug 2015 20:10:42 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:33532 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751722AbbHOAKj (ORCPT ); Fri, 14 Aug 2015 20:10:39 -0400 Date: Fri, 14 Aug 2015 17:10:39 -0700 From: "gregkh@linuxfoundation.org" To: Taichi Kageyama 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 Message-ID: <20150815001039.GA4885@kroah.com> References: <1439165707-2082-1-git-send-email-t-kageyama@cp.jp.nec.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1439165707-2082-1-git-send-email-t-kageyama@cp.jp.nec.com> User-Agent: Mutt/1.5.23+102 (2ca89bed6448) (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2939 Lines: 73 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? thanks, greg k-h -- 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/