espressif / esp-idf

Espressif IoT Development Framework. Official development framework for Espressif SoCs.
Apache License 2.0
13.37k stars 7.21k forks source link

esp_vfs_register_fd_range message at warning level (IDFGH-13421) #14327

Open ammaree opened 1 month ago

ammaree commented 1 month ago

Answers checklist.

General issue report

For some time, possibly last 12 months, we have been receiving a message at warning level.

'vfs esp_vfs_register_fd_range is successful for range <54; 64) and VFS ID 5'

Surely this message should be output at debug or verbose level?

The source is here: https://github.com/espressif/esp-idf/blob/8e4454b285ad39881c5bf3f440d8be617a2f18a8/components/vfs/vfs.c#L161

Any chance this can be change to debug or verbose?

igrr commented 1 month ago

I see that I changed the log level in when merging https://github.com/espressif/esp-idf/pull/7413, perhaps this was a mistake. Thanks for bringing this up.

I'm surprised, though, that littlefs would call esp_vfs_register_fd_range. This function is only necessary for LwIP...

BrianPugh commented 1 month ago

we don't call esp_vfs_register_fd_range in esp_littlefs, so maybe it's coming from another filesystem @ammaree is using? I don't have my esp-hardware handy to test out.

ammaree commented 1 month ago

@igrr

Thanks for the feedback, I will also check if anything else in our FW might call esp_vfs_register_fd_range.

On a similar note, another new wifi message at incorrect Error level has started coming through. wifi Password length matches WPA2 standards, authmode threshold changes from OPEN to WPA2

I would guess this should also be at DEBUG or VERBOSE level?

ammaree commented 1 month ago

@igrr @BrianPugh

Have checked our code. We only call esp_vfs_register that calls esp_vfs_register_common

Nothing else that calls esp_vfs_register_fd_range to trigger this message, so I assume from LWIP ?