os/bluestore: fix the problem that estimate the log size incorrectly

wanglinke · wanglinke · commit 61058bf102f9 · 2024-03-28T10:15:17.000+08:00
In BlueFS::_estimate_log_size_N, the total size of the dir was calculated incorrectly. Fixes: https://tracker.ceph.com/issues/65176 co-author: Jrchyang Yu <yuzhiqiang_yewu@cmss.chinamobile.com> Signed-off-by: Wang Linke <wanglinke_yewu@cmss.chinamobile.com>
diff --git a/src/os/bluestore/BlueFS.cc b/src/os/bluestore/BlueFS.cc
@@ -2429,7 +2429,7 @@ uint64_t BlueFS::_estimate_log_size_N()
   int avg_file_size = 12;
   uint64_t size = 4096 * 2;
   size += nodes.file_map.size() * (1 + sizeof(bluefs_fnode_t));
-  size += nodes.dir_map.size() + (1 + avg_dir_size);
+  size += nodes.dir_map.size() * (1 + avg_dir_size);
   size += nodes.file_map.size() * (1 + avg_dir_size + avg_file_size);
   return round_up_to(size, super.block_size);
 }

Original file line number	Diff line number	Diff line change
`@@ -2429,7 +2429,7 @@ uint64_t BlueFS::_estimate_log_size_N()`
`2429`	`2429`	`int avg_file_size = 12;`
`2430`	`2430`	`uint64_t size = 4096 * 2;`
`2431`	`2431`	`size += nodes.file_map.size() * (1 + sizeof(bluefs_fnode_t));`
`2432`		`- size += nodes.dir_map.size() + (1 + avg_dir_size);`
	`2432`	`+ size += nodes.dir_map.size() * (1 + avg_dir_size);`
`2433`	`2433`	`size += nodes.file_map.size() * (1 + avg_dir_size + avg_file_size);`
`2434`	`2434`	`return round_up_to(size, super.block_size);`
`2435`	`2435`	`}`