fdb_kvdb_init 第一次上电初始化冗余的双重格式化

[crazy3min]

fdb_kvdb_init()
  │
  ├─ _fdb_init_ex()                 ← 读取分区信息，很快
  ├─ 找 oldest sector               ← 读一次各扇区头部，很快
  │
  ├─ _fdb_kv_load()                 ★★★★★ 这里最慢 ★★★★★
  │   │
  │   ├─ sector_iterator() 遍历64个扇区
  │   │   │
  │   │   ├─ 第1扇区: read_sector_info → magic校验失败 → check_sec_hdr_cb()
  │   │   │   └─ format_sector() → ①擦除+②写头部     ← 第一次擦除！
  │   │   ├─ 第2扇区: read_sector_info → magic校验失败 → check_sec_hdr_cb()
  │   │   │   └─ format_sector() → ①擦除+②写头部     ← 第一次擦除！
  │   │   ├─ ...
  │   │   └─ 第64扇区: ...                            ← 第一次擦除！
  │   │
  │   ├─ check_failed_count (64) == SECTOR_NUM (64)
  │   │   └─ fdb_kv_set_default()                     ★ 冗余操作 ★
  │   │       ├─ 第1扇区: format_sector → ①擦除+②写头部  ← 第二次擦除！
  │   │       ├─ 第2扇区: format_sector → ①擦除+②写头部  ← 第二次擦除！
  │   │       ├─ ...
  │   │       ├─ 第64扇区: format_sector               ← 第二次擦除！
  │   │       └─ 逐条写入 default KV                   ← 少量写入
  │   │
  │   ├─ sector_iterator() (GC恢复检查)                 ← 已格式化，基本无操作
  │   └─ kv_iterator() (KV恢复检查)                     ← 已格式化，基本无操作
  │
  └─ kv_auto_update() (若启用)                        ← 写入 __ver_num__

[armink]

是的，确实存在这个优化点

[crazy3min]

/**
 * Check and load the flash KV.
 *
 * @return result
 */
static fdb_err_t _fdb_kv_load(fdb_kvdb_t db)
{
    fdb_err_t result = FDB_NO_ERR;
    struct fdb_kv kv;
    struct kvdb_sec_info sector;
    size_t check_failed_count = 0;

    db->in_recovery_check = true;
    /* check all sector header */
    sector_iterator(db, &sector, FDB_SECTOR_STORE_UNUSED, &check_failed_count, db, check_sec_hdr_cb, false);
    if (db->parent.not_formatable && check_failed_count > 0) {
        return FDB_READ_ERR;
    }
    /* all sector header check failed */
    if (check_failed_count == SECTOR_NUM) {
        FDB_INFO("All sector header is incorrect. Set it to default.\n");
        // fdb_kv_set_default(db);

/// 解决第一次上电冗余格式化，替换 fdb_kv_set_default =============================
        uint32_t  i, value_len;
        /* lock the KV cache */
        db_lock(db);
#ifdef FDB_KV_USING_CACHE
        for (i = 0; i < FDB_KV_CACHE_TABLE_SIZE; i++) {
            db->kv_cache_table[i].addr = FDB_DATA_UNUSED;
        }
#endif /* FDB_KV_USING_CACHE */

        /* create default KV */
        for (i = 0; i < db->default_kvs.num; i++) {
            /* It seems to be a string when value length is 0.
            * This mechanism is for compatibility with older versions (less then V4.0). */
            if (db->default_kvs.kvs[i].value_len == 0) {
                value_len = strlen(db->default_kvs.kvs[i].value);
            } else {
                value_len = db->default_kvs.kvs[i].value_len;
            }
            sector.empty_kv = FAILED_ADDR;
            create_kv_blob(db, &sector, db->default_kvs.kvs[i].key, db->default_kvs.kvs[i].value, value_len);
            if (result != FDB_NO_ERR) {
                break;
            }
        }
        db_oldest_addr(db) = 0;
        /* unlock the KV cache */
        db_unlock(db);
/// 解决第一次上电冗余格式化，替换 fdb_kv_set_default =============================
    }

    /* check all sector header for recovery GC */
    sector_iterator(db, &sector, FDB_SECTOR_STORE_UNUSED, db, NULL, check_and_recovery_gc_cb, false);

__retry:
    /* check all KV for recovery */
    kv_iterator(db, &kv, db, NULL, check_and_recovery_kv_cb);
    if (db->gc_request) {
        gc_collect(db);
        goto __retry;
    }

    db->in_recovery_check = false;

    return result;
}

能否接受这种解决方案？直接提取 fdb_kv_set_default的操作替换。