revert(store): drop the page-cache hoist from process init

Measurement retired the theory it was built on: the slab installs correctly now
(config_failed=0) and the leak was unchanged, because the cause was thread
heaps never being released rather than page-cache fragmentation.

It also runs at the top of main() for every process, including the launcher,
and the managed reinstall guard went red. A change that fixes nothing it
claimed does not get to stay on a release-blocking PR while a Windows guard is
failing. cbm_store_configure_pagecache remains available for a future attempt
wired ahead of sqlite3_initialize deliberately.

Signed-off-by: Martin Vogel <martin.vogel.tech@gmail.com>
This commit is contained in:
Martin Vogel
2026-07-26 05:21:16 +02:00
parent 32b2c2dde5
commit 80ea64a7dc
-3
View File
@@ -1706,9 +1706,6 @@ static int main_run_daemon_ctl(int argc, char **argv, const cbm_daemon_ipc_endpo
int main(int argc, char **argv) {
/* Must remain the first statement: see allocator binding contract above. */
cbm_alloc_init();
/* Immediately after, and before anything can touch SQLite: the page-cache
* slab can only be installed ahead of sqlite3_initialize (#581). */
cbm_store_configure_pagecache();
#ifndef _WIN32
pid_t process_initial_ppid = getppid();
#endif