Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SL-TEMP] Increasing the lwip thread priority to 32 (osPriorityAboveNormal) #312

Open
wants to merge 1 commit into
base: release_2.5-1.4
Choose a base branch
from

Conversation

chirag-silabs
Copy link
Contributor

Testing

After the move to the cmsis os the lwip priority was still 2 i.e., below the application (osPriorityNormal)
Increasing the lwip task priority to 32

Current task priorities list is as follows:

Timer SVC - 55 (Highest)
si91x_event - 49 : Handles the notifications event coming from the wifi sdk itself
si91x_bus - 48 : Handles the Tx and Rx events coming from the wifi sdk itself
rsi_ble - 40 : Matter platform layer for the ble events, this internalls calls APIs from the wifi sdk and vice versa
wlan_rsi - 39 : Matter platform layer for the wlan events, this internalls calls APIs from the wifi sdk and vice versa
LIT - 24 : Application thread
Shell - 16 : Matter CLI
UART - 16 : Logging
LWIP - 2 : Lwip packet handling low level input and output handling
CHIP - 2 : Matter thread common for all the platforms. Handling the data model read and writes. Provided by chip stack itself.
IDLE - 0

After discussing with the wifi sdk team it was suggested to keep the priority more than the application, there can be possibility that the lwip is blocked by the application thread if throughtput is more and your application is processing multiple events simulateously.

I have increased the priority to 32 and have validated commissioning and commands are working fine.

Currently since we don't get many events on the application thread like btn and all which is not affecting it but it can affect in the future when the application is more events blocking lwip.

Testing
Tested with 917 SoC and Wf200 RCP

Multiple commissioning runs and commands transfer.

Note:

This is a temporary change for Matter 1.4, the CSA code have chip task of priority osPriorityHigh so it should be analyzed and all the other tasks should also be modified.

@chirag-silabs chirag-silabs requested review from a team as code owners March 4, 2025 05:26
@chirag-silabs chirag-silabs changed the base branch from main to release_2.5-1.4 March 4, 2025 05:27
@mkardous-silabs mkardous-silabs removed the request for review from a team March 4, 2025 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant