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
This commit is contained in:
Martin Vychodil 2023-11-22 22:41:18 +08:00
commit 3e04fba477
6 changed files with 18 additions and 0 deletions

View File

@ -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

View File

@ -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)

View File

@ -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})

View File

@ -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

6
components/vfs/linker.lf Normal file
View File

@ -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)

View File

@ -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;
}