Skip to content

Commit 3541e43

Browse files
Matthew MacovskyKyle Kearney
authored andcommitted
Generalize KVStore phase 1/2 test BlockDevice sizes
1 parent e3e74b2 commit 3541e43

File tree

2 files changed

+24
-16
lines changed
  • features/storage/TESTS/kvstore

2 files changed

+24
-16
lines changed

features/storage/TESTS/kvstore/general_tests_phase_1/main.cpp

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -63,9 +63,6 @@ static const char *kv_prefix[] = {"TDB_", "FS_", "SEC_"};
6363

6464
static int kv_setup = TDBStoreSet;
6565

66-
static const size_t ul_bd_size = 16 * 4096;
67-
static const size_t rbp_bd_size = 8 * 4096;
68-
6966
static const int heap_alloc_threshold_size = 4096;
7067

7168
/*----------------initialization------------------*/
@@ -74,6 +71,8 @@ static const int heap_alloc_threshold_size = 4096;
7471
static void kvstore_init()
7572
{
7673
int res;
74+
size_t erase_size, ul_bd_size, rbp_bd_size;
75+
BlockDevice *sec_bd;
7776

7877
res = bd->init();
7978
TEST_ASSERT_EQUAL_ERROR_CODE(0, res);
@@ -102,14 +101,19 @@ static void kvstore_init()
102101

103102
#if SECURESTORE_ENABLED
104103
if (kv_setup == SecStoreSet) {
104+
sec_bd = bd;
105105
if (erase_val == -1) {
106106
flash_bd = new FlashSimBlockDevice(bd);
107-
ul_bd = new SlicingBlockDevice(flash_bd, 0, ul_bd_size);
108-
rbp_bd = new SlicingBlockDevice(flash_bd, ul_bd_size, ul_bd_size + rbp_bd_size);
109-
} else {
110-
ul_bd = new SlicingBlockDevice(bd, 0, ul_bd_size);
111-
rbp_bd = new SlicingBlockDevice(bd, ul_bd_size, ul_bd_size + rbp_bd_size);
107+
sec_bd = flash_bd;
112108
}
109+
110+
erase_size = sec_bd->get_erase_size();
111+
ul_bd_size = erase_size * 4;
112+
rbp_bd_size = erase_size * 2;
113+
114+
ul_bd = new SlicingBlockDevice(sec_bd, 0, ul_bd_size);
115+
rbp_bd = new SlicingBlockDevice(sec_bd, ul_bd_size, ul_bd_size + rbp_bd_size);
116+
113117
TDBStore *ul_kv = new TDBStore(ul_bd);
114118
TDBStore *rbp_kv = new TDBStore(rbp_bd);
115119
kvstore = new SecureStore(ul_kv, rbp_kv);

features/storage/TESTS/kvstore/general_tests_phase_2/main.cpp

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -63,9 +63,6 @@ static const char *kv_prefix[] = {"TDB_", "FS_", "SEC_"};
6363

6464
static int kv_setup = TDBStoreSet;
6565

66-
static const size_t ul_bd_size = 16 * 4096;
67-
static const size_t rbp_bd_size = 8 * 4096;
68-
6966
static const int heap_alloc_threshold_size = 4096;
7067

7168
/*----------------initialization------------------*/
@@ -74,6 +71,8 @@ static const int heap_alloc_threshold_size = 4096;
7471
static void kvstore_init()
7572
{
7673
int res;
74+
size_t erase_size, ul_bd_size, rbp_bd_size;
75+
BlockDevice *sec_bd;
7776

7877
res = bd->init();
7978
TEST_ASSERT_EQUAL_ERROR_CODE(0, res);
@@ -102,14 +101,19 @@ static void kvstore_init()
102101

103102
#if SECURESTORE_ENABLED
104103
if (kv_setup == SecStoreSet) {
104+
sec_bd = bd;
105105
if (erase_val == -1) {
106106
flash_bd = new FlashSimBlockDevice(bd);
107-
ul_bd = new SlicingBlockDevice(flash_bd, 0, ul_bd_size);
108-
rbp_bd = new SlicingBlockDevice(flash_bd, ul_bd_size, ul_bd_size + rbp_bd_size);
109-
} else {
110-
ul_bd = new SlicingBlockDevice(bd, 0, ul_bd_size);
111-
rbp_bd = new SlicingBlockDevice(bd, ul_bd_size, ul_bd_size + rbp_bd_size);
107+
sec_bd = flash_bd;
112108
}
109+
110+
erase_size = sec_bd->get_erase_size();
111+
ul_bd_size = erase_size * 4;
112+
rbp_bd_size = erase_size * 2;
113+
114+
ul_bd = new SlicingBlockDevice(sec_bd, 0, ul_bd_size);
115+
rbp_bd = new SlicingBlockDevice(sec_bd, ul_bd_size, ul_bd_size + rbp_bd_size);
116+
113117
TDBStore *ul_kv = new TDBStore(ul_bd);
114118
TDBStore *rbp_kv = new TDBStore(rbp_bd);
115119
kvstore = new SecureStore(ul_kv, rbp_kv);

0 commit comments

Comments
 (0)