[30.x] Backports #35232

1. 
   fanquake commented at 9:30 AM on May 7, 2026: member

   Backports:

   • #34228
   • #35228
   • #35283
   • #35313 (only https://github.com/bitcoin-core/leveldb-subtree/pull/61)
   • #35378
   • #35408
2. fanquake added this to the milestone 30.3 on May 7, 2026
3. DrahtBot added the label Backport on May 7, 2026
4. 
   DrahtBot commented at 9:31 AM on May 7, 2026: contributor

   <!--e57a25ab6845829454e8d69fc972939a-->

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

   <!--006a51241073e994b41acfe9ec718e94-->

   Code Coverage & Benchmarks

   For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/35232.

   <!--021abf342d371248e50ceaed478a90ca-->

   Reviews

   See the guideline for information on the review process.

   Type	Reviewers
   ACK	willcl-ark

   If your review is incorrectly listed, please copy-paste <code>&lt;!--meta-tag:bot-skip--&gt;</code> into the comment that the bot should ignore.

   <!--5faf32d7da4f0f540f40219e4f7537a3-->

   LLM Linter (✨ experimental)

   Possible places where named args for integral literals may be used (e.g. func(x, /*named_arg=*/0) in C++, and func(x, named_arg=0) in Python):

   • MaxInputWeight(*desc, CTxIn{outpoint}, coin_control, true, can_grind_r) in src/wallet/spend.cpp

   ^{2026-05-29 15:24:33}

5. depends: Unset `SOURCE_DATE_EPOCH` in `gen_id` script

   Github-Pull: #34228
   Rebased-From: 9f7a2293c48f8afb5e87765a327a0c89fd21fa56

   c7034f4854
6. fanquake force-pushed on May 8, 2026
7. doc: mention -DWITH_ZMQ=ON in BSD build guides

   The FreeBSD, NetBSD, and OpenBSD build guides state that ZMQ support is compiled in when the package is installed. Since WITH_ZMQ defaults to OFF, update the wording to mention the required CMake option.
   
   Github-Pull: #35283
   Rebased-From: ca93ab808c488324990eb91ed8297a37dd10d580

   f92bd8b8ba
8. fanquake force-pushed on May 14, 2026
9. wallet: use outpoint when estimating input size

   `CalculateMaximumSignedInputSize()` is passed the outpoint being sized, but that context was not used when estimating the signed input size.
   Pass the outpoint through so externally selected inputs are not underestimated.
   
   Co-authored-by: Antoine Poinsot <darosior@protonmail.com>
   
   Github-Pull: #35228
   Rebased-From: cd8d3bd937b5515ea000408eb07d2ae3cd1aa417

   1dba05e7f6
10. fanquake force-pushed on May 20, 2026
11. Disable seek compaction

    Seek compaction is causing a cascade effect in the chainstate DB, causing large parts of the database to be rewritten every ~hour.
    
    Every periodic flush writes around 2 MiB. Since this is roughly the `write_buffer_size`, these writes regularly cause the memtable to rotate into a small L0 file. This file has a small seek budget, and with the random UTXO reads done during validation, it can get scheduled for seek compaction quickly.
    
    That seek compaction pushes the small file down to L1. Since most UTXOs are already lower down in L4/L5, many reads that consult this file do not find the key there and continue downward. The bloom filter makes those misses cheap, but LevelDB still decrements the file's seek budget. The file then gets scheduled for another seek compaction, and the same pattern pushes it down through L2 and L3.
    
    The expensive part happens around L3/L4. L4 has many ~32 MiB files holding the bulk of the UTXO set. When LevelDB compacts into L3, it may split the output into many smaller L3 files to limit how much L4 "grandparent" data any one output overlaps. Each of these small L3 files then gets its own small seek budget. Because chainstate keys are hash-random, each small L3 file can still have a broad key range, so many random reads consult it and quickly drain its budget. Once seek-compacted into L4, each tiny L3 file can overlap many L4 files, so compacting a few hundred KiB from L3 can require rewriting hundreds of MiB from L4. Repeating that across many small L3 files can rewrite most of the chainstate.
    
    This is a poor fit for chainstate because UTXO keys are hash-random, the DB is large enough to have many levels, writes are relatively small and periodic, and reads are frequent. The result is that read misses trigger compactions much earlier than size pressure would, and those compactions have very high write amplification.
    
    Disabling seek compaction may leave more files in upper levels for longer, so reads could theoretically consult more files. But Bitcoin Core enables bloom filters for all its LevelDB instances, so these misses are usually cheap in-memory filter checks rather than disk reads.
    
    For the other DBs, the risk is much smaller. They also use bloom filters, and most are smaller and less read-heavy. With fewer levels and less random read pressure, disabling seek compaction should have little effect there.
    
    Co-authored-by: l0rinc <pap.lorinc@gmail.com>

    b4a066ca40
12. fanquake force-pushed on May 28, 2026
13. fanquake force-pushed on May 29, 2026
14. fanquake force-pushed on May 29, 2026
15. fanquake marked this as ready for review on May 29, 2026
16. in .github/workflows/ci.yml:49 in 3ca4d94cf7 outdated

      48 |            fi
      49 |  
      50 |    test-each-commit:
      51 |      name: 'test each commit'
      52 | -    runs-on: ubuntu-24.04
      53 | +    runs-on: ${{ needs.runners.outputs.provider == 'warp' && 'warp-ubuntu-latest-x64-8x' || 'ubuntu-latest' }}
    

    ---

    ---

    

    willcl-ark commented at 3:02 PM on May 29, 2026:

    As this now needs runners, I think we must wait for runners to finish first;

    Should add to test-each-commit:

        needs: runners
    

    ---

    

    fanquake commented at 3:24 PM on May 29, 2026:

    Should add to test-each-commit:

    Added.

17. willcl-ark changes_requested
18. 
    willcl-ark commented at 3:05 PM on May 29, 2026: member

    Just one miss in test-each-commit that I can see. Everythign else looks ok.

    leveldb is missing the usual Github-Pull or Rebased-From, but that's because it was a cherry-pick. the content appears correct

19. ci: switch runners from cirrus to warpbuild

    Github-Pull: #35378
    Rebased-From: 4bdd46ace37f02da062a53a2943caeddca4ed8f9

    c562778a9a
20. doc: remove reference to cirrus

    Github-Pull: #35408
    Rebased-From: 265563bf75c0b8b615e28d47398098020ff0b109

    5e205b4561
21. ci: use ubuntu-latest instead of ubuntu-24.04

    To match the usage of -latest for the warp runners.
    
    Github-Pull: #35408
    Rebased-From: 5700a61b73342b506b0114b342499da7642c1c10

    bc6051b23c
22. doc: update release notes for v30.x cdd7c79225
23. fanquake force-pushed on May 29, 2026
24. willcl-ark approved
25. 
    willcl-ark commented at 7:20 PM on May 29, 2026: member

    ACK cdd7c79225965e68f7a5764dd0599f5f68a89094

    Failing lint is expected here from a subtree cherry-pick (and not entire pull)

Contributors

fanquakeDrahtBotwillcl-ark

Milestone
30.3