12016-09-02T00:03:02 *** gabridome has joined #bitcoin-core-dev
22016-09-02T00:06:50 *** Ylbam has quit IRC
32016-09-02T00:12:07 *** Alopex has quit IRC
42016-09-02T00:13:12 *** Alopex has joined #bitcoin-core-dev
52016-09-02T00:16:47 *** ghtdak has joined #bitcoin-core-dev
62016-09-02T00:23:04 <GitHub92> [bitcoin] gmaxwell opened pull request #8644: [0.13 backport] Check for compatibility with download in FindNextBlocksToDownload (0.13...findnext_backport) https://github.com/bitcoin/bitcoin/pull/8644
72016-09-02T00:24:58 <gmaxwell> Do we want to backport, "Reduce default number of blocks to check at startup"? It's a trivial fix to long startup time complaints, and arguably the dbcache increase in 0.13 was a startup time regression for some people.
82016-09-02T00:25:59 <GitHub29> [bitcoin] droark opened pull request #8645: Remove unused Qt 4.6 patch. (master...master) https://github.com/bitcoin/bitcoin/pull/8645
92016-09-02T00:26:35 <sipa> i would be fime with that, though calling it a bugfix is a stretch
102016-09-02T00:27:15 <gmaxwell> well I gave a pragmatic argument for it being one.
112016-09-02T00:27:33 <gmaxwell> but I don't think I needed to, it's pretty obviously riskless with no "interface" impact.
122016-09-02T00:28:10 <sipa> yes, i agree it has very low risk
132016-09-02T00:28:26 <gmaxwell> which I think beyond 'fix' should be the goal for point releases, nothing that is going to turn a working deployment into a broken one.
142016-09-02T00:32:19 <luke-jr> sounds reasonable to do, in any case
152016-09-02T00:36:01 <GitHub176> [bitcoin] gmaxwell opened pull request #8646: [0.13 backport] Reduce default number of blocks to check at startup (0.13...reduceblocks_backport) https://github.com/bitcoin/bitcoin/pull/8646
162016-09-02T00:39:33 *** gabridome has quit IRC
172016-09-02T00:41:16 *** gabridome has joined #bitcoin-core-dev
182016-09-02T00:43:44 *** gabridome has joined #bitcoin-core-dev
192016-09-02T00:44:16 <GitHub102> [bitcoin] gmaxwell opened pull request #8648: [0.13 backport] Precompute sighashes (0.13...precompute_backport) https://github.com/bitcoin/bitcoin/pull/8648
202016-09-02T00:44:21 <gmaxwell> wumpus: feel free to close my backport prs if you don't find them helpful, I was just feeling usless simply saying X or Y should be backported rather than doing it.
212016-09-02T00:55:17 *** Alopex has quit IRC
222016-09-02T00:56:22 *** Alopex has joined #bitcoin-core-dev
232016-09-02T00:57:11 *** gabridome has quit IRC
242016-09-02T01:00:26 *** TomMc has joined #bitcoin-core-dev
252016-09-02T01:13:10 *** grubles has quit IRC
262016-09-02T01:16:32 *** Chris_St1 has joined #bitcoin-core-dev
272016-09-02T01:17:04 *** justanotheruser has quit IRC
282016-09-02T01:17:34 *** justanotheruser has joined #bitcoin-core-dev
292016-09-02T01:34:13 *** Giszmo has quit IRC
302016-09-02T01:34:29 *** achow101 has quit IRC
312016-09-02T01:35:07 *** achow101 has joined #bitcoin-core-dev
322016-09-02T01:38:42 <jl2012> We need a nulldummy only bip?
332016-09-02T01:38:49 *** Chris_St1 has quit IRC
342016-09-02T01:42:03 <jl2012> And I already have a PR for empty failing signature https://github.com/bitcoin/bitcoin/pull/8634
352016-09-02T01:47:37 *** dcousens has joined #bitcoin-core-dev
362016-09-02T01:51:36 *** Chris_St1 has joined #bitcoin-core-dev
372016-09-02T01:57:01 *** Alopex has quit IRC
382016-09-02T01:58:06 *** Alopex has joined #bitcoin-core-dev
392016-09-02T02:01:00 *** Chris_St1 has quit IRC
402016-09-02T02:04:56 <cfields> sdaftuar: did you say you were able to reproduce the segwit.py failure locally?
412016-09-02T02:05:39 <cfields> sdaftuar: if so: https://github.com/theuni/bitcoin/commit/fa88a425c0c1089ab75ea9223699286f40c8eea7
422016-09-02T02:06:10 <cfields> it's pretty hackish, needs a cleanup. But should be enough to determine if it fixes the problem
432016-09-02T02:07:18 <cfields> wumpus: let me know if ^^ is what you had in mind
442016-09-02T02:09:13 <cfields> that not only eliminates polling, but also drops cs_main locking, so i should think it'd be a good bit quicker
452016-09-02T02:19:22 *** Alopex has quit IRC
462016-09-02T02:20:27 *** Alopex has joined #bitcoin-core-dev
472016-09-02T02:21:14 *** ghtdak has quit IRC
482016-09-02T02:25:34 *** Samdney has left #bitcoin-core-dev
492016-09-02T02:25:44 *** achow101 has quit IRC
502016-09-02T02:27:04 *** TomMc has quit IRC
512016-09-02T02:29:04 *** jtimon has quit IRC
522016-09-02T02:30:21 *** Alopex has quit IRC
532016-09-02T02:31:26 *** Alopex has joined #bitcoin-core-dev
542016-09-02T02:35:46 *** adiabat has joined #bitcoin-core-dev
552016-09-02T02:41:01 *** Alopex has quit IRC
562016-09-02T02:42:07 *** Alopex has joined #bitcoin-core-dev
572016-09-02T02:55:01 *** Alopex has quit IRC
582016-09-02T02:55:29 *** fengling has joined #bitcoin-core-dev
592016-09-02T02:56:06 *** Alopex has joined #bitcoin-core-dev
602016-09-02T03:00:26 *** fengling has quit IRC
612016-09-02T03:04:20 *** justan0theruser has joined #bitcoin-core-dev
622016-09-02T03:05:57 *** justanotheruser has quit IRC
632016-09-02T03:24:06 *** Alopex has quit IRC
642016-09-02T03:25:12 *** Alopex has joined #bitcoin-core-dev
652016-09-02T03:36:38 *** jcorgan has joined #bitcoin-core-dev
662016-09-02T03:45:32 *** dcousens has quit IRC
672016-09-02T03:47:48 *** TomMc has joined #bitcoin-core-dev
682016-09-02T03:55:21 *** Alopex has quit IRC
692016-09-02T03:56:27 *** Alopex has joined #bitcoin-core-dev
702016-09-02T03:57:16 *** fengling has joined #bitcoin-core-dev
712016-09-02T04:01:35 *** spudowiar has quit IRC
722016-09-02T04:02:06 *** fengling has quit IRC
732016-09-02T04:05:22 *** TomMc has quit IRC
742016-09-02T04:11:14 *** JackH has quit IRC
752016-09-02T04:11:49 *** JackH has joined #bitcoin-core-dev
762016-09-02T04:22:16 *** Alopex has quit IRC
772016-09-02T04:23:21 *** Alopex has joined #bitcoin-core-dev
782016-09-02T04:33:40 *** kadoban has quit IRC
792016-09-02T04:37:25 *** cfields has quit IRC
802016-09-02T04:40:40 *** achow101 has joined #bitcoin-core-dev
812016-09-02T04:42:38 *** achow101 has quit IRC
822016-09-02T04:44:06 *** Alopex has quit IRC
832016-09-02T04:45:12 *** Alopex has joined #bitcoin-core-dev
842016-09-02T04:48:33 *** cfields has joined #bitcoin-core-dev
852016-09-02T04:52:07 *** achow101 has joined #bitcoin-core-dev
862016-09-02T04:57:49 *** achow101 has quit IRC
872016-09-02T04:58:58 *** fengling has joined #bitcoin-core-dev
882016-09-02T05:03:30 <GitHub186> [bitcoin] JeremyRubin opened pull request #8650: Make tests much faster by replacing BOOST_CHECK with FAST_CHECK (master...faster_tests) https://github.com/bitcoin/bitcoin/pull/8650
892016-09-02T05:03:46 *** fengling has quit IRC
902016-09-02T05:10:51 *** justan0theruser has quit IRC
912016-09-02T05:13:55 <gmaxwell> down with 'test frameworks'.
922016-09-02T05:18:43 *** gabridome has joined #bitcoin-core-dev
932016-09-02T05:19:04 *** paveljanik has quit IRC
942016-09-02T05:22:29 *** Ylbam has joined #bitcoin-core-dev
952016-09-02T05:29:21 *** Alopex has quit IRC
962016-09-02T05:30:26 *** Alopex has joined #bitcoin-core-dev
972016-09-02T05:31:12 *** gabridome has quit IRC
982016-09-02T05:45:01 *** dcousens has joined #bitcoin-core-dev
992016-09-02T05:47:00 *** fengling has joined #bitcoin-core-dev
1002016-09-02T05:48:56 <jeremyrubin> gmaxwell: Hear, hear!
1012016-09-02T05:51:09 <GitHub124> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/f061415d12ce...91990ee01dab
1022016-09-02T05:51:10 <GitHub124> bitcoin/master 854f1af Pieter Wuille: Make the dummy argument to getaddednodeinfo optional
1032016-09-02T05:51:10 <GitHub124> bitcoin/master 91990ee Wladimir J. van der Laan: Merge #8272: Make the dummy argument to getaddednodeinfo optional...
1042016-09-02T05:51:14 <GitHub133> [bitcoin] laanwj closed pull request #8272: Make the dummy argument to getaddednodeinfo optional (master...optionaladdnodedummy) https://github.com/bitcoin/bitcoin/pull/8272
1052016-09-02T05:52:07 <gmaxwell> jeremyrubin: I had someone 'testframeworkize' one of my tests in another project and it caused it to go from 45 second runtime something like an hour.
1062016-09-02T05:55:19 *** jannes has quit IRC
1072016-09-02T06:02:32 *** Alopex has quit IRC
1082016-09-02T06:03:37 *** Alopex has joined #bitcoin-core-dev
1092016-09-02T06:05:02 *** assder has joined #bitcoin-core-dev
1102016-09-02T06:15:00 <jeremyrubin> gross :/
1112016-09-02T06:26:47 *** ghtdak has joined #bitcoin-core-dev
1122016-09-02T06:44:46 *** fengling has quit IRC
1132016-09-02T06:52:13 *** gabridome has joined #bitcoin-core-dev
1142016-09-02T06:54:29 *** gabridome has quit IRC
1152016-09-02T06:56:54 *** BashCo has quit IRC
1162016-09-02T07:03:49 *** kyletorpey has quit IRC
1172016-09-02T07:10:48 *** gabridome1 has quit IRC
1182016-09-02T07:11:16 *** Alopex has quit IRC
1192016-09-02T07:12:22 *** Alopex has joined #bitcoin-core-dev
1202016-09-02T07:15:45 *** BashCo has joined #bitcoin-core-dev
1212016-09-02T07:20:49 *** pmienk has quit IRC
1222016-09-02T07:22:05 *** paveljanik has joined #bitcoin-core-dev
1232016-09-02T07:32:54 *** pmienk has joined #bitcoin-core-dev
1242016-09-02T07:37:31 *** pmienk has quit IRC
1252016-09-02T07:37:50 *** BashCo_ has joined #bitcoin-core-dev
1262016-09-02T07:41:00 *** BashCo has quit IRC
1272016-09-02T07:41:42 *** laurentmt has joined #bitcoin-core-dev
1282016-09-02T07:44:05 <jonasschnelli> Hmm... IBDing from a local peer (same host) seems to take much longer the bootstraping from random peers.
1292016-09-02T07:46:21 <gmaxwell> early in the sync it fetches to few a number of blocks at a time, so fetching from a single peer is slow. hardly matters overall.
1302016-09-02T07:46:31 <gmaxwell> other than that, they'll obviously contend a bit for IO
1312016-09-02T07:46:50 <gmaxwell> I'm not aware of any other reasons for slowdowns other than that.
1322016-09-02T07:47:23 <paveljanik> jonasschnelli, OS X<
1332016-09-02T07:47:24 <paveljanik> ?
1342016-09-02T07:48:02 <jonasschnelli> paveljanik: no debian on my Intel(R) Xeon(R) CPU E3-1275 v5 @ 3.60GHz
1352016-09-02T07:48:28 <paveljanik> I had the same issue on OS X in the past. unplugging the network cable, turning off firewall helped 8)
1362016-09-02T07:48:43 <gmaxwell> are you only looking at the first 150k blocks? or are you seeing the rate per block slower later?
1372016-09-02T07:48:47 <paveljanik> the process named powerd took 100% of CPU power
1382016-09-02T07:49:29 <jonasschnelli> gmaxwell: It took more then 1h for progress=0.035480
1392016-09-02T07:49:53 <jonasschnelli> (where is did sync with random peers up to progress 1.0 in ~3h [2 month ago though])
1402016-09-02T07:49:55 *** pmienk has joined #bitcoin-core-dev
1412016-09-02T07:50:56 <gmaxwell> better to look at time vs height, e.g. grab a chunk of 100 blocks wherever it is and comare the blocks per second at that height with the same height in your prior logs.
1422016-09-02T07:50:59 <jonasschnelli> Also... my extremely scientific measure of how fast the tail -f output scrolls though.... did tell me its slower
1432016-09-02T07:51:57 <jonasschnelli> I guess i let it sync up to h400'000 and then compare it against a sync from random peers
1442016-09-02T07:51:59 *** laurentmt has quit IRC
1452016-09-02T07:52:28 <jonasschnelli> The only differences are: -prune=550 and connecting to a node thats running on the same machine (in sync, should not cause a huge slowdown)
1462016-09-02T07:52:49 * jonasschnelli needs to write a IBD benchmark tool
1472016-09-02T07:53:00 <gmaxwell> prune might make it slower.
1482016-09-02T07:53:20 <jonasschnelli> the prune data eviction seems pretty fast
1492016-09-02T07:53:21 <gmaxwell> after all it's writing things to only delete it after.
1502016-09-02T07:53:35 <jonasschnelli> it's running on a 1GB/s SSD
1512016-09-02T07:55:18 <gmaxwell> I doubt anyone has benmarked prune vs not, if you told me that it was doing some linear scan of all the blocks to decide what to prune, every block-- and was thus massively slower, I wouldn't not be too surprised.
1522016-09-02T07:57:00 <GitHub55> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/91990ee01dab...6f939c9080a3
1532016-09-02T07:57:01 <GitHub55> bitcoin/master cdd79eb Jorge Timón: C++11: s/boost::scoped_ptr/std::unique_ptr/
1542016-09-02T07:57:01 <GitHub55> bitcoin/master 6f939c9 Wladimir J. van der Laan: Merge #8629: C++11: s/boost::scoped_ptr/std::unique_ptr/...
1552016-09-02T07:57:08 * jonasschnelli is trying again without prune
1562016-09-02T07:57:15 <GitHub51> [bitcoin] laanwj closed pull request #8629: C++11: s/boost::scoped_ptr/std::unique_ptr/ (master...0.13-boost-scoped-ptr) https://github.com/bitcoin/bitcoin/pull/8629
1572016-09-02T08:00:14 <sipa> pruning causes flushing
1582016-09-02T08:01:11 <gmaxwell> oh obviously, it needs to make sure that after a restart it can continue, so it hast to at least be consistent to the point where it pruned.
1592016-09-02T08:07:12 *** AaronvanW has joined #bitcoin-core-dev
1602016-09-02T08:07:40 *** AaronvanW has quit IRC
1612016-09-02T08:07:40 *** AaronvanW has joined #bitcoin-core-dev
1622016-09-02T08:10:44 <jonasschnelli> What is the reason for the txn_count=0x00 in the headers p2p message?
1632016-09-02T08:11:49 *** fengling has joined #bitcoin-core-dev
1642016-09-02T08:13:10 <sipa> because CBlockHeader did not originally exist
1652016-09-02T08:13:30 <sipa> and headers messages just contained a CBlock with 0 transactions
1662016-09-02T08:13:55 <jonasschnelli> thanks.
1672016-09-02T08:20:16 <jeremyrubin> Is it correct that the maximum number of items deleted from sigcache while checking a block is max sigops?
1682016-09-02T08:20:58 <sipa> i believe so
1692016-09-02T08:21:55 <jeremyrubin> And is that "MAX_BLOCK_SIGOPS_COST" or is that something else
1702016-09-02T08:22:03 <jeremyrubin> because I see a 20k number floating around too
1712016-09-02T08:22:08 <jeremyrubin> but MAX_BLOCK_SIGOPS_COST=80k
1722016-09-02T08:22:32 <sipa> yes, 4x factor from segwit
1732016-09-02T08:22:57 <sipa> sigops in non-witness part count as 4, in witness part they count as 1
1742016-09-02T08:23:39 <jeremyrubin> so if you have 0 sigops in witness, you can have the full 80k?
1752016-09-02T08:24:13 <sipa> if you have 0 sigops in non-witness
1762016-09-02T08:24:15 <sipa> yes
1772016-09-02T08:24:18 <jeremyrubin> ok cool
1782016-09-02T08:24:22 <sipa> pre segwit the max is 20k
1792016-09-02T08:24:29 <sipa> post segwit it is 80k
1802016-09-02T08:39:01 *** Alopex has quit IRC
1812016-09-02T08:40:06 *** Alopex has joined #bitcoin-core-dev
1822016-09-02T08:40:32 *** JackH has quit IRC
1832016-09-02T08:58:43 <jonasschnelli> with disabled pruning it took also >1h for progress=0.031407... now trying a sync from random peers
1842016-09-02T09:00:16 <gmaxwell> you're killin me with the testing with just the first 3%.
1852016-09-02T09:00:41 <jonasschnelli> gmaxwell: heh. You think comparing the first 3% does not make sense?
1862016-09-02T09:01:14 <jonasschnelli> I just try to not waste time. :)
1872016-09-02T09:01:33 <gmaxwell> it's very unrepresentative, as I explaned earlier. We only fetch 16 blocks per peer at a time ... early in the chain this is far too few to keep the system well pipelined, because the blocks take no work to process. :)
1882016-09-02T09:02:42 <jonasschnelli> gmaxwell: Okay. I see. But anyways, a sync with a single local peer will then take always longer, right?
1892016-09-02T09:03:18 <gmaxwell> yes, somewhat. the difference will only be in the initial part of the chain, but go away later-- at least, whatever issue was due to that.
1902016-09-02T09:04:09 <gmaxwell> it's like the 0.13 vs 0.12 reindex, the 0.13 reindex spends 20 minutes at the front reading headers. So it starts off 20 minutes behind, it still finishes earlier.
1912016-09-02T09:04:35 <gmaxwell> but if you were to compare reintext for 0.13 vs 0.12 for the first 3% you would likely find 0.13 was much slower. :)
1922016-09-02T09:11:30 *** AaronvanW has quit IRC
1932016-09-02T09:16:36 *** AaronvanW has joined #bitcoin-core-dev
1942016-09-02T09:16:37 *** AaronvanW has quit IRC
1952016-09-02T09:16:37 *** AaronvanW has joined #bitcoin-core-dev
1962016-09-02T09:33:46 *** spudowiar has joined #bitcoin-core-dev
1972016-09-02T09:36:00 *** murch has joined #bitcoin-core-dev
1982016-09-02T09:38:31 *** jtimon has joined #bitcoin-core-dev
1992016-09-02T09:44:15 <GitHub197> [bitcoin] sipa opened pull request #8651: Predeclare PrecomputedTransactionData as struct (master...classtructblah) https://github.com/bitcoin/bitcoin/pull/8651
2002016-09-02T09:48:03 <GitHub77> [bitcoin] yurizhykin opened pull request #8652: [qa]: remove root test directory for RPC tests (master...cleanup) https://github.com/bitcoin/bitcoin/pull/8652
2012016-09-02T09:52:46 *** fengling has quit IRC
2022016-09-02T10:04:35 *** fengling has joined #bitcoin-core-dev
2032016-09-02T10:19:20 *** jannes has joined #bitcoin-core-dev
2042016-09-02T10:26:26 <GitHub76> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/6f939c9080a3...381d0ddc8aaa
2052016-09-02T10:26:27 <GitHub76> bitcoin/master cce3024 Pavel JanÃk: Do not shadow local variable, cleanup
2062016-09-02T10:26:28 <GitHub76> bitcoin/master a159f25 Pavel JanÃk: Remove redundand (and shadowing) declaration
2072016-09-02T10:26:28 <GitHub76> bitcoin/master 381d0dd Wladimir J. van der Laan: Merge #8449: [Trivial] Do not shadow local variable, cleanup...
2082016-09-02T10:26:32 <GitHub190> [bitcoin] laanwj closed pull request #8449: [Trivial] Do not shadow local variable, cleanup (master...20160803_shadow_blockencodings) https://github.com/bitcoin/bitcoin/pull/8449
2092016-09-02T10:30:06 *** BashCo has joined #bitcoin-core-dev
2102016-09-02T10:31:52 *** BashCo_ has quit IRC
2112016-09-02T10:43:26 *** fengling has quit IRC
2122016-09-02T10:45:52 *** owowo has quit IRC
2132016-09-02T10:52:18 <GitHub29> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/381d0ddc8aaa...cbe9ae8c69b9
2142016-09-02T10:52:19 <GitHub29> bitcoin/master b7c349d Pavel JanÃk: Do not shadow variables in networking code
2152016-09-02T10:52:19 <GitHub29> bitcoin/master cbe9ae8 Wladimir J. van der Laan: Merge #8466: [Trivial] Do not shadow variables in networking code...
2162016-09-02T10:52:23 <GitHub92> [bitcoin] laanwj closed pull request #8466: [Trivial] Do not shadow variables in networking code (master...20160805_Wshadow_net) https://github.com/bitcoin/bitcoin/pull/8466
2172016-09-02T10:54:59 *** fengling has joined #bitcoin-core-dev
2182016-09-02T11:03:03 *** jcorgan_ has joined #bitcoin-core-dev
2192016-09-02T11:06:02 *** timothy has quit IRC
2202016-09-02T11:06:04 *** drizztbsd has joined #bitcoin-core-dev
2212016-09-02T11:06:36 *** drizztbsd is now known as timothy
2222016-09-02T11:07:31 *** jcorgan has quit IRC
2232016-09-02T11:07:32 *** mn3monic has quit IRC
2242016-09-02T11:07:32 *** sanada` has quit IRC
2252016-09-02T11:07:33 *** Bootvis has quit IRC
2262016-09-02T11:07:34 *** crudel has quit IRC
2272016-09-02T11:07:34 *** harding has quit IRC
2282016-09-02T11:07:34 *** Taek has quit IRC
2292016-09-02T11:09:54 *** mn3monic has joined #bitcoin-core-dev
2302016-09-02T11:09:54 *** sanada` has joined #bitcoin-core-dev
2312016-09-02T11:09:54 *** Bootvis has joined #bitcoin-core-dev
2322016-09-02T11:09:54 *** crudel has joined #bitcoin-core-dev
2332016-09-02T11:09:54 *** harding has joined #bitcoin-core-dev
2342016-09-02T11:09:54 *** Taek has joined #bitcoin-core-dev
2352016-09-02T11:19:46 *** fengling has quit IRC
2362016-09-02T11:30:26 *** fengling has joined #bitcoin-core-dev
2372016-09-02T11:38:48 <cfields> jonasschnelli: so your comment on #8085 may not be relevant to those changes?
2382016-09-02T11:42:08 <GitHub97> [bitcoin] jl2012 closed pull request #8533: Implement LOW_S and NULLDUMMY softfork (BIP146) (master...bip146) https://github.com/bitcoin/bitcoin/pull/8533
2392016-09-02T11:42:56 *** BashCo has quit IRC
2402016-09-02T11:55:06 *** fengling has quit IRC
2412016-09-02T11:55:26 *** cryptapus_afk is now known as cryptapus
2422016-09-02T12:22:04 *** dcousens has quit IRC
2432016-09-02T12:36:42 *** paveljanik has quit IRC
2442016-09-02T12:37:15 *** LeMiner2 has joined #bitcoin-core-dev
2452016-09-02T12:39:07 *** LeMiner has quit IRC
2462016-09-02T12:39:07 *** LeMiner2 is now known as LeMiner
2472016-09-02T12:45:49 *** owowo has joined #bitcoin-core-dev
2482016-09-02T12:58:12 <cfields> wumpus: re-ping incase you missed in backog. what do you think of the approach here to fix the travis timeouts? https://github.com/theuni/bitcoin/commit/fa88a425c0c1089ab75ea9223699286f40c8eea7
2492016-09-02T13:03:42 *** dcousens has joined #bitcoin-core-dev
2502016-09-02T13:20:51 *** jcorgan_ is now known as jcorgan
2512016-09-02T13:23:57 <GitHub117> [bitcoin] sstone opened pull request #8653: doc (trivial): add tip for cross-builds on ubuntu (master...wip-doccrosscompile) https://github.com/bitcoin/bitcoin/pull/8653
2522016-09-02T13:29:44 *** Guyver2 has joined #bitcoin-core-dev
2532016-09-02T13:41:14 <sipa> cfields: looks good to me
2542016-09-02T13:41:58 <sipa> jonasschnelli: 3%, how many blocks is that?
2552016-09-02T13:42:15 * jonasschnelli checking the logs
2562016-09-02T13:42:54 <jonasschnelli> sipa: aprox 211000
2572016-09-02T13:43:15 <jonasschnelli> My assumption is that IBD with a single peer is much slower... but can't prove it right now.
2582016-09-02T13:43:18 <sipa> jonasschnelli: that seems very wrong
2592016-09-02T13:43:25 <sipa> it should take minutes to get to 211000
2602016-09-02T13:43:58 <sipa> not an hour
2612016-09-02T13:44:12 <sipa> the effect greg talks about it real, but shouldn't last more than a few minutes
2622016-09-02T13:44:35 <jonasschnelli> 2016-09-02 08:58:49 UpdateTip: new best=0000000000000345b371caa3f829cacbe2b4d38ecd15a5a02031efae79934d15 height=211000
2632016-09-02T13:44:38 <jonasschnelli> 2016-09-02 07:56:40 Bitcoin version v0.13.99.0-df98230
2642016-09-02T13:45:00 *** paveljanik has joined #bitcoin-core-dev
2652016-09-02T13:48:12 *** warren has quit IRC
2662016-09-02T13:49:58 *** warren has joined #bitcoin-core-dev
2672016-09-02T13:54:37 *** shesek has quit IRC
2682016-09-02T13:56:16 *** grubles has joined #bitcoin-core-dev
2692016-09-02T13:57:00 <jonasschnelli> sipa: I guess its caused by --enable-debug
2702016-09-02T13:58:43 <sipa> oh, yes, that will slow things down tremendously
2712016-09-02T14:00:02 <jonasschnelli> sry for the noise... I should finally remember to disable debug mode when benachmarking.
2722016-09-02T14:04:31 *** jtimon has quit IRC
2732016-09-02T14:16:36 *** TomMc has joined #bitcoin-core-dev
2742016-09-02T14:17:14 *** Cheeseo has quit IRC
2752016-09-02T14:17:34 *** Cheeseo has joined #bitcoin-core-dev
2762016-09-02T14:17:34 *** Cheeseo has joined #bitcoin-core-dev
2772016-09-02T14:28:37 *** grubles has quit IRC
2782016-09-02T14:29:58 *** Samdney has joined #bitcoin-core-dev
2792016-09-02T14:31:48 *** Giszmo has joined #bitcoin-core-dev
2802016-09-02T14:37:22 *** warren has quit IRC
2812016-09-02T14:38:06 *** grubles has joined #bitcoin-core-dev
2822016-09-02T14:39:48 *** warren has joined #bitcoin-core-dev
2832016-09-02T14:43:39 *** grubles has quit IRC
2842016-09-02T14:48:11 *** gabridome has joined #bitcoin-core-dev
2852016-09-02T14:48:42 *** gabridome has quit IRC
2862016-09-02T14:48:57 *** gabridome has joined #bitcoin-core-dev
2872016-09-02T14:51:14 *** grubles has joined #bitcoin-core-dev
2882016-09-02T14:53:37 *** grubles has quit IRC
2892016-09-02T14:56:39 *** grubles has joined #bitcoin-core-dev
2902016-09-02T15:05:33 *** Giszmo has quit IRC
2912016-09-02T15:07:36 *** gabridome has quit IRC
2922016-09-02T15:09:27 *** Giszmo has joined #bitcoin-core-dev
2932016-09-02T15:12:56 *** achow101 has joined #bitcoin-core-dev
2942016-09-02T15:15:47 <sturles> a/away
2952016-09-02T15:25:40 <cfields> sipa: thanks
2962016-09-02T15:26:19 <cfields> jonasschnelli: so I should ignore https://github.com/bitcoin/bitcoin/pull/8085#issuecomment-244077083 ?
2972016-09-02T15:26:40 <jonasschnelli> cfields: I think so...
2982016-09-02T15:26:50 <jonasschnelli> I'm working on more authentic IBD benchmarks.
2992016-09-02T15:27:39 <cfields> jonasschnelli: ok. apples to apples IBD is a good idea though. I'll do some syncs over the weekend
3002016-09-02T15:27:39 *** achow101 has quit IRC
3012016-09-02T15:28:11 <jonasschnelli> cfields: Yes. I was fooled (again) by the bias of --enable-debug and --prune
3022016-09-02T15:30:12 <cfields> jonasschnelli: ah, heh. I've fallen into the same trap. Turns out profiles generated with -O0 (for better info) aren't at all representative of real-world usage
3032016-09-02T15:30:28 <cfields> so it makes sense that it'd be a major performance killer
3042016-09-02T15:33:27 <gmaxwell> jonasschnelli: I think you got caught on debug before, you realize now I'm gonna start asking you? "Are you benchmarking with enable debug again?" :P
3052016-09-02T15:34:05 <jonasschnelli> heh... yes.
3062016-09-02T15:34:07 <btcdrak> Jonas 'debug' schnelli
3072016-09-02T15:34:09 <sdaftuar> cfields: i don't think i have seen segwit.py fail locally for the reason mentioned in #8532
3082016-09-02T15:34:19 <sdaftuar> gmaxwell: please set up an irc autoresponder so the rest of us don't forget to ask him too :)
3092016-09-02T15:34:49 <cfields> sdaftuar: ah, ok
3102016-09-02T15:35:20 *** cryptapus has quit IRC
3112016-09-02T15:35:45 *** cryptapus has joined #bitcoin-core-dev
3122016-09-02T15:36:11 <sdaftuar> it seems like the most common reason my rpc tests fail is the communicat() timeout thing i reported in #8649, and then new test runs failing when old bitcoind's haven't been killed from prior test runs (sigh)
3132016-09-02T15:44:57 <GitHub85> [bitcoin] jl2012 opened pull request #8654: Reuse sighash computations across evaluation (rebase of #4562) (master...sighashcache) https://github.com/bitcoin/bitcoin/pull/8654
3142016-09-02T15:56:23 <wumpus> cfields: looking
3152016-09-02T15:59:04 <wumpus> cfields: looks good to me, doesn't even require any new signals!
3162016-09-02T16:00:35 <wumpus> cfields: any reason to not put the signal subscription/unsubscription in StartRPC/StopRPC?
3172016-09-02T16:01:09 <wumpus> cfields: oh duh, those are general methods
3182016-09-02T16:01:29 <wumpus> cfields: if there was a rpcblockchain.init/deinit, it would fit there
3192016-09-02T16:02:32 <wumpus> cfields: but this is ok; maybe mark the RPC calls that they are primarily meant for testing and/or set them as hidden?
3202016-09-02T16:02:59 <wumpus> (e.g. I can see complaints from people hanging the RPC server this way with no clue what they're doing)
3212016-09-02T16:03:28 <sipa> agree
3222016-09-02T16:05:25 <cfields> wumpus: yep, makes sense
3232016-09-02T16:05:44 <cfields> and yes, i kinda abused the existing signals there. I figured you'd be grumpier about it, hence the poke rather than PR :)
3242016-09-02T16:07:48 <wumpus> cfields: well the important thing is that all other processing has been done at the time of the signal; does this guarantee that?
3252016-09-02T16:08:21 <wumpus> (I guess the answer is yes, as there is no async processing, although it could depend on the order in which the handlers are called?)
3262016-09-02T16:09:20 <sipa> by default i think handlers are called in the order they were added
3272016-09-02T16:09:37 *** jtimon has joined #bitcoin-core-dev
3282016-09-02T16:09:43 <cfields> wumpus: i believe it's fine, but i'll double-check.
3292016-09-02T16:10:15 *** kadoban has joined #bitcoin-core-dev
3302016-09-02T16:10:21 <cfields> wumpus: since it abuses the ui signal, i'm not sure we need to worry about the order? Looking to see who else receives that
3312016-09-02T16:11:28 <wumpus> sipa: yes, but that is a really fragile thing to rely on
3322016-09-02T16:11:58 <wumpus> cfields: ok, yes that makes sense, if the UI signal is last
3332016-09-02T16:12:17 <wumpus> cfields: indeed there's no need to be worried doing things concurrently with the GUI
3342016-09-02T16:12:33 <wumpus> (usually that won't be running anyhow during RPC testing, although it's possible)
3352016-09-02T16:12:47 <cfields> it doesn't guarantee that we've acted on relaying appropriately, though all we could guarantee there anyway is that we've queued up some messages
3362016-09-02T16:13:31 <cfields> right
3372016-09-02T16:15:59 <cfields> yes, only other user is qt for updating gui
3382016-09-02T16:27:58 <GitHub78> [bitcoin] paveljanik opened pull request #8655: Do not shadow variables (trivials) (master...20160902_Wshadow_trivials) https://github.com/bitcoin/bitcoin/pull/8655
3392016-09-02T16:32:20 *** jujumax has joined #bitcoin-core-dev
3402016-09-02T16:32:34 *** aalex has quit IRC
3412016-09-02T16:41:38 *** aalex has joined #bitcoin-core-dev
3422016-09-02T16:52:13 <phantomcircuit> jonasschnelli, the p2p network contains a bunch of nodes which exist only to monitor people and as a side effect have bizarre behaviour
3432016-09-02T16:58:48 *** jujumax has quit IRC
3442016-09-02T16:59:02 *** Chris_Stewart_5 has joined #bitcoin-core-dev
3452016-09-02T17:18:18 *** gabridome has joined #bitcoin-core-dev
3462016-09-02T17:20:27 *** gabridome has quit IRC
3472016-09-02T17:21:21 *** Chris_St1 has joined #bitcoin-core-dev
3482016-09-02T17:22:09 *** Chris_Stewart_5 has quit IRC
3492016-09-02T17:22:44 *** jtimon has quit IRC
3502016-09-02T17:23:17 *** jtimon has joined #bitcoin-core-dev
3512016-09-02T17:35:54 *** spudowiar has quit IRC
3522016-09-02T17:59:06 *** spudowiar has joined #bitcoin-core-dev
3532016-09-02T18:00:34 *** Chris_St1 has quit IRC
3542016-09-02T18:08:59 *** spudowiar has quit IRC
3552016-09-02T18:16:09 *** Chris_St1 has joined #bitcoin-core-dev
3562016-09-02T18:27:25 *** spudowiar has joined #bitcoin-core-dev
3572016-09-02T18:31:29 *** Chris_St1 has quit IRC
3582016-09-02T18:37:20 *** btcdrak has quit IRC
3592016-09-02T18:47:16 *** jtimon has quit IRC
3602016-09-02T18:47:28 *** Chris_St1 has joined #bitcoin-core-dev
3612016-09-02T18:55:11 *** CubicEarth has joined #bitcoin-core-dev
3622016-09-02T18:58:04 <GitHub12> [bitcoin] paveljanik opened pull request #8656: Do not shadow global variable fileout (master...20160902_Wshadow_fileout) https://github.com/bitcoin/bitcoin/pull/8656
3632016-09-02T19:01:24 <instagibbs> sipa, where is chainActive loaded on init? Having trouble tracking that down.
3642016-09-02T19:04:19 <sipa> instagibbs: ActivateBestChain?
3652016-09-02T19:04:26 <sipa> LoadBlockIndex?
3662016-09-02T19:06:49 <instagibbs> eh yes, but it got moved around and I'm flailing
3672016-09-02T19:07:04 <instagibbs> I'll track it down
3682016-09-02T19:07:51 *** CubicEar_ has joined #bitcoin-core-dev
3692016-09-02T19:09:58 <instagibbs> ah i misunderstood what wallet init was doing with it, nevermind, thanks
3702016-09-02T19:10:02 *** btcdrak has joined #bitcoin-core-dev
3712016-09-02T19:10:03 *** CubicEarth has quit IRC
3722016-09-02T19:16:31 *** e4xit has quit IRC
3732016-09-02T19:17:01 *** CubicEar_ has quit IRC
3742016-09-02T19:18:04 *** CubicEarth has joined #bitcoin-core-dev
3752016-09-02T19:19:33 *** TomMc has quit IRC
3762016-09-02T19:21:36 *** spudowiar has quit IRC
3772016-09-02T19:27:02 *** CubicEarth has quit IRC
3782016-09-02T19:30:31 *** e4xit has joined #bitcoin-core-dev
3792016-09-02T19:35:19 *** laurentmt has joined #bitcoin-core-dev
3802016-09-02T19:37:20 *** laurentmt has quit IRC
3812016-09-02T19:38:57 *** laurentmt has joined #bitcoin-core-dev
3822016-09-02T19:39:23 *** laurentmt has quit IRC
3832016-09-02T19:57:33 *** e4xit has quit IRC
3842016-09-02T19:58:08 *** e4xit has joined #bitcoin-core-dev
3852016-09-02T20:08:40 *** drizztbsd has joined #bitcoin-core-dev
3862016-09-02T20:08:55 *** timothy has quit IRC
3872016-09-02T20:09:06 *** drizztbsd is now known as timothy
3882016-09-02T20:09:26 <GitHub70> [bitcoin] paveljanik opened pull request #8658: WIP/DO NOT MERGE: Remove unused statements in serialization (master...20160902_nVersion_serialization_cleanup) https://github.com/bitcoin/bitcoin/pull/8658
3892016-09-02T20:25:17 *** shesek has joined #bitcoin-core-dev
3902016-09-02T20:34:33 *** shesek has quit IRC
3912016-09-02T20:47:19 *** shesek has joined #bitcoin-core-dev
3922016-09-02T21:01:01 *** Guyver2 has quit IRC
3932016-09-02T21:18:09 *** veleiro has joined #bitcoin-core-dev
3942016-09-02T21:58:39 *** gluytium has quit IRC
3952016-09-02T22:03:08 *** pmienk has quit IRC
3962016-09-02T22:03:13 *** gluytium has joined #bitcoin-core-dev
3972016-09-02T22:15:01 *** veleiro has quit IRC
3982016-09-02T22:15:50 *** pmienk has joined #bitcoin-core-dev
3992016-09-02T22:20:32 *** cryptapus is now known as cryptapus_afk
4002016-09-02T22:20:37 *** justanotheruser has joined #bitcoin-core-dev
4012016-09-02T22:33:34 *** pmienk has quit IRC
4022016-09-02T22:46:23 *** pmienk has joined #bitcoin-core-dev
4032016-09-02T22:59:23 *** spudowiar has joined #bitcoin-core-dev
4042016-09-02T23:13:45 *** murch has quit IRC
4052016-09-02T23:14:58 *** jtimon has joined #bitcoin-core-dev
4062016-09-02T23:18:19 *** crudel has quit IRC
4072016-09-02T23:24:28 *** slackircbridge has quit IRC
4082016-09-02T23:49:53 *** CyrusV has joined #bitcoin-core-dev
4092016-09-02T23:50:12 *** AaronvanW has quit IRC