Refactor CreateNewBlock to be a method of the BlockAssembler class
(cherry picked from commit bitcoin/bitcoin@4dc94d1036) Zcash: Adapted to include our miner changes.
This commit is contained in:
parent
4bb399a8f5
commit
e6d4027888
701
src/miner.cpp
701
src/miner.cpp
|
@ -331,18 +331,52 @@ CMutableTransaction CreateCoinbaseTransaction(const CChainParams& chainparams, C
|
||||||
return mtx;
|
return mtx;
|
||||||
}
|
}
|
||||||
|
|
||||||
CBlockTemplate* CreateNewBlock(const CChainParams& chainparams, const MinerAddress& minerAddress, const std::optional<CMutableTransaction>& next_cb_mtx)
|
BlockAssembler::BlockAssembler(const CChainParams& _chainparams)
|
||||||
|
: chainparams(_chainparams)
|
||||||
{
|
{
|
||||||
// Create new block
|
// Largest block you're willing to create:
|
||||||
std::unique_ptr<CBlockTemplate> pblocktemplate(new CBlockTemplate());
|
nBlockMaxSize = GetArg("-blockmaxsize", DEFAULT_BLOCK_MAX_SIZE);
|
||||||
|
// Limit to between 1K and MAX_BLOCK_SIZE-1K for sanity:
|
||||||
|
nBlockMaxSize = std::max((unsigned int)1000, std::min((unsigned int)(MAX_BLOCK_SIZE-1000), nBlockMaxSize));
|
||||||
|
|
||||||
|
// Minimum block size you want to create; block will be filled with free transactions
|
||||||
|
// until there are no more or the block reaches this size:
|
||||||
|
nBlockMinSize = GetArg("-blockminsize", DEFAULT_BLOCK_MIN_SIZE);
|
||||||
|
nBlockMinSize = std::min(nBlockMaxSize, nBlockMinSize);
|
||||||
|
}
|
||||||
|
|
||||||
|
void BlockAssembler::resetBlock()
|
||||||
|
{
|
||||||
|
inBlock.clear();
|
||||||
|
|
||||||
|
// Reserve space for coinbase tx
|
||||||
|
nBlockSize = 1000;
|
||||||
|
nBlockSigOps = 100;
|
||||||
|
|
||||||
|
// These counters do not include coinbase tx
|
||||||
|
nBlockTx = 0;
|
||||||
|
nFees = 0;
|
||||||
|
|
||||||
|
sproutValue = 0;
|
||||||
|
saplingValue = 0;
|
||||||
|
orchardValue = 0;
|
||||||
|
monitoring_pool_balances = true;
|
||||||
|
|
||||||
|
lastFewTxs = 0;
|
||||||
|
blockFinished = false;
|
||||||
|
}
|
||||||
|
|
||||||
|
CBlockTemplate* BlockAssembler::CreateNewBlock(
|
||||||
|
const MinerAddress& minerAddress,
|
||||||
|
const std::optional<CMutableTransaction>& next_cb_mtx)
|
||||||
|
{
|
||||||
|
resetBlock();
|
||||||
|
|
||||||
|
pblocktemplate.reset(new CBlockTemplate());
|
||||||
|
|
||||||
if(!pblocktemplate.get())
|
if(!pblocktemplate.get())
|
||||||
return NULL;
|
return NULL;
|
||||||
CBlock *pblock = &pblocktemplate->block; // pointer for convenience
|
pblock = &pblocktemplate->block; // pointer for convenience
|
||||||
|
|
||||||
// -regtest only: allow overriding block.nVersion with
|
|
||||||
// -blockversion=N to test forking scenarios
|
|
||||||
if (chainparams.MineBlocksOnDemand())
|
|
||||||
pblock->nVersion = GetArg("-blockversion", pblock->nVersion);
|
|
||||||
|
|
||||||
// Add dummy coinbase tx as first transaction
|
// Add dummy coinbase tx as first transaction
|
||||||
pblock->vtx.push_back(CTransaction());
|
pblock->vtx.push_back(CTransaction());
|
||||||
|
@ -351,26 +385,325 @@ CBlockTemplate* CreateNewBlock(const CChainParams& chainparams, const MinerAddre
|
||||||
|
|
||||||
// If we're given a coinbase tx, it's been precomputed, its fees are zero,
|
// If we're given a coinbase tx, it's been precomputed, its fees are zero,
|
||||||
// so we can't include any mempool transactions; this will be an empty block.
|
// so we can't include any mempool transactions; this will be an empty block.
|
||||||
bool nonEmptyBlock = !next_cb_mtx;
|
blockFinished = blockFinished || next_cb_mtx;
|
||||||
|
|
||||||
// Largest block you're willing to create:
|
LOCK2(cs_main, mempool.cs);
|
||||||
unsigned int nBlockMaxSize = GetArg("-blockmaxsize", DEFAULT_BLOCK_MAX_SIZE);
|
CBlockIndex* pindexPrev = chainActive.Tip();
|
||||||
// Limit to between 1K and MAX_BLOCK_SIZE-1K for sanity:
|
nHeight = pindexPrev->nHeight + 1;
|
||||||
nBlockMaxSize = std::max((unsigned int)1000, std::min((unsigned int)(MAX_BLOCK_SIZE-1000), nBlockMaxSize));
|
uint32_t consensusBranchId = CurrentEpochBranchId(nHeight, chainparams.GetConsensus());
|
||||||
|
|
||||||
|
// -regtest only: allow overriding block.nVersion with
|
||||||
|
// -blockversion=N to test forking scenarios
|
||||||
|
if (chainparams.MineBlocksOnDemand())
|
||||||
|
pblock->nVersion = GetArg("-blockversion", pblock->nVersion);
|
||||||
|
|
||||||
|
pblock->nTime = GetTime();
|
||||||
|
const int64_t nMedianTimePast = pindexPrev->GetMedianTimePast();
|
||||||
|
CCoinsViewCache view(pcoinsTip);
|
||||||
|
|
||||||
|
SaplingMerkleTree sapling_tree;
|
||||||
|
assert(view.GetSaplingAnchorAt(view.GetBestAnchor(SAPLING), sapling_tree));
|
||||||
|
|
||||||
|
nLockTimeCutoff = (STANDARD_LOCKTIME_VERIFY_FLAGS & LOCKTIME_MEDIAN_TIME_PAST)
|
||||||
|
? nMedianTimePast
|
||||||
|
: pblock->GetBlockTime();
|
||||||
|
|
||||||
|
// We want to track the value pool, but if the miner gets
|
||||||
|
// invoked on an old block before the hardcoded fallback
|
||||||
|
// is active we don't want to trip up any assertions. So,
|
||||||
|
// we only adhere to the turnstile (as a miner) if we
|
||||||
|
// actually have all of the information necessary to do
|
||||||
|
// so.
|
||||||
|
if (chainparams.ZIP209Enabled()) {
|
||||||
|
if (pindexPrev->nChainSproutValue) {
|
||||||
|
sproutValue = *pindexPrev->nChainSproutValue;
|
||||||
|
} else {
|
||||||
|
monitoring_pool_balances = false;
|
||||||
|
}
|
||||||
|
if (pindexPrev->nChainSaplingValue) {
|
||||||
|
saplingValue = *pindexPrev->nChainSaplingValue;
|
||||||
|
} else {
|
||||||
|
monitoring_pool_balances = false;
|
||||||
|
}
|
||||||
|
if (pindexPrev->nChainOrchardValue) {
|
||||||
|
orchardValue = *pindexPrev->nChainOrchardValue;
|
||||||
|
} else {
|
||||||
|
monitoring_pool_balances = false;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
addPriorityTxs();
|
||||||
|
addScoreTxs();
|
||||||
|
|
||||||
|
last_block_num_txs = nBlockTx;
|
||||||
|
last_block_size = nBlockSize;
|
||||||
|
LogPrintf("%s: total size %u (excluding coinbase) txs: %u fees: %ld sigops %d", __func__, nBlockSize, nBlockTx, nFees, nBlockSigOps);
|
||||||
|
|
||||||
|
// Create coinbase tx
|
||||||
|
if (next_cb_mtx) {
|
||||||
|
pblock->vtx[0] = *next_cb_mtx;
|
||||||
|
} else {
|
||||||
|
pblock->vtx[0] = CreateCoinbaseTransaction(chainparams, nFees, minerAddress, nHeight);
|
||||||
|
}
|
||||||
|
pblocktemplate->vTxFees[0] = -nFees;
|
||||||
|
|
||||||
|
// Update the Sapling commitment tree.
|
||||||
|
for (const CTransaction& tx : pblock->vtx) {
|
||||||
|
for (const OutputDescription& odesc : tx.vShieldedOutput) {
|
||||||
|
sapling_tree.append(odesc.cmu);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
// Randomise nonce
|
||||||
|
arith_uint256 nonce = UintToArith256(GetRandHash());
|
||||||
|
// Clear the top and bottom 16 bits (for local use as thread flags and counters)
|
||||||
|
nonce <<= 32;
|
||||||
|
nonce >>= 16;
|
||||||
|
pblock->nNonce = ArithToUint256(nonce);
|
||||||
|
|
||||||
|
uint32_t prevConsensusBranchId = CurrentEpochBranchId(pindexPrev->nHeight, chainparams.GetConsensus());
|
||||||
|
|
||||||
|
// Fill in header
|
||||||
|
pblock->hashPrevBlock = pindexPrev->GetBlockHash();
|
||||||
|
if (chainparams.GetConsensus().NetworkUpgradeActive(nHeight, Consensus::UPGRADE_NU5)) {
|
||||||
|
// hashBlockCommitments depends on the block transactions, so we have to
|
||||||
|
// update it whenever the coinbase transaction changes.
|
||||||
|
//
|
||||||
|
// - For the internal miner (either directly or via the `generate` RPC), this
|
||||||
|
// will occur in `IncrementExtraNonce()`, like for `hashMerkleRoot`.
|
||||||
|
// - For `getblocktemplate`, we have two sets of fields to handle:
|
||||||
|
// - The `defaultroots` fields, which contain both the default value (if
|
||||||
|
// nothing in the template is altered), and the roots that can be used to
|
||||||
|
// recalculate it (if some or all of the template is altered).
|
||||||
|
// - The legacy `finalsaplingroothash`, `lightclientroothash`, and
|
||||||
|
// `blockcommitmentshash` fields, which had the semantics of "place this
|
||||||
|
// value into the block header and things will work" (except for in
|
||||||
|
// v4.6.0 where they were accidentally set to always be the NU5 value).
|
||||||
|
//
|
||||||
|
// To accommodate all use cases, we calculate the `hashBlockCommitments`
|
||||||
|
// default value here (unlike `hashMerkleRoot`), and additionally cache the
|
||||||
|
// values necessary to recalculate it.
|
||||||
|
pblocktemplate->hashChainHistoryRoot = view.GetHistoryRoot(prevConsensusBranchId);
|
||||||
|
pblocktemplate->hashAuthDataRoot = pblock->BuildAuthDataMerkleTree();
|
||||||
|
pblock->hashBlockCommitments = DeriveBlockCommitmentsHash(
|
||||||
|
pblocktemplate->hashChainHistoryRoot,
|
||||||
|
pblocktemplate->hashAuthDataRoot);
|
||||||
|
} else if (IsActivationHeight(nHeight, chainparams.GetConsensus(), Consensus::UPGRADE_HEARTWOOD)) {
|
||||||
|
pblocktemplate->hashChainHistoryRoot.SetNull();
|
||||||
|
pblocktemplate->hashAuthDataRoot.SetNull();
|
||||||
|
pblock->hashBlockCommitments.SetNull();
|
||||||
|
} else if (chainparams.GetConsensus().NetworkUpgradeActive(nHeight, Consensus::UPGRADE_HEARTWOOD)) {
|
||||||
|
pblocktemplate->hashChainHistoryRoot = view.GetHistoryRoot(prevConsensusBranchId);
|
||||||
|
pblocktemplate->hashAuthDataRoot.SetNull();
|
||||||
|
pblock->hashBlockCommitments = pblocktemplate->hashChainHistoryRoot;
|
||||||
|
} else {
|
||||||
|
pblocktemplate->hashChainHistoryRoot.SetNull();
|
||||||
|
pblocktemplate->hashAuthDataRoot.SetNull();
|
||||||
|
pblock->hashBlockCommitments = sapling_tree.root();
|
||||||
|
}
|
||||||
|
UpdateTime(pblock, chainparams.GetConsensus(), pindexPrev);
|
||||||
|
pblock->nBits = GetNextWorkRequired(pindexPrev, pblock, chainparams.GetConsensus());
|
||||||
|
pblock->nSolution.clear();
|
||||||
|
pblocktemplate->vTxSigOps[0] = GetLegacySigOpCount(pblock->vtx[0]);
|
||||||
|
|
||||||
|
CValidationState state;
|
||||||
|
if (!TestBlockValidity(state, chainparams, *pblock, pindexPrev, true)) {
|
||||||
|
throw std::runtime_error(strprintf("%s: TestBlockValidity failed: %s", __func__, FormatStateMessage(state)));
|
||||||
|
}
|
||||||
|
|
||||||
|
return pblocktemplate.release();
|
||||||
|
}
|
||||||
|
|
||||||
|
bool BlockAssembler::isStillDependent(CTxMemPool::txiter iter)
|
||||||
|
{
|
||||||
|
for (CTxMemPool::txiter parent : mempool.GetMemPoolParents(iter))
|
||||||
|
{
|
||||||
|
if (!inBlock.count(parent)) {
|
||||||
|
return true;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
|
|
||||||
|
bool BlockAssembler::TestForBlock(CTxMemPool::txiter iter)
|
||||||
|
{
|
||||||
|
if (nBlockSize + iter->GetTxSize() >= nBlockMaxSize) {
|
||||||
|
// If the block is so close to full that no more txs will fit
|
||||||
|
// or if we've tried more than 50 times to fill remaining space
|
||||||
|
// then flag that the block is finished
|
||||||
|
if (nBlockSize > nBlockMaxSize - 100 || lastFewTxs > 50) {
|
||||||
|
blockFinished = true;
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
// Once we're within 1000 bytes of a full block, only look at 50 more txs
|
||||||
|
// to try to fill the remaining space.
|
||||||
|
if (nBlockSize > nBlockMaxSize - 1000) {
|
||||||
|
lastFewTxs++;
|
||||||
|
}
|
||||||
|
LogPrintf("%s: skipping tx %s: exceeded maximum block size %u.",
|
||||||
|
__func__,
|
||||||
|
iter->GetTx().GetHash().GetHex(),
|
||||||
|
nBlockMaxSize);
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (nBlockSigOps + iter->GetSigOpCount() >= MAX_BLOCK_SIGOPS) {
|
||||||
|
// If the block has room for no more sig ops then
|
||||||
|
// flag that the block is finished
|
||||||
|
if (nBlockSigOps > MAX_BLOCK_SIGOPS - 2) {
|
||||||
|
blockFinished = true;
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
// Otherwise attempt to find another tx with fewer sigops
|
||||||
|
// to put in the block.
|
||||||
|
LogPrintf("%s: skipping tx %s: exceeds legacy max sigops %u.",
|
||||||
|
__func__,
|
||||||
|
iter->GetTx().GetHash().GetHex(),
|
||||||
|
MAX_BLOCK_SIGOPS);
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
|
||||||
|
// Must check that lock times are still valid
|
||||||
|
// This can be removed once MTP is always enforced
|
||||||
|
// as long as reorgs keep the mempool consistent.
|
||||||
|
if (!IsFinalTx(iter->GetTx(), nHeight, nLockTimeCutoff))
|
||||||
|
return false;
|
||||||
|
|
||||||
|
// Must check that expiry heights are still valid.
|
||||||
|
if (IsExpiredTx(iter->GetTx(), nHeight))
|
||||||
|
return false;
|
||||||
|
|
||||||
|
if (chainparams.ZIP209Enabled() && monitoring_pool_balances) {
|
||||||
|
// Does this transaction lead to a turnstile violation?
|
||||||
|
|
||||||
|
CAmount sproutValueDummy = sproutValue;
|
||||||
|
CAmount saplingValueDummy = saplingValue;
|
||||||
|
CAmount orchardValueDummy = orchardValue;
|
||||||
|
|
||||||
|
saplingValueDummy += -iter->GetTx().GetValueBalanceSapling();
|
||||||
|
orchardValueDummy += -iter->GetTx().GetOrchardBundle().GetValueBalance();
|
||||||
|
|
||||||
|
for (auto js : iter->GetTx().vJoinSplit) {
|
||||||
|
sproutValueDummy += js.vpub_old;
|
||||||
|
sproutValueDummy -= js.vpub_new;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (sproutValueDummy < 0) {
|
||||||
|
LogPrintf("CreateNewBlock: tx %s appears to violate Sprout turnstile\n",
|
||||||
|
iter->GetTx().GetHash().GetHex());
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
if (saplingValueDummy < 0) {
|
||||||
|
LogPrintf("CreateNewBlock: tx %s appears to violate Sapling turnstile\n",
|
||||||
|
iter->GetTx().GetHash().GetHex());
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
if (orchardValueDummy < 0) {
|
||||||
|
LogPrintf("CreateNewBlock: tx %s appears to violate Orchard turnstile\n",
|
||||||
|
iter->GetTx().GetHash().GetHex());
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
|
||||||
|
// We update this here instead of in AddToBlock to avoid recalculating
|
||||||
|
// the deltas, because there are no more checks and we know that the
|
||||||
|
// transaction will be added to the block.
|
||||||
|
sproutValue = sproutValueDummy;
|
||||||
|
saplingValue = saplingValueDummy;
|
||||||
|
orchardValue = orchardValueDummy;
|
||||||
|
}
|
||||||
|
|
||||||
|
return true;
|
||||||
|
}
|
||||||
|
|
||||||
|
void BlockAssembler::AddToBlock(CTxMemPool::txiter iter)
|
||||||
|
{
|
||||||
|
pblock->vtx.push_back(iter->GetTx());
|
||||||
|
pblocktemplate->vTxFees.push_back(iter->GetFee());
|
||||||
|
pblocktemplate->vTxSigOps.push_back(iter->GetSigOpCount());
|
||||||
|
nBlockSize += iter->GetTxSize();
|
||||||
|
++nBlockTx;
|
||||||
|
nBlockSigOps += iter->GetSigOpCount();
|
||||||
|
nFees += iter->GetFee();
|
||||||
|
inBlock.insert(iter);
|
||||||
|
|
||||||
|
bool fPrintPriority = GetBoolArg("-printpriority", DEFAULT_PRINTPRIORITY);
|
||||||
|
if (fPrintPriority) {
|
||||||
|
double dPriority = iter->GetPriority(nHeight);
|
||||||
|
CAmount dummy;
|
||||||
|
mempool.ApplyDeltas(iter->GetTx().GetHash(), dPriority, dummy);
|
||||||
|
LogPrintf("%s: priority %.1f fee %s txid %s\n",
|
||||||
|
__func__,
|
||||||
|
dPriority,
|
||||||
|
CFeeRate(iter->GetModifiedFee(), iter->GetTxSize()).ToString(),
|
||||||
|
iter->GetTx().GetHash().ToString());
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
void BlockAssembler::addScoreTxs()
|
||||||
|
{
|
||||||
|
std::priority_queue<CTxMemPool::txiter, std::vector<CTxMemPool::txiter>, ScoreCompare> clearedTxs;
|
||||||
|
CTxMemPool::setEntries waitSet;
|
||||||
|
CTxMemPool::indexed_transaction_set::index<mining_score>::type::iterator mi = mempool.mapTx.get<mining_score>().begin();
|
||||||
|
CTxMemPool::txiter iter;
|
||||||
|
while (!blockFinished && (mi != mempool.mapTx.get<mining_score>().end() || !clearedTxs.empty()))
|
||||||
|
{
|
||||||
|
// If no txs that were previously postponed are available to try
|
||||||
|
// again, then try the next highest score tx
|
||||||
|
if (clearedTxs.empty()) {
|
||||||
|
iter = mempool.mapTx.project<0>(mi);
|
||||||
|
mi++;
|
||||||
|
}
|
||||||
|
// If a previously postponed tx is available to try again, then it
|
||||||
|
// has higher score than all untried so far txs
|
||||||
|
else {
|
||||||
|
iter = clearedTxs.top();
|
||||||
|
clearedTxs.pop();
|
||||||
|
}
|
||||||
|
|
||||||
|
// If tx is dependent on other mempool txs which haven't yet been included
|
||||||
|
// then put it in the waitSet
|
||||||
|
if (isStillDependent(iter)) {
|
||||||
|
waitSet.insert(iter);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
|
// If the fee rate is below the min fee rate for mining, then we're done
|
||||||
|
// adding txs based on score (fee rate)
|
||||||
|
if ((iter->GetModifiedFee() < ::minRelayTxFee.GetFee(iter->GetTxSize())) &&
|
||||||
|
(iter->GetModifiedFee() < DEFAULT_FEE) &&
|
||||||
|
(nBlockSize >= nBlockMinSize)) {
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
// If this tx fits in the block add it, otherwise keep looping
|
||||||
|
if (TestForBlock(iter)) {
|
||||||
|
AddToBlock(iter);
|
||||||
|
|
||||||
|
// This tx was successfully added, so
|
||||||
|
// add transactions that depend on this one to the priority queue to try again
|
||||||
|
for (CTxMemPool::txiter child : mempool.GetMemPoolChildren(iter))
|
||||||
|
{
|
||||||
|
if (waitSet.count(child)) {
|
||||||
|
clearedTxs.push(child);
|
||||||
|
waitSet.erase(child);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
void BlockAssembler::addPriorityTxs()
|
||||||
|
{
|
||||||
// How much of the block should be dedicated to high-priority transactions,
|
// How much of the block should be dedicated to high-priority transactions,
|
||||||
// included regardless of the fees they pay
|
// included regardless of the fees they pay
|
||||||
unsigned int nBlockPrioritySize = GetArg("-blockprioritysize", DEFAULT_BLOCK_PRIORITY_SIZE);
|
unsigned int nBlockPrioritySize = GetArg("-blockprioritysize", DEFAULT_BLOCK_PRIORITY_SIZE);
|
||||||
nBlockPrioritySize = std::min(nBlockMaxSize, nBlockPrioritySize);
|
nBlockPrioritySize = std::min(nBlockMaxSize, nBlockPrioritySize);
|
||||||
|
|
||||||
// Minimum block size you want to create; block will be filled with free transactions
|
if (nBlockPrioritySize == 0) {
|
||||||
// until there are no more or the block reaches this size:
|
return;
|
||||||
unsigned int nBlockMinSize = GetArg("-blockminsize", DEFAULT_BLOCK_MIN_SIZE);
|
}
|
||||||
nBlockMinSize = std::min(nBlockMaxSize, nBlockMinSize);
|
|
||||||
|
|
||||||
// Collect memory pool transactions into the block
|
|
||||||
CTxMemPool::setEntries inBlock;
|
|
||||||
CTxMemPool::setEntries waitSet;
|
|
||||||
|
|
||||||
// This vector will be sorted into a priority queue:
|
// This vector will be sorted into a priority queue:
|
||||||
vector<TxCoinAgePriority> vecPriority;
|
vector<TxCoinAgePriority> vecPriority;
|
||||||
|
@ -379,304 +712,60 @@ CBlockTemplate* CreateNewBlock(const CChainParams& chainparams, const MinerAddre
|
||||||
typedef std::map<CTxMemPool::txiter, double, CTxMemPool::CompareIteratorByHash>::iterator waitPriIter;
|
typedef std::map<CTxMemPool::txiter, double, CTxMemPool::CompareIteratorByHash>::iterator waitPriIter;
|
||||||
double actualPriority = -1;
|
double actualPriority = -1;
|
||||||
|
|
||||||
std::priority_queue<CTxMemPool::txiter, std::vector<CTxMemPool::txiter>, ScoreCompare> clearedTxs;
|
vecPriority.reserve(mempool.mapTx.size());
|
||||||
bool fPrintPriority = GetBoolArg("-printpriority", DEFAULT_PRINTPRIORITY);
|
for (CTxMemPool::indexed_transaction_set::iterator mi = mempool.mapTx.begin();
|
||||||
uint64_t nBlockSize = 1000;
|
mi != mempool.mapTx.end(); ++mi)
|
||||||
uint64_t nBlockTx = 0;
|
|
||||||
unsigned int nBlockSigOps = 100;
|
|
||||||
int lastFewTxs = 0;
|
|
||||||
CAmount nFees = 0;
|
|
||||||
|
|
||||||
{
|
{
|
||||||
LOCK2(cs_main, mempool.cs);
|
double dPriority = mi->GetPriority(nHeight);
|
||||||
CBlockIndex* pindexPrev = chainActive.Tip();
|
CAmount dummy;
|
||||||
const int nHeight = pindexPrev->nHeight + 1;
|
mempool.ApplyDeltas(mi->GetTx().GetHash(), dPriority, dummy);
|
||||||
uint32_t consensusBranchId = CurrentEpochBranchId(nHeight, chainparams.GetConsensus());
|
vecPriority.push_back(TxCoinAgePriority(dPriority, mi));
|
||||||
pblock->nTime = GetTime();
|
}
|
||||||
const int64_t nMedianTimePast = pindexPrev->GetMedianTimePast();
|
std::make_heap(vecPriority.begin(), vecPriority.end(), pricomparer);
|
||||||
CCoinsViewCache view(pcoinsTip);
|
|
||||||
|
|
||||||
SaplingMerkleTree sapling_tree;
|
CTxMemPool::txiter iter;
|
||||||
assert(view.GetSaplingAnchorAt(view.GetBestAnchor(SAPLING), sapling_tree));
|
while (!vecPriority.empty() && !blockFinished) { // add a tx from priority queue to fill the blockprioritysize
|
||||||
|
iter = vecPriority.front().second;
|
||||||
|
actualPriority = vecPriority.front().first;
|
||||||
|
std::pop_heap(vecPriority.begin(), vecPriority.end(), pricomparer);
|
||||||
|
vecPriority.pop_back();
|
||||||
|
|
||||||
int64_t nLockTimeCutoff = (STANDARD_LOCKTIME_VERIFY_FLAGS & LOCKTIME_MEDIAN_TIME_PAST)
|
// If tx already in block, skip
|
||||||
? nMedianTimePast
|
if (inBlock.count(iter)) {
|
||||||
: pblock->GetBlockTime();
|
assert(false); // shouldn't happen for priority txs
|
||||||
|
continue;
|
||||||
bool fPriorityBlock = nBlockPrioritySize > 0;
|
|
||||||
if (nonEmptyBlock && fPriorityBlock) {
|
|
||||||
vecPriority.reserve(mempool.mapTx.size());
|
|
||||||
for (CTxMemPool::indexed_transaction_set::iterator mi = mempool.mapTx.begin();
|
|
||||||
mi != mempool.mapTx.end(); ++mi)
|
|
||||||
{
|
|
||||||
double dPriority = mi->GetPriority(nHeight);
|
|
||||||
CAmount dummy;
|
|
||||||
mempool.ApplyDeltas(mi->GetTx().GetHash(), dPriority, dummy);
|
|
||||||
vecPriority.push_back(TxCoinAgePriority(dPriority, mi));
|
|
||||||
}
|
|
||||||
std::make_heap(vecPriority.begin(), vecPriority.end(), pricomparer);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
CTxMemPool::indexed_transaction_set::index<mining_score>::type::iterator mi = mempool.mapTx.get<mining_score>().begin();
|
// If tx is dependent on other mempool txs which haven't yet been included
|
||||||
CTxMemPool::txiter iter;
|
// then put it in the waitSet
|
||||||
|
if (isStillDependent(iter)) {
|
||||||
// We want to track the value pool, but if the miner gets
|
waitPriMap.insert(std::make_pair(iter, actualPriority));
|
||||||
// invoked on an old block before the hardcoded fallback
|
continue;
|
||||||
// is active we don't want to trip up any assertions. So,
|
|
||||||
// we only adhere to the turnstile (as a miner) if we
|
|
||||||
// actually have all of the information necessary to do
|
|
||||||
// so.
|
|
||||||
CAmount sproutValue = 0;
|
|
||||||
CAmount saplingValue = 0;
|
|
||||||
CAmount orchardValue = 0;
|
|
||||||
bool monitoring_pool_balances = true;
|
|
||||||
if (chainparams.ZIP209Enabled()) {
|
|
||||||
if (pindexPrev->nChainSproutValue) {
|
|
||||||
sproutValue = *pindexPrev->nChainSproutValue;
|
|
||||||
} else {
|
|
||||||
monitoring_pool_balances = false;
|
|
||||||
}
|
|
||||||
if (pindexPrev->nChainSaplingValue) {
|
|
||||||
saplingValue = *pindexPrev->nChainSaplingValue;
|
|
||||||
} else {
|
|
||||||
monitoring_pool_balances = false;
|
|
||||||
}
|
|
||||||
if (pindexPrev->nChainOrchardValue) {
|
|
||||||
orchardValue = *pindexPrev->nChainOrchardValue;
|
|
||||||
} else {
|
|
||||||
monitoring_pool_balances = false;
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
while (nonEmptyBlock && (mi != mempool.mapTx.get<mining_score>().end() || !clearedTxs.empty()))
|
// If this tx fits in the block add it, otherwise keep looping
|
||||||
{
|
if (TestForBlock(iter)) {
|
||||||
bool priorityTx = false;
|
AddToBlock(iter);
|
||||||
if (fPriorityBlock && !vecPriority.empty()) { // add a tx from priority queue to fill the blockprioritysize
|
|
||||||
priorityTx = true;
|
// If now that this txs is added we've surpassed our desired priority size
|
||||||
iter = vecPriority.front().second;
|
// or have dropped below the AllowFreeThreshold, then we're done adding priority txs
|
||||||
actualPriority = vecPriority.front().first;
|
if (nBlockSize + iter->GetTxSize() >= nBlockPrioritySize || !AllowFree(actualPriority)) {
|
||||||
std::pop_heap(vecPriority.begin(), vecPriority.end(), pricomparer);
|
return;
|
||||||
vecPriority.pop_back();
|
|
||||||
}
|
|
||||||
else if (clearedTxs.empty()) { // add tx with next highest score
|
|
||||||
iter = mempool.mapTx.project<0>(mi);
|
|
||||||
mi++;
|
|
||||||
}
|
|
||||||
else { // try to add a previously postponed child tx
|
|
||||||
iter = clearedTxs.top();
|
|
||||||
clearedTxs.pop();
|
|
||||||
}
|
}
|
||||||
|
|
||||||
if (inBlock.count(iter))
|
// This tx was successfully added, so
|
||||||
continue; // could have been added to the priorityBlock
|
// add transactions that depend on this one to the priority queue to try again
|
||||||
|
|
||||||
const CTransaction& tx = iter->GetTx();
|
|
||||||
const uint256& hash = tx.GetHash();
|
|
||||||
|
|
||||||
bool fOrphan = false;
|
|
||||||
for (CTxMemPool::txiter parent : mempool.GetMemPoolParents(iter))
|
|
||||||
{
|
|
||||||
if (!inBlock.count(parent)) {
|
|
||||||
fOrphan = true;
|
|
||||||
break;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
if (fOrphan) {
|
|
||||||
if (priorityTx)
|
|
||||||
waitPriMap.insert(std::make_pair(iter,actualPriority));
|
|
||||||
else
|
|
||||||
waitSet.insert(iter);
|
|
||||||
continue;
|
|
||||||
}
|
|
||||||
|
|
||||||
unsigned int nTxSize = iter->GetTxSize();
|
|
||||||
if (fPriorityBlock &&
|
|
||||||
(nBlockSize + nTxSize >= nBlockPrioritySize || !AllowFree(actualPriority))) {
|
|
||||||
fPriorityBlock = false;
|
|
||||||
waitPriMap.clear();
|
|
||||||
}
|
|
||||||
if (!priorityTx &&
|
|
||||||
(iter->GetModifiedFee() < ::minRelayTxFee.GetFee(nTxSize)) &&
|
|
||||||
(iter->GetModifiedFee() < DEFAULT_FEE) &&
|
|
||||||
(nBlockSize >= nBlockMinSize)) {
|
|
||||||
break;
|
|
||||||
}
|
|
||||||
if (nBlockSize + nTxSize >= nBlockMaxSize) {
|
|
||||||
if (nBlockSize > nBlockMaxSize - 100 || lastFewTxs > 50) {
|
|
||||||
break;
|
|
||||||
}
|
|
||||||
// Once we're within 1000 bytes of a full block, only look at 50 more txs
|
|
||||||
// to try to fill the remaining space.
|
|
||||||
if (nBlockSize > nBlockMaxSize - 1000) {
|
|
||||||
lastFewTxs++;
|
|
||||||
}
|
|
||||||
LogPrintf("%s: skipping tx %s: exceeded maximum block size %u.", __func__, hash.GetHex(), nBlockMaxSize);
|
|
||||||
continue;
|
|
||||||
}
|
|
||||||
|
|
||||||
if (!IsFinalTx(tx, nHeight, nLockTimeCutoff) || IsExpiredTx(tx, nHeight))
|
|
||||||
continue;
|
|
||||||
|
|
||||||
unsigned int nTxSigOps = iter->GetSigOpCount();
|
|
||||||
if (nBlockSigOps + nTxSigOps >= MAX_BLOCK_SIGOPS) {
|
|
||||||
if (nBlockSigOps > MAX_BLOCK_SIGOPS - 2) {
|
|
||||||
break;
|
|
||||||
}
|
|
||||||
LogPrintf("%s: skipping tx %s: exceeds legacy max sigops %u.", __func__, hash.GetHex(), MAX_BLOCK_SIGOPS);
|
|
||||||
continue;
|
|
||||||
}
|
|
||||||
|
|
||||||
if (chainparams.ZIP209Enabled() && monitoring_pool_balances) {
|
|
||||||
// Does this transaction lead to a turnstile violation?
|
|
||||||
|
|
||||||
CAmount sproutValueDummy = sproutValue;
|
|
||||||
CAmount saplingValueDummy = saplingValue;
|
|
||||||
CAmount orchardValueDummy = orchardValue;
|
|
||||||
|
|
||||||
saplingValueDummy += -tx.GetValueBalanceSapling();
|
|
||||||
orchardValueDummy += -tx.GetOrchardBundle().GetValueBalance();
|
|
||||||
|
|
||||||
for (auto js : tx.vJoinSplit) {
|
|
||||||
sproutValueDummy += js.vpub_old;
|
|
||||||
sproutValueDummy -= js.vpub_new;
|
|
||||||
}
|
|
||||||
|
|
||||||
if (sproutValueDummy < 0) {
|
|
||||||
LogPrintf("%s: tx %s appears to violate Sprout turnstile\n", __func__, hash.GetHex());
|
|
||||||
continue;
|
|
||||||
}
|
|
||||||
if (saplingValueDummy < 0) {
|
|
||||||
LogPrintf("%s: tx %s appears to violate Sapling turnstile\n", __func__, hash.GetHex());
|
|
||||||
continue;
|
|
||||||
}
|
|
||||||
if (orchardValueDummy < 0) {
|
|
||||||
LogPrintf("%s: tx %s appears to violate Orchard turnstile\n", __func__, hash.GetHex());
|
|
||||||
continue;
|
|
||||||
}
|
|
||||||
|
|
||||||
sproutValue = sproutValueDummy;
|
|
||||||
saplingValue = saplingValueDummy;
|
|
||||||
orchardValue = orchardValueDummy;
|
|
||||||
}
|
|
||||||
|
|
||||||
CAmount nTxFees = iter->GetFee();
|
|
||||||
// Added
|
|
||||||
pblock->vtx.push_back(tx);
|
|
||||||
pblocktemplate->vTxFees.push_back(nTxFees);
|
|
||||||
pblocktemplate->vTxSigOps.push_back(nTxSigOps);
|
|
||||||
nBlockSize += nTxSize;
|
|
||||||
++nBlockTx;
|
|
||||||
nBlockSigOps += nTxSigOps;
|
|
||||||
nFees += nTxFees;
|
|
||||||
|
|
||||||
if (fPrintPriority)
|
|
||||||
{
|
|
||||||
double dPriority = iter->GetPriority(nHeight);
|
|
||||||
CAmount dummy;
|
|
||||||
mempool.ApplyDeltas(tx.GetHash(), dPriority, dummy);
|
|
||||||
LogPrintf("%s: priority %.1f fee %s txid %s\n",
|
|
||||||
__func__, dPriority , CFeeRate(iter->GetModifiedFee(), nTxSize).ToString(), hash.GetHex());
|
|
||||||
}
|
|
||||||
|
|
||||||
inBlock.insert(iter);
|
|
||||||
|
|
||||||
// Add transactions that depend on this one to the priority queue
|
|
||||||
for (CTxMemPool::txiter child : mempool.GetMemPoolChildren(iter))
|
for (CTxMemPool::txiter child : mempool.GetMemPoolChildren(iter))
|
||||||
{
|
{
|
||||||
if (fPriorityBlock) {
|
waitPriIter wpiter = waitPriMap.find(child);
|
||||||
waitPriIter wpiter = waitPriMap.find(child);
|
if (wpiter != waitPriMap.end()) {
|
||||||
if (wpiter != waitPriMap.end()) {
|
vecPriority.push_back(TxCoinAgePriority(wpiter->second,child));
|
||||||
vecPriority.push_back(TxCoinAgePriority(wpiter->second,child));
|
std::push_heap(vecPriority.begin(), vecPriority.end(), pricomparer);
|
||||||
std::push_heap(vecPriority.begin(), vecPriority.end(), pricomparer);
|
waitPriMap.erase(wpiter);
|
||||||
waitPriMap.erase(wpiter);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
else {
|
|
||||||
if (waitSet.count(child)) {
|
|
||||||
clearedTxs.push(child);
|
|
||||||
waitSet.erase(child);
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
last_block_num_txs = nBlockTx;
|
|
||||||
last_block_size = nBlockSize;
|
|
||||||
LogPrintf("%s: total size %u (excluding coinbase) txs: %u fees: %ld sigops %d", __func__, nBlockSize, nBlockTx, nFees, nBlockSigOps);
|
|
||||||
|
|
||||||
// Create coinbase tx
|
|
||||||
if (next_cb_mtx) {
|
|
||||||
pblock->vtx[0] = *next_cb_mtx;
|
|
||||||
} else {
|
|
||||||
pblock->vtx[0] = CreateCoinbaseTransaction(chainparams, nFees, minerAddress, nHeight);
|
|
||||||
}
|
|
||||||
pblocktemplate->vTxFees[0] = -nFees;
|
|
||||||
|
|
||||||
// Update the Sapling commitment tree.
|
|
||||||
for (const CTransaction& tx : pblock->vtx) {
|
|
||||||
for (const OutputDescription& odesc : tx.vShieldedOutput) {
|
|
||||||
sapling_tree.append(odesc.cmu);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
// Randomise nonce
|
|
||||||
arith_uint256 nonce = UintToArith256(GetRandHash());
|
|
||||||
// Clear the top and bottom 16 bits (for local use as thread flags and counters)
|
|
||||||
nonce <<= 32;
|
|
||||||
nonce >>= 16;
|
|
||||||
pblock->nNonce = ArithToUint256(nonce);
|
|
||||||
|
|
||||||
uint32_t prevConsensusBranchId = CurrentEpochBranchId(pindexPrev->nHeight, chainparams.GetConsensus());
|
|
||||||
|
|
||||||
// Fill in header
|
|
||||||
pblock->hashPrevBlock = pindexPrev->GetBlockHash();
|
|
||||||
if (chainparams.GetConsensus().NetworkUpgradeActive(nHeight, Consensus::UPGRADE_NU5)) {
|
|
||||||
// hashBlockCommitments depends on the block transactions, so we have to
|
|
||||||
// update it whenever the coinbase transaction changes.
|
|
||||||
//
|
|
||||||
// - For the internal miner (either directly or via the `generate` RPC), this
|
|
||||||
// will occur in `IncrementExtraNonce()`, like for `hashMerkleRoot`.
|
|
||||||
// - For `getblocktemplate`, we have two sets of fields to handle:
|
|
||||||
// - The `defaultroots` fields, which contain both the default value (if
|
|
||||||
// nothing in the template is altered), and the roots that can be used to
|
|
||||||
// recalculate it (if some or all of the template is altered).
|
|
||||||
// - The legacy `finalsaplingroothash`, `lightclientroothash`, and
|
|
||||||
// `blockcommitmentshash` fields, which had the semantics of "place this
|
|
||||||
// value into the block header and things will work" (except for in
|
|
||||||
// v4.6.0 where they were accidentally set to always be the NU5 value).
|
|
||||||
//
|
|
||||||
// To accommodate all use cases, we calculate the `hashBlockCommitments`
|
|
||||||
// default value here (unlike `hashMerkleRoot`), and additionally cache the
|
|
||||||
// values necessary to recalculate it.
|
|
||||||
pblocktemplate->hashChainHistoryRoot = view.GetHistoryRoot(prevConsensusBranchId);
|
|
||||||
pblocktemplate->hashAuthDataRoot = pblock->BuildAuthDataMerkleTree();
|
|
||||||
pblock->hashBlockCommitments = DeriveBlockCommitmentsHash(
|
|
||||||
pblocktemplate->hashChainHistoryRoot,
|
|
||||||
pblocktemplate->hashAuthDataRoot);
|
|
||||||
} else if (IsActivationHeight(nHeight, chainparams.GetConsensus(), Consensus::UPGRADE_HEARTWOOD)) {
|
|
||||||
pblocktemplate->hashChainHistoryRoot.SetNull();
|
|
||||||
pblocktemplate->hashAuthDataRoot.SetNull();
|
|
||||||
pblock->hashBlockCommitments.SetNull();
|
|
||||||
} else if (chainparams.GetConsensus().NetworkUpgradeActive(nHeight, Consensus::UPGRADE_HEARTWOOD)) {
|
|
||||||
pblocktemplate->hashChainHistoryRoot = view.GetHistoryRoot(prevConsensusBranchId);
|
|
||||||
pblocktemplate->hashAuthDataRoot.SetNull();
|
|
||||||
pblock->hashBlockCommitments = pblocktemplate->hashChainHistoryRoot;
|
|
||||||
} else {
|
|
||||||
pblocktemplate->hashChainHistoryRoot.SetNull();
|
|
||||||
pblocktemplate->hashAuthDataRoot.SetNull();
|
|
||||||
pblock->hashBlockCommitments = sapling_tree.root();
|
|
||||||
}
|
|
||||||
UpdateTime(pblock, chainparams.GetConsensus(), pindexPrev);
|
|
||||||
pblock->nBits = GetNextWorkRequired(pindexPrev, pblock, chainparams.GetConsensus());
|
|
||||||
pblock->nSolution.clear();
|
|
||||||
pblocktemplate->vTxSigOps[0] = GetLegacySigOpCount(pblock->vtx[0]);
|
|
||||||
|
|
||||||
CValidationState state;
|
|
||||||
if (!TestBlockValidity(state, chainparams, *pblock, pindexPrev, true)) {
|
|
||||||
throw std::runtime_error(strprintf("%s: TestBlockValidity failed: %s", __func__, FormatStateMessage(state)));
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
return pblocktemplate.release();
|
|
||||||
}
|
}
|
||||||
|
|
||||||
//////////////////////////////////////////////////////////////////////////////
|
//////////////////////////////////////////////////////////////////////////////
|
||||||
|
@ -874,7 +963,7 @@ void static BitcoinMiner(const CChainParams& chainparams)
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
|
|
||||||
unique_ptr<CBlockTemplate> pblocktemplate(CreateNewBlock(chainparams, minerAddress));
|
unique_ptr<CBlockTemplate> pblocktemplate(BlockAssembler(chainparams).CreateNewBlock(minerAddress));
|
||||||
if (!pblocktemplate.get())
|
if (!pblocktemplate.get())
|
||||||
{
|
{
|
||||||
if (GetArg("-mineraddress", "").empty()) {
|
if (GetArg("-mineraddress", "").empty()) {
|
||||||
|
|
63
src/miner.h
63
src/miner.h
|
@ -8,8 +8,10 @@
|
||||||
#define BITCOIN_MINER_H
|
#define BITCOIN_MINER_H
|
||||||
|
|
||||||
#include "primitives/block.h"
|
#include "primitives/block.h"
|
||||||
|
#include "txmempool.h"
|
||||||
|
|
||||||
#include <stdint.h>
|
#include <stdint.h>
|
||||||
|
#include <memory>
|
||||||
#include <variant>
|
#include <variant>
|
||||||
|
|
||||||
#include <boost/shared_ptr.hpp>
|
#include <boost/shared_ptr.hpp>
|
||||||
|
@ -17,6 +19,7 @@
|
||||||
class CBlockIndex;
|
class CBlockIndex;
|
||||||
class CChainParams;
|
class CChainParams;
|
||||||
class CScript;
|
class CScript;
|
||||||
|
|
||||||
namespace Consensus { struct Params; };
|
namespace Consensus { struct Params; };
|
||||||
|
|
||||||
static const bool DEFAULT_GENERATE = false;
|
static const bool DEFAULT_GENERATE = false;
|
||||||
|
@ -95,7 +98,65 @@ struct CBlockTemplate
|
||||||
CMutableTransaction CreateCoinbaseTransaction(const CChainParams& chainparams, CAmount nFees, const MinerAddress& minerAddress, int nHeight);
|
CMutableTransaction CreateCoinbaseTransaction(const CChainParams& chainparams, CAmount nFees, const MinerAddress& minerAddress, int nHeight);
|
||||||
|
|
||||||
/** Generate a new block, without valid proof-of-work */
|
/** Generate a new block, without valid proof-of-work */
|
||||||
CBlockTemplate* CreateNewBlock(const CChainParams& chainparams, const MinerAddress& minerAddress, const std::optional<CMutableTransaction>& next_coinbase_mtx = std::nullopt);
|
class BlockAssembler
|
||||||
|
{
|
||||||
|
private:
|
||||||
|
// The constructed block template
|
||||||
|
std::unique_ptr<CBlockTemplate> pblocktemplate;
|
||||||
|
// A convenience pointer that always refers to the CBlock in pblocktemplate
|
||||||
|
CBlock* pblock;
|
||||||
|
|
||||||
|
// Configuration parameters for the block size
|
||||||
|
unsigned int nBlockMaxSize, nBlockMinSize;
|
||||||
|
|
||||||
|
// Information on the current status of the block
|
||||||
|
uint64_t nBlockSize;
|
||||||
|
uint64_t nBlockTx;
|
||||||
|
unsigned int nBlockSigOps;
|
||||||
|
CAmount nFees;
|
||||||
|
CTxMemPool::setEntries inBlock;
|
||||||
|
|
||||||
|
// Information on the current chain state after this block
|
||||||
|
CAmount sproutValue;
|
||||||
|
CAmount saplingValue;
|
||||||
|
CAmount orchardValue;
|
||||||
|
bool monitoring_pool_balances;
|
||||||
|
|
||||||
|
// Chain context for the block
|
||||||
|
int nHeight;
|
||||||
|
int64_t nLockTimeCutoff;
|
||||||
|
const CChainParams& chainparams;
|
||||||
|
|
||||||
|
// Variables used for addScoreTxs and addPriorityTxs
|
||||||
|
int lastFewTxs;
|
||||||
|
bool blockFinished;
|
||||||
|
|
||||||
|
public:
|
||||||
|
BlockAssembler(const CChainParams& chainparams);
|
||||||
|
/** Construct a new block template with coinbase to minerAddress */
|
||||||
|
CBlockTemplate* CreateNewBlock(
|
||||||
|
const MinerAddress& minerAddress,
|
||||||
|
const std::optional<CMutableTransaction>& next_coinbase_mtx = std::nullopt);
|
||||||
|
|
||||||
|
private:
|
||||||
|
// utility functions
|
||||||
|
/** Clear the block's state and prepare for assembling a new block */
|
||||||
|
void resetBlock();
|
||||||
|
/** Add a tx to the block */
|
||||||
|
void AddToBlock(CTxMemPool::txiter iter);
|
||||||
|
|
||||||
|
// Methods for how to add transactions to a block.
|
||||||
|
/** Add transactions based on modified feerate */
|
||||||
|
void addScoreTxs();
|
||||||
|
/** Add transactions based on tx "priority" */
|
||||||
|
void addPriorityTxs();
|
||||||
|
|
||||||
|
// helper function for addScoreTxs and addPriorityTxs
|
||||||
|
/** Test if tx will still "fit" in the block */
|
||||||
|
bool TestForBlock(CTxMemPool::txiter iter);
|
||||||
|
/** Test if tx still has unconfirmed parents not yet in block */
|
||||||
|
bool isStillDependent(CTxMemPool::txiter iter);
|
||||||
|
};
|
||||||
|
|
||||||
#ifdef ENABLE_MINING
|
#ifdef ENABLE_MINING
|
||||||
/** Get -mineraddress */
|
/** Get -mineraddress */
|
||||||
|
|
|
@ -221,7 +221,7 @@ UniValue generate(const UniValue& params, bool fHelp)
|
||||||
unsigned int k = Params().GetConsensus().nEquihashK;
|
unsigned int k = Params().GetConsensus().nEquihashK;
|
||||||
while (nHeight < nHeightEnd)
|
while (nHeight < nHeightEnd)
|
||||||
{
|
{
|
||||||
std::unique_ptr<CBlockTemplate> pblocktemplate(CreateNewBlock(Params(), minerAddress));
|
std::unique_ptr<CBlockTemplate> pblocktemplate(BlockAssembler(Params()).CreateNewBlock(minerAddress));
|
||||||
if (!pblocktemplate.get())
|
if (!pblocktemplate.get())
|
||||||
throw JSONRPCError(RPC_INTERNAL_ERROR, "Couldn't create new block");
|
throw JSONRPCError(RPC_INTERNAL_ERROR, "Couldn't create new block");
|
||||||
CBlock *pblock = &pblocktemplate->block;
|
CBlock *pblock = &pblocktemplate->block;
|
||||||
|
@ -700,7 +700,7 @@ UniValue getblocktemplate(const UniValue& params, bool fHelp)
|
||||||
throw JSONRPCError(RPC_INTERNAL_ERROR, "No miner address available (mining requires a wallet or -mineraddress)");
|
throw JSONRPCError(RPC_INTERNAL_ERROR, "No miner address available (mining requires a wallet or -mineraddress)");
|
||||||
}
|
}
|
||||||
|
|
||||||
pblocktemplate = CreateNewBlock(Params(), minerAddress, next_cb_mtx);
|
pblocktemplate = BlockAssembler(Params()).CreateNewBlock(minerAddress, next_cb_mtx);
|
||||||
if (!pblocktemplate)
|
if (!pblocktemplate)
|
||||||
throw JSONRPCError(RPC_OUT_OF_MEMORY, "Out of memory");
|
throw JSONRPCError(RPC_OUT_OF_MEMORY, "Out of memory");
|
||||||
|
|
||||||
|
|
|
@ -168,7 +168,7 @@ BOOST_AUTO_TEST_CASE(CreateNewBlock_validity)
|
||||||
fCoinbaseEnforcedShieldingEnabled = false;
|
fCoinbaseEnforcedShieldingEnabled = false;
|
||||||
|
|
||||||
// Simple block creation, nothing special yet:
|
// Simple block creation, nothing special yet:
|
||||||
BOOST_CHECK(pblocktemplate = CreateNewBlock(chainparams, scriptPubKey));
|
BOOST_CHECK(pblocktemplate = BlockAssembler(chainparams).CreateNewBlock(scriptPubKey));
|
||||||
|
|
||||||
// We can't make transactions until we have inputs
|
// We can't make transactions until we have inputs
|
||||||
// Therefore, load 100 blocks :)
|
// Therefore, load 100 blocks :)
|
||||||
|
@ -294,7 +294,7 @@ BOOST_AUTO_TEST_CASE(CreateNewBlock_validity)
|
||||||
delete pblocktemplate;
|
delete pblocktemplate;
|
||||||
|
|
||||||
// Just to make sure we can still make simple blocks
|
// Just to make sure we can still make simple blocks
|
||||||
BOOST_CHECK(pblocktemplate = CreateNewBlock(chainparams, scriptPubKey));
|
BOOST_CHECK(pblocktemplate = BlockAssembler(chainparams).CreateNewBlock(scriptPubKey));
|
||||||
delete pblocktemplate;
|
delete pblocktemplate;
|
||||||
|
|
||||||
const CAmount BLOCKSUBSIDY = 50000; // First slow start subsidy. Upstream: 50*COIN
|
const CAmount BLOCKSUBSIDY = 50000; // First slow start subsidy. Upstream: 50*COIN
|
||||||
|
@ -327,7 +327,7 @@ BOOST_AUTO_TEST_CASE(CreateNewBlock_validity)
|
||||||
mempool.addUnchecked(hash, entry.Fee(LOWFEE).Time(GetTime()).SpendsCoinbase(spendsCoinbase).FromTx(tx));
|
mempool.addUnchecked(hash, entry.Fee(LOWFEE).Time(GetTime()).SpendsCoinbase(spendsCoinbase).FromTx(tx));
|
||||||
tx.vin[0].prevout.hash = hash;
|
tx.vin[0].prevout.hash = hash;
|
||||||
}
|
}
|
||||||
BOOST_CHECK_EXCEPTION(CreateNewBlock(chainparams, scriptPubKey), std::runtime_error, err_is("bad-blk-sigops"));
|
BOOST_CHECK_EXCEPTION(BlockAssembler(chainparams).CreateNewBlock(scriptPubKey), std::runtime_error, err_is("bad-blk-sigops"));
|
||||||
mempool.clear();
|
mempool.clear();
|
||||||
|
|
||||||
tx.vin[0].prevout.hash = txFirst[0]->GetHash();
|
tx.vin[0].prevout.hash = txFirst[0]->GetHash();
|
||||||
|
@ -341,7 +341,7 @@ BOOST_AUTO_TEST_CASE(CreateNewBlock_validity)
|
||||||
mempool.addUnchecked(hash, entry.Fee(LOWFEE).Time(GetTime()).SpendsCoinbase(spendsCoinbase).SigOps(20).FromTx(tx));
|
mempool.addUnchecked(hash, entry.Fee(LOWFEE).Time(GetTime()).SpendsCoinbase(spendsCoinbase).SigOps(20).FromTx(tx));
|
||||||
tx.vin[0].prevout.hash = hash;
|
tx.vin[0].prevout.hash = hash;
|
||||||
}
|
}
|
||||||
BOOST_CHECK(pblocktemplate = CreateNewBlock(chainparams, scriptPubKey));
|
BOOST_CHECK(pblocktemplate = BlockAssembler(chainparams).CreateNewBlock(scriptPubKey));
|
||||||
delete pblocktemplate;
|
delete pblocktemplate;
|
||||||
mempool.clear();
|
mempool.clear();
|
||||||
|
|
||||||
|
@ -362,14 +362,14 @@ BOOST_AUTO_TEST_CASE(CreateNewBlock_validity)
|
||||||
mempool.addUnchecked(hash, entry.Fee(LOWFEE).Time(GetTime()).SpendsCoinbase(spendsCoinbase).FromTx(tx));
|
mempool.addUnchecked(hash, entry.Fee(LOWFEE).Time(GetTime()).SpendsCoinbase(spendsCoinbase).FromTx(tx));
|
||||||
tx.vin[0].prevout.hash = hash;
|
tx.vin[0].prevout.hash = hash;
|
||||||
}
|
}
|
||||||
BOOST_CHECK(pblocktemplate = CreateNewBlock(chainparams, scriptPubKey));
|
BOOST_CHECK(pblocktemplate = BlockAssembler(chainparams).CreateNewBlock(scriptPubKey));
|
||||||
delete pblocktemplate;
|
delete pblocktemplate;
|
||||||
mempool.clear();
|
mempool.clear();
|
||||||
|
|
||||||
// orphan in mempool, template creation fails
|
// orphan in mempool, template creation fails
|
||||||
hash = tx.GetHash();
|
hash = tx.GetHash();
|
||||||
mempool.addUnchecked(hash, entry.Fee(LOWFEE).Time(GetTime()).FromTx(tx));
|
mempool.addUnchecked(hash, entry.Fee(LOWFEE).Time(GetTime()).FromTx(tx));
|
||||||
BOOST_CHECK_EXCEPTION(CreateNewBlock(chainparams, scriptPubKey), std::runtime_error, err_is("bad-txns-inputs-missingorspent"));
|
BOOST_CHECK_EXCEPTION(BlockAssembler(chainparams).CreateNewBlock(scriptPubKey), std::runtime_error, err_is("bad-txns-inputs-missingorspent"));
|
||||||
mempool.clear();
|
mempool.clear();
|
||||||
|
|
||||||
// child with higher priority than parent
|
// child with higher priority than parent
|
||||||
|
@ -386,7 +386,7 @@ BOOST_AUTO_TEST_CASE(CreateNewBlock_validity)
|
||||||
tx.vout[0].nValue = tx.vout[0].nValue+BLOCKSUBSIDY-HIGHERFEE; //First txn output + fresh coinbase - new txn fee
|
tx.vout[0].nValue = tx.vout[0].nValue+BLOCKSUBSIDY-HIGHERFEE; //First txn output + fresh coinbase - new txn fee
|
||||||
hash = tx.GetHash();
|
hash = tx.GetHash();
|
||||||
mempool.addUnchecked(hash, entry.Fee(HIGHERFEE).Time(GetTime()).SpendsCoinbase(true).FromTx(tx));
|
mempool.addUnchecked(hash, entry.Fee(HIGHERFEE).Time(GetTime()).SpendsCoinbase(true).FromTx(tx));
|
||||||
BOOST_CHECK(pblocktemplate = CreateNewBlock(chainparams, scriptPubKey));
|
BOOST_CHECK(pblocktemplate = BlockAssembler(chainparams).CreateNewBlock(scriptPubKey));
|
||||||
delete pblocktemplate;
|
delete pblocktemplate;
|
||||||
mempool.clear();
|
mempool.clear();
|
||||||
|
|
||||||
|
@ -398,7 +398,7 @@ BOOST_AUTO_TEST_CASE(CreateNewBlock_validity)
|
||||||
hash = tx.GetHash();
|
hash = tx.GetHash();
|
||||||
// give it a fee so it'll get mined
|
// give it a fee so it'll get mined
|
||||||
mempool.addUnchecked(hash, entry.Fee(LOWFEE).Time(GetTime()).SpendsCoinbase(false).FromTx(tx));
|
mempool.addUnchecked(hash, entry.Fee(LOWFEE).Time(GetTime()).SpendsCoinbase(false).FromTx(tx));
|
||||||
BOOST_CHECK_EXCEPTION(CreateNewBlock(chainparams, scriptPubKey), std::runtime_error, err_is("bad-cb-multiple"));
|
BOOST_CHECK_EXCEPTION(BlockAssembler(chainparams).CreateNewBlock(scriptPubKey), std::runtime_error, err_is("bad-cb-multiple"));
|
||||||
mempool.clear();
|
mempool.clear();
|
||||||
|
|
||||||
// P2SH txn in mempool (valid, Zcash always had P2SH enabled)
|
// P2SH txn in mempool (valid, Zcash always had P2SH enabled)
|
||||||
|
@ -415,7 +415,7 @@ BOOST_AUTO_TEST_CASE(CreateNewBlock_validity)
|
||||||
tx.vout[0].nValue -= LOWFEE;
|
tx.vout[0].nValue -= LOWFEE;
|
||||||
hash = tx.GetHash();
|
hash = tx.GetHash();
|
||||||
mempool.addUnchecked(hash, entry.Fee(LOWFEE).Time(GetTime()).SpendsCoinbase(false).FromTx(tx));
|
mempool.addUnchecked(hash, entry.Fee(LOWFEE).Time(GetTime()).SpendsCoinbase(false).FromTx(tx));
|
||||||
BOOST_CHECK(pblocktemplate = CreateNewBlock(chainparams, scriptPubKey));
|
BOOST_CHECK(pblocktemplate = BlockAssembler(chainparams).CreateNewBlock(scriptPubKey));
|
||||||
delete pblocktemplate;
|
delete pblocktemplate;
|
||||||
mempool.clear();
|
mempool.clear();
|
||||||
|
|
||||||
|
@ -429,16 +429,16 @@ BOOST_AUTO_TEST_CASE(CreateNewBlock_validity)
|
||||||
tx.vout[0].scriptPubKey = CScript() << OP_2;
|
tx.vout[0].scriptPubKey = CScript() << OP_2;
|
||||||
hash = tx.GetHash();
|
hash = tx.GetHash();
|
||||||
mempool.addUnchecked(hash, entry.Fee(HIGHFEE).Time(GetTime()).SpendsCoinbase(true).FromTx(tx));
|
mempool.addUnchecked(hash, entry.Fee(HIGHFEE).Time(GetTime()).SpendsCoinbase(true).FromTx(tx));
|
||||||
BOOST_CHECK_EXCEPTION(CreateNewBlock(chainparams, scriptPubKey), std::runtime_error, err_is("bad-txns-inputs-missingorspent"));
|
BOOST_CHECK_EXCEPTION(BlockAssembler(chainparams).CreateNewBlock(scriptPubKey), std::runtime_error, err_is("bad-txns-inputs-missingorspent"));
|
||||||
mempool.clear();
|
mempool.clear();
|
||||||
|
|
||||||
// subsidy changing
|
// subsidy changing
|
||||||
int nHeight = chainActive.Height();
|
int nHeight = chainActive.Height();
|
||||||
chainActive.Tip()->nHeight = 209999;
|
chainActive.Tip()->nHeight = 209999;
|
||||||
BOOST_CHECK(pblocktemplate = CreateNewBlock(chainparams, scriptPubKey));
|
BOOST_CHECK(pblocktemplate = BlockAssembler(chainparams).CreateNewBlock(scriptPubKey));
|
||||||
delete pblocktemplate;
|
delete pblocktemplate;
|
||||||
chainActive.Tip()->nHeight = 210000;
|
chainActive.Tip()->nHeight = 210000;
|
||||||
BOOST_CHECK(pblocktemplate = CreateNewBlock(chainparams, scriptPubKey));
|
BOOST_CHECK(pblocktemplate = BlockAssembler(chainparams).CreateNewBlock(scriptPubKey));
|
||||||
delete pblocktemplate;
|
delete pblocktemplate;
|
||||||
chainActive.Tip()->nHeight = nHeight;
|
chainActive.Tip()->nHeight = nHeight;
|
||||||
|
|
||||||
|
@ -473,7 +473,7 @@ BOOST_AUTO_TEST_CASE(CreateNewBlock_validity)
|
||||||
mempool.addUnchecked(hash, entry.Fee(HIGHFEE).Time(GetTime()).SpendsCoinbase(true).FromTx(tx2));
|
mempool.addUnchecked(hash, entry.Fee(HIGHFEE).Time(GetTime()).SpendsCoinbase(true).FromTx(tx2));
|
||||||
BOOST_CHECK(!CheckFinalTx(tx2, LOCKTIME_MEDIAN_TIME_PAST));
|
BOOST_CHECK(!CheckFinalTx(tx2, LOCKTIME_MEDIAN_TIME_PAST));
|
||||||
|
|
||||||
BOOST_CHECK(pblocktemplate = CreateNewBlock(chainparams, scriptPubKey));
|
BOOST_CHECK(pblocktemplate = BlockAssembler(chainparams).CreateNewBlock(scriptPubKey));
|
||||||
|
|
||||||
// Neither tx should have made it into the template.
|
// Neither tx should have made it into the template.
|
||||||
BOOST_CHECK_EQUAL(pblocktemplate->block.vtx.size(), 1);
|
BOOST_CHECK_EQUAL(pblocktemplate->block.vtx.size(), 1);
|
||||||
|
@ -488,7 +488,7 @@ BOOST_AUTO_TEST_CASE(CreateNewBlock_validity)
|
||||||
//BOOST_CHECK(CheckFinalTx(tx));
|
//BOOST_CHECK(CheckFinalTx(tx));
|
||||||
//BOOST_CHECK(CheckFinalTx(tx2));
|
//BOOST_CHECK(CheckFinalTx(tx2));
|
||||||
|
|
||||||
BOOST_CHECK(pblocktemplate = CreateNewBlock(chainparams, scriptPubKey));
|
BOOST_CHECK(pblocktemplate = BlockAssembler(chainparams).CreateNewBlock(scriptPubKey));
|
||||||
BOOST_CHECK_EQUAL(pblocktemplate->block.vtx.size(), 2);
|
BOOST_CHECK_EQUAL(pblocktemplate->block.vtx.size(), 2);
|
||||||
delete pblocktemplate;
|
delete pblocktemplate;
|
||||||
|
|
||||||
|
|
|
@ -174,7 +174,7 @@ TestChain100Setup::CreateAndProcessBlock(const std::vector<CMutableTransaction>&
|
||||||
boost::shared_ptr<CReserveScript> mAddr(new CReserveScript());
|
boost::shared_ptr<CReserveScript> mAddr(new CReserveScript());
|
||||||
mAddr->reserveScript = scriptPubKey;
|
mAddr->reserveScript = scriptPubKey;
|
||||||
|
|
||||||
CBlockTemplate *pblocktemplate = CreateNewBlock(chainparams, mAddr);
|
CBlockTemplate *pblocktemplate = BlockAssembler(chainparams).CreateNewBlock(mAddr);
|
||||||
CBlock& block = pblocktemplate->block;
|
CBlock& block = pblocktemplate->block;
|
||||||
|
|
||||||
// Replace mempool-selected txns with just coinbase plus passed-in txns:
|
// Replace mempool-selected txns with just coinbase plus passed-in txns:
|
||||||
|
|
Loading…
Reference in New Issue