ci: Use clang-19 from apt.llvm.org #30634

pull maflcko wants to merge 1 commits into bitcoin:master from maflcko:2408-ci-clang-19 changing 4 files +23 −9
  1. maflcko commented at 10:34 am on August 12, 2024: member

    A new clang version generally comes with bugfixes, new sanitizer features, deprecations, as well as new features.

    Upgrade the sanitizer tasks to use the new version.

  2. DrahtBot commented at 10:34 am on August 12, 2024: contributor

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Code Coverage

    For detailed information about the code coverage, see the test coverage report.

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK TheCharlatan, hebasto
    Concept ACK l0rinc

    If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #30997 (build: Switch to Qt 6 by hebasto)
    • #28710 (Remove the legacy wallet and BDB dependency by achow101)

    If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

  3. DrahtBot added the label Tests on Aug 12, 2024
  4. maflcko force-pushed on Aug 12, 2024
  5. DrahtBot added the label CI failed on Aug 12, 2024
  6. DrahtBot commented at 12:49 pm on August 12, 2024: contributor

    🚧 At least one of the CI tasks failed. Debug: https://github.com/bitcoin/bitcoin/runs/28646383146

    Make sure to run all tests locally, according to the documentation.

    The failure may happen due to a number of reasons, for example:

    • Possibly due to a silent merge conflict (the changes in this pull request being incompatible with the current code in the target branch). If so, make sure to rebase on the latest commit of the target branch.

    • A sanitizer issue, which can only be found by compiling with the sanitizer and running the affected test.

    • An intermittent issue.

    Leave a comment here, if you need help tracking down a confusing failure.

  7. DrahtBot removed the label CI failed on Aug 12, 2024
  8. hebasto added the label Needs CMake port on Aug 16, 2024
  9. DrahtBot added the label Needs rebase on Aug 28, 2024
  10. maflcko removed the label Needs CMake port on Aug 29, 2024
  11. maflcko force-pushed on Aug 29, 2024
  12. DrahtBot removed the label Needs rebase on Aug 29, 2024
  13. maflcko commented at 7:34 am on August 29, 2024: member

    rebased.

    A follow-up can explore if the tidy task, or the valgrind tasks can also use a newer compiler version.

  14. maflcko force-pushed on Aug 29, 2024
  15. maflcko force-pushed on Sep 6, 2024
  16. maflcko force-pushed on Sep 6, 2024
  17. DrahtBot added the label CI failed on Sep 7, 2024
  18. DrahtBot removed the label CI failed on Sep 15, 2024
  19. maflcko force-pushed on Sep 17, 2024
  20. maflcko commented at 7:45 pm on September 17, 2024: member

    testing tsan on aarch gives me:

      0src/wallet/test/wallet_tests.cpp(75): Leaving test case "scan_for_wallet_transactions"; testing time: 2316025us
  1src/wallet/test/wallet_tests.cpp(921): Entering test case "CreateWalletWithoutChain"
  2==================
  3WARNING: ThreadSanitizer: lock-order-inversion (potential deadlock) (pid=62275)
  4  Cycle in lock order graph: M0 (0xffff74604608) => M1 (0xffff7460cb58) => M2 (0xffff7460cd08) => M0
  5
  6  Mutex M1 acquired here while holding mutex M0 in main thread:
  7    [#0](/bitcoin-bitcoin/0/) pthread_rwlock_wrlock <null> (test_bitcoin+0x16b704) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
  8    [#1](/bitcoin-bitcoin/1/) __db_pthread_mutex_lock <null> (test_bitcoin+0x1744afc) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
  9    [#2](/bitcoin-bitcoin/2/) wallet::BerkeleyBatch::BerkeleyBatch(wallet::BerkeleyDatabase&, bool, bool) ci/scratch/build-aarch64-unknown-linux-gnu/src/wallet/./src/wallet/bdb.cpp:366:14 (test_bitcoin+0x12ee4e8) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 10    [#3](/bitcoin-bitcoin/3/) std::__1::__unique_if<wallet::BerkeleyBatch>::__unique_single std::__1::make_unique[abi:ne190100]<wallet::BerkeleyBatch, wallet::BerkeleyDatabase&, bool, bool&>(wallet::BerkeleyDatabase&, bool&&, bool&) /usr/lib/llvm-19/bin/../include/c++/v1/__memory/unique_ptr.h:634:30 (test_bitcoin+0x12f44e0) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 11    [#4](/bitcoin-bitcoin/4/) wallet::BerkeleyDatabase::MakeBatch(bool) ci/scratch/build-aarch64-unknown-linux-gnu/src/wallet/./src/wallet/bdb.cpp:945:12 (test_bitcoin+0x12f44e0)
 12    [#5](/bitcoin-bitcoin/5/) wallet::WalletBatch::WalletBatch(wallet::WalletDatabase&, bool) ci/scratch/build-aarch64-unknown-linux-gnu/src/wallet/./src/wallet/walletdb.h:221:26 (test_bitcoin+0x1287688) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 13    [#6](/bitcoin-bitcoin/6/) wallet::CWallet::LoadWallet() ci/scratch/build-aarch64-unknown-linux-gnu/src/wallet/./src/wallet/wallet.cpp:2363:31 (test_bitcoin+0x1287688)
 14    [#7](/bitcoin-bitcoin/7/) wallet::CWallet::Create(wallet::WalletContext&, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char>> const&, std::__1::unique_ptr<wallet::WalletDatabase, std::__1::default_delete<wallet::WalletDatabase>>, unsigned long, bilingual_str&, std::__1::vector<bilingual_str, std::__1::allocator<bilingual_str>>&) ci/scratch/build-aarch64-unknown-linux-gnu/src/wallet/./src/wallet/wallet.cpp:2987:47 (test_bitcoin+0x126e2cc) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 15    [#8](/bitcoin-bitcoin/8/) wallet::TestLoadWallet(std::__1::unique_ptr<wallet::WalletDatabase, std::__1::default_delete<wallet::WalletDatabase>>, wallet::WalletContext&, unsigned long) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/util/./src/wallet/test/util.cpp:54:19 (test_bitcoin+0xcc8c40) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 16    [#9](/bitcoin-bitcoin/9/) wallet::TestLoadWallet(wallet::WalletContext&) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/util/./src/wallet/test/util.cpp:70:12 (test_bitcoin+0xcc8fa8) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 17    [#10](/bitcoin-bitcoin/10/) wallet::wallet_tests::CreateWalletWithoutChain::test_method() ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./src/wallet/test/wallet_tests.cpp:925:19 (test_bitcoin+0xc59a68) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 18    [#11](/bitcoin-bitcoin/11/) wallet::wallet_tests::CreateWalletWithoutChain_invoker() ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./src/wallet/test/wallet_tests.cpp:921:1 (test_bitcoin+0xc5920c) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 19    [#12](/bitcoin-bitcoin/12/) boost::detail::function::void_function_invoker0<void (*)(), void>::invoke(boost::detail::function::function_buffer&) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/function/function_template.hpp:117:11 (test_bitcoin+0x2eb3ec) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 20    [#13](/bitcoin-bitcoin/13/) boost::function0<void>::operator()() const ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/function/function_template.hpp:763:14 (test_bitcoin+0x28117c) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 21    [#14](/bitcoin-bitcoin/14/) boost::detail::forward::operator()() ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/execution_monitor.ipp:1388:32 (test_bitcoin+0x28117c)
 22    [#15](/bitcoin-bitcoin/15/) boost::detail::function::function_obj_invoker0<boost::detail::forward, int>::invoke(boost::detail::function::function_buffer&) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/function/function_template.hpp:137:18 (test_bitcoin+0x28117c)
 23    [#16](/bitcoin-bitcoin/16/) boost::function0<int>::operator()() const ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/function/function_template.hpp:763:14 (test_bitcoin+0x211b04) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 24    [#17](/bitcoin-bitcoin/17/) int boost::detail::do_invoke<boost::shared_ptr<boost::detail::translator_holder_base>, boost::function<int ()>>(boost::shared_ptr<boost::detail::translator_holder_base> const&, boost::function<int ()> const&) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/execution_monitor.ipp:301:30 (test_bitcoin+0x211b04)
 25    [#18](/bitcoin-bitcoin/18/) boost::execution_monitor::catch_signals(boost::function<int ()> const&) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/execution_monitor.ipp:903:16 (test_bitcoin+0x211b04)
 26    [#19](/bitcoin-bitcoin/19/) boost::execution_monitor::execute(boost::function<int ()> const&) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/execution_monitor.ipp:1301:16 (test_bitcoin+0x211e14) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 27    [#20](/bitcoin-bitcoin/20/) boost::execution_monitor::vexecute(boost::function<void ()> const&) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/execution_monitor.ipp:1397:5 (test_bitcoin+0x20e0a8) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 28    [#21](/bitcoin-bitcoin/21/) boost::unit_test::unit_test_monitor_t::execute_and_translate(boost::function<void ()> const&, unsigned long) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/unit_test_monitor.ipp:49:9 (test_bitcoin+0x20e0a8)
 29    [#22](/bitcoin-bitcoin/22/) boost::unit_test::framework::state::execute_test_tree(unsigned long, unsigned long, boost::unit_test::framework::state::random_generator_helper const*) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/framework.ipp:815:44 (test_bitcoin+0x23c8ec) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 30    [#23](/bitcoin-bitcoin/23/) boost::unit_test::framework::state::execute_test_tree(unsigned long, unsigned long, boost::unit_test::framework::state::random_generator_helper const*) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/framework.ipp:784:58 (test_bitcoin+0x23ce08) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 31    [#24](/bitcoin-bitcoin/24/) boost::unit_test::framework::state::execute_test_tree(unsigned long, unsigned long, boost::unit_test::framework::state::random_generator_helper const*) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/framework.ipp:784:58 (test_bitcoin+0x23ce08) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 32    [#25](/bitcoin-bitcoin/25/) boost::unit_test::framework::run(unsigned long, bool) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/framework.ipp:1722:29 (test_bitcoin+0x20cfa4) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 33    [#26](/bitcoin-bitcoin/26/) boost::unit_test::unit_test_main(boost::unit_test::test_suite* (*)(int, char**), int, char**) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/unit_test_main.ipp:250:9 (test_bitcoin+0x224914) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 34    [#27](/bitcoin-bitcoin/27/) main ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/unit_test_main.ipp:306:12 (test_bitcoin+0x22512c) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 35
 36    Hint: use TSAN_OPTIONS=second_deadlock_stack=1 to get more informative warning message
 37
 38  Mutex M2 acquired here while holding mutex M1 in main thread:
 39    [#0](/bitcoin-bitcoin/0/) pthread_rwlock_wrlock <null> (test_bitcoin+0x16b704) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 40    [#1](/bitcoin-bitcoin/1/) __db_pthread_mutex_lock <null> (test_bitcoin+0x1744afc) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 41    [#2](/bitcoin-bitcoin/2/) wallet::BerkeleyBatch::BerkeleyBatch(wallet::BerkeleyDatabase&, bool, bool) ci/scratch/build-aarch64-unknown-linux-gnu/src/wallet/./src/wallet/bdb.cpp:366:14 (test_bitcoin+0x12ee4e8) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 42    [#3](/bitcoin-bitcoin/3/) std::__1::__unique_if<wallet::BerkeleyBatch>::__unique_single std::__1::make_unique[abi:ne190100]<wallet::BerkeleyBatch, wallet::BerkeleyDatabase&, bool, bool&>(wallet::BerkeleyDatabase&, bool&&, bool&) /usr/lib/llvm-19/bin/../include/c++/v1/__memory/unique_ptr.h:634:30 (test_bitcoin+0x12f44e0) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 43    [#4](/bitcoin-bitcoin/4/) wallet::BerkeleyDatabase::MakeBatch(bool) ci/scratch/build-aarch64-unknown-linux-gnu/src/wallet/./src/wallet/bdb.cpp:945:12 (test_bitcoin+0x12f44e0)
 44    [#5](/bitcoin-bitcoin/5/) wallet::WalletBatch::WalletBatch(wallet::WalletDatabase&, bool) ci/scratch/build-aarch64-unknown-linux-gnu/src/wallet/./src/wallet/walletdb.h:221:26 (test_bitcoin+0x1287688) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 45    [#6](/bitcoin-bitcoin/6/) wallet::CWallet::LoadWallet() ci/scratch/build-aarch64-unknown-linux-gnu/src/wallet/./src/wallet/wallet.cpp:2363:31 (test_bitcoin+0x1287688)
 46    [#7](/bitcoin-bitcoin/7/) wallet::CWallet::Create(wallet::WalletContext&, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char>> const&, std::__1::unique_ptr<wallet::WalletDatabase, std::__1::default_delete<wallet::WalletDatabase>>, unsigned long, bilingual_str&, std::__1::vector<bilingual_str, std::__1::allocator<bilingual_str>>&) ci/scratch/build-aarch64-unknown-linux-gnu/src/wallet/./src/wallet/wallet.cpp:2987:47 (test_bitcoin+0x126e2cc) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 47    [#8](/bitcoin-bitcoin/8/) wallet::TestLoadWallet(std::__1::unique_ptr<wallet::WalletDatabase, std::__1::default_delete<wallet::WalletDatabase>>, wallet::WalletContext&, unsigned long) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/util/./src/wallet/test/util.cpp:54:19 (test_bitcoin+0xcc8c40) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 48    [#9](/bitcoin-bitcoin/9/) wallet::TestLoadWallet(wallet::WalletContext&) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/util/./src/wallet/test/util.cpp:70:12 (test_bitcoin+0xcc8fa8) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 49    [#10](/bitcoin-bitcoin/10/) wallet::wallet_tests::CreateWalletWithoutChain::test_method() ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./src/wallet/test/wallet_tests.cpp:925:19 (test_bitcoin+0xc59a68) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 50    [#11](/bitcoin-bitcoin/11/) wallet::wallet_tests::CreateWalletWithoutChain_invoker() ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./src/wallet/test/wallet_tests.cpp:921:1 (test_bitcoin+0xc5920c) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 51    [#12](/bitcoin-bitcoin/12/) boost::detail::function::void_function_invoker0<void (*)(), void>::invoke(boost::detail::function::function_buffer&) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/function/function_template.hpp:117:11 (test_bitcoin+0x2eb3ec) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 52    [#13](/bitcoin-bitcoin/13/) boost::function0<void>::operator()() const ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/function/function_template.hpp:763:14 (test_bitcoin+0x28117c) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 53    [#14](/bitcoin-bitcoin/14/) boost::detail::forward::operator()() ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/execution_monitor.ipp:1388:32 (test_bitcoin+0x28117c)
 54    [#15](/bitcoin-bitcoin/15/) boost::detail::function::function_obj_invoker0<boost::detail::forward, int>::invoke(boost::detail::function::function_buffer&) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/function/function_template.hpp:137:18 (test_bitcoin+0x28117c)
 55    [#16](/bitcoin-bitcoin/16/) boost::function0<int>::operator()() const ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/function/function_template.hpp:763:14 (test_bitcoin+0x211b04) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 56    [#17](/bitcoin-bitcoin/17/) int boost::detail::do_invoke<boost::shared_ptr<boost::detail::translator_holder_base>, boost::function<int ()>>(boost::shared_ptr<boost::detail::translator_holder_base> const&, boost::function<int ()> const&) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/execution_monitor.ipp:301:30 (test_bitcoin+0x211b04)
 57    [#18](/bitcoin-bitcoin/18/) boost::execution_monitor::catch_signals(boost::function<int ()> const&) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/execution_monitor.ipp:903:16 (test_bitcoin+0x211b04)
 58    [#19](/bitcoin-bitcoin/19/) boost::execution_monitor::execute(boost::function<int ()> const&) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/execution_monitor.ipp:1301:16 (test_bitcoin+0x211e14) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 59    [#20](/bitcoin-bitcoin/20/) boost::execution_monitor::vexecute(boost::function<void ()> const&) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/execution_monitor.ipp:1397:5 (test_bitcoin+0x20e0a8) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 60    [#21](/bitcoin-bitcoin/21/) boost::unit_test::unit_test_monitor_t::execute_and_translate(boost::function<void ()> const&, unsigned long) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/unit_test_monitor.ipp:49:9 (test_bitcoin+0x20e0a8)
 61    [#22](/bitcoin-bitcoin/22/) boost::unit_test::framework::state::execute_test_tree(unsigned long, unsigned long, boost::unit_test::framework::state::random_generator_helper const*) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/framework.ipp:815:44 (test_bitcoin+0x23c8ec) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 62    [#23](/bitcoin-bitcoin/23/) boost::unit_test::framework::state::execute_test_tree(unsigned long, unsigned long, boost::unit_test::framework::state::random_generator_helper const*) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/framework.ipp:784:58 (test_bitcoin+0x23ce08) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 63    [#24](/bitcoin-bitcoin/24/) boost::unit_test::framework::state::execute_test_tree(unsigned long, unsigned long, boost::unit_test::framework::state::random_generator_helper const*) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/framework.ipp:784:58 (test_bitcoin+0x23ce08) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 64    [#25](/bitcoin-bitcoin/25/) boost::unit_test::framework::run(unsigned long, bool) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/framework.ipp:1722:29 (test_bitcoin+0x20cfa4) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 65    [#26](/bitcoin-bitcoin/26/) boost::unit_test::unit_test_main(boost::unit_test::test_suite* (*)(int, char**), int, char**) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/unit_test_main.ipp:250:9 (test_bitcoin+0x224914) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 66    [#27](/bitcoin-bitcoin/27/) main ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/unit_test_main.ipp:306:12 (test_bitcoin+0x22512c) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 67
 68  Mutex M0 acquired here while holding mutex M2 in main thread:
 69    [#0](/bitcoin-bitcoin/0/) pthread_rwlock_rdlock <null> (test_bitcoin+0x16b184) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 70    [#1](/bitcoin-bitcoin/1/) __db_pthread_mutex_readlock <null> (test_bitcoin+0x1744d7c) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 71    [#2](/bitcoin-bitcoin/2/) wallet::BerkeleyBatch::BerkeleyBatch(wallet::BerkeleyDatabase&, bool, bool) ci/scratch/build-aarch64-unknown-linux-gnu/src/wallet/./src/wallet/bdb.cpp:366:14 (test_bitcoin+0x12ee4e8) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 72    [#3](/bitcoin-bitcoin/3/) std::__1::__unique_if<wallet::BerkeleyBatch>::__unique_single std::__1::make_unique[abi:ne190100]<wallet::BerkeleyBatch, wallet::BerkeleyDatabase&, bool, bool&>(wallet::BerkeleyDatabase&, bool&&, bool&) /usr/lib/llvm-19/bin/../include/c++/v1/__memory/unique_ptr.h:634:30 (test_bitcoin+0x12f44e0) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 73    [#4](/bitcoin-bitcoin/4/) wallet::BerkeleyDatabase::MakeBatch(bool) ci/scratch/build-aarch64-unknown-linux-gnu/src/wallet/./src/wallet/bdb.cpp:945:12 (test_bitcoin+0x12f44e0)
 74    [#5](/bitcoin-bitcoin/5/) wallet::WalletBatch::WalletBatch(wallet::WalletDatabase&, bool) ci/scratch/build-aarch64-unknown-linux-gnu/src/wallet/./src/wallet/walletdb.h:221:26 (test_bitcoin+0x1287688) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 75    [#6](/bitcoin-bitcoin/6/) wallet::CWallet::LoadWallet() ci/scratch/build-aarch64-unknown-linux-gnu/src/wallet/./src/wallet/wallet.cpp:2363:31 (test_bitcoin+0x1287688)
 76    [#7](/bitcoin-bitcoin/7/) wallet::CWallet::Create(wallet::WalletContext&, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char>> const&, std::__1::unique_ptr<wallet::WalletDatabase, std::__1::default_delete<wallet::WalletDatabase>>, unsigned long, bilingual_str&, std::__1::vector<bilingual_str, std::__1::allocator<bilingual_str>>&) ci/scratch/build-aarch64-unknown-linux-gnu/src/wallet/./src/wallet/wallet.cpp:2987:47 (test_bitcoin+0x126e2cc) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 77    [#8](/bitcoin-bitcoin/8/) wallet::TestLoadWallet(std::__1::unique_ptr<wallet::WalletDatabase, std::__1::default_delete<wallet::WalletDatabase>>, wallet::WalletContext&, unsigned long) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/util/./src/wallet/test/util.cpp:54:19 (test_bitcoin+0xcc8c40) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 78    [#9](/bitcoin-bitcoin/9/) wallet::TestLoadWallet(wallet::WalletContext&) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/util/./src/wallet/test/util.cpp:70:12 (test_bitcoin+0xcc8fa8) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 79    [#10](/bitcoin-bitcoin/10/) wallet::wallet_tests::CreateWalletWithoutChain::test_method() ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./src/wallet/test/wallet_tests.cpp:925:19 (test_bitcoin+0xc59a68) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 80    [#11](/bitcoin-bitcoin/11/) wallet::wallet_tests::CreateWalletWithoutChain_invoker() ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./src/wallet/test/wallet_tests.cpp:921:1 (test_bitcoin+0xc5920c) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 81    [#12](/bitcoin-bitcoin/12/) boost::detail::function::void_function_invoker0<void (*)(), void>::invoke(boost::detail::function::function_buffer&) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/function/function_template.hpp:117:11 (test_bitcoin+0x2eb3ec) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 82    [#13](/bitcoin-bitcoin/13/) boost::function0<void>::operator()() const ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/function/function_template.hpp:763:14 (test_bitcoin+0x28117c) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 83    [#14](/bitcoin-bitcoin/14/) boost::detail::forward::operator()() ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/execution_monitor.ipp:1388:32 (test_bitcoin+0x28117c)
 84    [#15](/bitcoin-bitcoin/15/) boost::detail::function::function_obj_invoker0<boost::detail::forward, int>::invoke(boost::detail::function::function_buffer&) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/function/function_template.hpp:137:18 (test_bitcoin+0x28117c)
 85    [#16](/bitcoin-bitcoin/16/) boost::function0<int>::operator()() const ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/function/function_template.hpp:763:14 (test_bitcoin+0x211b04) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 86    [#17](/bitcoin-bitcoin/17/) int boost::detail::do_invoke<boost::shared_ptr<boost::detail::translator_holder_base>, boost::function<int ()>>(boost::shared_ptr<boost::detail::translator_holder_base> const&, boost::function<int ()> const&) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/execution_monitor.ipp:301:30 (test_bitcoin+0x211b04)
 87    [#18](/bitcoin-bitcoin/18/) boost::execution_monitor::catch_signals(boost::function<int ()> const&) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/execution_monitor.ipp:903:16 (test_bitcoin+0x211b04)
 88    [#19](/bitcoin-bitcoin/19/) boost::execution_monitor::execute(boost::function<int ()> const&) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/execution_monitor.ipp:1301:16 (test_bitcoin+0x211e14) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 89    [#20](/bitcoin-bitcoin/20/) boost::execution_monitor::vexecute(boost::function<void ()> const&) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/execution_monitor.ipp:1397:5 (test_bitcoin+0x20e0a8) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 90    [#21](/bitcoin-bitcoin/21/) boost::unit_test::unit_test_monitor_t::execute_and_translate(boost::function<void ()> const&, unsigned long) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/unit_test_monitor.ipp:49:9 (test_bitcoin+0x20e0a8)
 91    [#22](/bitcoin-bitcoin/22/) boost::unit_test::framework::state::execute_test_tree(unsigned long, unsigned long, boost::unit_test::framework::state::random_generator_helper const*) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/framework.ipp:815:44 (test_bitcoin+0x23c8ec) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 92    [#23](/bitcoin-bitcoin/23/) boost::unit_test::framework::state::execute_test_tree(unsigned long, unsigned long, boost::unit_test::framework::state::random_generator_helper const*) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/framework.ipp:784:58 (test_bitcoin+0x23ce08) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 93    [#24](/bitcoin-bitcoin/24/) boost::unit_test::framework::state::execute_test_tree(unsigned long, unsigned long, boost::unit_test::framework::state::random_generator_helper const*) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/framework.ipp:784:58 (test_bitcoin+0x23ce08) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 94    [#25](/bitcoin-bitcoin/25/) boost::unit_test::framework::run(unsigned long, bool) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/framework.ipp:1722:29 (test_bitcoin+0x20cfa4) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 95    [#26](/bitcoin-bitcoin/26/) boost::unit_test::unit_test_main(boost::unit_test::test_suite* (*)(int, char**), int, char**) ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/unit_test_main.ipp:250:9 (test_bitcoin+0x224914) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 96    [#27](/bitcoin-bitcoin/27/) main ci/scratch/build-aarch64-unknown-linux-gnu/src/test/./depends/aarch64-unknown-linux-gnu/include/boost/test/impl/unit_test_main.ipp:306:12 (test_bitcoin+0x22512c) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572)
 97
 98SUMMARY: ThreadSanitizer: lock-order-inversion (potential deadlock) (/ci_container_base/ci/scratch/build-aarch64-unknown-linux-gnu/src/test/test_bitcoin+0x16b704) (BuildId: f04f46ca28a9a2c623cb88a2e41ac6a1381f7572) in pthread_rwlock_wrlock
 99==================
100
101127/137 Test [#118](/bitcoin-bitcoin/118/): validation_chainstatemanager_tests ...   Passed   73.51 sec
102128/137 Test [#136](/bitcoin-bitcoin/136/): walletload_tests .....................   Passed   40.86 sec
103129/137 Test [#111](/bitcoin-bitcoin/111/): txvalidationcache_tests ..............   Passed  111.55 sec
104130/137 Test [#116](/bitcoin-bitcoin/116/): validation_block_tests ...............   Passed  107.92 sec
105131/137 Test  [#77](/bitcoin-bitcoin/77/): random_tests .........................   Passed  201.19 sec
106132/137 Test   [#7](/bitcoin-bitcoin/7/): exhaustive_tests .....................   Passed  253.06 sec
107133/137 Test   [#9](/bitcoin-bitcoin/9/): bench_sanity_check_high_priority .....   Passed  286.50 sec
108134/137 Test  [#31](/bitcoin-bitcoin/31/): coins_tests ..........................   Passed  328.75 sec
109135/137 Test [#123](/bitcoin-bitcoin/123/): coinselector_tests ...................   Passed  245.65 sec
110136/137 Test   [#5](/bitcoin-bitcoin/5/): noverify_tests .......................   Passed  616.08 sec
111137/137 Test   [#6](/bitcoin-bitcoin/6/): tests ................................   Passed  1415.82 sec
112
11398% tests passed, 3 tests failed out of 137
114
115Total Test time (real) = 1415.98 sec
116
117The following tests FAILED:
118	133 - wallet_tests (Failed)
119	135 - walletdb_tests (Failed)
120	137 - db_tests (Failed)
121Errors while running CTest

    Not sure if this is a regression, or similar to https://github.com/bitcoin/bitcoin/issues/30764

  21. maflcko commented at 8:19 pm on September 17, 2024: member
    Testing on aarch64, Asan and Fuzz pass. The Tsan issue on aarch64 is pre-existing and unrelated to this pull request, so can be ignored.
  22. fanquake referenced this in commit fd08fded63 on Sep 18, 2024
  23. maflcko force-pushed on Sep 19, 2024
  24. DrahtBot added the label Needs rebase on Sep 19, 2024
  25. maflcko force-pushed on Sep 19, 2024
  26. DrahtBot removed the label Needs rebase on Sep 19, 2024
  27. DrahtBot added the label Needs rebase on Sep 30, 2024
  28. maflcko force-pushed on Oct 1, 2024
  29. DrahtBot removed the label Needs rebase on Oct 1, 2024
  30. maflcko force-pushed on Oct 16, 2024
  31. maflcko force-pushed on Oct 23, 2024
  32. DrahtBot added the label Needs rebase on Oct 28, 2024
  33. ci: Use clang-19 from apt.llvm.org fabe90c824
  34. maflcko force-pushed on Oct 28, 2024
  35. DrahtBot removed the label Needs rebase on Oct 28, 2024
  36. maflcko added the label Needs rebase on Oct 28, 2024
  37. DrahtBot removed the label Needs rebase on Oct 28, 2024
  38. l0rinc commented at 7:59 pm on October 28, 2024: contributor
    Concept ACK
  39. TheCharlatan approved
  40. TheCharlatan commented at 1:51 pm on October 29, 2024: contributor
    ACK fabe90c8242aa45a8b9925347ca6fc11d5852ffd
  41. DrahtBot requested review from l0rinc on Oct 29, 2024
  42. hebasto approved
  43. hebasto commented at 2:19 pm on October 29, 2024: member
    ACK fabe90c8242aa45a8b9925347ca6fc11d5852ffd, I have reviewed the code and it looks OK.
  44. fanquake merged this on Nov 6, 2024
  45. fanquake closed this on Nov 6, 2024
  46. maflcko deleted the branch on Nov 6, 2024


maflcko DrahtBot l0rinc TheCharlatan hebasto


l0rinc

Labels
Tests

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-11-23 12:12 UTC
This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me