-
Notifications
You must be signed in to change notification settings - Fork 22
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
Improve memory footprint #71
Conversation
3538c5a
to
934ff51
Compare
934ff51
to
3ac9f88
Compare
@AzonInc I installed it now, I'll update if I experience stability issues. |
So no restarts within the week and event logs enabled? Also were there any API disconnects or no issues at all as it happened in #66 |
No restarts while event logs enabled. I didn't experience any API disconnects, but couldn't monitor for the whole week. |
Okay cool. |
I tried to reduce the memory footprint and improve efficiency.
Would be nice if someone could give it a try and report about the stability.
Changes
std::string
with a fixed-size character array, usingmemset
,strcpy
, andstrcmp
to save additional bytes.deviceName
log line.Additions
CONFIG_BT_NIMBLE_MEM_ALLOC_MODE_EXTERNAL
when using thepsram
component to enable PSRAM usage for the NimBLE host.Validation Enhancements
Miscellaneous