From: Zygo Blaxell <bees@furryterror.org>
Date: Thu, 6 Feb 2025 06:17:09 +0000 (-0500)
Subject: main: unconditionally enable workaround for the logical_ino-vs-clone kernel bug
X-Git-Url: http://git.hungrycats.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=88b1e4ca6e6af092c5b1e58860acdcf85a2ecb6a;p=bees

main: unconditionally enable workaround for the logical_ino-vs-clone kernel bug

This obviously doesn't fix or prevent the kernel bug, but it does prevent
bees from triggering the bug without assitance from another application.

The bug can still be triggered by running bees at the same time as an
application which uses clone or LOGICAL_INO.  `btdu` uses LOGICAL_INO,
while `cp` from coreutils (and many others) use clone (reflink copy).

Signed-off-by: Zygo Blaxell <bees@furryterror.org>
---

diff --git a/src/bees.cc b/src/bees.cc
index 7d51651e..20c6db72 100644
--- a/src/bees.cc
+++ b/src/bees.cc
@@ -875,12 +875,8 @@ bees_main(int argc, char *argv[])
 	// Set root scan mode
 	bc->roots()->set_scan_mode(root_scan_mode);
 
-	if (root_scan_mode == BeesRoots::SCAN_MODE_EXTENT) {
-		MultiLocker::enable_locking(false);
-	} else {
-		// Workaround for a kernel bug that the subvol-based crawlers keep triggering
-		MultiLocker::enable_locking(true);
-	}
+	// Workaround for the logical-ino-vs-clone kernel bug
+	MultiLocker::enable_locking(true);
 
 	// Start crawlers
 	bc->start();