From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from picard.linux.it (picard.linux.it [213.254.12.146]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 4B1A7C4321E for ; Mon, 5 Dec 2022 11:14:02 +0000 (UTC) Received: from picard.linux.it (localhost [IPv6:::1]) by picard.linux.it (Postfix) with ESMTP id 38BBC3CC213 for ; Mon, 5 Dec 2022 12:14:00 +0100 (CET) Received: from in-5.smtp.seeweb.it (in-5.smtp.seeweb.it [217.194.8.5]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-384)) (No client certificate requested) by picard.linux.it (Postfix) with ESMTPS id D6CFE3CC44E for ; Thu, 1 Dec 2022 19:52:59 +0100 (CET) Received: from mail.enpas.org (zhong.enpas.org [IPv6:2a03:4000:2:537::1]) by in-5.smtp.seeweb.it (Postfix) with ESMTP id 7E6CE644E88 for ; Thu, 1 Dec 2022 19:52:56 +0100 (CET) Received: from [127.0.0.1] (localhost [127.0.0.1]) by mail.enpas.org (Postfix) with ESMTPSA id A3484100AD8; Thu, 1 Dec 2022 18:52:49 +0000 (UTC) Date: Fri, 2 Dec 2022 03:52:42 +0900 From: Max Staudt To: "Jiri Slaby (SUSE)" Message-ID: <20221202035242.155d54f4.max@enpas.org> In-Reply-To: <20221201073426.17328-1-jirislaby@kernel.org> References: <20221201073426.17328-1-jirislaby@kernel.org> MIME-Version: 1.0 X-Virus-Scanned: clamav-milter 0.102.4 at in-5.smtp.seeweb.it X-Virus-Status: Clean X-Mailman-Approved-At: Mon, 05 Dec 2022 12:13:38 +0100 Subject: Re: [LTP] [PATCH] can: slcan: fix freed work crash X-BeenThere: ltp@lists.linux.it X-Mailman-Version: 2.1.29 Precedence: list List-Id: Linux Test Project List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Paolo Abeni , netdev@vger.kernel.org, linux-can@vger.kernel.org, linux-kernel@vger.kernel.org, stable@vger.kernel.org, Petr Vorel , Eric Dumazet , Marc Kleine-Budde , ltp@lists.linux.it, linux-serial@vger.kernel.org, Jakub Kicinski , dario.binacchi@amarulasolutions.com, "David S. Miller" , Wolfgang Grandegger , Richard Palethorpe Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ltp-bounces+ltp=archiver.kernel.org@lists.linux.it Sender: "ltp" (CC: ltp@lists.linux.it because Petr did so.) Hi Jiry, Thanks for finding this! Your patch looks correct to me, so please have a Reviewed-by: Max Staudt for both this patch to slcan, as well as an 1:1 patch to can327. Some history: This is actually my code from can327, which was backported to slcan as part of Dario's larger modernisation effort. The rationale for moving it was to flush the UART TX buffer in case of ndo_close(), in order to bring the device into a more predictable state between ndo_close() and ndo_open(). I guess that's actually counterproductive - whatever is in the TX buffer at that time should likely be fully sent. For example, can327 sends one last byte to abort any running chatty monitoring mode from the adapter. So your patch also fixes this as well ;) Of course, this resulted in calling flush_worker() in both ndo_stop() and ldisc_close(), so I wanted to avoid code duplication, and relied on ndo_stop(). Oops. Thanks, Max -- Mailing list info: https://lists.linux.it/listinfo/ltp