From 93dbdd4a621e2ecb2fe69ca76f5460e1bf8c7780 Mon Sep 17 00:00:00 2001 From: YAMAMOTO Takashi Date: Tue, 14 Jan 2025 15:49:28 +0900 Subject: [PATCH] esp32c3: apply wifi deadlock fix from esp32s3 cf. https://github.com/apache/nuttx/pull/15445 --- arch/risc-v/src/esp32c3/esp_wifi_adapter.c | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/arch/risc-v/src/esp32c3/esp_wifi_adapter.c b/arch/risc-v/src/esp32c3/esp_wifi_adapter.c index 44fa965b641..533fb7c0b20 100644 --- a/arch/risc-v/src/esp32c3/esp_wifi_adapter.c +++ b/arch/risc-v/src/esp32c3/esp_wifi_adapter.c @@ -2576,6 +2576,13 @@ static void esp_evt_work_cb(void *arg) break; } + /* Some of the following logic (eg. esp_wlan_sta_set_linkstatus) + * can take net_lock(). To maintain the consistent locking order, + * we take net_lock() here before taking esp_wifi_lock. Note that + * net_lock() is a recursive lock. + */ + + net_lock(); esp_wifi_lock(true); switch (evt_adpt->id) @@ -2697,6 +2704,7 @@ static void esp_evt_work_cb(void *arg) } esp_wifi_lock(false); + net_unlock(); kmm_free(evt_adpt); }