1 2016-12-24 00:54:10 0|phantomcircuit|im running master on an older slightly slower system
2 2016-12-24 00:54:16 0|phantomcircuit|it's got an ssd in it though
3 2016-12-24 00:54:27 0|phantomcircuit|and it's not running ar anywhere near 100% cu during ibd
4 2016-12-24 00:54:33 0|phantomcircuit|at block height 188k
5 2016-12-24 00:54:53 0|phantomcircuit|seems something is broken with ibd block downloads maybe?
6 2016-12-24 00:57:48 0|phantomcircuit|yeah i have one peer that is significantly slower than the others
7 2016-12-24 00:59:16 0|gmaxwell|it won't use much cpu before it starts verifying signatures.
8 2016-12-24 00:59:33 0|gmaxwell|early in the chain its latency bound because it only fetches 16 blocks at a time from each peer.
9 2016-12-24 00:59:55 0|gmaxwell|it doesn't make that much difference in the time for overall synchronization.
10 2016-12-24 01:01:14 0|phantomcircuit|gmaxwell, yeah i was only at ~3% cpu
11 2016-12-24 01:01:19 0|phantomcircuit|which is definitely not right
12 2016-12-24 01:01:36 0|phantomcircuit|i disconnected a peer and now im at 100%
13 2016-12-24 01:01:39 0|gmaxwell|well thats goofy.
14 2016-12-24 01:01:44 0|phantomcircuit|net thread is spinning doing deserialization
15 2016-12-24 01:02:10 0|gmaxwell|it'll disconnect a peer if they're stalling your download, and I know that works because it's a little twitchy about it.
16 2016-12-24 01:02:29 0|phantomcircuit|gmaxwell, doesn't seem to have worked here for whatever reason
17 2016-12-24 05:11:54 0|bitcoin-git|[13bitcoin] 15rebroad closed pull request #9414: Skip processing of TXs that we already have. (06master...06DoNotProcessNewTxsMoreThanOnce) 02https://github.com/bitcoin/bitcoin/pull/9414
18 2016-12-24 05:57:21 0|rebroad|I have a question about mapAlreadyAskedFor - it seems that entries are erased from it only when a tx is received...
19 2016-12-24 05:57:49 0|rebroad|what is the maximum size it can reach? and why aren't tx hashes erased when a block arrives?
20 2016-12-24 05:58:08 0|rebroad|and what is to stop a node inventing random tx hashes and filling the map?
21 2016-12-24 05:58:20 0|rebroad|(which will never be erased as the txs don't exist)
22 2016-12-24 05:59:53 0|gmaxwell|rebroad: it is a limitedmap.
23 2016-12-24 06:08:42 0|rebroad_|gmaxwell, ah ok. so limited to 50,000 entries which is the most invs that any node can send in one go without misbehaving
24 2016-12-24 06:09:33 0|rebroad_|so i guess a node could send 50,000 and replace the map with useless entries, but the worst that would happen is that the node would then request every tx hash received rather than queue/delay them by 2 minute increments
25 2016-12-24 06:10:09 0|rebroad_|i would have thought the threshold for misbehaviour perhaps could be a fraction of MAX_INV_SZ rather than MAX_INV_SZ itself
26 2016-12-24 06:10:58 0|rebroad_|sorry, thinking aloud
27 2016-12-24 06:58:59 0|bitcoin-git|[13bitcoin] 15rebroad opened pull request #9418: Exit when no direct fetch (06master...06ExitWhenNoDirectFetch) 02https://github.com/bitcoin/bitcoin/pull/9418
28 2016-12-24 07:03:22 0|rebroad|#9418 is a very minimal PR.. just adds a return and reduces the indentation accordingly - hopefully not treading on the toes of {m,}any other PRs
29 2016-12-24 07:03:23 0|gribble|https://github.com/bitcoin/bitcoin/issues/9418 | Exit when no direct fetch by rebroad ÷ Pull Request #9418 ÷ bitcoin/bitcoin ÷ GitHub
30 2016-12-24 07:03:53 0|rebroad|(although it does tread on the toes of various other commits of mine, but I shall rebase them as need be)
31 2016-12-24 07:46:54 0|cfields|gmaxwell: heh, fixing this busted process loop has been my primary target for over a year now :)
32 2016-12-24 07:48:04 0|gmaxwell|cfields: ya. well I hope that a simple and expediant tweak isn't too distastful, I know you're working on big redesigns around it.
33 2016-12-24 07:48:08 0|cfields|no issues with quick changes if they help, but we're really close to having it fixed cleanly (imo)
34 2016-12-24 07:49:52 0|cfields|gmaxwell: my only request would be to base it on #9289, since that reworks all of that sleeping code
35 2016-12-24 07:49:54 0|gribble|https://github.com/bitcoin/bitcoin/issues/9289 | net: drop boost::thread_group by theuni ÷ Pull Request #9289 ÷ bitcoin/bitcoin ÷ GitHub
36 2016-12-24 07:50:10 0|cfields|(no real logical changes there though)
37 2016-12-24 07:50:35 0|gmaxwell|I'd gladly do that.
38 2016-12-24 07:51:05 0|cfields|thanks :)
39 2016-12-24 07:51:40 0|cfields|BlueMatt and I have been squabbling over syntactic sugar on that one, but I think it's clear that it will go in in some form
40 2016-12-24 08:11:54 0|luke-jr|oh well, fixed a bunch I could easily guess.
41 2016-12-24 08:32:27 0|luke-jr|hm
42 2016-12-24 08:32:47 0|luke-jr|it's actually possible to get a N-of-17 multisig P2SH
43 2016-12-24 08:32:54 0|luke-jr|OP_DUP OP_HASH160 <pubKeyHash> OP_EQUALVERIFY OP_CHECKSIG ( OP_ROT OP_ROT OP_DUP OP_HASH160 <pubKeyHash> OP_EQUALVERIFY OP_CHECKSIG OP_ADD )*(n-1) n OP_EQUAL
44 2016-12-24 08:42:29 0|jl2012|luke-jr: having more than 15 sigop is non standard, I think
45 2016-12-24 08:46:06 0|sipa|2 subsequent OP_ROTs?
46 2016-12-24 08:47:02 0|sipa|ah, i confused it with OP_SWAP, nvm
47 2016-12-24 08:55:11 0|luke-jr|jl2012: well, it probably ought to be lifted if anyone wants to make use of this
48 2016-12-24 08:55:36 0|luke-jr|otoh, with segwit around the corner.. no real need?
49 2016-12-24 08:56:02 0|luke-jr|(reason for coming up with this script initially is that cannon-c wanted to multisig without revealing the pubkeys to participants)
50 2016-12-24 09:13:18 0|luke-jr|hmm, it seems like Transifex's translations of plurals has degraded. seeing a lot of "blok(ken)" where it used to be "blok" or "blokken" depending on the number :x
51 2016-12-24 15:23:28 0|BlueMatt|cfields/gmaxwell well I think you've got a version of #9289 that I'm happy with up now
52 2016-12-24 15:23:30 0|BlueMatt|:)
53 2016-12-24 15:23:32 0|gribble|https://github.com/bitcoin/bitcoin/issues/9289 | net: drop boost::thread_group by theuni ÷ Pull Request #9289 ÷ bitcoin/bitcoin ÷ GitHub
54 2016-12-24 16:20:57 0|bitcoin-git|[13bitcoin] 15TheBlueMatt opened pull request #9419: Stop Using cs_main for CNodeState/State() (06master...062016-12-nodestate) 02https://github.com/bitcoin/bitcoin/pull/9419
55 2016-12-24 18:25:33 0|bitcoin-git|[13bitcoin] 15rebroad reopened pull request #9402: Allow per network configuration file (06master...06PerNetworkConfig) 02https://github.com/bitcoin/bitcoin/pull/9402
56 2016-12-24 20:30:56 0|bitcoin-git|[13bitcoin] 15droark opened pull request #9420: Fix linker error when configured with --enable-lcov (06master...06fixlcov) 02https://github.com/bitcoin/bitcoin/pull/9420