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 persistence demonstration to SM example #152
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's the cost of littlefs in term of flash and RAM consumed ?
BLE_SM/source/main.cpp
Outdated
SMDevicePeripheral peripheral(ble, queue, peer_address); | ||
peripheral.run(); | ||
/* if filesystem creation fails or there is no filesystem the security manager | ||
* will fallback to storing the security databse in memory */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo databse
-> database
@@ -159,9 +162,19 @@ class SMDevice : private mbed::NonCopyable<SMDevice>, | |||
return; | |||
} | |||
|
|||
/* This path will be used to store bonding information but will fallback | |||
* to storing in memory if file access fails (for example due to lack of a filesystem) */ | |||
const char* db_path = "/fs/bt_sec_db"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
May we use a configuration parameter ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you mean for the path in the example or to toggle the feature as a whole in the mbed-os?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added to config
@pan- memory usage of fs: |
Please note for OOB, we're working with mbed-os-5.9.0-oob branch. |
What is the status of this PR? |
be90095
to
6e94bc5
Compare
4127b58
to
78a115b
Compare
Waiting for these to go in before merging: |
doesn't change the flow, and doesn't implement actual persistence but shows how