Revert "Put lock logging behind DEBUG_LOCKCONTENTION preprocessor directive"

This reverts commit 2b7faf0.

Author: psgreco

src/logging.cpp

@@ -160,9 +160,7 @@ const CLogCategoryDesc LogCategories[] =
     {BCLog::VALIDATION, "validation"},
     {BCLog::I2P, "i2p"},
     {BCLog::IPC, "ipc"},
-#ifdef DEBUG_LOCKCONTENTION
     {BCLog::LOCK, "lock"},
-#endif
     {BCLog::UTIL, "util"},
     {BCLog::BLOCKSTORE, "blockstorage"},
     {BCLog::ALL, "1"},

src/logging.h

@@ -60,9 +60,7 @@ namespace BCLog {
         VALIDATION  = (1 << 21),
         I2P         = (1 << 22),
         IPC         = (1 << 23),
-#ifdef DEBUG_LOCKCONTENTION
         LOCK        = (1 << 24),
-#endif
         UTIL        = (1 << 25),
         BLOCKSTORE  = (1 << 26),
         ALL         = ~(uint32_t)0,

src/net.h

@@ -13,7 +13,6 @@
 #include <crypto/siphash.h>
 #include <hash.h>
 #include <i2p.h>
-#include <logging.h>
 #include <net_permissions.h>
 #include <netaddress.h>
 #include <netbase.h>
@@ -33,7 +32,6 @@
 #include <cstdint>
 #include <deque>
 #include <functional>
-#include <list>
 #include <map>
 #include <memory>
 #include <optional>

src/sync.h

@@ -6,11 +6,8 @@
 #ifndef BITCOIN_SYNC_H
 #define BITCOIN_SYNC_H
 
-#ifdef DEBUG_LOCKCONTENTION
 #include <logging.h>
 #include <logging/timer.h>
-#endif
-
 #include <threadsafety.h>
 #include <util/macros.h>
 
@@ -139,10 +136,8 @@ class SCOPED_LOCKABLE UniqueLock : public Base
     void Enter(const char* pszName, const char* pszFile, int nLine)
     {
         EnterCritical(pszName, pszFile, nLine, Base::mutex());
-#ifdef DEBUG_LOCKCONTENTION
         if (Base::try_lock()) return;
         LOG_TIME_MICROS_WITH_CATEGORY(strprintf("lock contention %s, %s:%d", pszName, pszFile, nLine), BCLog::LOCK);
-#endif
         Base::lock();
     }
 

src/test/checkqueue_tests.cpp

@@ -19,17 +19,13 @@
 #include <vector>
 
 /**
- * Identical to TestingSetup but excludes lock contention logging if
- * `DEBUG_LOCKCONTENTION` is defined, as some of these tests are designed to be
- * heavily contested to trigger race conditions or other issues.
+ * Identical to TestingSetup but excludes lock contention logging, as some of
+ * these tests are designed to be heavily contested to trigger race conditions
+ * or other issues.
  */
 struct NoLockLoggingTestingSetup : public TestingSetup {
     NoLockLoggingTestingSetup()
-#ifdef DEBUG_LOCKCONTENTION
         : TestingSetup{CBaseChainParams::MAIN, /*extra_args=*/{"-debugexclude=lock"}} {}
-#else
-        : TestingSetup{CBaseChainParams::MAIN} {}
-#endif
 };
 
 BOOST_FIXTURE_TEST_SUITE(checkqueue_tests, NoLockLoggingTestingSetup)

test/functional/rpc_misc.py

@@ -56,6 +56,9 @@ def run_test(self):
 
         self.log.info("test logging rpc and help")
 
+        # Test logging RPC returns the expected number of logging categories.
+        assert_equal(len(node.logging()), 27)
+
         # Test toggling a logging category on/off/on with the logging RPC.
         assert_equal(node.logging()['qt'], True)
         node.logging(exclude=['qt'])