Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751629AbbGHXfh (ORCPT ); Wed, 8 Jul 2015 19:35:37 -0400 Received: from mail-qk0-f174.google.com ([209.85.220.174]:35460 "EHLO mail-qk0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751014AbbGHXf2 (ORCPT ); Wed, 8 Jul 2015 19:35:28 -0400 Message-ID: <559DB3BE.5080306@hurleysoftware.com> Date: Wed, 08 Jul 2015 19:35:26 -0400 From: Peter Hurley User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 MIME-Version: 1.0 To: Taichi Kageyama , "gregkh@linuxfoundation.org" CC: "linux-serial@vger.kernel.org" , "jslaby@suse.cz" , "linux-kernel@vger.kernel.org" , "prarit@redhat.com" , Naoya Horiguchi Subject: Re: [PATCH 1/2] serial: 8250: Fix autoconfig_irq() to avoid race conditions References: <55717224.9060104@cp.jp.nec.com> <55717294.8020905@cp.jp.nec.com> In-Reply-To: <55717294.8020905@cp.jp.nec.com> 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: 1689 Lines: 34 On 06/05/2015 05:57 AM, Taichi Kageyama wrote: > 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. Reviewed-by: Peter Hurley -- 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/