From 37cc2a2d953c072a236b657bfd7de5167092a47a Mon Sep 17 00:00:00 2001
From: stickies-v <stickies-v@protonmail.com>
Date: Mon, 26 Jan 2026 18:48:08 +0000
Subject: [PATCH] logging: use util/log.h where possible

Preparation for a future commit where kernel's dependency
on logging.cpp is removed completely.

Replace usage of logging\.h with util/log\.h where it
suffices, and fix wrong includes according to iwyu.
---
 src/checkqueue.h               |  2 +-
 src/coins.cpp                  |  2 +-
 src/dbwrapper.cpp              | 35 +++++++++++++++++-----------------
 src/flatfile.cpp               |  7 ++++---
 src/index/base.cpp             |  2 +-
 src/index/blockfilterindex.cpp |  2 +-
 src/index/coinstatsindex.cpp   |  2 +-
 src/index/txindex.cpp          |  2 +-
 src/kernel/chainparams.cpp     |  2 +-
 src/kernel/coinstats.cpp       |  2 +-
 src/kernel/context.cpp         |  2 +-
 src/logging/timer.h            |  2 +-
 src/node/blockstorage.cpp      |  2 +-
 src/node/chainstate.cpp        |  2 +-
 src/node/utxo_snapshot.cpp     |  2 +-
 src/policy/truc_policy.cpp     |  1 -
 src/random.cpp                 |  2 +-
 src/script/sigcache.cpp        |  2 +-
 src/signet.cpp                 |  2 +-
 src/streams.h                  |  2 +-
 src/sync.cpp                   |  2 +-
 src/sync.h                     |  1 -
 src/txdb.cpp                   |  2 +-
 src/txmempool.cpp              |  2 +-
 src/util/asmap.cpp             |  2 +-
 src/util/batchpriority.cpp     |  2 +-
 src/util/exception.cpp         |  2 +-
 src/util/fs_helpers.cpp        |  6 +++---
 src/util/sock.cpp              |  6 ++++--
 src/util/sock.h                |  1 +
 src/util/thread.cpp            |  2 +-
 src/validation.cpp             |  4 ++--
 src/validationinterface.cpp    |  2 +-
 33 files changed, 57 insertions(+), 54 deletions(-)

diff --git a/src/checkqueue.h b/src/checkqueue.h
index 037023ee3a2..5258d7113e6 100644
--- a/src/checkqueue.h
+++ b/src/checkqueue.h
@@ -5,9 +5,9 @@
 #ifndef BITCOIN_CHECKQUEUE_H
 #define BITCOIN_CHECKQUEUE_H
 
-#include <logging.h>
 #include <sync.h>
 #include <tinyformat.h>
+#include <util/log.h>
 #include <util/threadnames.h>
 
 #include <algorithm>
diff --git a/src/coins.cpp b/src/coins.cpp
index a3bc369de30..fc33c521617 100644
--- a/src/coins.cpp
+++ b/src/coins.cpp
@@ -5,9 +5,9 @@
 #include <coins.h>
 
 #include <consensus/consensus.h>
-#include <logging.h>
 #include <random.h>
 #include <uint256.h>
+#include <util/log.h>
 #include <util/trace.h>
 
 TRACEPOINT_SEMAPHORE(utxocache, add);
diff --git a/src/dbwrapper.cpp b/src/dbwrapper.cpp
index b3f08cb20d6..eb222078b5e 100644
--- a/src/dbwrapper.cpp
+++ b/src/dbwrapper.cpp
@@ -4,21 +4,6 @@
 
 #include <dbwrapper.h>
 
-#include <logging.h>
-#include <random.h>
-#include <serialize.h>
-#include <span.h>
-#include <streams.h>
-#include <util/fs.h>
-#include <util/fs_helpers.h>
-#include <util/obfuscation.h>
-#include <util/strencodings.h>
-
-#include <algorithm>
-#include <cassert>
-#include <cstdarg>
-#include <cstdint>
-#include <cstdio>
 #include <leveldb/cache.h>
 #include <leveldb/db.h>
 #include <leveldb/env.h>
@@ -29,6 +14,22 @@
 #include <leveldb/slice.h>
 #include <leveldb/status.h>
 #include <leveldb/write_batch.h>
+#include <logging.h>
+#include <random.h>
+#include <serialize.h>
+#include <span.h>
+#include <streams.h>
+#include <util/fs.h>
+#include <util/fs_helpers.h>
+#include <util/log.h>
+#include <util/obfuscation.h>
+#include <util/strencodings.h>
+
+#include <algorithm>
+#include <cassert>
+#include <cstdarg>
+#include <cstdint>
+#include <cstdio>
 #include <memory>
 #include <optional>
 #include <utility>
@@ -57,7 +58,7 @@ public:
     // This code is adapted from posix_logger.h, which is why it is using vsprintf.
     // Please do not do this in normal code
     void Logv(const char * format, va_list ap) override {
-            if (!LogAcceptCategory(BCLog::LEVELDB, BCLog::Level::Debug)) {
+            if (!LogAcceptCategory(BCLog::LEVELDB, util::log::Level::Debug)) {
                 return;
             }
             char buffer[500];
@@ -276,7 +277,7 @@ CDBWrapper::~CDBWrapper()
 
 void CDBWrapper::WriteBatch(CDBBatch& batch, bool fSync)
 {
-    const bool log_memory = LogAcceptCategory(BCLog::LEVELDB, BCLog::Level::Debug);
+    const bool log_memory = LogAcceptCategory(BCLog::LEVELDB, util::log::Level::Debug);
     double mem_before = 0;
     if (log_memory) {
         mem_before = DynamicMemoryUsage() / 1024.0 / 1024;
diff --git a/src/flatfile.cpp b/src/flatfile.cpp
index 140217b0603..056fb9c1f92 100644
--- a/src/flatfile.cpp
+++ b/src/flatfile.cpp
@@ -3,12 +3,13 @@
 // Distributed under the MIT software license, see the accompanying
 // file COPYING or http://www.opensource.org/licenses/mit-license.php.
 
-#include <stdexcept>
-
 #include <flatfile.h>
-#include <logging.h>
+
 #include <tinyformat.h>
 #include <util/fs_helpers.h>
+#include <util/log.h>
+
+#include <stdexcept>
 
 FlatFileSeq::FlatFileSeq(fs::path dir, const char* prefix, size_t chunk_size) :
     m_dir(std::move(dir)),
diff --git a/src/index/base.cpp b/src/index/base.cpp
index 7f77d13a574..fba2f4f6c10 100644
--- a/src/index/base.cpp
+++ b/src/index/base.cpp
@@ -10,7 +10,6 @@
 #include <interfaces/chain.h>
 #include <interfaces/types.h>
 #include <kernel/types.h>
-#include <logging.h>
 #include <node/abort.h>
 #include <node/blockstorage.h>
 #include <node/context.h>
@@ -22,6 +21,7 @@
 #include <uint256.h>
 #include <undo.h>
 #include <util/fs.h>
+#include <util/log.h>
 #include <util/string.h>
 #include <util/thread.h>
 #include <util/threadinterrupt.h>
diff --git a/src/index/blockfilterindex.cpp b/src/index/blockfilterindex.cpp
index caf6d592e7e..67fa1fab318 100644
--- a/src/index/blockfilterindex.cpp
+++ b/src/index/blockfilterindex.cpp
@@ -14,7 +14,6 @@
 #include <index/db_key.h>
 #include <interfaces/chain.h>
 #include <interfaces/types.h>
-#include <logging.h>
 #include <serialize.h>
 #include <streams.h>
 #include <sync.h>
@@ -22,6 +21,7 @@
 #include <util/check.h>
 #include <util/fs.h>
 #include <util/hasher.h>
+#include <util/log.h>
 #include <util/syserror.h>
 
 #include <cerrno>
diff --git a/src/index/coinstatsindex.cpp b/src/index/coinstatsindex.cpp
index 6be3db9a897..319ffcadc63 100644
--- a/src/index/coinstatsindex.cpp
+++ b/src/index/coinstatsindex.cpp
@@ -17,7 +17,6 @@
 #include <interfaces/chain.h>
 #include <interfaces/types.h>
 #include <kernel/coinstats.h>
-#include <logging.h>
 #include <primitives/block.h>
 #include <primitives/transaction.h>
 #include <script/script.h>
@@ -26,6 +25,7 @@
 #include <undo.h>
 #include <util/check.h>
 #include <util/fs.h>
+#include <util/log.h>
 #include <validation.h>
 
 #include <compare>
diff --git a/src/index/txindex.cpp b/src/index/txindex.cpp
index d7d8cb56652..d0e88021325 100644
--- a/src/index/txindex.cpp
+++ b/src/index/txindex.cpp
@@ -10,7 +10,6 @@
 #include <index/base.h>
 #include <index/disktxpos.h>
 #include <interfaces/chain.h>
-#include <logging.h>
 #include <node/blockstorage.h>
 #include <primitives/block.h>
 #include <primitives/transaction.h>
@@ -18,6 +17,7 @@
 #include <streams.h>
 #include <uint256.h>
 #include <util/fs.h>
+#include <util/log.h>
 #include <validation.h>
 
 #include <cassert>
diff --git a/src/kernel/chainparams.cpp b/src/kernel/chainparams.cpp
index 16ebffb9a98..79cf2ff7023 100644
--- a/src/kernel/chainparams.cpp
+++ b/src/kernel/chainparams.cpp
@@ -12,13 +12,13 @@
 #include <crypto/hex_base.h>
 #include <hash.h>
 #include <kernel/messagestartchars.h>
-#include <logging.h>
 #include <primitives/block.h>
 #include <primitives/transaction.h>
 #include <script/interpreter.h>
 #include <script/script.h>
 #include <uint256.h>
 #include <util/chaintype.h>
+#include <util/log.h>
 #include <util/strencodings.h>
 
 #include <algorithm>
diff --git a/src/kernel/coinstats.cpp b/src/kernel/coinstats.cpp
index 13c0bee836f..d287ec4be6e 100644
--- a/src/kernel/coinstats.cpp
+++ b/src/kernel/coinstats.cpp
@@ -8,7 +8,6 @@
 #include <coins.h>
 #include <crypto/muhash.h>
 #include <hash.h>
-#include <logging.h>
 #include <node/blockstorage.h>
 #include <primitives/transaction.h>
 #include <script/script.h>
@@ -17,6 +16,7 @@
 #include <sync.h>
 #include <uint256.h>
 #include <util/check.h>
+#include <util/log.h>
 #include <util/overflow.h>
 #include <validation.h>
 
diff --git a/src/kernel/context.cpp b/src/kernel/context.cpp
index e74f29cbc1a..9148bf57331 100644
--- a/src/kernel/context.cpp
+++ b/src/kernel/context.cpp
@@ -5,8 +5,8 @@
 #include <kernel/context.h>
 
 #include <crypto/sha256.h>
-#include <logging.h>
 #include <random.h>
+#include <util/log.h>
 
 #include <mutex>
 #include <string>
diff --git a/src/logging/timer.h b/src/logging/timer.h
index 52a86e714ad..2b183822e96 100644
--- a/src/logging/timer.h
+++ b/src/logging/timer.h
@@ -6,7 +6,7 @@
 #ifndef BITCOIN_LOGGING_TIMER_H
 #define BITCOIN_LOGGING_TIMER_H
 
-#include <logging.h>
+#include <util/log.h>
 #include <util/macros.h>
 #include <util/time.h>
 #include <util/types.h>
diff --git a/src/node/blockstorage.cpp b/src/node/blockstorage.cpp
index 37bf1799aa4..ddaaad19c76 100644
--- a/src/node/blockstorage.cpp
+++ b/src/node/blockstorage.cpp
@@ -16,7 +16,6 @@
 #include <kernel/messagestartchars.h>
 #include <kernel/notifications_interface.h>
 #include <kernel/types.h>
-#include <logging.h>
 #include <pow.h>
 #include <primitives/block.h>
 #include <primitives/transaction.h>
@@ -31,6 +30,7 @@
 #include <util/check.h>
 #include <util/expected.h>
 #include <util/fs.h>
+#include <util/log.h>
 #include <util/obfuscation.h>
 #include <util/overflow.h>
 #include <util/result.h>
diff --git a/src/node/chainstate.cpp b/src/node/chainstate.cpp
index 930d843121a..b3e9753e5a7 100644
--- a/src/node/chainstate.cpp
+++ b/src/node/chainstate.cpp
@@ -9,7 +9,6 @@
 #include <coins.h>
 #include <consensus/params.h>
 #include <kernel/caches.h>
-#include <logging.h>
 #include <node/blockstorage.h>
 #include <sync.h>
 #include <threadsafety.h>
@@ -17,6 +16,7 @@
 #include <txdb.h>
 #include <uint256.h>
 #include <util/fs.h>
+#include <util/log.h>
 #include <util/signalinterrupt.h>
 #include <util/time.h>
 #include <util/translation.h>
diff --git a/src/node/utxo_snapshot.cpp b/src/node/utxo_snapshot.cpp
index f11e979a357..e5997b7b65b 100644
--- a/src/node/utxo_snapshot.cpp
+++ b/src/node/utxo_snapshot.cpp
@@ -4,12 +4,12 @@
 
 #include <node/utxo_snapshot.h>
 
-#include <logging.h>
 #include <streams.h>
 #include <sync.h>
 #include <tinyformat.h>
 #include <uint256.h>
 #include <util/fs.h>
+#include <util/log.h>
 #include <validation.h>
 
 #include <cassert>
diff --git a/src/policy/truc_policy.cpp b/src/policy/truc_policy.cpp
index 597d08f7063..2ecb763e6c3 100644
--- a/src/policy/truc_policy.cpp
+++ b/src/policy/truc_policy.cpp
@@ -6,7 +6,6 @@
 
 #include <coins.h>
 #include <consensus/amount.h>
-#include <logging.h>
 #include <tinyformat.h>
 #include <util/check.h>
 
diff --git a/src/random.cpp b/src/random.cpp
index aeaf6baa8e8..f1c4bb86ff2 100644
--- a/src/random.cpp
+++ b/src/random.cpp
@@ -12,12 +12,12 @@
 #include <crypto/chacha20.h>
 #include <crypto/sha256.h>
 #include <crypto/sha512.h>
-#include <logging.h>
 #include <randomenv.h>
 #include <span.h>
 #include <support/allocators/secure.h>
 #include <support/cleanse.h>
 #include <sync.h>
+#include <util/log.h>
 #include <util/time.h>
 
 #include <array>
diff --git a/src/script/sigcache.cpp b/src/script/sigcache.cpp
index 77ffbabdf09..90d1c845c36 100644
--- a/src/script/sigcache.cpp
+++ b/src/script/sigcache.cpp
@@ -6,12 +6,12 @@
 #include <script/sigcache.h>
 
 #include <crypto/sha256.h>
-#include <logging.h>
 #include <pubkey.h>
 #include <random.h>
 #include <script/interpreter.h>
 #include <span.h>
 #include <uint256.h>
+#include <util/log.h>
 
 #include <mutex>
 #include <shared_mutex>
diff --git a/src/signet.cpp b/src/signet.cpp
index d874ddc1a65..d0f8aa28515 100644
--- a/src/signet.cpp
+++ b/src/signet.cpp
@@ -7,7 +7,6 @@
 #include <consensus/merkle.h>
 #include <consensus/params.h>
 #include <consensus/validation.h>
-#include <logging.h>
 #include <primitives/block.h>
 #include <primitives/transaction.h>
 #include <script/interpreter.h>
@@ -15,6 +14,7 @@
 #include <streams.h>
 #include <uint256.h>
 #include <util/check.h>
+#include <util/log.h>
 
 #include <algorithm>
 #include <cstddef>
diff --git a/src/streams.h b/src/streams.h
index 466084e9fa0..81dbf40591a 100644
--- a/src/streams.h
+++ b/src/streams.h
@@ -6,11 +6,11 @@
 #ifndef BITCOIN_STREAMS_H
 #define BITCOIN_STREAMS_H
 
-#include <logging.h>
 #include <serialize.h>
 #include <span.h>
 #include <support/allocators/zeroafterfree.h>
 #include <util/check.h>
+#include <util/log.h>
 #include <util/obfuscation.h>
 #include <util/overflow.h>
 #include <util/syserror.h>
diff --git a/src/sync.cpp b/src/sync.cpp
index b661aad94b6..e59f86b94fc 100644
--- a/src/sync.cpp
+++ b/src/sync.cpp
@@ -4,8 +4,8 @@
 
 #include <sync.h>
 
-#include <logging.h>
 #include <tinyformat.h>
+#include <util/log.h>
 #include <util/strencodings.h>
 #include <util/threadnames.h>
 
diff --git a/src/sync.h b/src/sync.h
index 01e92e6e42c..548b51d9d16 100644
--- a/src/sync.h
+++ b/src/sync.h
@@ -7,7 +7,6 @@
 #define BITCOIN_SYNC_H
 
 #ifdef DEBUG_LOCKCONTENTION
-#include <logging.h>
 #include <logging/timer.h>
 #endif
 
diff --git a/src/txdb.cpp b/src/txdb.cpp
index 141c31c5d51..21f6eb93867 100644
--- a/src/txdb.cpp
+++ b/src/txdb.cpp
@@ -7,11 +7,11 @@
 
 #include <coins.h>
 #include <dbwrapper.h>
-#include <logging.h>
 #include <primitives/transaction.h>
 #include <random.h>
 #include <serialize.h>
 #include <uint256.h>
+#include <util/log.h>
 #include <util/vector.h>
 
 #include <cassert>
diff --git a/src/txmempool.cpp b/src/txmempool.cpp
index 2486270bc0d..55c332769d4 100644
--- a/src/txmempool.cpp
+++ b/src/txmempool.cpp
@@ -11,13 +11,13 @@
 #include <consensus/consensus.h>
 #include <consensus/tx_verify.h>
 #include <consensus/validation.h>
-#include <logging.h>
 #include <policy/policy.h>
 #include <policy/settings.h>
 #include <random.h>
 #include <tinyformat.h>
 #include <util/check.h>
 #include <util/feefrac.h>
+#include <util/log.h>
 #include <util/moneystr.h>
 #include <util/overflow.h>
 #include <util/result.h>
diff --git a/src/util/asmap.cpp b/src/util/asmap.cpp
index 4d4ed4fe869..f8b5a527a93 100644
--- a/src/util/asmap.cpp
+++ b/src/util/asmap.cpp
@@ -6,11 +6,11 @@
 
 #include <clientversion.h>
 #include <hash.h>
-#include <logging.h>
 #include <serialize.h>
 #include <streams.h>
 #include <uint256.h>
 #include <util/fs.h>
+#include <util/log.h>
 
 #include <algorithm>
 #include <bit>
diff --git a/src/util/batchpriority.cpp b/src/util/batchpriority.cpp
index 86abc011513..e6bb43b4aca 100644
--- a/src/util/batchpriority.cpp
+++ b/src/util/batchpriority.cpp
@@ -4,7 +4,7 @@
 
 #include <util/batchpriority.h>
 
-#include <logging.h>
+#include <util/log.h>
 #include <util/syserror.h>
 
 #include <string>
diff --git a/src/util/exception.cpp b/src/util/exception.cpp
index a4421226551..cd2bd231879 100644
--- a/src/util/exception.cpp
+++ b/src/util/exception.cpp
@@ -5,8 +5,8 @@
 
 #include <util/exception.h>
 
-#include <logging.h>
 #include <tinyformat.h>
+#include <util/log.h>
 
 #include <exception>
 #include <iostream>
diff --git a/src/util/fs_helpers.cpp b/src/util/fs_helpers.cpp
index 98646c021ea..e7780d743c1 100644
--- a/src/util/fs_helpers.cpp
+++ b/src/util/fs_helpers.cpp
@@ -3,13 +3,13 @@
 // Distributed under the MIT software license, see the accompanying
 // file COPYING or http://www.opensource.org/licenses/mit-license.php.
 
-#include <util/fs_helpers.h>
-
 #include <bitcoin-build-config.h> // IWYU pragma: keep
 
-#include <logging.h>
+#include <util/fs_helpers.h>
+
 #include <sync.h>
 #include <util/fs.h>
+#include <util/log.h>
 #include <util/syserror.h>
 
 #include <cerrno>
diff --git a/src/util/sock.cpp b/src/util/sock.cpp
index ba822a0eaf7..f13406ebe5e 100644
--- a/src/util/sock.cpp
+++ b/src/util/sock.cpp
@@ -2,11 +2,13 @@
 // Distributed under the MIT software license, see the accompanying
 // file COPYING or http://www.opensource.org/licenses/mit-license.php.
 
+#include <util/sock.h>
+
 #include <common/system.h>
 #include <compat/compat.h>
-#include <logging.h>
+#include <span.h>
 #include <tinyformat.h>
-#include <util/sock.h>
+#include <util/log.h>
 #include <util/syserror.h>
 #include <util/threadinterrupt.h>
 #include <util/time.h>
diff --git a/src/util/sock.h b/src/util/sock.h
index 69ec1b8e01d..8c51b8ea6ea 100644
--- a/src/util/sock.h
+++ b/src/util/sock.h
@@ -11,6 +11,7 @@
 
 #include <chrono>
 #include <memory>
+#include <span>
 #include <string>
 #include <unordered_map>
 
diff --git a/src/util/thread.cpp b/src/util/thread.cpp
index 7f8fa20a237..0fde73c4e29 100644
--- a/src/util/thread.cpp
+++ b/src/util/thread.cpp
@@ -4,8 +4,8 @@
 
 #include <util/thread.h>
 
-#include <logging.h>
 #include <util/exception.h>
+#include <util/log.h>
 #include <util/threadnames.h>
 
 #include <exception>
diff --git a/src/validation.cpp b/src/validation.cpp
index f2819695da2..c200c3d6cd8 100644
--- a/src/validation.cpp
+++ b/src/validation.cpp
@@ -28,7 +28,6 @@
 #include <kernel/notifications_interface.h>
 #include <kernel/types.h>
 #include <kernel/warning.h>
-#include <logging.h>
 #include <logging/timer.h>
 #include <node/blockstorage.h>
 #include <node/utxo_snapshot.h>
@@ -53,6 +52,7 @@
 #include <util/fs.h>
 #include <util/fs_helpers.h>
 #include <util/hasher.h>
+#include <util/log.h>
 #include <util/moneystr.h>
 #include <util/rbf.h>
 #include <util/result.h>
@@ -2870,7 +2870,7 @@ static void UpdateTipLog(
     AssertLockHeld(::cs_main);
 
     // Disable rate limiting in LogPrintLevel_ so this source location may log during IBD.
-    LogPrintLevel_(BCLog::LogFlags::ALL, BCLog::Level::Info, /*should_ratelimit=*/false, "%s%s: new best=%s height=%d version=0x%08x log2_work=%f tx=%lu date='%s' progress=%f cache=%.1fMiB(%utxo)%s\n",
+    LogPrintLevel_(BCLog::LogFlags::ALL, util::log::Level::Info, /*should_ratelimit=*/false, "%s%s: new best=%s height=%d version=0x%08x log2_work=%f tx=%lu date='%s' progress=%f cache=%.1fMiB(%utxo)%s\n",
                    prefix, func_name,
                    tip->GetBlockHash().ToString(), tip->nHeight, tip->nVersion,
                    log(tip->nChainWork.getdouble()) / log(2.0), tip->m_chain_tx_count,
diff --git a/src/validationinterface.cpp b/src/validationinterface.cpp
index 586d29f4806..c7be6abc3a1 100644
--- a/src/validationinterface.cpp
+++ b/src/validationinterface.cpp
@@ -10,10 +10,10 @@
 #include <kernel/mempool_entry.h>
 #include <kernel/mempool_removal_reason.h>
 #include <kernel/types.h>
-#include <logging.h>
 #include <primitives/block.h>
 #include <primitives/transaction.h>
 #include <util/check.h>
+#include <util/log.h>
 #include <util/task_runner.h>
 
 #include <future>