From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from dispatch1-us1.ppe-hosted.com (dispatch1-us1.ppe-hosted.com [67.231.154.184]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 6530E26FA50 for ; Mon, 17 Nov 2025 19:21:01 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=67.231.154.184 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763407264; cv=none; b=sfYWM9pudh8kmuXT7OCHuq3Y968iJzpCFhRyv8ZNAGPmsCRiUWe+0ThGEVH6Tbp3BMbbr94MJqVtjTrinlOHDs7HkfMgAY50aQu6Bj+M4uO3qNwqfUB2cN5eG84/7vXzdpw1qUfecEgnObUzW/whPzTwSBw95ZOBldbVMaMGspo= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763407264; c=relaxed/simple; bh=6A+TrBEj0IvjlzggxexZGZR1wM0bXpsxcrXQ4grw9M8=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=cxDs/MvhIMwFM84tQBNdxO1rW+S26SOIe5EQzaGQtdzU1NonOqEk1Y6qqd78I+LBw5EsGwV9+tHEMwK+iEHzFV+IwsIQqqKyMnsSGzRno4s3eGUnixfbGnMznSK+7IqnYrHFUeb+21vZxB+K/YpOTCp9F8nn9XmGOheFphMmSqw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=candelatech.com; spf=pass smtp.mailfrom=candelatech.com; dkim=pass (1024-bit key) header.d=candelatech.com header.i=@candelatech.com header.b=lpuf+Zb5; arc=none smtp.client-ip=67.231.154.184 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=candelatech.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=candelatech.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=candelatech.com header.i=@candelatech.com header.b="lpuf+Zb5" Received: from dispatch1-us1.ppe-hosted.com (ip6-localhost [127.0.0.1]) by dispatch1-us1.ppe-hosted.com (PPE Hosted ESMTP Server) with ESMTP id C47CD506413 for ; Mon, 17 Nov 2025 19:15:21 +0000 (UTC) X-Virus-Scanned: Proofpoint Essentials engine Received: from mail3.candelatech.com (mail.candelatech.com [208.74.158.173]) by mx1-us1.ppe-hosted.com (PPE Hosted ESMTP Server) with ESMTP id C86F280009C; Mon, 17 Nov 2025 19:15:12 +0000 (UTC) Received: from [172.18.0.200] (unknown [64.190.177.183]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail3.candelatech.com (Postfix) with ESMTPSA id 09F0013C2B0; Mon, 17 Nov 2025 11:14:58 -0800 (PST) DKIM-Filter: OpenDKIM Filter v2.11.0 mail3.candelatech.com 09F0013C2B0 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=candelatech.com; s=default; t=1763406909; bh=6A+TrBEj0IvjlzggxexZGZR1wM0bXpsxcrXQ4grw9M8=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=lpuf+Zb5GPwfhy1848cXSYrmsh9GZ9ziGL4LYyCFYckQHzCFce5AKaoS+5PJ3O0BF 9uzMWuK8HKBS8J/mnNGNwwWEihds5g4fBvXf7eGiJaSeQuUgbZolKIA8IyvD5mVxze NPF7X5qF2F2vzl7YhMIUKAcq1puRpnPgalq1fbQw= Message-ID: <344fc8c1-484c-421a-a73e-9e46ed7480ec@candelatech.com> Date: Mon, 17 Nov 2025 11:14:55 -0800 Precedence: bulk X-Mailing-List: linux-wireless@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH mt76] wifi: mt76: mt7996: Do not run ieee80211_iter_keys() for scanning links in mt7996_vif_link_remove() To: Lorenzo Bianconi Cc: Felix Fietkau , Ryder Lee , Shayne Chen , Sean Wang , Matthias Brugger , AngeloGioacchino Del Regno , linux-wireless@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org References: <20251115-mt7996-key-iter-link-remove-fix-v1-1-4f3f4e1eaa78@kernel.org> <49fdf52b-32cb-427e-ab95-a81856840a1e@candelatech.com> Content-Language: en-MW From: Ben Greear Organization: Candela Technologies In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-MDID: 1763406914-QDe2gbazx2At X-PPE-STACK: {"stack":"us5"} X-MDID-O: us5;at1;1763406914;QDe2gbazx2At;;40f3b03bb24907b736a33b345f288e14 X-PPE-TRUSTED: V=1;DIR=OUT; On 11/17/25 11:11 AM, Lorenzo Bianconi wrote: >> On 11/15/25 2:41 AM, Lorenzo Bianconi wrote: >>> mt7996_vif_link_remove routine is executed by mt76_scan_complete() >>> without holding the wiphy mutex triggering the following lockdep warning. >> >> Hello Lorenzo, >> >> Thank you for the patch. This appears to fix the lockdep splats I was >> seeing, and kernel is stable in at least initial testing. > > thx Ben for testing the patch. Are you ok with adding your Tested-by tag? Sure: Tested-by: Ben Greear Thanks, Ben > > Regards, > Lorenzo > >> >> Thanks, >> Ben >> >>> >>> WARNING: CPU: 0 PID: 72 at net/mac80211/key.c:1029 ieee80211_iter_keys+0xe4/0x1a0 [mac80211] >>> CPU: 0 UID: 0 PID: 72 Comm: kworker/u32:2 Tainted: G S 6.18.0-rc5+ #27 PREEMPT(full) >>> Tainted: [S]=CPU_OUT_OF_SPEC >>> Hardware name: Default string Default string/SKYBAY, BIOS 5.12 02/15/2023 >>> Workqueue: phy3 mt76_scan_work [mt76] >>> RIP: 0010:ieee80211_iter_keys+0xe4/0x1a0 [mac80211] >>> Code: 4c 48 83 c4 10 5b 5d 41 5c 41 5d 41 5e 41 5f c3 48 8b 47 48 be ff ff ff ff 48 8d 78 68 e8 b4 eb 1e e1 85 c0 0f 85 49 >>> ff ff ff 4c 8b ab 90 1a 00 00 48 8d 83 90 >>> RSP: 0018:ffffc900002f7cb0 EFLAGS: 00010246 >>> RAX: 0000000000000000 RBX: ffff888127e00ee0 RCX: 0000000000000000 >>> RDX: 0000000000000000 RSI: ffff888127e00788 RDI: ffff88811132b5c8 >>> RBP: ffffffffa0ddf400 R08: 0000000000000001 R09: 000000009dcc1dac >>> R10: 0000000000000001 R11: ffff88811132b5a0 R12: ffffc900002f7d00 >>> R13: ffff8882581e6a80 R14: ffff888127e0afc8 R15: ffff888158832038 >>> FS: 0000000000000000(0000) GS:ffff8884da486000(0000) knlGS:0000000000000000 >>> CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033 >>> CR2: 0000000030a0fd90 CR3: 0000000002c52004 CR4: 00000000003706f0 >>> Call Trace: >>> >>> ? lock_acquire+0xc2/0x2c0 >>> mt7996_vif_link_remove+0x64/0x2b0 [mt7996e] >>> mt76_put_vif_phy_link+0x41/0x50 [mt76] >>> mt76_scan_complete+0x77/0x100 [mt76] >>> mt76_scan_work+0x2eb/0x3f0 [mt76] >>> ? process_one_work+0x1e5/0x6d0 >>> process_one_work+0x221/0x6d0 >>> worker_thread+0x19a/0x340 >>> ? rescuer_thread+0x450/0x450 >>> kthread+0x108/0x220 >>> ? kthreads_online_cpu+0x110/0x110 >>> ret_from_fork+0x1c6/0x220 >>> ? kthreads_online_cpu+0x110/0x110 >>> ret_from_fork_asm+0x11/0x20 >>> >>> irq event stamp: 45471 >>> hardirqs last enabled at (45477): [] __up_console_sem+0x5e/0x70 >>> hardirqs last disabled at (45482): [] __up_console_sem+0x43/0x70 >>> softirqs last enabled at (44500): [] napi_pp_put_page+0xac/0xd0 >>> softirqs last disabled at (44498): [] page_pool_put_unrefed_netmem+0x290/0x3d0 >>> ---[ end trace 0000000000000000 ]--- >>> >>> Fix the issue skipping ieee80211_iter_keys() for scanning links in >>> mt7996_vif_link_remove routine since we have not uploaded any hw keys >>> for these links. >>> >>> Fixes: 04414d7bba78 ("wifi: mt76: mt7996: delete vif keys when requested") >>> Signed-off-by: Lorenzo Bianconi >>> --- >>> drivers/net/wireless/mediatek/mt76/mt7996/main.c | 3 ++- >>> 1 file changed, 2 insertions(+), 1 deletion(-) >>> >>> diff --git a/drivers/net/wireless/mediatek/mt76/mt7996/main.c b/drivers/net/wireless/mediatek/mt76/mt7996/main.c >>> index 2a45db398fd597180c0b2bfc12e019b514f24fd6..beed795edb24c67e1b7b44fe87fd5de125a21d94 100644 >>> --- a/drivers/net/wireless/mediatek/mt76/mt7996/main.c >>> +++ b/drivers/net/wireless/mediatek/mt76/mt7996/main.c >>> @@ -392,7 +392,8 @@ void mt7996_vif_link_remove(struct mt76_phy *mphy, struct ieee80211_vif *vif, >>> }; >>> int idx = msta_link->wcid.idx; >>> - ieee80211_iter_keys(mphy->hw, vif, mt7996_key_iter, &it); >>> + if (!mlink->wcid->offchannel) >>> + ieee80211_iter_keys(mphy->hw, vif, mt7996_key_iter, &it); >>> mt7996_mcu_add_sta(dev, link_conf, NULL, link, NULL, >>> CONN_STATE_DISCONNECT, false); >>> >>> --- >>> base-commit: 09600421ffe2acc1a00ec466b25485f11b6bb957 >>> change-id: 20251115-mt7996-key-iter-link-remove-fix-7cf4d529c5a0 >>> >>> Best regards, >> >> >> -- >> Ben Greear >> Candela Technologies Inc http://www.candelatech.com -- Ben Greear Candela Technologies Inc http://www.candelatech.com