Merge branch 'bugfix/uart_vfs_select_in_iram_v5.1' into 'release/v5.1'

fix: add UART VFS select callback in IRAM when CONFIG_UART_ISR_IN_IRAM is enabled (v5.1) See merge request espressif/esp-idf!27298
espressif · Nov 22, 2023 · 3e04fba · 3e04fba
2 parents 2a227b8 + 633a3b0
commit 3e04fba
Show file tree

Hide file tree

Showing 6 changed files with 18 additions and 0 deletions.
diff --git a/components/driver/Kconfig b/components/driver/Kconfig
@@ -230,6 +230,7 @@ menu "Driver Configurations"
         config UART_ISR_IN_IRAM
             bool "Place UART ISR function into IRAM"
             depends on !RINGBUF_PLACE_ISR_FUNCTIONS_INTO_FLASH
+            select VFS_SELECT_IN_RAM if VFS_SUPPORT_SELECT
             default n
             help
                 If this option is not selected, UART interrupt will be disabled for a long time and

diff --git a/components/lwip/linker.lf b/components/lwip/linker.lf
@@ -134,3 +134,5 @@ entries:
       memp:do_memp_malloc_pool (noflash_text)
   if ESP_ALLOW_BSS_SEG_EXTERNAL_MEMORY = y:
     * (extram_bss)
+  if VFS_SELECT_IN_RAM = y:
+    vfs_lwip:lwip_stop_socket_select_isr (noflash)
diff --git a/components/vfs/CMakeLists.txt b/components/vfs/CMakeLists.txt
@@ -8,6 +8,7 @@ list(APPEND pr driver
                esp_timer)
 
 idf_component_register(SRCS ${sources}
+                       LDFRAGMENTS "linker.lf"
                        INCLUDE_DIRS include
                        PRIV_INCLUDE_DIRS private_include
                        PRIV_REQUIRES ${pr})

diff --git a/components/vfs/Kconfig b/components/vfs/Kconfig
@@ -63,6 +63,13 @@ menu "Virtual file system"
             It is possible to suppress these debug outputs by enabling this
             option.
 
+    config VFS_SELECT_IN_RAM
+        bool "Make VFS driver select() callbacks IRAM-safe"
+        default n
+        depends on VFS_SUPPORT_SELECT
+        help
+            If enabled, VFS driver select() callback function will be placed in IRAM.
+
     config VFS_SUPPORT_TERMIOS
         bool "Provide termios.h functions"
         default y

diff --git a/components/vfs/linker.lf b/components/vfs/linker.lf
@@ -0,0 +1,6 @@
+[mapping:vfs]
+archive: libvfs.a
+entries:
+  if VFS_SELECT_IN_RAM = y:
+    vfs:esp_vfs_select_triggered_isr (noflash)
+    vfs_uart:select_notif_callback_isr (noflash)
diff --git a/components/vfs/vfs.c b/components/vfs/vfs.c
@@ -1133,6 +1133,7 @@ void esp_vfs_select_triggered_isr(esp_vfs_select_sem_t sem, BaseType_t *woken)
             // matter here stop_socket_select() will be called for only valid VFS drivers.
             const vfs_entry_t *vfs = s_vfs[i];
             if (vfs != NULL && vfs->vfs.stop_socket_select_isr != NULL) {
+                // Note: If the UART ISR resides in IRAM, the function referenced by stop_socket_select_isr should also be placed in IRAM.
                 vfs->vfs.stop_socket_select_isr(sem.sem, woken);
                 break;
             }