From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754219AbcAVQ2m (ORCPT ); Fri, 22 Jan 2016 11:28:42 -0500 Received: from www.linutronix.de ([62.245.132.108]:47111 "EHLO Galois.linutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754157AbcAVQ2j (ORCPT ); Fri, 22 Jan 2016 11:28:39 -0500 From: John Ogness To: gregkh@linuxfoundation.org Cc: peter@hurleysoftware.com Cc: bigeasy@linutronix.de Cc: tony@atomide.com Cc: nsekhar@ti.com Cc: peter.ujfalusi@ti.com Cc: linux-serial@vger.kernel.org Cc: linux-kernel@vger.kernel.org Subject: [PATCH REPOST] tty: serial: 8250: omap: restore registers on shutdown Date: Fri, 22 Jan 2016 17:28:34 +0100 Message-ID: <87vb6llgst.fsf@linutronix.de> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/23.4 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Linutronix-Spam-Score: -1.0 X-Linutronix-Spam-Level: - X-Linutronix-Spam-Status: No , -1.0 points, 5.0 required, ALL_TRUSTED=-1,SHORTCIRCUIT=-0.0001,URIBL_BLOCKED=0.001 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org If DMA is active during a shutdown, a delayed restore of the registers may be pending. The restore must be performed after the DMA is stopped, otherwise the delayed restore remains pending and will fire upon the first DMA TX complete of a totally different serial session. Signed-off-by: John Ogness --- This patch was originally posted and discussed here: http://lkml.kernel.org/r/87egjp2r4a.fsf@linutronix.de The patch was not accepted because Peter expressed dissatisfaction with the entire deferred implementation. (This patch fixes a bug in that implementation.) However, no alternative solution was determined and the bug continues to exist. Although we can certainly continue to debate about an alternate solution to the deferred implementation, I think it makes sense to at least fix the existing bug until an alternative exists. The bug can be easily produced with a `kill -9` on a process transmitting on the serial port. patch against next-20160122 drivers/tty/serial/8250/8250_omap.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/drivers/tty/serial/8250/8250_omap.c b/drivers/tty/serial/8250/8250_omap.c index a2c0734..0ffec67 100644 --- a/drivers/tty/serial/8250/8250_omap.c +++ b/drivers/tty/serial/8250/8250_omap.c @@ -664,9 +664,15 @@ static void omap_8250_shutdown(struct uart_port *port) up->ier = 0; serial_out(up, UART_IER, 0); - if (up->dma) + if (up->dma) { serial8250_release_dma(up); + if (priv->delayed_restore) { + priv->delayed_restore = 0; + omap8250_restore_regs(up); + } + } + /* * Disable break condition and FIFOs */ -- 1.7.10.4