aboutsummaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAgeFilesLines
* Merge #9183: Final Preparation for main.cpp SplitPieter Wuille2016-12-012-18/+59
|\ | | | | | | | | | | | | | | 2c8c57e Document cs_main status when calling into PNB or PNBH (Matt Corallo) 58a215c Use ProcessNewBlockHeaders in CMPCTBLOCK processing (Matt Corallo) a8b936d Use exposed ProcessNewBlockHeaders from ProcessMessages (Matt Corallo) 63fd101 Split ::HEADERS processing into two separate cs_main locks (Matt Corallo) 4a6b1f3 Expose AcceptBlockHeader through main.h (Matt Corallo)
| * Document cs_main status when calling into PNB or PNBHMatt Corallo2016-12-011-0/+4
| |
| * Use ProcessNewBlockHeaders in CMPCTBLOCK processingMatt Corallo2016-12-011-2/+7
| |
| * Use exposed ProcessNewBlockHeaders from ProcessMessagesMatt Corallo2016-12-011-12/+15
| |
| * Split ::HEADERS processing into two separate cs_main locksMatt Corallo2016-11-231-4/+7
| | | | | | | | | | This will allow NotifyHeaderTip to be called from an AcceptBlockHeader wrapper function without holding cs_main.
| * Expose AcceptBlockHeader through main.hMatt Corallo2016-11-232-1/+27
| |
* | Merge #9188: Make orphan parent fetching ask for witnesses.Pieter Wuille2016-12-011-1/+2
|\ \ | | | | | | | | | 5b0150a Make orphan parent fetching ask for witnesses. (Gregory Maxwell)
| * | Make orphan parent fetching ask for witnesses.Gregory Maxwell2016-11-181-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | In 0.13 orphan transactions began being treated as implicit INVs for their parents. But the resulting getdata were not getting the witness flag. This fixes issue #9182 reported by chjj and roasbeef on IRC.
* | | Merge #9253: Fix calculation of number of bound sockets to usePieter Wuille2016-12-011-1/+3
|\ \ \ | | | | | | | | | | | | 9e1f468 Fix calculation of number of bound sockets to use (Matt Corallo)
| * | | Fix calculation of number of bound sockets to useMatt Corallo2016-11-301-1/+3
| | | |
* | | | Merge #9230: Fix some benign races in timestamp loggingWladimir J. van der Laan2016-12-012-3/+4
|\ \ \ \ | |/ / / |/| | | | | | | | | | | 8b22efb Make fStartedNewLine an std::atomic_bool (Matt Corallo) 507145d Fix race when accessing std::locale::classic() (Matt Corallo)
| * | | Make fStartedNewLine an std::atomic_boolMatt Corallo2016-11-271-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | While this doesnt really fix the race of adding timestamps mid-logical-line, it avoids the undefined behavior of using a bool in multiple threads.
| * | | Fix race when accessing std::locale::classic()Matt Corallo2016-11-271-1/+2
| | | | | | | | | | | | | | | | See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=78552
* | | | Merge #9226: Remove fNetworkNode and pnodeLocalHost.Pieter Wuille2016-11-303-22/+3
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | bdb922b Remove pnodeLocalHost. (Gregory Maxwell) 083f203 Remove fNetworkNode. (Gregory Maxwell)
| * | | | Remove pnodeLocalHost.Gregory Maxwell2016-11-271-15/+0
| | | | | | | | | | | | | | | | | | | | Mostly a legacy of the long removed pub/sub system.
| * | | | Remove fNetworkNode.Gregory Maxwell2016-11-273-7/+3
| |/ / / | | | | | | | | | | | | Matt pointed out to me that this appeared to be doing nothing (except involving itself in data races).
* | | | Merge #9244: Trivial refactor: Remove extern keyword from function declarationsPieter Wuille2016-11-301-11/+11
|\ \ \ \ | | | | | | | | | | | | | | | 446a8f9 Trivial refactor: Remove extern keyword from function declarations, as they are extern by default. (Karl-Johan Alm)
| * | | | Trivial refactor: Remove extern keyword from function declarations, as they ↵Karl-Johan Alm2016-11-301-11/+11
| | | | | | | | | | | | | | | | | | | | are extern by default.
* | | | | Merge #9010: Split up AppInit2 into multiple phases, daemonize after datadir ↵Pieter Wuille2016-11-306-35/+121
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | lock errors deec83f init: Get rid of fServer flag (Wladimir J. van der Laan) 16ca0bf init: Try to aquire datadir lock before and after daemonization (Wladimir J. van der Laan) 0cc8b6b init: Split up AppInit2 into multiple phases (Wladimir J. van der Laan)
| * | | | | init: Get rid of fServer flagWladimir J. van der Laan2016-11-293-5/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There is no need to store this flag globally, the variable is only used inside the initialization process. Thanks to Alex Morcos for the idea.
| * | | | | init: Try to aquire datadir lock before and after daemonizationWladimir J. van der Laan2016-11-291-11/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Before daemonization, just probe the data directory lock and print an early error message if possible. After daemonization get the data directory lock again and hold on to it until exit This creates a slight window for a race condition to happen, however this condition is harmless: it will at most make us exit without printing a message to console. $ src/bitcoind -testnet -daemon Bitcoin server starting $ src/bitcoind -testnet -daemon Error: Cannot obtain a lock on data directory /home/orion/.bitcoin/testnet3. Bitcoin Core is probably already running.
| * | | | | init: Split up AppInit2 into multiple phasesWladimir J. van der Laan2016-11-294-21/+95
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This allows doing some of the steps before e.g. daemonization and some fater.
* | | | | | Merge #9234: torcontrol: Explicitly request RSA1024 private keyWladimir J. van der Laan2016-11-301-1/+1
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | | | | | | | 7d3b627 torcontrol: Explicitly request RSA1024 private key (Wladimir J. van der Laan)
| * | | | | torcontrol: Explicitly request RSA1024 private keyWladimir J. van der Laan2016-11-281-1/+1
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When generating a new service key, explicitly request a RSA1024 one. The bitcoin P2P protocol has no support for the longer hidden service names that will come with ed25519 keys, until it does, we depend on the old hidden service type so make this explicit. See #9214.
* | | | | Merge #9202: bench: Add support for measuring CPU cyclesWladimir J. van der Laan2016-11-295-5/+121
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | 3532818 bench: Add support for measuring CPU cycles (Wladimir J. van der Laan)
| * | | | | bench: Add support for measuring CPU cyclesWladimir J. van der Laan2016-11-225-5/+121
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds cycle min/max/avg to the statistics. Supported on x86 and x86_64 (natively through rdtsc), as well as Linux (perf syscall).
* | | | | | Merge #9225: Fix some benign racesWladimir J. van der Laan2016-11-295-36/+32
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | dfed983 Fix unlocked access to vNodes.size() (Matt Corallo) 3033522 Remove double brackets in addrman (Matt Corallo) dbfaade Fix AddrMan locking (Matt Corallo) 047ea10 Make fImporting an std::atomic (Matt Corallo) 42071ca Make fDisconnect an std::atomic (Matt Corallo)
| * | | | | | Fix unlocked access to vNodes.size()Matt Corallo2016-11-261-2/+7
| | | | | | |
| * | | | | | Remove double brackets in addrmanMatt Corallo2016-11-261-18/+12
| | | | | | |
| * | | | | | Fix AddrMan lockingMatt Corallo2016-11-261-13/+10
| | | | | | |
| * | | | | | Make fImporting an std::atomicMatt Corallo2016-11-262-2/+2
| | | | | | |
| * | | | | | Make fDisconnect an std::atomicMatt Corallo2016-11-251-1/+1
| | | | | | |
* | | | | | | Merge #9224: Prevent FD_SETSIZE error building on OpenBSDWladimir J. van der Laan2016-11-291-0/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 498a1d7 Include select.h when WIN32 is not defined (Ivo van der Sangen)
| * | | | | | | Include select.h when WIN32 is not definedIvo van der Sangen2016-11-291-0/+1
| | |_|/ / / / | |/| | | | |
* / | | | | | Fix some typosfsb40002016-11-288-8/+8
|/ / / / / /
* | | | | | Merge #9128: net: Decouple CConnman and message serializationPieter Wuille2016-11-257-152/+308
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | c7be56d net: push only raw data into CConnman (Cory Fields) 2ec935d net: add CVectorWriter and CNetMsgMaker (Cory Fields) b7695c2 net: No need to check individually for disconnection anymore (Cory Fields) fedea8a net: don't send any messages before handshake or after requested disconnect (Cory Fields) d74e352 net: Set feelers to disconnect at the end of the version message (Cory Fields)
| * | | | | | net: push only raw data into CConnmanCory Fields2016-11-253-119/+93
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes one of the last major layer violations in the networking stack. The network side is no longer in charge of message serialization, so it is now decoupled from Bitcoin structures. Only the header is serialized and attached to the payload.
| * | | | | | net: add CVectorWriter and CNetMsgMakerCory Fields2016-11-255-0/+178
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | CVectorWriter is useful for overwriting or appending an existing byte vector. CNetMsgMaker is a shortcut for creating messages on-the-fly which are suitable for pushing to CConnman.
| * | | | | | net: No need to check individually for disconnection anymoreCory Fields2016-11-251-7/+7
| | | | | | |
| * | | | | | net: don't send any messages before handshake or after requested disconnectCory Fields2016-11-231-21/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also, send reject messages earlier in SendMessages(), so that disconnections are processed earlier. These changes combined should ensure that no message is ever sent after fDisconnect is set.
| * | | | | | net: Set feelers to disconnect at the end of the version messageCory Fields2016-11-231-6/+6
| | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This way we're not relying on messages going out after fDisconnect has been set. This should not cause any real behavioral changes, though feelers should arguably disconnect earlier in the process. That can be addressed in a later functional change.
* | | | | | Merge #9220: Refactor: Stop using namespace std (bitcoin-cli/-tx).MarcoFalke2016-11-252-112/+108
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | 2f2625a Removed using namespace std from bitcoin-cli/-tx and added std:: in appropriate places. (Karl-Johan Alm)
| * | | | | | Removed using namespace std from bitcoin-cli/-tx and added std:: in ↵Karl-Johan Alm2016-11-252-112/+108
| | |/ / / / | |/| | | | | | | | | | | | | | | | appropriate places.
* | | | | | Merge #9100: tx_valid: re-order inputs to how they are encodedMarcoFalke2016-11-251-9/+9
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | 5262a15 tx_valid: re-order inputs to how they are encoded (Daniel Cousens)
| * | | | | | tx_valid: re-order inputs to how they are encodedDaniel Cousens2016-11-081-9/+9
| | | | | | |
* | | | | | | Merge #9189: Always add default_witness_commitment with GBT client supportWladimir J. van der Laan2016-11-253-9/+5
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | | | | | | | | | | | | | | | | 95f4a03 [qa] Test getblocktemplate default_witness_commitment (Suhas Daftuar) ad04d1c Always add default_witness_commitment with GBT client support (Pieter Wuille)
| * | | | | | Always add default_witness_commitment with GBT client supportPieter Wuille2016-11-213-9/+5
| | | | | | |
* | | | | | | Merge #9216: Doc: Fix copypasted commentMarcoFalke2016-11-241-1/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | f26da35 Fix copypasted comment. (Pavel Janík)
| * | | | | | | Fix copypasted comment.Pavel Janík2016-11-241-1/+1
| | |_|/ / / / | |/| | | | |
* | | | | | | Correct help output for waitfor RPC commandsfanquake2016-11-241-10/+10
| | | | | | |