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

add dynamic stacksize for procBLETask #2137

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

Lukaswnd
Copy link

add OVERRIDE_PROC_BLE_TASK_STACK_SIZE to set the stacksize for the procBLETask by a define in the build_flags

Description:

When using the ESP32s3-devkit-c1 with Arduino and using it as a ble gateway, the task watchdog for the stacksize gets triggerd, if you receive a request the same time it scans for new devices. This results in a crash. I testes it with 13500 and it works fine. I think a build flag which each user can set seperatly but doesn't have to is the best option to solve it.

Checklist:

  • The pull request is done against the latest development branch
  • Only one feature/fix was added per PR and the code change compiles without warnings
  • I accept the DCO.

add OVERRIDE_PROC_BLE_TASK_STACK_SIZE to set the stacksize for the procBLETask by a define in the build_flags
@1technophile
Copy link
Owner

Would be interesting to adjust the stack size from the 8500 low limit, I would prefer to increase this one rather than going directly to the 13500.

If you put the log level to trace you will have a view of how much stack is remaining at the end of the scan. You may want to add other trace but this first log should give you an idea.

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