You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
KVstore: use _storage_config_xxxx() in kv_config.cpp can just create one securestore instance,which is not accordant with "#define MAX_ATTACHED_KVS 3" in KVMap.h
#12881
Closed
bianyu91 opened this issue
Apr 29, 2020
· 3 comments
In mbed-os/features/storage/kvstore/KVMap.h has the following macro
#define MAX_ATTACHED_KVS 3
I think this macro defines the max number of kvstore_main_instances attached to KVMap.
But in kv_config.cpp
static kvstore_config_t kvstore_config;
and all the _ storage_config_XXXXAPIs,such as
int _storage_config_filesystem_common()
{
ret =kv_map.attach(STR(MBED_CONF_STORAGE_DEFAULT_KV),& kvstore_config);
}
just can define only one kvstore_main_instance with the default partition name"/kv/",what if need to define two or more kvstore_main_instances whose partition name should be different?
Target(s) affected by this defect ?
N/A
Toolchain(s) (name and version) displaying this defect ?
N/A
What version of Mbed-os are you using (tag or sha) ?
mbed-os-5.11.5
What version(s) of tools are you using. List all that apply (E.g. mbed-cli)
mbed-cli
How is this defect reproduced ?
Just view the source code
The text was updated successfully, but these errors were encountered:
Thank you for raising this detailed GitHub issue. I am now notifying our internal issue triagers. Internal Jira reference: https://jira.arm.com/browse/IOTOSM-2157
We closed this issue because it has been inactive for quite some time and we believe it to be low priority. If you think that the priority should be higher, then please reopen with your justification for increasing the priority.
Description of defect
In mbed-os/features/storage/kvstore/KVMap.h has the following macro
#define MAX_ATTACHED_KVS 3
I think this macro defines the max number of kvstore_main_instances attached to KVMap.
But in kv_config.cpp
static kvstore_config_t kvstore_config;
and all the _ storage_config_XXXXAPIs,such as
int _storage_config_filesystem_common()
{
just can define only one kvstore_main_instance with the default partition name"/kv/",what if need to define two or more kvstore_main_instances whose partition name should be different?
Target(s) affected by this defect ?
N/A
Toolchain(s) (name and version) displaying this defect ?
N/A
What version of Mbed-os are you using (tag or sha) ?
mbed-os-5.11.5
What version(s) of tools are you using. List all that apply (E.g. mbed-cli)
mbed-cli
How is this defect reproduced ?
Just view the source code
The text was updated successfully, but these errors were encountered: