mirror of
https://github.com/bitcoin/bitcoin.git
synced 2025-12-13 20:36:21 +01:00
b8d279a81cdoc: add comment to explain correctness of GatherClusters() (Suhas Daftuar)aba7500a30Fix parameter name in getmempoolcluster rpc (Suhas Daftuar)6c1325a091Rename weight -> clusterweight in RPC output, and add doc explaining mempool terminology (Suhas Daftuar)bc2eb931daRequire mempool lock to be held when invoking TRUC checks (Suhas Daftuar)957ae23241Improve comments for getTransactionAncestry to reference cluster counts instead of descendants (Suhas Daftuar)d97d6199ceFix comment to reference cluster limits, not chain limits (Suhas Daftuar)a1b341ef98Sanity check feerate diagram in CTxMemPool::check() (Suhas Daftuar)23d6f457c4rpc: improve getmempoolcluster output (Suhas Daftuar)d2dcd37aacAvoid using mapTx.modify() to update modified fees (Suhas Daftuar)d84ffc24d2doc: add release notes snippet for cluster mempool (Suhas Daftuar)b0417ba944doc: Add design notes for cluster mempool and explain new mempool limits (Suhas Daftuar)2d88966e43miner: replace "package" with "chunk" (Suhas Daftuar)6f3e8eb300Add a GetFeePerVSize() accessor to CFeeRate, and use it in the BlockAssembler (Suhas Daftuar)b5f245f6f2Remove unused DEFAULT_ANCESTOR_SIZE_LIMIT_KVB and DEFAULT_DESCENDANT_SIZE_LIMIT_KVB (Suhas Daftuar)1dac54d506Use cluster size limit instead of ancestor size limit in txpackage unit test (Suhas Daftuar)04f65488caUse cluster size limit instead of ancestor/descendant size limits when sanity checking TRUC policy limits (Suhas Daftuar)634291a7dcUse cluster limits instead of ancestor/descendant limits when sanity checking package policy limits (Suhas Daftuar)fc18ef1f3fRemove ancestor and descendant vsize limits from MemPoolLimits (Suhas Daftuar)ed8e819121Warn user if using -limitancestorsize/-limitdescendantsize that the options have no effect (Suhas Daftuar)80d8df2d47Invoke removeUnchecked() directly in removeForBlock() (Suhas Daftuar)9292570f4cRewrite GetChildren without sets (Suhas Daftuar)3e39ea8c30Rewrite removeForReorg to avoid using sets (Suhas Daftuar)a3c31dfd71scripted-diff: rename AddToMempool -> TryAddToMempool (Suhas Daftuar)a5a7905d83Simplify removeRecursive (Suhas Daftuar)01d8520038Remove unused argument to RemoveStaged (Suhas Daftuar)bc64013e6fRemove unused variable (cacheMap) in mempool (Suhas Daftuar) Pull request description: As suggested in the main cluster mempool PR (https://github.com/bitcoin/bitcoin/pull/28676#pullrequestreview-3177119367), I've pulled out some of the non-essential optimizations and cleanups into this separate PR. Will continue to add more commits here to address non-blocking suggestions/improvements as they come up. ACKs for top commit: instagibbs: ACKb8d279a81csipa: ACKb8d279a81cTree-SHA512: 1a05e99eaf8db2e274a1801307fed5d82f8f917e75ccb9ab0e1b0eb2f9672b13c79d691d78ea7cd96900d0e7d5031a3dd582ebcccc9b1d66eb7455b1d3642235
249 lines
11 KiB
Python
Executable File
249 lines
11 KiB
Python
Executable File
#!/usr/bin/env python3
|
|
# Copyright (c) 2014-2022 The Bitcoin Core developers
|
|
# Distributed under the MIT software license, see the accompanying
|
|
# file COPYING or http://www.opensource.org/licenses/mit-license.php.
|
|
"""Test mempool re-org scenarios.
|
|
|
|
Test re-org scenarios with a mempool that contains transactions
|
|
that spend (directly or indirectly) coinbase transactions.
|
|
"""
|
|
|
|
import time
|
|
|
|
from test_framework.messages import (
|
|
CInv,
|
|
MSG_WTX,
|
|
msg_getdata,
|
|
)
|
|
from test_framework.p2p import (
|
|
P2PTxInvStore,
|
|
p2p_lock,
|
|
)
|
|
from test_framework.test_framework import BitcoinTestFramework
|
|
from test_framework.util import assert_equal, assert_raises_rpc_error
|
|
from test_framework.wallet import MiniWallet
|
|
from test_framework.blocktools import (
|
|
create_empty_fork,
|
|
)
|
|
|
|
# Number of blocks to create in temporary blockchain branch for reorg testing
|
|
# needs to be long enough to allow MTP to move arbitrarily forward
|
|
FORK_LENGTH = 20
|
|
|
|
class MempoolCoinbaseTest(BitcoinTestFramework):
|
|
def set_test_params(self):
|
|
self.num_nodes = 2
|
|
self.extra_args = [
|
|
[
|
|
'-whitelist=noban@127.0.0.1', # immediate tx relay
|
|
],
|
|
[]
|
|
]
|
|
|
|
def trigger_reorg(self, fork_blocks, node):
|
|
"""Trigger reorg of the fork blocks."""
|
|
for block in fork_blocks:
|
|
node.submitblock(block.serialize().hex())
|
|
assert_equal(self.nodes[0].getbestblockhash(), fork_blocks[-1].hash_hex)
|
|
|
|
def test_reorg_relay(self):
|
|
self.log.info("Test that transactions from disconnected blocks are available for relay immediately")
|
|
# Prevent time from moving forward
|
|
self.nodes[1].setmocktime(int(time.time()))
|
|
self.connect_nodes(0, 1)
|
|
self.generate(self.wallet, 3)
|
|
|
|
# Disconnect node0 and node1 to create different chains.
|
|
self.disconnect_nodes(0, 1)
|
|
# Connect a peer to node1, which doesn't have immediate tx relay
|
|
peer1 = self.nodes[1].add_p2p_connection(P2PTxInvStore())
|
|
|
|
# Create a transaction that is included in a block.
|
|
tx_disconnected = self.wallet.send_self_transfer(from_node=self.nodes[1])
|
|
self.generate(self.nodes[1], 1, sync_fun=self.no_op)
|
|
|
|
# Create a transaction and submit it to node1's mempool.
|
|
tx_before_reorg = self.wallet.send_self_transfer(from_node=self.nodes[1])
|
|
|
|
# Create a child of that transaction and submit it to node1's mempool.
|
|
tx_child = self.wallet.send_self_transfer(utxo_to_spend=tx_disconnected["new_utxo"], from_node=self.nodes[1])
|
|
assert_equal(self.nodes[1].getmempoolentry(tx_child["txid"])["ancestorcount"], 1)
|
|
assert_equal(len(peer1.get_invs()), 0)
|
|
|
|
# node0 has a longer chain in which tx_disconnected was not confirmed.
|
|
self.generate(self.nodes[0], 3, sync_fun=self.no_op)
|
|
|
|
# Reconnect the nodes and sync chains. node0's chain should win.
|
|
self.connect_nodes(0, 1)
|
|
self.sync_blocks()
|
|
|
|
# Child now has an ancestor from the disconnected block
|
|
assert_equal(self.nodes[1].getmempoolentry(tx_child["txid"])["ancestorcount"], 2)
|
|
assert_equal(self.nodes[1].getmempoolentry(tx_before_reorg["txid"])["ancestorcount"], 1)
|
|
|
|
# peer1 should not have received an inv for any of the transactions during this time, as no
|
|
# mocktime has elapsed for those transactions to be announced. Likewise, it cannot
|
|
# request very recent, unanounced transactions.
|
|
assert_equal(len(peer1.get_invs()), 0)
|
|
# It's too early to request these two transactions
|
|
requests_too_recent = msg_getdata([CInv(t=MSG_WTX, h=tx["tx"].wtxid_int) for tx in [tx_before_reorg, tx_child]])
|
|
peer1.send_and_ping(requests_too_recent)
|
|
for _ in range(len(requests_too_recent.inv)):
|
|
peer1.sync_with_ping()
|
|
with p2p_lock:
|
|
assert "tx" not in peer1.last_message
|
|
assert "notfound" in peer1.last_message
|
|
|
|
# Request the tx from the disconnected block
|
|
request_disconnected_tx = msg_getdata([CInv(t=MSG_WTX, h=tx_disconnected["tx"].wtxid_int)])
|
|
peer1.send_and_ping(request_disconnected_tx)
|
|
|
|
# The tx from the disconnected block was never announced, and it entered the mempool later
|
|
# than the transactions that are too recent.
|
|
assert_equal(len(peer1.get_invs()), 0)
|
|
with p2p_lock:
|
|
# However, the node will answer requests for the tx from the recently-disconnected block.
|
|
assert_equal(peer1.last_message["tx"].tx.wtxid_hex,tx_disconnected["tx"].wtxid_hex)
|
|
|
|
self.nodes[1].setmocktime(int(time.time()) + 300)
|
|
peer1.sync_with_ping()
|
|
# the transactions are now announced
|
|
assert_equal(len(peer1.get_invs()), 3)
|
|
for _ in range(3):
|
|
# make sure all tx requests have been responded to
|
|
peer1.sync_with_ping()
|
|
last_tx_received = peer1.last_message["tx"]
|
|
|
|
tx_after_reorg = self.wallet.send_self_transfer(from_node=self.nodes[1])
|
|
request_after_reorg = msg_getdata([CInv(t=MSG_WTX, h=tx_after_reorg["tx"].wtxid_int)])
|
|
assert tx_after_reorg["txid"] in self.nodes[1].getrawmempool()
|
|
peer1.send_and_ping(request_after_reorg)
|
|
with p2p_lock:
|
|
assert_equal(peer1.last_message["tx"], last_tx_received)
|
|
|
|
def run_test(self):
|
|
self.wallet = MiniWallet(self.nodes[0])
|
|
wallet = self.wallet
|
|
|
|
# Prevent clock from moving blocks further forward in time
|
|
now = int(time.time())
|
|
self.nodes[0].setmocktime(now)
|
|
|
|
# Start with a 200 block chain
|
|
assert_equal(self.nodes[0].getblockcount(), 200)
|
|
|
|
self.log.info("Add 4 coinbase utxos to the miniwallet")
|
|
# Block 76 contains the first spendable coinbase txs.
|
|
first_block = 76
|
|
|
|
# Three scenarios for re-orging coinbase spends in the memory pool:
|
|
# 1. Direct coinbase spend : spend_1
|
|
# 2. Indirect (coinbase spend in chain, child in mempool) : spend_2 and spend_2_1
|
|
# 3. Indirect (coinbase and child both in chain) : spend_3 and spend_3_1
|
|
# Use re-org to make all of the above coinbase spends invalid (immature coinbase),
|
|
# and make sure the mempool code behaves correctly.
|
|
b = [self.nodes[0].getblockhash(n) for n in range(first_block, first_block+4)]
|
|
coinbase_txids = [self.nodes[0].getblock(h)['tx'][0] for h in b]
|
|
utxo_1 = wallet.get_utxo(txid=coinbase_txids[1])
|
|
utxo_2 = wallet.get_utxo(txid=coinbase_txids[2])
|
|
utxo_3 = wallet.get_utxo(txid=coinbase_txids[3])
|
|
self.log.info("Create three transactions spending from coinbase utxos: spend_1, spend_2, spend_3")
|
|
spend_1 = wallet.create_self_transfer(utxo_to_spend=utxo_1)
|
|
spend_2 = wallet.create_self_transfer(utxo_to_spend=utxo_2)
|
|
spend_3 = wallet.create_self_transfer(utxo_to_spend=utxo_3)
|
|
|
|
self.log.info("Create another transaction which is time-locked to 300 seconds in the future")
|
|
future = now + 300
|
|
utxo = wallet.get_utxo(txid=coinbase_txids[0])
|
|
timelock_tx = wallet.create_self_transfer(
|
|
utxo_to_spend=utxo,
|
|
locktime=future,
|
|
)['hex']
|
|
|
|
self.log.info("Check that the time-locked transaction is too immature to spend")
|
|
assert_raises_rpc_error(-26, "non-final", self.nodes[0].sendrawtransaction, timelock_tx)
|
|
|
|
self.log.info("Broadcast and mine spend_2 and spend_3")
|
|
spend_2_id = wallet.sendrawtransaction(from_node=self.nodes[0], tx_hex=spend_2['hex'])
|
|
wallet.sendrawtransaction(from_node=self.nodes[0], tx_hex=spend_3['hex'])
|
|
self.log.info("Generate a block")
|
|
self.generate(self.nodes[0], 1)
|
|
self.log.info("Check that time-locked transaction is still too immature to spend")
|
|
assert_raises_rpc_error(-26, 'non-final', self.nodes[0].sendrawtransaction, timelock_tx)
|
|
|
|
self.log.info("Create spend_2_1 and spend_3_1")
|
|
spend_2_1 = wallet.create_self_transfer(utxo_to_spend=spend_2["new_utxo"], version=1)
|
|
spend_3_1 = wallet.create_self_transfer(utxo_to_spend=spend_3["new_utxo"])
|
|
|
|
self.log.info("Broadcast and mine spend_3_1")
|
|
spend_3_1_id = self.nodes[0].sendrawtransaction(spend_3_1['hex'])
|
|
self.log.info("Generate a block")
|
|
|
|
# Prep for fork, only go FORK_LENGTH seconds into the MTP future max
|
|
fork_blocks = create_empty_fork(self.nodes[0], fork_length=FORK_LENGTH)
|
|
|
|
# Jump node and MTP 300 seconds and generate a slightly weaker chain than reorg one
|
|
self.nodes[0].setmocktime(future)
|
|
self.generate(self.nodes[0], FORK_LENGTH - 1)
|
|
block_time = self.nodes[0].getblock(self.nodes[0].getbestblockhash())['time']
|
|
assert(block_time >= now + 300)
|
|
|
|
# generate() implicitly syncs blocks, so that peer 1 gets the block before timelock_tx
|
|
# Otherwise, peer 1 would put the timelock_tx in m_lazy_recent_rejects
|
|
self.log.info("The time-locked transaction can now be spent")
|
|
timelock_tx_id = self.nodes[0].sendrawtransaction(timelock_tx)
|
|
|
|
self.log.info("Add spend_1 and spend_2_1 to the mempool")
|
|
spend_1_id = self.nodes[0].sendrawtransaction(spend_1['hex'])
|
|
spend_2_1_id = self.nodes[0].sendrawtransaction(spend_2_1['hex'])
|
|
|
|
assert_equal(set(self.nodes[0].getrawmempool()), {spend_1_id, spend_2_1_id, timelock_tx_id})
|
|
self.sync_all()
|
|
|
|
self.trigger_reorg(fork_blocks, self.nodes[0])
|
|
self.sync_blocks()
|
|
|
|
# We went backwards in time to boot timelock_tx_id
|
|
fork_block_time = self.nodes[0].getblock(self.nodes[0].getbestblockhash())['time']
|
|
assert fork_block_time < block_time
|
|
|
|
self.log.info("The time-locked transaction is now too immature and has been removed from the mempool")
|
|
self.log.info("spend_3_1 has been re-orged out of the chain and is back in the mempool")
|
|
assert_equal(set(self.nodes[0].getrawmempool()), {spend_1_id, spend_2_1_id, spend_3_1_id})
|
|
|
|
self.log.info("Reorg out enough blocks to get spend_2 back in the mempool, along with its child")
|
|
|
|
while (spend_2_id not in self.nodes[0].getrawmempool()):
|
|
b = self.nodes[0].getbestblockhash()
|
|
for node in self.nodes:
|
|
node.invalidateblock(b)
|
|
|
|
assert(spend_2_id in self.nodes[0].getrawmempool())
|
|
assert(spend_2_1_id in self.nodes[0].getrawmempool())
|
|
|
|
# Chain 10 more transactions off of spend_2_1
|
|
self.log.info("Give spend_2 some more descendants by creating a chain of 10 transactions spending from it")
|
|
parent_utxo = spend_2_1["new_utxo"]
|
|
for i in range(10):
|
|
tx = wallet.create_self_transfer(utxo_to_spend=parent_utxo, version=1)
|
|
self.nodes[0].sendrawtransaction(tx['hex'])
|
|
parent_utxo = tx["new_utxo"]
|
|
|
|
self.log.info("Use invalidateblock to re-org back and make all those coinbase spends immature/invalid")
|
|
b = self.nodes[0].getblockhash(first_block + 100)
|
|
|
|
# Use invalidateblock to go backwards in MTP time.
|
|
# invalidateblock actually moves MTP backwards, making timelock_tx_id valid again.
|
|
for node in self.nodes:
|
|
node.invalidateblock(b)
|
|
|
|
self.log.info("Check that the mempool is empty")
|
|
assert_equal(set(self.nodes[0].getrawmempool()), set())
|
|
self.sync_all()
|
|
|
|
self.test_reorg_relay()
|
|
|
|
|
|
if __name__ == '__main__':
|
|
MempoolCoinbaseTest(__file__).main()
|