ISAR/libmdbx
3
0
Fork 0
mirror of https://github.com/isar/libmdbx.git synced 2025-01-04 18:24:12 +08:00
Code Issues Packages Projects Releases Wiki Activity

mdbx: добавление FIXME для MDBX_NOMETASYNC.

Browse Source
This commit is contained in:
Леонид Юрьев (Leonid Yuriev) 2022-12-01 03:00:40 +03:00
parent 512e6dbd08
commit 163486fa3a
1 changed files with 18 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
src/core.c
View File

@ -11244,9 +11244,26 @@ int mdbx_txn_commit_ex(MDBX_txn *txn, MDBX_commit_latency *latency) {
} }
const meta_ptr_t head = meta_recent(env, &txn->tw.troika); const meta_ptr_t head = meta_recent(env, &txn->tw.troika);
/* sync prev meta */
if (head.is_steady && atomic_load32(&env->me_lck->mti_meta_sync_txnid, if (head.is_steady && atomic_load32(&env->me_lck->mti_meta_sync_txnid,
mo_Relaxed) != (uint32_t)head.txnid) { mo_Relaxed) != (uint32_t)head.txnid) {
/* sync prev meta */ /* FIXME: Тут есть унаследованный от LMDB недочет.
*
* Проблем нет, если все процессы работающие с БД не используют WRITEMAP.
* Тогда мета-страница (обновленная, но не сброшенная на диск) будет
* сохранена в результате fdatasync() при записи данных этой транзакции.
*
* Проблем нет, если все процессы работающие с БД используют WRITEMAP
* без MDBX_AVOID_MSYNC.
* Тогда мета-страница (обновленная, но не сброшенная на диск) будет
* сохранена в результате msync() при записи данных этой транзакции.
*
* Если же происходит комбинирование WRITEMAP и записи через файловый
* дескриптор, то требуется явно обновлять мета-страницу. Однако,
* так полностью теряется выгода от NOMETASYNC.
*
* Дефект же в том, что сейчас нет возможности отличить последний случай от
* двух предыдущих и поэтому приходится всегда задействовать meta_sync(). */
rc = meta_sync(env, head); rc = meta_sync(env, head);
if (unlikely(rc != MDBX_SUCCESS)) { if (unlikely(rc != MDBX_SUCCESS)) {
ERROR("txn-%s: error %d", "presync-meta", rc); ERROR("txn-%s: error %d", "presync-meta", rc);