Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:3 as permitted sender) client-ip=2620:137:e000::3:3;
From:   Paul Geurts <paul_geurts@live.nl>
To:     gregkh@linuxfoundation.org, jirislaby@kernel.org,
        shawnguo@kernel.org, s.hauer@pengutronix.de, kernel@pengutronix.de,
        festevam@gmail.com, linux-imx@nxp.com,
        linux-kernel@vger.kernel.org, linux-serial@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org
Cc:     Paul Geurts <paul_geurts@live.nl>
Subject: [PATCH] serial: imx: fix tx statemachine deadlock
Date:   Wed,  4 Oct 2023 11:57:22 +0200
Message-ID: <AM0PR09MB26750E782F81CD447058FE7D95CBA@AM0PR09MB2675.eurprd09.prod.outlook.com>
Content-Transfer-Encoding: 7BIT
Content-Type:   text/plain; charset=US-ASCII
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
X-MS-PublicTrafficType: Email
X-MS-TrafficTypeDiagnostic: AM0PR09MB2675:EE_|AM0PR09MB3585:EE_
X-MS-Office365-Filtering-Correlation-Id: 4360296e-620b-4187-1c46-08dbc4c05c2f
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info: eJnSjbvZLyvd2L69cQII9KTPBZu5jkZ+QAorh5cByU29CiaGvgM4vJQXo0WCRmwa8vWS5VjTBH8sH5mvSgb8XCAXC5weoYCZZ+bcKsLJGO4LJQ5x1YxIb7X2rJ2piX/HxKapG96MHB2LUQ9lBJczCGZI1vpYk0ase+QqcdqkzibCZSBoqELMYWG0C8Bckrph6hoE2QLu99InMliqfOCPNZwfaYR/X56SavM51zIRBo8yQ6wzUw2l71WkgWIcM05L5Cd2m2ODRT17J+o1Lo7h/Al7aklvzzfTxNVTLTSAABS/Njes2+yQw2lwkSnqoaeVuajLyTJRSE9Gq27u3WejiPAKhpWGVY1rHymtfjbyAikmOqQ6po86i8JpLwgkKA/qRH1K537fmWi/5E1DFa+4sf0S3zftY9LbVKV+VNiYO+/BV45x3J2CW/WSAZMJOKuDJhzxRkM8Vcv9OA47bjf8kx8YimiVNQky4e5KiOOTQnC5H7CCoxAeG3sM/qhIGhv7ARS0h492EkX/y42rP7rQAz63o4aWt1gqQvyc8rOVAOBdgOJsIN/gWUkVVEi+j9SMr7I+OCJHWsXs6hsn7ZIh1Apwbq85KaPzD74FcZGxXTRjPfzkCw8nvVTcp3LKnZna
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?OSF591OI80e5QgUDtvqQo1SqW08WQWE4rWx47b9UmIakqiPunTpfEbEGYSPR?=
 =?us-ascii?Q?0swFoYvEA6ClTHqtV64e0oNk5DOUu+iQxK8PWPx5EsRQhmnnKW8loC3WWvgM?=
 =?us-ascii?Q?GcL9MG3GJfeYo5md+R+1WNK/8oNP6SqoP9LResw4cSy9MIwkO9GatZuRgOK4?=
 =?us-ascii?Q?BAT6ejOkKa7QIT3FQ81ftnOK4/IkdzwEX4fPJY2FcB7bOAA9Sez0KhPFqv6p?=
 =?us-ascii?Q?yGFqgjNT8qaQu4OfZMQBW7fKUhR1Vy2qklhKcd7AysyAkZg+TVtGUi5xA2+W?=
 =?us-ascii?Q?AXbJlpuEmHW35bHDjFVfjqUITNHDSNxRg14ZCqUmperGLLhWYqXOEo6bOjfn?=
 =?us-ascii?Q?cCSRWZXTy0JglzxQ8+mEr4V66R9FBTm5XdKvS8k+4UnmGNcUxkxcVo1eHBlW?=
 =?us-ascii?Q?yCqajkzvPx2bo+xkU1MVN51N8N8iTX6Jk773aASg3DxyUWvbHaGp4IzKlGOB?=
 =?us-ascii?Q?+zSc5mbZ1LjrsBWeYMiYiqAB3EBqQiidtxp79EanHvwqASKNRp6donRkrLk3?=
 =?us-ascii?Q?ISc7T9PNsikTHfVCTnpSk3LdFKX++PnnVOw88t/wWwTSOKVIj96UPU0gFuDj?=
 =?us-ascii?Q?MFLYbg7R5938yF1Qh1jNWn14TOHBSZxgxFfNFyw/SczJ/VXYdVNUVTyNwaOu?=
 =?us-ascii?Q?94OpknzJNw91hYPn7KbdfVq/4tX9mFXMcyArb8bwMjRgncJMCW2pio+5wOZq?=
 =?us-ascii?Q?Ba1+z6uHzGWlCdcF3BzmVb37c1ZMjhPcLB6ESsMYGfizvCwKdpWlTvCsFO1c?=
 =?us-ascii?Q?r31+jBnfjH1FSkHFpcWOGCzjMvYWDo2f3xbSjbC8shDbySo6QB9jWGXn+bxp?=
 =?us-ascii?Q?Suj7xJoIzDOqvBVKWEVEczDIZHPjoqUsOGOrIobpmO0RytiA/RBK1ENgeXbi?=
 =?us-ascii?Q?PWigzPX9TvRxuneSzuGSBX9JTiRzhESFpmwJGiiX63k4CcHCwTHCNEafI5k7?=
 =?us-ascii?Q?KdcRgLmNsmDHYOmv9qPOSXxex40iZ/0v5jj7jmKSi/4DsIzYHro49aFrnRWG?=
 =?us-ascii?Q?X7mN/yg2b3IIxb4HLsjxRF82PWdFzY1nLTy444O00c2aDQSVSUJ1HQWY3gCz?=
 =?us-ascii?Q?ncRzmbhg+jBl5wFFAIBrgjpJ96zdR82Ydj2YvQsgOP5o404uNColUoRwtK/W?=
 =?us-ascii?Q?rMVhjbr3EYwn3E6+qyZUsmWPaJAzN6COJIjvVWa3JdrpTGy0gb5wvDAHBQyi?=
 =?us-ascii?Q?7jtFeB0r3WkYY+PwYlSM6L5zPdgT8KA/hRNVATvNXae3kvln10svlls8ipwY?=
 =?us-ascii?Q?9eWkVb1QMdBf6FBPBYHChkBjdf+VTxsTlQgWrin/oQ=3D=3D?=
X-OriginatorOrg: sct-15-20-4755-11-msonline-outlook-64da6.templateTenant
X-MS-Exchange-CrossTenant-Network-Message-Id: 4360296e-620b-4187-1c46-08dbc4c05c2f
X-MS-Exchange-CrossTenant-AuthSource: AM0PR09MB2675.eurprd09.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Oct 2023 09:57:46.7393
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa
X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM0PR09MB3585
X-Spam-Status: No, score=-0.5 required=5.0 tests=FREEMAIL_FORGED_FROMDOMAIN,
	FREEMAIL_FROM,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,
	SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no
	version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lipwig.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (lipwig.vger.email [0.0.0.0]); Wed, 04 Oct 2023 02:58:15 -0700 (PDT)

When using the serial port as RS485 port, the tx statemachine is used to
control the RTS pin to drive the RS485 transceiver TX_EN pin. When the
TTY port is closed in the middle of a transmission (for instance during
userland application crash), imx_uart_shutdown disables the interface
and disables the Transmission Complete interrupt. afer that,
imx_uart_stop_tx bails on an incomplete transmission, to be retriggered
by the TC interrupt. This interrupt is disabled and therefore the tx
statemachine never transitions out of SEND. The statemachine is in
deadlock now, and the TX_EN remains low, making the interface useless.

imx_uart_stop_tx now checks for incomplete transmission AND whether TC
interrupts are enabled before bailing to be retriggered. This makes sure
the state machine handling is reached, and is properly set to
WAIT_AFTER_SEND.

Signed-off-by: Paul Geurts <paul_geurts@live.nl>
---
 drivers/tty/serial/imx.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/tty/serial/imx.c b/drivers/tty/serial/imx.c
index 13cb78340709..90a4b7841030 100644
--- a/drivers/tty/serial/imx.c
+++ b/drivers/tty/serial/imx.c
@@ -421,13 +421,13 @@ static void imx_uart_stop_tx(struct uart_port *port)
 	ucr1 = imx_uart_readl(sport, UCR1);
 	imx_uart_writel(sport, ucr1 & ~UCR1_TRDYEN, UCR1);
 
+	ucr4 = imx_uart_readl(sport, UCR4);
 	usr2 = imx_uart_readl(sport, USR2);
-	if (!(usr2 & USR2_TXDC)) {
+	if ((!(usr2 & USR2_TXDC)) && (ucr4 & UCR4_TCEN)) {
 		/* The shifter is still busy, so retry once TC triggers */
 		return;
 	}
 
-	ucr4 = imx_uart_readl(sport, UCR4);
 	ucr4 &= ~UCR4_TCEN;
 	imx_uart_writel(sport, ucr4, UCR4);
 
-- 
2.20.1