sikkienl/dash
1
0
Fork 0
mirror of https://github.com/dashpay/dash.git synced 2024-12-26 04:22:55 +01:00
Code Issues Actions Packages Projects Releases Wiki Activity

Make CMasternodeSync members atomic, remove CCriticalSection (#4462)

Browse Source
This commit is contained in:
PastaPastaPasta 2021-09-28 17:10:37 -04:00 committed by GitHub
parent 8fb5e07c4e
commit 02d7a5f965
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 20 additions and 39 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
src/masternode/masternode-sync.cpp
View File

@ -24,20 +24,16 @@ void CMasternodeSync::Reset(bool fForce, bool fNotifyReset)
{
// Avoid resetting the sync process if we just "recently" received a new block
if (!fForce) {
LOCK(cs);
if (GetTime() - nTimeLastUpdateBlockTip < MASTERNODE_SYNC_RESET_SECONDS) {
return;
}
}
{
LOCK(cs);
nCurrentAsset = MASTERNODE_SYNC_BLOCKCHAIN;
nTriedPeerCount = 0;
nTimeAssetSyncStarted = GetTime();
nTimeLastBumped = GetTime();
nTimeLastUpdateBlockTip = 0;
fReachedBestHeader = false;
}
if (fNotifyReset) {
uiInterface.NotifyAdditionalDataSyncProgressChanged(-1);
}
@ -46,14 +42,12 @@ void CMasternodeSync::Reset(bool fForce, bool fNotifyReset)
void CMasternodeSync::BumpAssetLastTime(const std::string& strFuncName)
{
if (IsSynced()) return;
LOCK(cs);
nTimeLastBumped = GetTime();
LogPrint(BCLog::MNSYNC, "CMasternodeSync::BumpAssetLastTime -- %s\n", strFuncName);
}
std::string CMasternodeSync::GetAssetName() const
{
LOCK(cs);
switch(nCurrentAsset)
{
case(MASTERNODE_SYNC_BLOCKCHAIN): return "MASTERNODE_SYNC_BLOCKCHAIN";
@ -65,7 +59,6 @@ std::string CMasternodeSync::GetAssetName() const
void CMasternodeSync::SwitchToNextAsset(CConnman& connman)
{
LOCK(cs);
switch(nCurrentAsset)
{
case(MASTERNODE_SYNC_BLOCKCHAIN):
@ -92,7 +85,6 @@ void CMasternodeSync::SwitchToNextAsset(CConnman& connman)
std::string CMasternodeSync::GetSyncStatus() const
{
LOCK(cs);
switch (nCurrentAsset) {
case MASTERNODE_SYNC_BLOCKCHAIN: return _("Synchronizing blockchain...");
case MASTERNODE_SYNC_GOVERNANCE: return _("Synchronizing governance objects...");
@ -148,8 +140,6 @@ void CMasternodeSync::ProcessTick(CConnman& connman)
return;
}
{
LOCK(cs);
// Calculate "progress" for LOG reporting / GUI notification
double nSyncProgress = double(nTriedPeerCount + (nCurrentAsset - 1) * 8) / (8*4);
LogPrint(BCLog::MNSYNC, "CMasternodeSync::ProcessTick -- nTick %d nCurrentAsset %d nTriedPeerCount %d nSyncProgress %f\n", nTick, nCurrentAsset, nTriedPeerCount, nSyncProgress);
@ -272,10 +262,9 @@ void CMasternodeSync::ProcessTick(CConnman& connman)
}
}
}
} // cs
if (WITH_LOCK(cs, return nCurrentAsset != MASTERNODE_SYNC_GOVERNANCE)) {
if (nCurrentAsset != MASTERNODE_SYNC_GOVERNANCE) {
// looped through all nodes and not syncing governance yet/already, release them
connman.ReleaseNodeVector(vNodesCopy);
return;
@ -334,7 +323,6 @@ void CMasternodeSync::SendGovernanceSyncRequest(CNode* pnode, CConnman& connman)
void CMasternodeSync::AcceptedBlockHeader(const CBlockIndex *pindexNew)
{
LogPrint(BCLog::MNSYNC, "CMasternodeSync::AcceptedBlockHeader -- pindexNew->nHeight: %d\n", pindexNew->nHeight);
LOCK(cs);
if (!IsBlockchainSynced()) {
// Postpone timeout each time new block header arrives while we are still syncing blockchain
@ -360,11 +348,7 @@ void CMasternodeSync::NotifyHeaderTip(const CBlockIndex *pindexNew, bool fInitia
void CMasternodeSync::UpdatedBlockTip(const CBlockIndex *pindexNew, bool fInitialDownload, CConnman& connman)
{
LogPrint(BCLog::MNSYNC, "CMasternodeSync::UpdatedBlockTip -- pindexNew->nHeight: %d fInitialDownload=%d\n", pindexNew->nHeight, fInitialDownload);
{
LOCK(cs);
nTimeLastUpdateBlockTip = GetAdjustedTime();
}
CBlockIndex* pindexTip;
{
@ -393,7 +377,6 @@ void CMasternodeSync::UpdatedBlockTip(const CBlockIndex *pindexNew, bool fInitia
// Note: since we sync headers first, it should be ok to use this
bool fReachedBestHeaderNew = pindexNew->GetBlockHash() == pindexTip->GetBlockHash();
LOCK(cs);
if (fReachedBestHeader && !fReachedBestHeaderNew) {
// Switching from true to false means that we previously stuck syncing headers for some reason,
// probably initial timeout was not enough,

26
src/masternode/masternode-sync.h
View File

@ -4,7 +4,7 @@
#ifndef BITCOIN_MASTERNODE_MASTERNODE_SYNC_H
#define BITCOIN_MASTERNODE_MASTERNODE_SYNC_H
#include <sync.h>
#include <atomic>
class CMasternodeSync;
class CBlockIndex;
@ -31,35 +31,33 @@ extern CMasternodeSync masternodeSync;
class CMasternodeSync
{
private:
mutable CCriticalSection cs;
// Keep track of current asset
int nCurrentAsset GUARDED_BY(cs) {MASTERNODE_SYNC_BLOCKCHAIN};
std::atomic<int> nCurrentAsset{MASTERNODE_SYNC_BLOCKCHAIN};
// Count peers we've requested the asset from
int nTriedPeerCount GUARDED_BY(cs) {0};
std::atomic<int> nTriedPeerCount{0};
// Time when current masternode asset sync started
int64_t nTimeAssetSyncStarted GUARDED_BY(cs) {0};
std::atomic<int64_t> nTimeAssetSyncStarted{0};
// ... last bumped
int64_t nTimeLastBumped GUARDED_BY(cs) {0};
std::atomic<int64_t> nTimeLastBumped{0};
/// Set to true if best header is reached in CMasternodeSync::UpdatedBlockTip
bool fReachedBestHeader GUARDED_BY(cs) {false};
std::atomic<bool> fReachedBestHeader{false};
/// Last time UpdateBlockTip has been called
int64_t nTimeLastUpdateBlockTip GUARDED_BY(cs) {0};
std::atomic<int64_t> nTimeLastUpdateBlockTip{0};
public:
CMasternodeSync();
static void SendGovernanceSyncRequest(CNode* pnode, CConnman& connman);
bool IsBlockchainSynced() const { LOCK(cs); return nCurrentAsset > MASTERNODE_SYNC_BLOCKCHAIN; }
bool IsSynced() const { LOCK(cs); return nCurrentAsset == MASTERNODE_SYNC_FINISHED; }
bool IsBlockchainSynced() const { return nCurrentAsset > MASTERNODE_SYNC_BLOCKCHAIN; }
bool IsSynced() const { return nCurrentAsset == MASTERNODE_SYNC_FINISHED; }
int GetAssetID() const { LOCK(cs); return nCurrentAsset; }
int GetAttempt() const { LOCK(cs); return nTriedPeerCount; }
int GetAssetID() const { return nCurrentAsset; }
int GetAttempt() const { return nTriedPeerCount; }
void BumpAssetLastTime(const std::string& strFuncName);
int64_t GetAssetStartTime() const { LOCK(cs); return nTimeAssetSyncStarted; }
int64_t GetAssetStartTime() const { return nTimeAssetSyncStarted; }
std::string GetAssetName() const;
std::string GetSyncStatus() const;