From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paul Gortmaker Subject: Re: [PATCH] 8250: add workaround for MPC8[356]xx UART break IRQ storm Date: Thu, 24 Nov 2011 10:26:59 -0500 Message-ID: <4ECE6243.3020604@windriver.com> References: <1267212301-26851-1-git-send-email-paul.gortmaker@windriver.com> <82B7F47D-C7BF-4BD1-9982-7E7AEDC23530@kernel.crashing.org> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Return-path: Received: from mail.windriver.com ([147.11.1.11]:42283 "EHLO mail.windriver.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752024Ab1KXP1J (ORCPT ); Thu, 24 Nov 2011 10:27:09 -0500 In-Reply-To: <82B7F47D-C7BF-4BD1-9982-7E7AEDC23530@kernel.crashing.org> Sender: linux-serial-owner@vger.kernel.org List-Id: linux-serial@vger.kernel.org To: Kumar Gala Cc: linuxppc-dev@lists.ozlabs.org, linux-serial@vger.kernel.org On 11-11-24 03:14 AM, Kumar Gala wrote: > > On Feb 26, 2010, at 1:25 PM, Paul Gortmaker wrote: > >> Sending a break on the SOC UARTs found in some MPC83xx/85xx/86xx >> chips seems to cause a short lived IRQ storm (/proc/interrupts >> typically shows somewhere between 300 and 1500 events). Unfortunately >> this renders SysRQ over the serial console completely inoperable. >> Testing with obvious things like ACKing the event doesn't seem to >> change anything vs. a completely dumb approach of just ignoring >> it and waiting for it to stop, so that is what is implemented here. >> >> Signed-off-by: Paul Gortmaker >> --- >> >> This is a refresh of a patch I'd done earlier -- I've tried to make >> the bug support as generic as possible to minimize having board >> specific ifdef crap in 8250.c -- any suggestions on how to further >> improve it are welcome. >> >> drivers/serial/8250.c | 6 ++++++ >> drivers/serial/8250.h | 20 ++++++++++++++++++++ >> drivers/serial/Kconfig | 14 ++++++++++++++ >> include/linux/serial_reg.h | 2 ++ >> 4 files changed, 42 insertions(+), 0 deletions(-) > > Did we ever decide what to do with this or trying to get it accepted upstream? That is an old version. ScottW gave me the errata information which allowed me to fix the problem in a cleaner way. http://patchwork.ozlabs.org/patch/46609/ I think the above version is OK as-is; the only thing I think we could do to improve it is to go and automatically select the thing based on known impacted CPU types (which could be a separate commit or commits, as various CPUs are confirmed to have the issue.) Paul. > > - k