Skip to content

Commit 097c66f

Browse files
committed
Merge bitcoin#30039: dbwrapper: Bump LevelDB max file size to 32 MiB to avoid system slowdown from high disk cache flush rate
b73d331 dbwrapper: Bump max file size to 32 MiB (Maciej S. Szmigiero) Pull request description: The default max file size for LevelDB is 2 MiB, which results in the LevelDB compaction code generating ~4 disk cache flushes per second when syncing with the Bitcoin network. These disk cache flushes are triggered by `fdatasync()` syscall issued by the LevelDB compaction code when reaching the max file size. If the database is on a HDD this flush rate brings the whole system to a crawl. It also results in very slow throughput since 2 MiB * 4 flushes per second is about 8 MiB / second max throughput, while even an old HDD can pull 100 - 200 MiB / second streaming throughput. Increase the max file size for LevelDB to 128 MiB instead so the flush rate drops to about 1 flush / 2 seconds and the system no longer gets so sluggish. The max file size value chosen also matches the `MAX_BLOCKFILE_SIZE` file size setting already used by the block storage. ACKs for top commit: l0rinc: ACK b73d331 davidgumberg: ACK bitcoin@b73d331 andrewtoth: ACK b73d331 TheCharlatan: ACK b73d331 willcl-ark: ACK b73d331 tdb3: ACK b73d331 laanwj: ACK b73d331 Tree-SHA512: 5d8fb9ad1ea643fb3e42a9c59f6fc90cc5cc3b82c06d9b8d59de3a5a926fabaeb78efb51b608b1e7925f49d82dfcbd5b72c552993879789f33201efe57c278f3
2 parents 68daaea + b73d331 commit 097c66f

File tree

2 files changed

+2
-0
lines changed

2 files changed

+2
-0
lines changed

src/dbwrapper.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -147,6 +147,7 @@ static leveldb::Options GetOptions(size_t nCacheSize)
147147
// on corruption in later versions.
148148
options.paranoid_checks = true;
149149
}
150+
options.max_file_size = std::max(options.max_file_size, DBWRAPPER_MAX_FILE_SIZE);
150151
SetMaxOpenFiles(&options);
151152
return options;
152153
}

src/dbwrapper.h

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@
2222

2323
static const size_t DBWRAPPER_PREALLOC_KEY_SIZE = 64;
2424
static const size_t DBWRAPPER_PREALLOC_VALUE_SIZE = 1024;
25+
static const size_t DBWRAPPER_MAX_FILE_SIZE = 32 << 20; // 32 MiB
2526

2627
//! User-controlled performance and debug options.
2728
struct DBOptions {

0 commit comments

Comments
 (0)