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

SnapProcessorRTOS: raise task priority by one #9

Merged
merged 1 commit into from
Feb 29, 2024

Conversation

h3ndrik
Copy link
Contributor

@h3ndrik h3ndrik commented Feb 29, 2024

I propose raising the task priority of the SnapProcessorRTOS to 2.
The Main Task is already running at priority level 1 (see: https://docs.espressif.com/projects/esp-idf/en/latest/esp32/api-reference/system/freertos.html#background-tasks)

Feel free to disregard this PR. This might starve the main task and have side-effects. However, I think copying real-time audio data to the peripheral warrants taking priority.

I'm tinkering around with ESPHome and there is too much going on with priority level 1. Raising the priority and setting large buffer sizes makes it almost usable (but not quite).

@pschatzmann
Copy link
Owner

I suggest to make it a parameter in SnapConfig.h

@h3ndrik
Copy link
Contributor Author

h3ndrik commented Feb 29, 2024

I force-pushed an update... I'm not sure if you like 1 or 2 as the default...

@pschatzmann pschatzmann merged commit 53539f8 into pschatzmann:main Feb 29, 2024
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.

2 participants