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 X-Spam-Level: X-Spam-Status: No, score=-4.5 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 39651C4727C for ; Tue, 29 Sep 2020 20:53:36 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id AE41C2074F for ; Tue, 29 Sep 2020 20:53:35 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="iiabMY9S" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org AE41C2074F Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=baikalelectronics.ru Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:Message-ID: Subject:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=0qcV3FWnD/Qc0vS9uaYbAHW1QahaZCZ9bIg46yaOV94=; b=iiabMY9SkorgrHSiVe3wPSsJJ deroPt/009GvmmSTumoHT7cfD0xhbxu6t66X5hbZHv3j9h8sS1Ad6V8BzloPoET5eKBTVeUKtrzxY 0cF2C6qn8Sc0zo1isk+KJlKx0d2TCUsP1iNLFCB0W0+YBZmJqTubzm4+hc+xR28xczk16UcAWhjLf sBrnCTPdsGFzQ422yoKEjy5y40+Jg2ColiRRobbF3kbsfcDKzVdBjvOaDzEQUO48S/DAV5yxNjgCq zwOb1mO+jbbL94jtvaXVrxSXobIhAGaLOg9hWiFzBvcH24zJCsaV5XCF1Y53MgtO00z9l4dYH2KGP n+u35tVeA==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kNMba-0005kR-Lv; Tue, 29 Sep 2020 20:52:10 +0000 Received: from mail.baikalelectronics.com ([87.245.175.226] helo=mail.baikalelectronics.ru) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kNMbN-0005ec-Ls for linux-arm-kernel@lists.infradead.org; Tue, 29 Sep 2020 20:52:00 +0000 Received: from localhost (unknown [127.0.0.1]) by mail.baikalelectronics.ru (Postfix) with ESMTP id 0AA59803086A; Tue, 29 Sep 2020 20:51:50 +0000 (UTC) X-Virus-Scanned: amavisd-new at baikalelectronics.ru Received: from mail.baikalelectronics.ru ([127.0.0.1]) by localhost (mail.baikalelectronics.ru [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id r4WGwg0cF7DZ; Tue, 29 Sep 2020 23:51:49 +0300 (MSK) Date: Tue, 29 Sep 2020 23:51:45 +0300 From: Serge Semin To: Hans de Goede Subject: Re: [PATCH 0/3] serial: 8250_dw: Fix clk-notifier/port suspend deadlock Message-ID: <20200929205145.23itpxwns32uhjnx@mobilestation> References: <20200923161950.6237-1-Sergey.Semin@baikalelectronics.ru> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-ClientProxiedBy: MAIL.baikal.int (192.168.51.25) To mail (192.168.51.25) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200929_165157_945152_BC04BC0F X-CRM114-Status: GOOD ( 22.53 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: linux-kernel@vger.kernel.org, Will Deacon , Greg Kroah-Hartman , Russell King , Maxime Ripard , Alexey Malahov , Andy Shevchenko , Pavel Parkhomenko , linux-serial@vger.kernel.org, Andy Shevchenko , Jiri Slaby , linux-arm-kernel@lists.infradead.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hello, On Sun, Sep 27, 2020 at 05:01:52PM +0200, Hans de Goede wrote: > Hi, > > On 9/23/20 6:19 PM, Serge Semin wrote: > > Hans has discovered that there is a potential deadlock between the ref > > clock change notifier and the port suspension procedures {see the link at > > the bottom of the letter}. Indeed the deadlock is possible if the port > > suspension is initiated during the ref clock rate change: > > > > CPU0 (suspend CPU/UART) CPU1 (update clock) > > ---- ---- > > lock(&port->mutex); > > lock((work_completion)(&data->clk_work)); > > lock(&port->mutex); > > lock((work_completion)(&data->clk_work)); > > > > *** DEADLOCK *** > > > > So the CPU performing the UART port shutdown procedure will wait until the > > ref clock change notifier is finished (worker is flushed), while the later > > will wait for a port mutex being released. > > > > A possible solution to bypass the deadlock is to move the worker flush out > > of the critical section protected by the TTY port mutex. For instance we > > can register and de-register the clock change notifier in the port probe > > and remove methods instead of having them called from the port > > startup/shutdown callbacks. But in order to do that we need to make sure > > that the serial8250_update_uartclk() method is safe to be used while the > > port is shutted down. Alas the current implementation doesn't provide that > > safety. The solution described above is introduced in the framework of > > this patchset. See individual patches for details. > > > > Link: https://lore.kernel.org/linux-serial/f1cd5c75-9cda-6896-a4e2-42c5bfc3f5c3@redhat.com > > > > Hans, could you test the patchset out on your Cherry Trail (x86)-based > > devices? After that we can merge it in into the kernels 5.8 and 5.9 if > > there is no objections against the fix. > > Done, I can confirm that this fixes the lockdep issue for me, so you > can add my: > > Tested-by: Hans de Goede Great! Thank you very much. Greg, could you merge the series in if you have no objection against the solution design? Seeing the bug has been introduced together with the original series integrated in the kernel 5.9, the fix provided by this patchset will be only needed in 5.9. -Sergey > > To the entire series. > > Regards, > > Hans > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel