Received: by 2002:a05:6a10:af89:0:0:0:0 with SMTP id iu9csp3600993pxb; Mon, 24 Jan 2022 13:14:28 -0800 (PST) X-Google-Smtp-Source: ABdhPJxFskqGTnXO78IBA9dL/C5Ci+QQ3v10ynp/8Qfs8kyWF/8niRQwIxHp0xsNr36ei4tWIdRu X-Received: by 2002:aa7:88c9:0:b0:4c9:319e:ecb7 with SMTP id k9-20020aa788c9000000b004c9319eecb7mr5803803pff.58.1643058868370; Mon, 24 Jan 2022 13:14:28 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1643058868; cv=none; d=google.com; s=arc-20160816; b=AnTdJfIhiOm0Sf1CLfWGc+32zzsmy8Vj7cXBKYZsbZjyMokyscwihdIuML/tt4q6BP d1tBt2E2+LX5sU1Zeo9zoHUuF0k0iZa7Hsj+6XJbvgsvDFG/XOrbl0qmmOyybAro99Et XKbjCTCIapDHXG8CKbYjciIxx5hV9TVDE4I48L50eQR3C/WOovbmZ5o3kPug7/+q9WGo jBGIzOBC0CWURuXN7G8BNy9pQsy0sPMdi0MleOz0WAC5U4YFxby9RY7751ktakcT4fX8 6zx3ddZf/X9UdNcLjo5PtUC9a7zZ8Fxx7DEewehrFxceK2RseoK31l0mN3wvvc+uBVT+ 1cyg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :user-agent:references:in-reply-to:message-id:date:subject:cc:to :from:dkim-signature; bh=KZfhRDl+YbLTMq/ZsV9VB+RfNW4RAISUmk/OCEIvMcw=; b=JNC9kPdQEN+IQq+QGyD5uAjFe2Qiptb9EdZzYn5PWPQP9GWAy7fu6oyTL1iObxu/y3 tcq1YiZ1FfvtdoNNJfIk18xAJ6IRlYnOMR/5DZHfHcc1lSkkxUml3SJlks8G4qd5+PUV fSntsrMmaa1fxoSYf5+7vQiCmGNPm07pqA1knfnj7zzDmwToUaHUcWDTcQ3+oVlbPhpf bskR/tLY1XTBx6fhltIZ/2CjxC4jVmlTY8pizF1Cyr8dNXZ1bOgnr79XD1V3x36xLaXN irIAr81n5MUVxQt5FnFrexc3jR5UtTQCgknUkKDMQ9BcQt5zIz47Dfx4GgLVxFBfR0l3 +okA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linuxfoundation.org header.s=korg header.b=fhuRwTXJ; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linuxfoundation.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id u5si14423476pga.242.2022.01.24.13.14.14; Mon, 24 Jan 2022 13:14:28 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@linuxfoundation.org header.s=korg header.b=fhuRwTXJ; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linuxfoundation.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1358334AbiAXUmD (ORCPT + 99 others); Mon, 24 Jan 2022 15:42:03 -0500 Received: from dfw.source.kernel.org ([139.178.84.217]:46800 "EHLO dfw.source.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1349067AbiAXUWs (ORCPT ); Mon, 24 Jan 2022 15:22:48 -0500 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 579CC61502; Mon, 24 Jan 2022 20:22:48 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 2F8E8C340E5; Mon, 24 Jan 2022 20:22:46 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1643055767; bh=RkngHeAeVCkGRFWJ2UEUeqVNrmIJmwasCnUlDu8quqk=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=fhuRwTXJ/vX6b3dKAjx5k0ucZ/OjqlAfwNJcI2WmPeJ4qZRAnGqBCDmuUmZvoVK3a t0JGgpia5CMAY1EDEvQ2zztQVSbeoAKFqVwqX3PGTVgtTYALDHY2WQAtFugNJZsGAR ndAL4gNLQmyKqkdAOwWdzPX0Q3ObMLCKUzsYHv8U= From: Greg Kroah-Hartman To: linux-kernel@vger.kernel.org Cc: Greg Kroah-Hartman , stable@vger.kernel.org, Douglas Anderson , TOTE Robot , Brian Norris , Kalle Valo , Sasha Levin Subject: [PATCH 5.15 226/846] mwifiex: Fix possible ABBA deadlock Date: Mon, 24 Jan 2022 19:35:43 +0100 Message-Id: <20220124184108.712569914@linuxfoundation.org> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20220124184100.867127425@linuxfoundation.org> References: <20220124184100.867127425@linuxfoundation.org> User-Agent: quilt/0.66 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Brian Norris [ Upstream commit 1b8bb8919ef81bfc8873d223b9361f1685f2106d ] Quoting Jia-Ju Bai : mwifiex_dequeue_tx_packet() spin_lock_bh(&priv->wmm.ra_list_spinlock); --> Line 1432 (Lock A) mwifiex_send_addba() spin_lock_bh(&priv->sta_list_spinlock); --> Line 608 (Lock B) mwifiex_process_sta_tx_pause() spin_lock_bh(&priv->sta_list_spinlock); --> Line 398 (Lock B) mwifiex_update_ralist_tx_pause() spin_lock_bh(&priv->wmm.ra_list_spinlock); --> Line 941 (Lock A) Similar report for mwifiex_process_uap_tx_pause(). While the locking expectations in this driver are a bit unclear, the Fixed commit only intended to protect the sta_ptr, so we can drop the lock as soon as we're done with it. IIUC, this deadlock cannot actually happen, because command event processing (which calls mwifiex_process_sta_tx_pause()) is sequentialized with TX packet processing (e.g., mwifiex_dequeue_tx_packet()) via the main loop (mwifiex_main_process()). But it's good not to leave this potential issue lurking. Fixes: f0f7c2275fb9 ("mwifiex: minor cleanups w/ sta_list_spinlock in cfg80211.c") Cc: Douglas Anderson Reported-by: TOTE Robot Link: https://lore.kernel.org/linux-wireless/0e495b14-efbb-e0da-37bd-af6bd677ee2c@gmail.com/ Signed-off-by: Brian Norris Reviewed-by: Douglas Anderson Signed-off-by: Kalle Valo Link: https://lore.kernel.org/r/YaV0pllJ5p/EuUat@google.com Signed-off-by: Sasha Levin --- drivers/net/wireless/marvell/mwifiex/sta_event.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/net/wireless/marvell/mwifiex/sta_event.c b/drivers/net/wireless/marvell/mwifiex/sta_event.c index 68c63268e2e6b..2b2e6e0166e14 100644 --- a/drivers/net/wireless/marvell/mwifiex/sta_event.c +++ b/drivers/net/wireless/marvell/mwifiex/sta_event.c @@ -365,10 +365,12 @@ static void mwifiex_process_uap_tx_pause(struct mwifiex_private *priv, sta_ptr = mwifiex_get_sta_entry(priv, tp->peermac); if (sta_ptr && sta_ptr->tx_pause != tp->tx_pause) { sta_ptr->tx_pause = tp->tx_pause; + spin_unlock_bh(&priv->sta_list_spinlock); mwifiex_update_ralist_tx_pause(priv, tp->peermac, tp->tx_pause); + } else { + spin_unlock_bh(&priv->sta_list_spinlock); } - spin_unlock_bh(&priv->sta_list_spinlock); } } @@ -400,11 +402,13 @@ static void mwifiex_process_sta_tx_pause(struct mwifiex_private *priv, sta_ptr = mwifiex_get_sta_entry(priv, tp->peermac); if (sta_ptr && sta_ptr->tx_pause != tp->tx_pause) { sta_ptr->tx_pause = tp->tx_pause; + spin_unlock_bh(&priv->sta_list_spinlock); mwifiex_update_ralist_tx_pause(priv, tp->peermac, tp->tx_pause); + } else { + spin_unlock_bh(&priv->sta_list_spinlock); } - spin_unlock_bh(&priv->sta_list_spinlock); } } } -- 2.34.1