aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | | | | | | | Do not shadow variable, use deprecated MAP_ANON if MAP_ANONYMOUS is not defined.Pavel Janík2016-11-021-3/+10
|/ / / / / / / / / / / /
* | | | | | | | | | | | Merge #8753: Locked memory managerWladimir J. van der Laan2016-11-0215-410/+966
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 444c673 bench: Add benchmark for lockedpool allocation/deallocation (Wladimir J. van der Laan) 6567999 rpc: Add `getmemoryinfo` call (Wladimir J. van der Laan) 4536148 support: Add LockedPool (Wladimir J. van der Laan) f4d1fc2 wallet: Get rid of LockObject and UnlockObject calls in key.h (Wladimir J. van der Laan) 999e4c9 wallet: Change CCrypter to use vectors with secure allocator (Wladimir J. van der Laan)
| * | | | | | | | | | | | bench: Add benchmark for lockedpool allocation/deallocationWladimir J. van der Laan2016-10-272-1/+49
| | | | | | | | | | | | |
| * | | | | | | | | | | | rpc: Add `getmemoryinfo` callWladimir J. van der Laan2016-10-271-0/+43
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ``` getmemoryinfo Returns an object containing information about memory usage. Result: { "locked": { (json object) Information about locked memory manager "used": xxxxx, (numeric) Number of bytes used "free": xxxxx, (numeric) Number of bytes available in current arenas "total": xxxxxxx, (numeric) Total number of bytes managed "locked": xxxxxx, (numeric) Amount of bytes that succeeded locking. If this number is smaller than total, locking pages failed at some point and key data could be swapped to disk. } } Examples: > bitcoin-cli getmemoryinfo > curl --user myusername --data-binary '{"jsonrpc": "1.0", "id":"curltest", "method": "getmemoryinfo", "params": [] }' -H 'content-type: text/plain;' http://127.0.0.1:8332/ ```
| * | | | | | | | | | | | support: Add LockedPoolWladimir J. van der Laan2016-10-277-328/+832
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a pool for locked memory chunks, replacing LockedPageManager. This is something I've been wanting to do for a long time. The current approach of locking objects where they happen to be on the stack or heap in-place causes a lot of mlock/munlock system call overhead, slowing down any handling of keys. Also locked memory is a limited resource on many operating systems (and using a lot of it bogs down the system), so the previous approach of locking every page that may contain any key information (but also other information) is wasteful.
| * | | | | | | | | | | | wallet: Get rid of LockObject and UnlockObject calls in key.hWladimir J. van der Laan2016-10-193-55/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Replace these with vectors allocated from the secure allocator. This avoids mlock syscall churn on stack pages, as well as makes it possible to get rid of these functions. Please review this commit and the previous one carefully that no `sizeof(vectortype)` remains in the memcpys and memcmps usage (ick!), and `.data()` or `&vec[x]` is used as appropriate instead of &vec.
| * | | | | | | | | | | | wallet: Change CCrypter to use vectors with secure allocatorWladimir J. van der Laan2016-10-193-26/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change CCrypter to use vectors with secure allocator instead of buffers on in the object itself which will end up on the stack. This avoids having to call LockedPageManager to lock stack memory pages to prevent the memory from being swapped to disk. This is wasteful.
* | | | | | | | | | | | | Merge #9032: test: Add format-dependent comparison to bctestWladimir J. van der Laan2016-11-021-3/+27
|\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6c5cd9d test: Add format-dependent comparison to bctest (Wladimir J. van der Laan)
| * | | | | | | | | | | | | test: Add format-dependent comparison to bctestWladimir J. van der Laan2016-10-281-3/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This splits the output comparison for `bitcoin-tx` into two steps: - First, check for data mismatch, parsing the data as json or hex depending on the extension of the output file - Then, check if the literal string matches For either of these cases give a different error. This prevents wild goose chases when e.g. a trailing space doesn't match exactly, and makes sure that both test output and examples are valid data of the purported format.
* | | | | | | | | | | | | | Merge #8448: Store mempool and prioritization data to diskWladimir J. van der Laan2016-11-026-7/+147
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 582068a Add mempool.dat to doc/files.md (Pieter Wuille) 3f78562 Add DumpMempool and LoadMempool (Pieter Wuille) ced7c94 Add AcceptToMemoryPoolWithTime function (Pieter Wuille) c3efb58 Add feedelta to TxMempoolInfo (Pieter Wuille)
| * | | | | | | | | | | | | Add mempool.dat to doc/files.mdPieter Wuille2016-10-311-0/+1
| | | | | | | | | | | | | |
| * | | | | | | | | | | | | Add DumpMempool and LoadMempoolPieter Wuille2016-10-313-0/+122
| | | | | | | | | | | | | |
| * | | | | | | | | | | | | Add AcceptToMemoryPoolWithTime functionPieter Wuille2016-10-302-5/+15
| | | | | | | | | | | | | |
| * | | | | | | | | | | | | Add feedelta to TxMempoolInfoPieter Wuille2016-10-302-2/+9
| | |_|_|/ / / / / / / / / | |/| | | | | | | | | | |
* | | | | | | | | | | | | Merge #9043: [qt] Return useful error message on ATMP failureJonas Schnelli2016-11-013-4/+8
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | 3333e5a [qt] Return useful error message on ATMP failure (MarcoFalke)
| * | | | | | | | | | | | [qt] Return useful error message on ATMP failureMarcoFalke2016-10-303-4/+8
| |/ / / / / / / / / / /
* | | | | | | | | | | | Merge #9041: keypoololdest denote Unix epoch, not GMTMarcoFalke2016-10-312-2/+2
|\ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | 7f61b49 Change all instance of 'GMT epoch' to 'Unix epoch' (matthias)
| * | | | | | | | | | | Change all instance of 'GMT epoch' to 'Unix epoch'matthias2016-10-312-2/+2
|/ / / / / / / / / / /
* | | | | | | | | | | Merge #8989: [Qt] overhaul smart-fee slider, adjust default confirmation targetWladimir J. van der Laan2016-10-287-13/+64
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | cfe77ef [Qt] overhaul smart-fee slider, adjust default confirmation target (Jonas Schnelli) 6f02899 [Qt] Hide nTxConfirmTarget behind WalletModel (Jonas Schnelli) 004168d CoinControl: add option for custom confirmation target (Jonas Schnelli)
| * | | | | | | | | | | [Qt] overhaul smart-fee slider, adjust default confirmation targetJonas Schnelli2016-10-283-11/+47
| | | | | | | | | | | |
| * | | | | | | | | | | [Qt] Hide nTxConfirmTarget behind WalletModelJonas Schnelli2016-10-282-0/+7
| | | | | | | | | | | |
| * | | | | | | | | | | CoinControl: add option for custom confirmation targetJonas Schnelli2016-10-282-2/+10
| | | | | | | | | | | |
* | | | | | | | | | | | Merge #9016: Return useful error message on ATMP failureWladimir J. van der Laan2016-10-284-13/+22
|\ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | 169bdab Return useful error message on ATMP failure (instagibbs)
| * | | | | | | | | | | Return useful error message on ATMP failureinstagibbs2016-10-264-13/+22
| | | | | | | | | | | |
* | | | | | | | | | | | Merge #9029: instance of 'mem pool' to 'mempool'Wladimir J. van der Laan2016-10-272-2/+2
|\ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | 1c3ecc7 instance of 'mem pool' to 'mempool' (S. Matthew English)
| * | | | | | | | | | | instance of 'mem pool' to 'mempool'S. Matthew English2016-10-272-2/+2
| | |/ / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | there was only one instance of 'mem pool' and not 'mempool', so I changed it to conform to the others
* | | | | | | | | | | Merge #9033: Update build notes for dropping osx 10.7 support (fanquake)Wladimir J. van der Laan2016-10-271-1/+1
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ba26d41 Update build notes for dropping osx 10.7 support (Michael Ford)
| * | | | | | | | | | | Update build notes for dropping osx 10.7 supportMichael Ford2016-10-271-1/+1
| |/ / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Github-Pull: #9022 Rebased-From: 1d1246314f761b3f84c869cb7b0af49cc25b55b3
* / / / / / / / / / / doc: Add release notes for 0.13.1 releaseWladimir J. van der Laan2016-10-271-0/+410
|/ / / / / / / / / /
* | | | | | | | | | Merge #9020: rpc: Remove invalid explanation from wallet fee messageWladimir J. van der Laan2016-10-261-1/+1
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 04c1c15 rpc: Remove invalid explanation from wallet fee message (Wladimir J. van der Laan)
| * | | | | | | | | | rpc: Remove invalid explanation from wallet fee messageWladimir J. van der Laan2016-10-261-1/+1
|/ / / / / / / / / /
* | | | | | | | | | Merge #9015: release: bump required osx version to 10.8. (jonasschnelli)Wladimir J. van der Laan2016-10-261-1/+1
|\ \ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / / |/| | | | | | | | | | | | | | | | | | | 339c4b6 release: bump required osx version to 10.8. Credit jonasschnelli. (Cory Fields)
| * | | | | | | | | release: bump required osx version to 10.8. Credit jonasschnelli.Cory Fields2016-10-251-1/+1
|/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | libc++ on 10.7 causes too many issues. See #8577 for discussion/details.
* | | | | | | | | Merge #8515: A few mempool removal optimizationsWladimir J. van der Laan2016-10-257-50/+51
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 0334430 Add some missing includes (Pieter Wuille) 4100499 Return shared_ptr<CTransaction> from mempool removes (Pieter Wuille) 51f2783 Make removed and conflicted arguments optional to remove (Pieter Wuille) f48211b Bypass removeRecursive in removeForReorg (Pieter Wuille)
| * | | | | | | | | Add some missing includesPieter Wuille2016-10-211-0/+4
| | | | | | | | | |
| * | | | | | | | | Return shared_ptr<CTransaction> from mempool removesPieter Wuille2016-10-215-15/+15
| | | | | | | | | |
| * | | | | | | | | Make removed and conflicted arguments optional to removePieter Wuille2016-10-217-35/+31
| | | | | | | | | |
| * | | | | | | | | Bypass removeRecursive in removeForReorgPieter Wuille2016-10-211-7/+8
| |/ / / / / / / /
* | | | | | | | | Merge #8948: [TRIVIAL] reorder Windows gitian build order to match LinuxWladimir J. van der Laan2016-10-251-1/+1
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3f7581d [TRIVIAL] reorder Windows gitian build order to match Linux (Micha)
| * | | | | | | | | [TRIVIAL] reorder Windows gitian build order to match LinuxMicha2016-10-181-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The consistency is helpful for gauging Gitian build progress. Right now it's necessary to remember which platform builds in which order, which can be confusing if you're attempting to get a quick idea of how far along your builds are.
* | | | | | | | | | Merge #9008: [net] Remove assert(nMaxInbound > 0)Wladimir J. van der Laan2016-10-251-1/+0
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fa1c3c2 [net] Remove assert(nMaxInbound > 0) (MarcoFalke)
| * | | | | | | | | | [net] Remove assert(nMaxInbound > 0)MarcoFalke2016-10-241-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | nMaxInbound might very well be 0 or -1, if the user prefers to keep a small number of maxconnections. Note: nMaxInbound of -1 means that the user set maxconnections to 8 or less, but we still want to keep an additional slot for the feeler connection.
* | | | | | | | | | | Merge #9002: Make connect=0 disable automatic outbound connections.Wladimir J. van der Laan2016-10-252-5/+6
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 515e264 Make connect=0 disable automatic outbound connections. (Gregory Maxwell)
| * | | | | | | | | | | Make connect=0 disable automatic outbound connections.Gregory Maxwell2016-10-252-5/+6
| | |_|/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Otherwise it just responds to this obvious bit of configuration by trying to connect to "0" in a loop.
* | | | | | | | | | | Merge #9004: Clarify `listenonion`Wladimir J. van der Laan2016-10-251-4/+4
|\ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | 3421e74 Clarify `listenonion` (unsystemizer)
| * | | | | | | | | | Clarify `listenonion`unsystemizer2016-10-241-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | > This new feature is enabled by default if Bitcoin Core is listening, and a connection to Tor can be made. It can be configured with the -listenonion, -torcontrol and -torpassword settings. To show verbose debugging information, pass -debug=tor. But it is correct to say that the feature is enabled *regardless* of whether a connection to Tor can be made. I propose to clarify that so that users can eliminate these in their logs (when `listen=1` and no Tor). And I think it's okay to clarify about the `listen` option, because on several occasions when I read this before I always assumed `listening` meant `server=1` which cost me a lot of time in troubleshooting. ``` 2016-10-24 06:19:22.551029 tor: Error connecting to Tor control socket 2016-10-24 06:19:22.551700 tor: Not connected to Tor control port 127.0.0.1:9051, trying to reconnect ``` ### What version of bitcoin-core are you using? 0.12.1
* | | | | | | | | | | Merge #8995: Add missing cs_main lock to ::GETBLOCKTXN processingWladimir J. van der Laan2016-10-241-0/+2
|\ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | dfe7906 Add missing cs_main lock to ::GETBLOCKTXN processing (Matt Corallo)
| * | | | | | | | | | Add missing cs_main lock to ::GETBLOCKTXN processingMatt Corallo2016-10-211-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Note that this is not a major issue as, in order for the missing lock to cause issues, you have to receive a GETBLOCKTXN message while reindexing, adding a block header via RPC, etc, which results in either a table rehash or an insert into the bucket which you are currently looking at.
* | | | | | | | | | | Merge #8993: Trivial: Fix doxygen comment: the transaction is returned in txOutWladimir J. van der Laan2016-10-241-1/+1
|\ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | 1d8e12b Fix doxygen comment: the transaction is returned in txOut (Pavel Janík)
| * | | | | | | | | | Fix doxygen comment: the transaction is returned in txOutPavel Janík2016-10-211-1/+1
|/ / / / / / / / / /