Re: [d-kernel] [PATCH 3/2] wifi: mt76: mt7921: fix potential deadlock in mt7921_roc_abort_sync

[Vasiliy Kovalev <kovalev@altlinux.org>]

On 6/11/26 14:15, Ajrat Makhmutov wrote:
> Привет, Василий!
> 
> 11.06.2026 14:07, Vasiliy Kovalev пишет:
>> Добрый день,
>>
>> On 6/11/26 13:44, Ajrat Makhmutov wrote:
>>> From: Sean Wang <sean.wang@mediatek.com>
>>>
>>
>> При бэкпортировании следует указывать хэш оригинального коммита из 
>> mainline, например:
>>
>> commit d5059e52fd8bc624ec4255c9fa01a266513d126b upstream.
>>
>>> roc_abort_sync() can deadlock with roc_work(). roc_work() holds
>>> dev->mt76.mutex, while cancel_work_sync() waits for roc_work()
>>> to finish. If the caller already owns the same mutex, both
>>> sides block and no progress is possible.
>>>
>>> This deadlock can occur during station removal when
>>> mt76_sta_state() -> mt76_sta_remove() -> mt7921_mac_sta_remove() ->
>>> mt7921_roc_abort_sync() invokes cancel_work_sync() while
>>> roc_work() is still running and holding dev->mt76.mutex.
>>>
>>> This avoids the mutex deadlock and preserves exactly-once
>>> work ownership.
>>>
>>> Fixes: 352d966126e6 ("wifi: mt76: mt7921: fix a potential association 
>>> failure upon resuming")
>>> Co-developed-by: Quan Zhou <quan.zhou@mediatek.com>
>>> Signed-off-by: Quan Zhou <quan.zhou@mediatek.com>
>>> Signed-off-by: Sean Wang <sean.wang@mediatek.com>
>>> Link: 
>>> https://patch.msgid.link/20260126180013.8167-1-sean.wang@kernel.org
>>> Signed-off-by: Felix Fietkau <nbd@nbd.name>
>>> (cherry picked from commit d5059e52fd8bc624ec4255c9fa01a266513d126b)
>>
>> cherry picked здесь не дает полезной информации, можно убрать
>>
>>> [ALT: keep del_timer_sync() instead of timer_delete_sync() — the
>>>   timer API rename is not present in 6.12.y. ]
>>
>> вместо "ALT" лучше указать бэкпортера "Ajrat"
>>
>>> ---
>>> Дополнение к ранее присланной серии из двух патчей.
>>>
>>> Саша Левин на ревью запроса в stable@ указал, что patch 2/2 (5ed54896,
>>> "fix a potential scan no APs") в одиночку вносит достижимый 
>>> self-deadlock
>>> по dev->mt76.mutex: путь удаления станции идёт под этим mutex (его берёт
>>> ядровая mt76_sta_remove()), а добавленный там roc_abort_sync() делает
>>> cancel_work_sync() по roc_work(), который тоже хочет этот mutex.
>>
>> Саша Левин вроде как не против [1] принять серию из 3х патчей в stable 
>> 6.12.y ветку, подготовьте исправленную версию и отправьте туда.
>>
>> [1] https://lore.kernel.org/all/20260610-stable-reply-0014@kernel.org/#t
> 
> Апстрим по сути ещё не принял. Это только rc 7.1 из linux-next.

Апстрим принял [https://github.com/torvalds/linux/commit/d5059e52fd8b], 
stable тоже:
  v7.0.y 
[https://github.com/gregkh/linux/commit/91e77840bf13de3add125060cf8b32ca24a52c8c]

v6.18.y 
[https://github.com/gregkh/linux/commit/35180c772f5e11e2fa4d80d3dfd50906cb6d9646]

Остальные патчи серии в 7.0.y и 6.18.y присутствуют, так что можно смело 
отправлять в v6.12.y

> Саша пишет, что либо я докажу, что исправление стабильное, либо нужно 
> ждать выпуска 7.1.
> Ок, попробую доказать.

>>
>>> Этот коммит (d5059e52) — обязательное лечение, применять вместе с 2/2.
>>> Таким образом серия теперь из трёх патчей.