[PATCH 1/2] serial: 8250: Fix autoconfig_irq() to avoid race conditions

[Taichi Kageyama <t-kageyama@cp.jp.nec.com>]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8", Size: 2373 bytes --]

The following race conditions can happen if a serial is used as console.
  Case1. CPU_B handles an interrupt from a serial
    autoconfig_irq() fails whether the interrupt is raised or not
    if CPU_B is disabled to handle interrupts for longer than it expects.
  Case2. CPU_B clears UART_IER just after CPU_A sets UART_IER
    A serial may not make an interrupt.
    autoconfig_irq() can fail if the interrupt is not raised.
  Case3. CPU_A sets UART_IER just after CPU_B clears UART_IER
    This is an unexpected behavior for uart_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,)
  -----------------------------------------------------------------

If autoconfig_irq() fails, the console doesn't work in interrupt mode,
the mode cannot be changed anymore, and "input overrun"
(which can make operation mistakes) happens easily.
This problem happens with high rate every boot once it occurs
because the boot sequence is always almost same.

Signed-off-by: Taichi Kageyama <t-kageyama@cp.jp.nec.com>
Cc: Naoya Horiguchi <n-horiguchi@ah.jp.nec.com>
---
  drivers/tty/serial/8250/8250_core.c |    6 ++++++
  1 files changed, 6 insertions(+), 0 deletions(-)

diff --git a/drivers/tty/serial/8250/8250_core.c b/drivers/tty/serial/8250/8250_core.c
index 4506e40..6bf31f2 100644
--- a/drivers/tty/serial/8250/8250_core.c
+++ b/drivers/tty/serial/8250/8250_core.c
@@ -1309,6 +1309,9 @@ static void autoconfig_irq(struct uart_8250_port *up)
  	unsigned long irqs;
  	int irq;

+	if (uart_console(port))
+		console_lock();
+
  	if (port->flags & UPF_FOURPORT) {
  		ICP = (port->iobase & 0xfe0) | 0x1f;
  		save_ICP = inb_p(ICP);
@@ -1347,6 +1350,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;
  }

-- 
1.7.1

ÿôèº{.nÇ+‰·Ÿ®‰­†+%ŠËÿ±éݶ\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dʇڙë,j\a­¢f£¢·hšïêÿ‘êçz_è®\x03(­éšŽŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨è­Ú&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥