1 2016-09-02T00:03:02  *** gabridome has joined #bitcoin-core-dev
  2 2016-09-02T00:06:50  *** Ylbam has quit IRC
  3 2016-09-02T00:12:07  *** Alopex has quit IRC
  4 2016-09-02T00:13:12  *** Alopex has joined #bitcoin-core-dev
  5 2016-09-02T00:16:47  *** ghtdak has joined #bitcoin-core-dev
  6 2016-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
  7 2016-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.
  8 2016-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
  9 2016-09-02T00:26:35  <sipa> i would be fime with that, though calling it a bugfix is a stretch
 10 2016-09-02T00:27:15  <gmaxwell> well I gave a pragmatic argument for it being one.
 11 2016-09-02T00:27:33  <gmaxwell> but I don't think I needed to, it's pretty obviously riskless with no "interface" impact.
 12 2016-09-02T00:28:10  <sipa> yes, i agree it has very low risk
 13 2016-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.
 14 2016-09-02T00:32:19  <luke-jr> sounds reasonable to do, in any case
 15 2016-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
 16 2016-09-02T00:39:33  *** gabridome has quit IRC
 17 2016-09-02T00:41:16  *** gabridome has joined #bitcoin-core-dev
 18 2016-09-02T00:43:44  *** gabridome has joined #bitcoin-core-dev
 19 2016-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
 20 2016-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.
 21 2016-09-02T00:55:17  *** Alopex has quit IRC
 22 2016-09-02T00:56:22  *** Alopex has joined #bitcoin-core-dev
 23 2016-09-02T00:57:11  *** gabridome has quit IRC
 24 2016-09-02T01:00:26  *** TomMc has joined #bitcoin-core-dev
 25 2016-09-02T01:13:10  *** grubles has quit IRC
 26 2016-09-02T01:16:32  *** Chris_St1 has joined #bitcoin-core-dev
 27 2016-09-02T01:17:04  *** justanotheruser has quit IRC
 28 2016-09-02T01:17:34  *** justanotheruser has joined #bitcoin-core-dev
 29 2016-09-02T01:34:13  *** Giszmo has quit IRC
 30 2016-09-02T01:34:29  *** achow101 has quit IRC
 31 2016-09-02T01:35:07  *** achow101 has joined #bitcoin-core-dev
 32 2016-09-02T01:38:42  <jl2012> We need a nulldummy only bip?
 33 2016-09-02T01:38:49  *** Chris_St1 has quit IRC
 34 2016-09-02T01:42:03  <jl2012> And I already have a PR for empty failing signature https://github.com/bitcoin/bitcoin/pull/8634
 35 2016-09-02T01:47:37  *** dcousens has joined #bitcoin-core-dev
 36 2016-09-02T01:51:36  *** Chris_St1 has joined #bitcoin-core-dev
 37 2016-09-02T01:57:01  *** Alopex has quit IRC
 38 2016-09-02T01:58:06  *** Alopex has joined #bitcoin-core-dev
 39 2016-09-02T02:01:00  *** Chris_St1 has quit IRC
 40 2016-09-02T02:04:56  <cfields> sdaftuar: did you say you were able to reproduce the segwit.py failure locally?
 41 2016-09-02T02:05:39  <cfields> sdaftuar: if so: https://github.com/theuni/bitcoin/commit/fa88a425c0c1089ab75ea9223699286f40c8eea7
 42 2016-09-02T02:06:10  <cfields> it's pretty hackish, needs a cleanup. But should be enough to determine if it fixes the problem
 43 2016-09-02T02:07:18  <cfields> wumpus: let me know if ^^ is what you had in mind
 44 2016-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
 45 2016-09-02T02:19:22  *** Alopex has quit IRC
 46 2016-09-02T02:20:27  *** Alopex has joined #bitcoin-core-dev
 47 2016-09-02T02:21:14  *** ghtdak has quit IRC
 48 2016-09-02T02:25:34  *** Samdney has left #bitcoin-core-dev
 49 2016-09-02T02:25:44  *** achow101 has quit IRC
 50 2016-09-02T02:27:04  *** TomMc has quit IRC
 51 2016-09-02T02:29:04  *** jtimon has quit IRC
 52 2016-09-02T02:30:21  *** Alopex has quit IRC
 53 2016-09-02T02:31:26  *** Alopex has joined #bitcoin-core-dev
 54 2016-09-02T02:35:46  *** adiabat has joined #bitcoin-core-dev
 55 2016-09-02T02:41:01  *** Alopex has quit IRC
 56 2016-09-02T02:42:07  *** Alopex has joined #bitcoin-core-dev
 57 2016-09-02T02:55:01  *** Alopex has quit IRC
 58 2016-09-02T02:55:29  *** fengling has joined #bitcoin-core-dev
 59 2016-09-02T02:56:06  *** Alopex has joined #bitcoin-core-dev
 60 2016-09-02T03:00:26  *** fengling has quit IRC
 61 2016-09-02T03:04:20  *** justan0theruser has joined #bitcoin-core-dev
 62 2016-09-02T03:05:57  *** justanotheruser has quit IRC
 63 2016-09-02T03:24:06  *** Alopex has quit IRC
 64 2016-09-02T03:25:12  *** Alopex has joined #bitcoin-core-dev
 65 2016-09-02T03:36:38  *** jcorgan has joined #bitcoin-core-dev
 66 2016-09-02T03:45:32  *** dcousens has quit IRC
 67 2016-09-02T03:47:48  *** TomMc has joined #bitcoin-core-dev
 68 2016-09-02T03:55:21  *** Alopex has quit IRC
 69 2016-09-02T03:56:27  *** Alopex has joined #bitcoin-core-dev
 70 2016-09-02T03:57:16  *** fengling has joined #bitcoin-core-dev
 71 2016-09-02T04:01:35  *** spudowiar has quit IRC
 72 2016-09-02T04:02:06  *** fengling has quit IRC
 73 2016-09-02T04:05:22  *** TomMc has quit IRC
 74 2016-09-02T04:11:14  *** JackH has quit IRC
 75 2016-09-02T04:11:49  *** JackH has joined #bitcoin-core-dev
 76 2016-09-02T04:22:16  *** Alopex has quit IRC
 77 2016-09-02T04:23:21  *** Alopex has joined #bitcoin-core-dev
 78 2016-09-02T04:33:40  *** kadoban has quit IRC
 79 2016-09-02T04:37:25  *** cfields has quit IRC
 80 2016-09-02T04:40:40  *** achow101 has joined #bitcoin-core-dev
 81 2016-09-02T04:42:38  *** achow101 has quit IRC
 82 2016-09-02T04:44:06  *** Alopex has quit IRC
 83 2016-09-02T04:45:12  *** Alopex has joined #bitcoin-core-dev
 84 2016-09-02T04:48:33  *** cfields has joined #bitcoin-core-dev
 85 2016-09-02T04:52:07  *** achow101 has joined #bitcoin-core-dev
 86 2016-09-02T04:57:49  *** achow101 has quit IRC
 87 2016-09-02T04:58:58  *** fengling has joined #bitcoin-core-dev
 88 2016-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
 89 2016-09-02T05:03:46  *** fengling has quit IRC
 90 2016-09-02T05:10:51  *** justan0theruser has quit IRC
 91 2016-09-02T05:13:55  <gmaxwell> down with 'test frameworks'.
 92 2016-09-02T05:18:43  *** gabridome has joined #bitcoin-core-dev
 93 2016-09-02T05:19:04  *** paveljanik has quit IRC
 94 2016-09-02T05:22:29  *** Ylbam has joined #bitcoin-core-dev
 95 2016-09-02T05:29:21  *** Alopex has quit IRC
 96 2016-09-02T05:30:26  *** Alopex has joined #bitcoin-core-dev
 97 2016-09-02T05:31:12  *** gabridome has quit IRC
 98 2016-09-02T05:45:01  *** dcousens has joined #bitcoin-core-dev
 99 2016-09-02T05:47:00  *** fengling has joined #bitcoin-core-dev
100 2016-09-02T05:48:56  <jeremyrubin> gmaxwell: Hear, hear!
101 2016-09-02T05:51:09  <GitHub124> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/f061415d12ce...91990ee01dab
102 2016-09-02T05:51:10  <GitHub124> bitcoin/master 854f1af Pieter Wuille: Make the dummy argument to getaddednodeinfo optional
103 2016-09-02T05:51:10  <GitHub124> bitcoin/master 91990ee Wladimir J. van der Laan: Merge #8272: Make the dummy argument to getaddednodeinfo optional...
104 2016-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
105 2016-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.
106 2016-09-02T05:55:19  *** jannes has quit IRC
107 2016-09-02T06:02:32  *** Alopex has quit IRC
108 2016-09-02T06:03:37  *** Alopex has joined #bitcoin-core-dev
109 2016-09-02T06:05:02  *** assder has joined #bitcoin-core-dev
110 2016-09-02T06:15:00  <jeremyrubin> gross :/
111 2016-09-02T06:26:47  *** ghtdak has joined #bitcoin-core-dev
112 2016-09-02T06:44:46  *** fengling has quit IRC
113 2016-09-02T06:52:13  *** gabridome has joined #bitcoin-core-dev
114 2016-09-02T06:54:29  *** gabridome has quit IRC
115 2016-09-02T06:56:54  *** BashCo has quit IRC
116 2016-09-02T07:03:49  *** kyletorpey has quit IRC
117 2016-09-02T07:10:48  *** gabridome1 has quit IRC
118 2016-09-02T07:11:16  *** Alopex has quit IRC
119 2016-09-02T07:12:22  *** Alopex has joined #bitcoin-core-dev
120 2016-09-02T07:15:45  *** BashCo has joined #bitcoin-core-dev
121 2016-09-02T07:20:49  *** pmienk has quit IRC
122 2016-09-02T07:22:05  *** paveljanik has joined #bitcoin-core-dev
123 2016-09-02T07:32:54  *** pmienk has joined #bitcoin-core-dev
124 2016-09-02T07:37:31  *** pmienk has quit IRC
125 2016-09-02T07:37:50  *** BashCo_ has joined #bitcoin-core-dev
126 2016-09-02T07:41:00  *** BashCo has quit IRC
127 2016-09-02T07:41:42  *** laurentmt has joined #bitcoin-core-dev
128 2016-09-02T07:44:05  <jonasschnelli> Hmm... IBDing from a local peer (same host) seems to take much longer the bootstraping from random peers.
129 2016-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.
130 2016-09-02T07:46:31  <gmaxwell> other than that, they'll obviously contend a bit for IO
131 2016-09-02T07:46:50  <gmaxwell> I'm not aware of any other reasons for slowdowns other than that.
132 2016-09-02T07:47:23  <paveljanik> jonasschnelli, OS X<
133 2016-09-02T07:47:24  <paveljanik> ?
134 2016-09-02T07:48:02  <jonasschnelli> paveljanik: no debian on my Intel(R) Xeon(R) CPU E3-1275 v5 @ 3.60GHz
135 2016-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)
136 2016-09-02T07:48:43  <gmaxwell> are you only looking at the first 150k blocks? or are you seeing the rate per block slower later?
137 2016-09-02T07:48:47  <paveljanik> the process named powerd took 100% of CPU power
138 2016-09-02T07:49:29  <jonasschnelli> gmaxwell: It took more then 1h for progress=0.035480
139 2016-09-02T07:49:53  <jonasschnelli> (where is did sync with random peers up to progress 1.0 in ~3h [2 month ago though])
140 2016-09-02T07:49:55  *** pmienk has joined #bitcoin-core-dev
141 2016-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.
142 2016-09-02T07:50:59  <jonasschnelli> Also... my extremely scientific measure of how fast the tail -f output scrolls though.... did tell me its slower
143 2016-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
144 2016-09-02T07:51:59  *** laurentmt has quit IRC
145 2016-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)
146 2016-09-02T07:52:49  * jonasschnelli needs to write a IBD benchmark tool
147 2016-09-02T07:53:00  <gmaxwell> prune might make it slower.
148 2016-09-02T07:53:20  <jonasschnelli> the prune data eviction seems pretty fast
149 2016-09-02T07:53:21  <gmaxwell> after all it's writing things to only delete it after.
150 2016-09-02T07:53:35  <jonasschnelli> it's running on a 1GB/s SSD
151 2016-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.
152 2016-09-02T07:57:00  <GitHub55> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/91990ee01dab...6f939c9080a3
153 2016-09-02T07:57:01  <GitHub55> bitcoin/master cdd79eb Jorge Timón: C++11: s/boost::scoped_ptr/std::unique_ptr/
154 2016-09-02T07:57:01  <GitHub55> bitcoin/master 6f939c9 Wladimir J. van der Laan: Merge #8629: C++11: s/boost::scoped_ptr/std::unique_ptr/...
155 2016-09-02T07:57:08  * jonasschnelli is trying again without prune
156 2016-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
157 2016-09-02T08:00:14  <sipa> pruning causes flushing
158 2016-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.
159 2016-09-02T08:07:12  *** AaronvanW has joined #bitcoin-core-dev
160 2016-09-02T08:07:40  *** AaronvanW has quit IRC
161 2016-09-02T08:07:40  *** AaronvanW has joined #bitcoin-core-dev
162 2016-09-02T08:10:44  <jonasschnelli> What is the reason for the txn_count=0x00 in the headers p2p message?
163 2016-09-02T08:11:49  *** fengling has joined #bitcoin-core-dev
164 2016-09-02T08:13:10  <sipa> because CBlockHeader did not originally exist
165 2016-09-02T08:13:30  <sipa> and headers messages just contained a CBlock with 0 transactions
166 2016-09-02T08:13:55  <jonasschnelli> thanks.
167 2016-09-02T08:20:16  <jeremyrubin> Is it correct that the maximum number of items deleted from sigcache while checking a block is max sigops?
168 2016-09-02T08:20:58  <sipa> i believe so
169 2016-09-02T08:21:55  <jeremyrubin> And is that "MAX_BLOCK_SIGOPS_COST" or is that something else
170 2016-09-02T08:22:03  <jeremyrubin> because I see a 20k number floating around too
171 2016-09-02T08:22:08  <jeremyrubin> but MAX_BLOCK_SIGOPS_COST=80k
172 2016-09-02T08:22:32  <sipa> yes, 4x factor from segwit
173 2016-09-02T08:22:57  <sipa> sigops in non-witness part count as 4, in witness part they count as 1
174 2016-09-02T08:23:39  <jeremyrubin> so if you have 0 sigops in witness, you can have the full 80k?
175 2016-09-02T08:24:13  <sipa> if you have 0 sigops in non-witness
176 2016-09-02T08:24:15  <sipa> yes
177 2016-09-02T08:24:18  <jeremyrubin> ok cool
178 2016-09-02T08:24:22  <sipa> pre segwit the max is 20k
179 2016-09-02T08:24:29  <sipa> post segwit it is 80k
180 2016-09-02T08:39:01  *** Alopex has quit IRC
181 2016-09-02T08:40:06  *** Alopex has joined #bitcoin-core-dev
182 2016-09-02T08:40:32  *** JackH has quit IRC
183 2016-09-02T08:58:43  <jonasschnelli> with disabled pruning it took also >1h for progress=0.031407... now trying a sync from random peers
184 2016-09-02T09:00:16  <gmaxwell> you're killin me with the testing with just the first 3%.
185 2016-09-02T09:00:41  <jonasschnelli> gmaxwell: heh. You think comparing the first 3% does not make sense?
186 2016-09-02T09:01:14  <jonasschnelli> I just try to not waste time. :)
187 2016-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. :)
188 2016-09-02T09:02:42  <jonasschnelli> gmaxwell: Okay. I see. But anyways, a sync with a single local peer will then take always longer, right?
189 2016-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.
190 2016-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.
191 2016-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. :)
192 2016-09-02T09:11:30  *** AaronvanW has quit IRC
193 2016-09-02T09:16:36  *** AaronvanW has joined #bitcoin-core-dev
194 2016-09-02T09:16:37  *** AaronvanW has quit IRC
195 2016-09-02T09:16:37  *** AaronvanW has joined #bitcoin-core-dev
196 2016-09-02T09:33:46  *** spudowiar has joined #bitcoin-core-dev
197 2016-09-02T09:36:00  *** murch has joined #bitcoin-core-dev
198 2016-09-02T09:38:31  *** jtimon has joined #bitcoin-core-dev
199 2016-09-02T09:44:15  <GitHub197> [bitcoin] sipa opened pull request #8651: Predeclare PrecomputedTransactionData as struct (master...classtructblah) https://github.com/bitcoin/bitcoin/pull/8651
200 2016-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
201 2016-09-02T09:52:46  *** fengling has quit IRC
202 2016-09-02T10:04:35  *** fengling has joined #bitcoin-core-dev
203 2016-09-02T10:19:20  *** jannes has joined #bitcoin-core-dev
204 2016-09-02T10:26:26  <GitHub76> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/6f939c9080a3...381d0ddc8aaa
205 2016-09-02T10:26:27  <GitHub76> bitcoin/master cce3024 Pavel Janík: Do not shadow local variable, cleanup
206 2016-09-02T10:26:28  <GitHub76> bitcoin/master a159f25 Pavel Janík: Remove redundand (and shadowing) declaration
207 2016-09-02T10:26:28  <GitHub76> bitcoin/master 381d0dd Wladimir J. van der Laan: Merge #8449: [Trivial] Do not shadow local variable, cleanup...
208 2016-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
209 2016-09-02T10:30:06  *** BashCo has joined #bitcoin-core-dev
210 2016-09-02T10:31:52  *** BashCo_ has quit IRC
211 2016-09-02T10:43:26  *** fengling has quit IRC
212 2016-09-02T10:45:52  *** owowo has quit IRC
213 2016-09-02T10:52:18  <GitHub29> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/381d0ddc8aaa...cbe9ae8c69b9
214 2016-09-02T10:52:19  <GitHub29> bitcoin/master b7c349d Pavel Janík: Do not shadow variables in networking code
215 2016-09-02T10:52:19  <GitHub29> bitcoin/master cbe9ae8 Wladimir J. van der Laan: Merge #8466: [Trivial] Do not shadow variables in networking code...
216 2016-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
217 2016-09-02T10:54:59  *** fengling has joined #bitcoin-core-dev
218 2016-09-02T11:03:03  *** jcorgan_ has joined #bitcoin-core-dev
219 2016-09-02T11:06:02  *** timothy has quit IRC
220 2016-09-02T11:06:04  *** drizztbsd has joined #bitcoin-core-dev
221 2016-09-02T11:06:36  *** drizztbsd is now known as timothy
222 2016-09-02T11:07:31  *** jcorgan has quit IRC
223 2016-09-02T11:07:32  *** mn3monic has quit IRC
224 2016-09-02T11:07:32  *** sanada` has quit IRC
225 2016-09-02T11:07:33  *** Bootvis has quit IRC
226 2016-09-02T11:07:34  *** crudel has quit IRC
227 2016-09-02T11:07:34  *** harding has quit IRC
228 2016-09-02T11:07:34  *** Taek has quit IRC
229 2016-09-02T11:09:54  *** mn3monic has joined #bitcoin-core-dev
230 2016-09-02T11:09:54  *** sanada` has joined #bitcoin-core-dev
231 2016-09-02T11:09:54  *** Bootvis has joined #bitcoin-core-dev
232 2016-09-02T11:09:54  *** crudel has joined #bitcoin-core-dev
233 2016-09-02T11:09:54  *** harding has joined #bitcoin-core-dev
234 2016-09-02T11:09:54  *** Taek has joined #bitcoin-core-dev
235 2016-09-02T11:19:46  *** fengling has quit IRC
236 2016-09-02T11:30:26  *** fengling has joined #bitcoin-core-dev
237 2016-09-02T11:38:48  <cfields> jonasschnelli: so your comment on #8085 may not be relevant to those changes?
238 2016-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
239 2016-09-02T11:42:56  *** BashCo has quit IRC
240 2016-09-02T11:55:06  *** fengling has quit IRC
241 2016-09-02T11:55:26  *** cryptapus_afk is now known as cryptapus
242 2016-09-02T12:22:04  *** dcousens has quit IRC
243 2016-09-02T12:36:42  *** paveljanik has quit IRC
244 2016-09-02T12:37:15  *** LeMiner2 has joined #bitcoin-core-dev
245 2016-09-02T12:39:07  *** LeMiner has quit IRC
246 2016-09-02T12:39:07  *** LeMiner2 is now known as LeMiner
247 2016-09-02T12:45:49  *** owowo has joined #bitcoin-core-dev
248 2016-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
249 2016-09-02T13:03:42  *** dcousens has joined #bitcoin-core-dev
250 2016-09-02T13:20:51  *** jcorgan_ is now known as jcorgan
251 2016-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
252 2016-09-02T13:29:44  *** Guyver2 has joined #bitcoin-core-dev
253 2016-09-02T13:41:14  <sipa> cfields: looks good to me
254 2016-09-02T13:41:58  <sipa> jonasschnelli: 3%, how many blocks is that?
255 2016-09-02T13:42:15  * jonasschnelli checking the logs
256 2016-09-02T13:42:54  <jonasschnelli> sipa: aprox 211000
257 2016-09-02T13:43:15  <jonasschnelli> My assumption is that IBD with a single peer is much slower... but can't prove it right now.
258 2016-09-02T13:43:18  <sipa> jonasschnelli: that seems very wrong
259 2016-09-02T13:43:25  <sipa> it should take minutes to get to 211000
260 2016-09-02T13:43:58  <sipa> not an hour
261 2016-09-02T13:44:12  <sipa> the effect greg talks about it real, but shouldn't last more than a few minutes
262 2016-09-02T13:44:35  <jonasschnelli> 2016-09-02 08:58:49 UpdateTip: new best=0000000000000345b371caa3f829cacbe2b4d38ecd15a5a02031efae79934d15 height=211000
263 2016-09-02T13:44:38  <jonasschnelli> 2016-09-02 07:56:40 Bitcoin version v0.13.99.0-df98230
264 2016-09-02T13:45:00  *** paveljanik has joined #bitcoin-core-dev
265 2016-09-02T13:48:12  *** warren has quit IRC
266 2016-09-02T13:49:58  *** warren has joined #bitcoin-core-dev
267 2016-09-02T13:54:37  *** shesek has quit IRC
268 2016-09-02T13:56:16  *** grubles has joined #bitcoin-core-dev
269 2016-09-02T13:57:00  <jonasschnelli> sipa: I guess its caused by --enable-debug
270 2016-09-02T13:58:43  <sipa> oh, yes, that will slow things down tremendously
271 2016-09-02T14:00:02  <jonasschnelli> sry for the noise... I should finally remember to disable debug mode when benachmarking.
272 2016-09-02T14:04:31  *** jtimon has quit IRC
273 2016-09-02T14:16:36  *** TomMc has joined #bitcoin-core-dev
274 2016-09-02T14:17:14  *** Cheeseo has quit IRC
275 2016-09-02T14:17:34  *** Cheeseo has joined #bitcoin-core-dev
276 2016-09-02T14:17:34  *** Cheeseo has joined #bitcoin-core-dev
277 2016-09-02T14:28:37  *** grubles has quit IRC
278 2016-09-02T14:29:58  *** Samdney has joined #bitcoin-core-dev
279 2016-09-02T14:31:48  *** Giszmo has joined #bitcoin-core-dev
280 2016-09-02T14:37:22  *** warren has quit IRC
281 2016-09-02T14:38:06  *** grubles has joined #bitcoin-core-dev
282 2016-09-02T14:39:48  *** warren has joined #bitcoin-core-dev
283 2016-09-02T14:43:39  *** grubles has quit IRC
284 2016-09-02T14:48:11  *** gabridome has joined #bitcoin-core-dev
285 2016-09-02T14:48:42  *** gabridome has quit IRC
286 2016-09-02T14:48:57  *** gabridome has joined #bitcoin-core-dev
287 2016-09-02T14:51:14  *** grubles has joined #bitcoin-core-dev
288 2016-09-02T14:53:37  *** grubles has quit IRC
289 2016-09-02T14:56:39  *** grubles has joined #bitcoin-core-dev
290 2016-09-02T15:05:33  *** Giszmo has quit IRC
291 2016-09-02T15:07:36  *** gabridome has quit IRC
292 2016-09-02T15:09:27  *** Giszmo has joined #bitcoin-core-dev
293 2016-09-02T15:12:56  *** achow101 has joined #bitcoin-core-dev
294 2016-09-02T15:15:47  <sturles> a/away
295 2016-09-02T15:25:40  <cfields> sipa: thanks
296 2016-09-02T15:26:19  <cfields> jonasschnelli: so I should ignore https://github.com/bitcoin/bitcoin/pull/8085#issuecomment-244077083 ?
297 2016-09-02T15:26:40  <jonasschnelli> cfields: I think so...
298 2016-09-02T15:26:50  <jonasschnelli> I'm working on more authentic IBD benchmarks.
299 2016-09-02T15:27:39  <cfields> jonasschnelli: ok. apples to apples IBD is a good idea though. I'll do some syncs over the weekend
300 2016-09-02T15:27:39  *** achow101 has quit IRC
301 2016-09-02T15:28:11  <jonasschnelli> cfields: Yes. I was fooled (again) by the bias of --enable-debug and --prune
302 2016-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
303 2016-09-02T15:30:28  <cfields> so it makes sense that it'd be a major performance killer
304 2016-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
305 2016-09-02T15:34:05  <jonasschnelli> heh... yes.
306 2016-09-02T15:34:07  <btcdrak> Jonas 'debug' schnelli
307 2016-09-02T15:34:09  <sdaftuar> cfields: i don't think i have seen segwit.py fail locally for the reason mentioned in #8532
308 2016-09-02T15:34:19  <sdaftuar> gmaxwell: please set up an irc autoresponder so the rest of us don't forget to ask him too :)
309 2016-09-02T15:34:49  <cfields> sdaftuar: ah, ok
310 2016-09-02T15:35:20  *** cryptapus has quit IRC
311 2016-09-02T15:35:45  *** cryptapus has joined #bitcoin-core-dev
312 2016-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)
313 2016-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
314 2016-09-02T15:56:23  <wumpus> cfields: looking
315 2016-09-02T15:59:04  <wumpus> cfields: looks good to me, doesn't even require any new signals!
316 2016-09-02T16:00:35  <wumpus> cfields: any reason to not put the signal subscription/unsubscription in StartRPC/StopRPC?
317 2016-09-02T16:01:09  <wumpus> cfields: oh duh, those are general methods
318 2016-09-02T16:01:29  <wumpus> cfields: if there was a rpcblockchain.init/deinit, it would fit there
319 2016-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?
320 2016-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)
321 2016-09-02T16:03:28  <sipa> agree
322 2016-09-02T16:05:25  <cfields> wumpus: yep, makes sense
323 2016-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 :)
324 2016-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?
325 2016-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?)
326 2016-09-02T16:09:20  <sipa> by default i think handlers are called in the order they were added
327 2016-09-02T16:09:37  *** jtimon has joined #bitcoin-core-dev
328 2016-09-02T16:09:43  <cfields> wumpus: i believe it's fine, but i'll double-check.
329 2016-09-02T16:10:15  *** kadoban has joined #bitcoin-core-dev
330 2016-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
331 2016-09-02T16:11:28  <wumpus> sipa: yes, but that is a really fragile thing to rely on
332 2016-09-02T16:11:58  <wumpus> cfields: ok, yes that makes sense, if the UI signal is last
333 2016-09-02T16:12:17  <wumpus> cfields: indeed there's no need to be worried doing things concurrently with the GUI
334 2016-09-02T16:12:33  <wumpus> (usually that won't be running anyhow during RPC testing, although it's possible)
335 2016-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
336 2016-09-02T16:13:31  <cfields> right
337 2016-09-02T16:15:59  <cfields> yes, only other user is qt for updating gui
338 2016-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
339 2016-09-02T16:32:20  *** jujumax has joined #bitcoin-core-dev
340 2016-09-02T16:32:34  *** aalex has quit IRC
341 2016-09-02T16:41:38  *** aalex has joined #bitcoin-core-dev
342 2016-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
343 2016-09-02T16:58:48  *** jujumax has quit IRC
344 2016-09-02T16:59:02  *** Chris_Stewart_5 has joined #bitcoin-core-dev
345 2016-09-02T17:18:18  *** gabridome has joined #bitcoin-core-dev
346 2016-09-02T17:20:27  *** gabridome has quit IRC
347 2016-09-02T17:21:21  *** Chris_St1 has joined #bitcoin-core-dev
348 2016-09-02T17:22:09  *** Chris_Stewart_5 has quit IRC
349 2016-09-02T17:22:44  *** jtimon has quit IRC
350 2016-09-02T17:23:17  *** jtimon has joined #bitcoin-core-dev
351 2016-09-02T17:35:54  *** spudowiar has quit IRC
352 2016-09-02T17:59:06  *** spudowiar has joined #bitcoin-core-dev
353 2016-09-02T18:00:34  *** Chris_St1 has quit IRC
354 2016-09-02T18:08:59  *** spudowiar has quit IRC
355 2016-09-02T18:16:09  *** Chris_St1 has joined #bitcoin-core-dev
356 2016-09-02T18:27:25  *** spudowiar has joined #bitcoin-core-dev
357 2016-09-02T18:31:29  *** Chris_St1 has quit IRC
358 2016-09-02T18:37:20  *** btcdrak has quit IRC
359 2016-09-02T18:47:16  *** jtimon has quit IRC
360 2016-09-02T18:47:28  *** Chris_St1 has joined #bitcoin-core-dev
361 2016-09-02T18:55:11  *** CubicEarth has joined #bitcoin-core-dev
362 2016-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
363 2016-09-02T19:01:24  <instagibbs> sipa, where is chainActive loaded on init? Having trouble tracking that down.
364 2016-09-02T19:04:19  <sipa> instagibbs: ActivateBestChain?
365 2016-09-02T19:04:26  <sipa> LoadBlockIndex?
366 2016-09-02T19:06:49  <instagibbs> eh yes, but it got moved around and I'm flailing
367 2016-09-02T19:07:04  <instagibbs> I'll track it down
368 2016-09-02T19:07:51  *** CubicEar_ has joined #bitcoin-core-dev
369 2016-09-02T19:09:58  <instagibbs> ah i misunderstood what wallet init was doing with it, nevermind, thanks
370 2016-09-02T19:10:02  *** btcdrak has joined #bitcoin-core-dev
371 2016-09-02T19:10:03  *** CubicEarth has quit IRC
372 2016-09-02T19:16:31  *** e4xit has quit IRC
373 2016-09-02T19:17:01  *** CubicEar_ has quit IRC
374 2016-09-02T19:18:04  *** CubicEarth has joined #bitcoin-core-dev
375 2016-09-02T19:19:33  *** TomMc has quit IRC
376 2016-09-02T19:21:36  *** spudowiar has quit IRC
377 2016-09-02T19:27:02  *** CubicEarth has quit IRC
378 2016-09-02T19:30:31  *** e4xit has joined #bitcoin-core-dev
379 2016-09-02T19:35:19  *** laurentmt has joined #bitcoin-core-dev
380 2016-09-02T19:37:20  *** laurentmt has quit IRC
381 2016-09-02T19:38:57  *** laurentmt has joined #bitcoin-core-dev
382 2016-09-02T19:39:23  *** laurentmt has quit IRC
383 2016-09-02T19:57:33  *** e4xit has quit IRC
384 2016-09-02T19:58:08  *** e4xit has joined #bitcoin-core-dev
385 2016-09-02T20:08:40  *** drizztbsd has joined #bitcoin-core-dev
386 2016-09-02T20:08:55  *** timothy has quit IRC
387 2016-09-02T20:09:06  *** drizztbsd is now known as timothy
388 2016-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
389 2016-09-02T20:25:17  *** shesek has joined #bitcoin-core-dev
390 2016-09-02T20:34:33  *** shesek has quit IRC
391 2016-09-02T20:47:19  *** shesek has joined #bitcoin-core-dev
392 2016-09-02T21:01:01  *** Guyver2 has quit IRC
393 2016-09-02T21:18:09  *** veleiro has joined #bitcoin-core-dev
394 2016-09-02T21:58:39  *** gluytium has quit IRC
395 2016-09-02T22:03:08  *** pmienk has quit IRC
396 2016-09-02T22:03:13  *** gluytium has joined #bitcoin-core-dev
397 2016-09-02T22:15:01  *** veleiro has quit IRC
398 2016-09-02T22:15:50  *** pmienk has joined #bitcoin-core-dev
399 2016-09-02T22:20:32  *** cryptapus is now known as cryptapus_afk
400 2016-09-02T22:20:37  *** justanotheruser has joined #bitcoin-core-dev
401 2016-09-02T22:33:34  *** pmienk has quit IRC
402 2016-09-02T22:46:23  *** pmienk has joined #bitcoin-core-dev
403 2016-09-02T22:59:23  *** spudowiar has joined #bitcoin-core-dev
404 2016-09-02T23:13:45  *** murch has quit IRC
405 2016-09-02T23:14:58  *** jtimon has joined #bitcoin-core-dev
406 2016-09-02T23:18:19  *** crudel has quit IRC
407 2016-09-02T23:24:28  *** slackircbridge has quit IRC
408 2016-09-02T23:49:53  *** CyrusV has joined #bitcoin-core-dev
409 2016-09-02T23:50:12  *** AaronvanW has quit IRC