12020-05-03T00:00:03  *** jes1 has quit IRC
  22020-05-03T00:01:04  *** whythat has joined #bitcoin-core-dev
  32020-05-03T00:02:04  *** promag has joined #bitcoin-core-dev
  42020-05-03T00:04:07  *** droark has quit IRC
  52020-05-03T00:04:29  *** droark has joined #bitcoin-core-dev
  62020-05-03T00:05:46  *** droark has quit IRC
  72020-05-03T00:06:10  *** droark has joined #bitcoin-core-dev
  82020-05-03T00:07:23  *** promag has quit IRC
  92020-05-03T00:07:38  *** droark has quit IRC
 102020-05-03T00:08:06  *** droark has joined #bitcoin-core-dev
 112020-05-03T00:12:03  *** droark has quit IRC
 122020-05-03T00:17:52  *** emilengler has quit IRC
 132020-05-03T00:22:15  *** patjameson has joined #bitcoin-core-dev
 142020-05-03T00:29:33  *** surja795 has joined #bitcoin-core-dev
 152020-05-03T00:33:02  *** surja795 has quit IRC
 162020-05-03T00:34:14  *** AaronvanW has joined #bitcoin-core-dev
 172020-05-03T00:34:57  *** surja795 has joined #bitcoin-core-dev
 182020-05-03T00:37:02  *** Guest35617 has quit IRC
 192020-05-03T00:37:09  *** Guest35617 has joined #bitcoin-core-dev
 202020-05-03T00:40:59  *** surja795_ has joined #bitcoin-core-dev
 212020-05-03T00:41:00  *** surja795 has quit IRC
 222020-05-03T01:03:44  *** AaronvanW has quit IRC
 232020-05-03T01:09:03  *** whythat has quit IRC
 242020-05-03T01:13:31  <hebasto> wumpus: sipa: could block nicebbs on the GH?
 252020-05-03T01:14:06  *** luke-jr has quit IRC
 262020-05-03T01:15:01  *** luke-jr has joined #bitcoin-core-dev
 272020-05-03T01:16:14  *** promag has joined #bitcoin-core-dev
 282020-05-03T01:18:38  <sipa> hebasto: i'm not convinced that's spam
 292020-05-03T01:20:14  <hebasto> ok, will try to translate it :D
 302020-05-03T01:21:53  <hebasto> sipa: sorry for false alarming
 312020-05-03T01:24:46  *** Nebraskka has quit IRC
 322020-05-03T01:24:47  <sipa> np
 332020-05-03T01:25:23  *** Nebraskka has joined #bitcoin-core-dev
 342020-05-03T01:26:44  *** brakmic has quit IRC
 352020-05-03T01:33:39  *** proofofkeags has quit IRC
 362020-05-03T01:34:14  *** proofofkeags has joined #bitcoin-core-dev
 372020-05-03T01:37:25  *** proofofk_ has joined #bitcoin-core-dev
 382020-05-03T01:41:43  *** proofofkeags has quit IRC
 392020-05-03T01:44:14  *** dviola has quit IRC
 402020-05-03T01:44:14  *** hebasto has quit IRC
 412020-05-03T01:44:51  <fanquake> yea I don't think that's spam either
 422020-05-03T01:45:23  *** hebasto has joined #bitcoin-core-dev
 432020-05-03T01:49:26  *** promag has quit IRC
 442020-05-03T02:07:44  *** dviola has joined #bitcoin-core-dev
 452020-05-03T02:11:35  <ossifrage> Should bitcoind be more aggressive in banning nodes that send high-hash or bad-diffbits?
 462020-05-03T02:18:59  *** dviola has quit IRC
 472020-05-03T02:20:00  *** dviola has joined #bitcoin-core-dev
 482020-05-03T02:28:14  *** mol_ has joined #bitcoin-core-dev
 492020-05-03T02:29:05  *** proofofk_ has quit IRC
 502020-05-03T02:29:40  *** proofofkeags has joined #bitcoin-core-dev
 512020-05-03T02:31:31  *** molz_ has quit IRC
 522020-05-03T02:34:29  *** proofofkeags has quit IRC
 532020-05-03T02:45:19  *** proofofkeags has joined #bitcoin-core-dev
 542020-05-03T02:49:54  *** proofofkeags has quit IRC
 552020-05-03T02:55:44  *** molz_ has joined #bitcoin-core-dev
 562020-05-03T02:58:16  *** proofofkeags has joined #bitcoin-core-dev
 572020-05-03T02:58:34  *** mol_ has quit IRC
 582020-05-03T02:59:29  *** proofofkeags has quit IRC
 592020-05-03T03:00:02  *** patjameson has quit IRC
 602020-05-03T03:00:05  *** proofofkeags has joined #bitcoin-core-dev
 612020-05-03T03:01:01  *** AaronvanW has joined #bitcoin-core-dev
 622020-05-03T03:04:56  *** proofofkeags has quit IRC
 632020-05-03T03:16:45  *** promag has joined #bitcoin-core-dev
 642020-05-03T03:16:48  *** promag has quit IRC
 652020-05-03T03:31:35  *** IGHOR has quit IRC
 662020-05-03T03:34:52  *** AaronvanW has quit IRC
 672020-05-03T03:41:29  *** IGHOR has joined #bitcoin-core-dev
 682020-05-03T03:46:11  *** bitcoin-git has joined #bitcoin-core-dev
 692020-05-03T03:46:11  <bitcoin-git> [bitcoin] achow101 opened pull request #18855: tests: feature_backwards_compatibility.py test downgrade after upgrade (master...test-backcompat-updowngrade) https://github.com/bitcoin/bitcoin/pull/18855
 702020-05-03T03:46:12  *** bitcoin-git has left #bitcoin-core-dev
 712020-05-03T03:48:54  *** OlyaAbram has joined #bitcoin-core-dev
 722020-05-03T03:51:21  *** OlyaAbram has quit IRC
 732020-05-03T03:56:22  *** keyboardsurfer has joined #bitcoin-core-dev
 742020-05-03T04:06:08  <luke-jr> ossifrage: no
 752020-05-03T04:06:13  <luke-jr> ossifrage: we're already too aggressive
 762020-05-03T04:06:48  <sipa> i don't see a good reason not to disconnect on invalid PoW, actually
 772020-05-03T04:07:22  <sipa> that's unambiguously invalid, for as long as the protocol existed
 782020-05-03T04:07:52  <luke-jr> disconnect != ban
 792020-05-03T04:08:18  <luke-jr> though having difficulty coming up with a scenario where banning would be bad
 802020-05-03T04:08:38  <luke-jr> oh, thought of one:
 812020-05-03T04:08:45  <luke-jr> a hardfork to reclaim always-zero bits in the header
 822020-05-03T04:09:27  <luke-jr> we'd of course reject the HF blocks, but we still want to share our best chain with such nodes
 832020-05-03T04:09:30  *** promag has joined #bitcoin-core-dev
 842020-05-03T04:09:48  <luke-jr> disconnection might be a problem there too
 852020-05-03T04:11:20  <sipa> i think banning is desired behavior between nodes on different sides of a HF
 862020-05-03T04:11:43  <sipa> to partition as quickly as possible, and not waste connection slots on either side with peers they disagree with
 872020-05-03T04:12:27  <luke-jr> partitioning is bad and should not happen with a HF
 882020-05-03T04:12:45  <sipa> ?!
 892020-05-03T04:12:56  <luke-jr> furthermore, it is at best a false sense of "security" since anyone can start a bridge later
 902020-05-03T04:13:09  <sipa> there is nothing to bridge
 912020-05-03T04:13:11  <luke-jr> sipa: I'm talking about real HFs, not scamcoins calling themselves HFs
 922020-05-03T04:13:33  <luke-jr> also HFs that *only* make invalid blocks valid
 932020-05-03T04:13:38  <luke-jr> in this example
 942020-05-03T04:13:54  <luke-jr> so the HF'd nodes would always prefer the pre-HF chain
 952020-05-03T04:14:00  <luke-jr> so long as they see it
 962020-05-03T04:14:07  *** promag has quit IRC
 972020-05-03T04:14:17  <luke-jr> and it's more work ofc
 982020-05-03T04:14:30  <sipa> as long as no such HF is defined, there is no need to accomodate ot
 992020-05-03T04:15:15  <sipa> right now, seeing a peer give you a block with invalid PoW is a sign that peer is, from your perspective, completely broken
1002020-05-03T04:15:38  <luke-jr> I'm not suggesting accomidation, just not going out of the way to disconnect it..
1012020-05-03T04:16:04  <luke-jr> outgoing connections, sure, since those are providing services to us primarily
1022020-05-03T04:16:12  <luke-jr> but inbound connections are primarily for the sake of the other node
1032020-05-03T04:44:04  <sipa> that's reasonable
1042020-05-03T04:58:40  *** surja795_ has quit IRC
1052020-05-03T04:58:59  *** surja795 has joined #bitcoin-core-dev
1062020-05-03T05:04:27  *** surja795 has quit IRC
1072020-05-03T05:31:50  *** AaronvanW has joined #bitcoin-core-dev
1082020-05-03T05:50:59  *** sonofhan has joined #bitcoin-core-dev
1092020-05-03T05:52:07  *** sonofhan has quit IRC
1102020-05-03T06:00:01  *** keyboardsurfer has quit IRC
1112020-05-03T06:04:02  *** AaronvanW has quit IRC
1122020-05-03T06:05:15  *** whythat has joined #bitcoin-core-dev
1132020-05-03T06:09:23  *** whythat has quit IRC
1142020-05-03T06:21:37  *** ski1 has joined #bitcoin-core-dev
1152020-05-03T07:15:57  *** AaronvanW has joined #bitcoin-core-dev
1162020-05-03T07:46:40  *** masteringbtc has joined #bitcoin-core-dev
1172020-05-03T08:19:35  *** AaronvanW has quit IRC
1182020-05-03T08:36:08  *** brakmic has joined #bitcoin-core-dev
1192020-05-03T08:37:57  *** brakmic_ has joined #bitcoin-core-dev
1202020-05-03T08:39:08  *** brakmic__ has joined #bitcoin-core-dev
1212020-05-03T08:40:44  *** brakmic has quit IRC
1222020-05-03T08:42:52  *** brakmic_ has quit IRC
1232020-05-03T09:00:02  *** ski1 has quit IRC
1242020-05-03T09:03:52  *** promag has joined #bitcoin-core-dev
1252020-05-03T09:08:35  *** proofofkeags has joined #bitcoin-core-dev
1262020-05-03T09:10:11  *** marcoagner has joined #bitcoin-core-dev
1272020-05-03T09:13:14  *** proofofkeags has quit IRC
1282020-05-03T09:21:00  *** afb has joined #bitcoin-core-dev
1292020-05-03T09:35:51  *** surja795 has joined #bitcoin-core-dev
1302020-05-03T09:39:59  *** surja795 has quit IRC
1312020-05-03T09:47:52  *** masteringbtc has quit IRC
1322020-05-03T09:49:32  *** AaronvanW has joined #bitcoin-core-dev
1332020-05-03T09:56:56  *** AaronvanW has quit IRC
1342020-05-03T10:09:21  *** AaronvanW has joined #bitcoin-core-dev
1352020-05-03T10:13:29  *** promag has quit IRC
1362020-05-03T10:29:46  *** emilengler has joined #bitcoin-core-dev
1372020-05-03T10:41:34  *** vasild_ has joined #bitcoin-core-dev
1382020-05-03T10:42:10  *** dmkathayat has quit IRC
1392020-05-03T10:44:43  *** vasild has quit IRC
1402020-05-03T10:44:44  *** vasild_ is now known as vasild
1412020-05-03T10:45:44  *** Dean_Guss has quit IRC
1422020-05-03T10:45:48  *** DeanWeen has joined #bitcoin-core-dev
1432020-05-03T11:02:34  *** surja795 has joined #bitcoin-core-dev
1442020-05-03T11:04:26  *** mdunnio has joined #bitcoin-core-dev
1452020-05-03T11:07:43  *** surja795 has quit IRC
1462020-05-03T11:08:59  *** mdunnio has quit IRC
1472020-05-03T11:16:28  *** Victor_sueca is now known as Victorsueca
1482020-05-03T11:18:03  *** Dean_Guss has joined #bitcoin-core-dev
1492020-05-03T11:19:23  *** DeanWeen has quit IRC
1502020-05-03T11:52:32  *** whythat has joined #bitcoin-core-dev
1512020-05-03T12:00:02  *** afb has quit IRC
1522020-05-03T12:04:21  *** jonatack has quit IRC
1532020-05-03T12:21:56  *** Nimda3 has joined #bitcoin-core-dev
1542020-05-03T12:30:02  *** Nimda3 has quit IRC
1552020-05-03T12:36:05  *** mol has joined #bitcoin-core-dev
1562020-05-03T12:38:38  *** molz_ has quit IRC
1572020-05-03T12:43:11  *** surja795 has joined #bitcoin-core-dev
1582020-05-03T12:49:03  *** mol has quit IRC
1592020-05-03T12:52:35  *** Nebraskka has quit IRC
1602020-05-03T12:55:55  *** gorthx has joined #bitcoin-core-dev
1612020-05-03T12:57:16  <yevaud> luke-jr: objectively, providing a connection to someone incoming who you know is on a different chain is a denial of service attack against them.
1622020-05-03T12:59:21  *** Highway61 has joined #bitcoin-core-dev
1632020-05-03T13:00:25  *** bitcoin-git has joined #bitcoin-core-dev
1642020-05-03T13:00:26  <bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/68ef9523d1bc...ddc0a600b38a
1652020-05-03T13:00:26  <bitcoin-git> bitcoin/master 2742c34 Harris: test: add factor option to adjust test timeouts
1662020-05-03T13:00:27  <bitcoin-git> bitcoin/master ddc0a60 MarcoFalke: Merge #18617: test: add factor option to adjust test timeouts
1672020-05-03T13:00:29  *** bitcoin-git has left #bitcoin-core-dev
1682020-05-03T13:00:51  *** bitcoin-git has joined #bitcoin-core-dev
1692020-05-03T13:00:51  <bitcoin-git> [bitcoin] MarcoFalke merged pull request #18617: test: add factor option to adjust test timeouts (master...test-timeouts-with-factor) https://github.com/bitcoin/bitcoin/pull/18617
1702020-05-03T13:00:52  *** bitcoin-git has left #bitcoin-core-dev
1712020-05-03T13:11:19  *** jonatack has joined #bitcoin-core-dev
1722020-05-03T13:14:12  *** wullon5 has quit IRC
1732020-05-03T13:17:11  *** masteringbtc has joined #bitcoin-core-dev
1742020-05-03T13:18:22  *** sudeep_ has joined #bitcoin-core-dev
1752020-05-03T13:18:56  *** amsudeep has quit IRC
1762020-05-03T13:25:23  *** vasild has quit IRC
1772020-05-03T13:31:23  *** whythat has quit IRC
1782020-05-03T13:34:09  *** mol has joined #bitcoin-core-dev
1792020-05-03T13:36:22  *** AnotherHaxker has joined #bitcoin-core-dev
1802020-05-03T13:37:25  *** vasild has joined #bitcoin-core-dev
1812020-05-03T13:39:16  *** whythat has joined #bitcoin-core-dev
1822020-05-03T13:41:20  *** AnotherHaxker has quit IRC
1832020-05-03T13:42:43  *** vasild has quit IRC
1842020-05-03T13:44:23  *** whythat has quit IRC
1852020-05-03T13:50:05  *** masteringbtc has quit IRC
1862020-05-03T13:56:53  *** mol_ has joined #bitcoin-core-dev
1872020-05-03T13:59:38  *** mol has quit IRC
1882020-05-03T14:03:45  *** belcher has joined #bitcoin-core-dev
1892020-05-03T14:06:23  *** jb55 has quit IRC
1902020-05-03T14:09:51  *** vasild has joined #bitcoin-core-dev
1912020-05-03T14:12:48  *** davec has quit IRC
1922020-05-03T14:13:25  *** davec has joined #bitcoin-core-dev
1932020-05-03T14:15:48  <luke-jr> yevaud: they're not on another chain in this scenario; it's *providing* a service, not denying one
1942020-05-03T14:18:49  *** whythat has joined #bitcoin-core-dev
1952020-05-03T14:19:39  *** bitcoin-git has joined #bitcoin-core-dev
1962020-05-03T14:19:39  <bitcoin-git> [bitcoin] vasild opened pull request #18857: build: avoid repetitions when enabling warnings in configure.ac (master...avoid_repetitions_in_configure.ac) https://github.com/bitcoin/bitcoin/pull/18857
1972020-05-03T14:19:50  *** bitcoin-git has left #bitcoin-core-dev
1982020-05-03T14:20:47  *** jb55 has joined #bitcoin-core-dev
1992020-05-03T14:23:23  *** whythat has quit IRC
2002020-05-03T14:28:57  *** whythat has joined #bitcoin-core-dev
2012020-05-03T14:32:12  *** bitcoin-git has joined #bitcoin-core-dev
2022020-05-03T14:32:13  <bitcoin-git> [bitcoin] MarcoFalke pushed 3 commits to master: https://github.com/bitcoin/bitcoin/compare/ddc0a600b38a...cce034b02800
2032020-05-03T14:32:13  <bitcoin-git> bitcoin/master fa8614a MarcoFalke: test: Fix intermittent p2p_segwit issue
2042020-05-03T14:32:14  <bitcoin-git> bitcoin/master fab7ee3 MarcoFalke: test: Fix p2p_leak intermittent issue
2052020-05-03T14:32:14  <bitcoin-git> bitcoin/master cce034b MarcoFalke: Merge #18839: test: Fix intermittent issues
2062020-05-03T14:32:16  *** bitcoin-git has left #bitcoin-core-dev
2072020-05-03T14:32:32  *** bitcoin-git has joined #bitcoin-core-dev
2082020-05-03T14:32:32  <bitcoin-git> [bitcoin] MarcoFalke merged pull request #18839: test: Fix intermittent issues (master...2004-qaSegwitNoIntermittentFail) https://github.com/bitcoin/bitcoin/pull/18839
2092020-05-03T14:32:33  *** bitcoin-git has left #bitcoin-core-dev
2102020-05-03T14:33:23  *** whythat has quit IRC
2112020-05-03T14:36:45  *** bitcoin-git has joined #bitcoin-core-dev
2122020-05-03T14:36:45  <bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/cce034b02800...2e6a16b968a9
2132020-05-03T14:36:46  <bitcoin-git> bitcoin/master 489ebfd Andrew Chow: tests: feature_backwards_compatibility.py test downgrade after upgrade
2142020-05-03T14:36:47  <bitcoin-git> bitcoin/master 2e6a16b MarcoFalke: Merge #18855: tests: feature_backwards_compatibility.py test downgrade aft...
2152020-05-03T14:36:48  *** bitcoin-git has left #bitcoin-core-dev
2162020-05-03T14:37:05  *** bitcoin-git has joined #bitcoin-core-dev
2172020-05-03T14:37:05  <bitcoin-git> [bitcoin] MarcoFalke merged pull request #18855: tests: feature_backwards_compatibility.py test downgrade after upgrade (master...test-backcompat-updowngrade) https://github.com/bitcoin/bitcoin/pull/18855
2182020-05-03T14:37:06  *** bitcoin-git has left #bitcoin-core-dev
2192020-05-03T14:42:32  *** mol has joined #bitcoin-core-dev
2202020-05-03T14:43:34  *** mol_ has quit IRC
2212020-05-03T14:55:15  *** whythat has joined #bitcoin-core-dev
2222020-05-03T15:00:01  *** gorthx has quit IRC
2232020-05-03T15:00:02  *** sdaftuar has quit IRC
2242020-05-03T15:00:24  *** sdaftuar has joined #bitcoin-core-dev
2252020-05-03T15:08:41  *** Guyver2 has joined #bitcoin-core-dev
2262020-05-03T15:11:04  *** whythat has quit IRC
2272020-05-03T15:20:13  *** jedi1 has joined #bitcoin-core-dev
2282020-05-03T15:20:33  <elichai2> jonasschnelli: Hi, I'm trying to review #18242, and I don't get something, it looks like you use AAD a lot in relation with chacha(ie encrypting the AAD), why is that? traditionally AAD is Additional Authenticated Data (meaning not encrypted) and also `CHACHA20_POLY1305_AEAD_AAD_LEN = 3` is that because the AAD is always the length which is capped at 24bits?
2292020-05-03T15:20:37  <gribble> https://github.com/bitcoin/bitcoin/issues/18242 | Add BIP324 encrypted p2p transport de-/serializer (only used in tests) by jonasschnelli · Pull Request #18242 · bitcoin/bitcoin · GitHub
2302020-05-03T15:24:18  *** promag has joined #bitcoin-core-dev
2312020-05-03T15:31:08  *** brakmic__ has quit IRC
2322020-05-03T15:31:26  *** brakmic has joined #bitcoin-core-dev
2332020-05-03T15:31:48  *** AaronvanW has quit IRC
2342020-05-03T15:39:23  *** vasild has quit IRC
2352020-05-03T15:41:08  <elichai2> I read the BIP again, and maybe I understand it now. you create 2 instances of chacha, `chacha_K1` and `chacha_K2` you encrypt the length of the data with `chacha_K1` and the data with chacha_K2`, but you use the encrypted `chacha_K1` length as the AAD of `chacha_K2`+Poly1305?
2362020-05-03T15:41:35  <elichai2> * I read the BIP again, and maybe I understand it now. you create 2 instances of chacha, `chacha_K1` and `chacha_K2` you encrypt the length of the data with `chacha_K1` and the data with `chacha_K2`, but you use the encrypted `chacha_K1` length as the AAD of `chacha_K2`+Poly1305?
2372020-05-03T15:45:55  *** promag has quit IRC
2382020-05-03T15:46:00  *** whythat has joined #bitcoin-core-dev
2392020-05-03T15:48:51  *** AaronvanW has joined #bitcoin-core-dev
2402020-05-03T15:49:59  *** mol has quit IRC
2412020-05-03T15:51:04  *** whythat has quit IRC
2422020-05-03T15:52:59  *** mol has joined #bitcoin-core-dev
2432020-05-03T16:21:20  *** whythat has joined #bitcoin-core-dev
2442020-05-03T16:25:43  *** whythat has quit IRC
2452020-05-03T16:28:42  *** promag has joined #bitcoin-core-dev
2462020-05-03T16:49:02  <jonasschnelli> elichai2: hi
2472020-05-03T16:49:29  <jonasschnelli> the AD is encrypted to keep the package length confidential
2482020-05-03T16:49:45  <jonasschnelli> it's based on the openssh chacha20poly1305 protocol
2492020-05-03T16:49:52  <jonasschnelli> https://github.com/openssh/openssh-portable/blob/master/PROTOCOL.chacha20poly1305
2502020-05-03T16:50:12  <jonasschnelli> and yes the AAD length is always 3 bytes.
2512020-05-03T16:50:23  <jonasschnelli> 23bits length plus 1 bit rekey flag
2522020-05-03T16:51:57  <jonasschnelli> with encrypted package length, it makes the stream pseudo-random... and it would allow to pad random data (to obfuscate size-based package detection, which is not in the scope of the BIP though)
2532020-05-03T16:56:19  *** whythat has joined #bitcoin-core-dev
2542020-05-03T16:57:22  *** promag has quit IRC
2552020-05-03T16:59:47  <elichai2> oh, openssh does this whole encrypt the length separately?
2562020-05-03T17:00:34  <elichai2> But do we really gain anything by encrypting the length? it will probably be really easy to see anyway
2572020-05-03T17:03:23  *** whythat has quit IRC
2582020-05-03T17:03:46  <elichai2> I haven't realized this is almost 100% openSSH protocol, I thought only their impl was used not the protocol itself, thanks!
2592020-05-03T17:06:52  *** promag has joined #bitcoin-core-dev
2602020-05-03T17:18:45  <jonasschnelli> elichai2: expect the reusing of the AAD keystream and the 3byte length, it's very much then openssh version
2612020-05-03T17:18:51  <jonasschnelli> *except
2622020-05-03T17:19:14  <elichai2> you mean limiting the length to constant size 3 bytes?
2632020-05-03T17:19:32  <jonasschnelli> that and mainly not throwing a way 60 bytes per AAD chacha round
2642020-05-03T17:30:43  *** whythat has joined #bitcoin-core-dev
2652020-05-03T17:34:19  *** justanotheruser has quit IRC
2662020-05-03T17:35:03  *** whythat has quit IRC
2672020-05-03T17:35:20  *** davterra has quit IRC
2682020-05-03T17:41:13  *** emilengler has quit IRC
2692020-05-03T17:49:31  *** justanotheruser has joined #bitcoin-core-dev
2702020-05-03T18:00:02  *** jedi1 has quit IRC
2712020-05-03T18:20:55  *** bitcoin-git has joined #bitcoin-core-dev
2722020-05-03T18:20:56  <bitcoin-git> [bitcoin] practicalswift opened pull request #18859: Remove CCoinsViewCache::GetValueIn(...) (master...GetValueIn) https://github.com/bitcoin/bitcoin/pull/18859
2732020-05-03T18:20:57  *** bitcoin-git has left #bitcoin-core-dev
2742020-05-03T18:21:10  *** paxed1 has joined #bitcoin-core-dev
2752020-05-03T18:26:09  *** vasild has joined #bitcoin-core-dev
2762020-05-03T18:28:25  *** kristapsk has joined #bitcoin-core-dev
2772020-05-03T18:30:39  *** Talkless has joined #bitcoin-core-dev
2782020-05-03T18:42:35  <vasild> sipa: wrt https://github.com/bitcoin/bitcoin/pull/18843#issuecomment-623158435 -- what would the current inline assembly do on a system that doesn't have that instruction? I guess it will cause SIGILL (illegal instruction) at runtime?
2792020-05-03T18:43:28  <sipa> yeah
2802020-05-03T18:43:53  <sipa> i would be very surprised that the compiler emits an rdrand instruction anywhere without invoking a corresponding intrinsic, though
2812020-05-03T18:44:35  <sipa> but that's specific to rdrand... e.g. the compiler will happily emit sse4 instructions for normal non-intrinsic c code when you compile with -msse4
2822020-05-03T18:45:37  <sipa> so in general, you always want to move code that needs certain hardware assumptions to a separate module that gets compiled with those flags, and only invoke its functions after testing at runtime the feature is available
2832020-05-03T18:46:59  <vasild> so there is room for improvement wrt the current inline assembly which assumes the instruction is present and if not would crash at runtime
2842020-05-03T18:47:51  <vasild> although I guess the instruction is available on all platforms where bitcoin core is being run, otherwise somebody would have reported the crash
2852020-05-03T18:48:42  <sipa> the current code is fine
2862020-05-03T18:49:37  <sipa> InitHardwareRand checks whether the instruction is available at runtime, and GetRdRand is only called if it reports true in g_rdrand_supported
2872020-05-03T18:49:59  <vasild> ah
2882020-05-03T18:50:15  <vasild> I was looking at GetRdRand() in isolation
2892020-05-03T18:50:42  <sipa> but if we'd compile the entirety of random.cpp with -mrdrnd, the compiler might also emit rdrand instructions in other functions (though i would be surprised if it did)
2902020-05-03T18:51:13  <sipa> there are compiler-specific extensions that let you compile just one function with different hardware assumptions, but i'm not sure they exist for all compilers we support
2912020-05-03T18:52:12  <vasild> "emit rdrand instructions in other functions" -- I don't get this, which other functions?
2922020-05-03T18:52:44  <sipa> literally anything
2932020-05-03T18:53:06  <vasild> if you have a function like `int f(int x) { return x * 2; }`?
2942020-05-03T18:53:21  <sipa> if you compile a module with -mrdrnd it means the compiler assumes all code in that module will be run on hardware that supports the rdrand instruction
2952020-05-03T18:53:34  <vasild> right
2962020-05-03T18:54:11  <sipa> as i said, i wouldn't expect a compiler to do that for rdrand... it's a very specific purpose instruction that likely wouldn't be used unless you explicitly invoke it using an intrinsic
2972020-05-03T18:54:34  <vasild> hmm, I see your point now
2982020-05-03T18:54:36  <sipa> but for other target flags it absolutely will
2992020-05-03T18:55:08  <sipa> this was the cause of #18456, btw
3002020-05-03T18:55:10  <gribble> https://github.com/bitcoin/bitcoin/issues/18456 | bitcoin-qt/bitcoind crashes upon startup of crypto/sha256_shani. Illegal instruction · Issue #18456 · bitcoin/bitcoin · GitHub
3012020-05-03T18:55:29  <sipa> sort of
3022020-05-03T18:55:32  <vasild> and even if we do a runtime check before using it, that instruction may be sneaked somewhere else where the code does not do a runtime check
3032020-05-03T18:55:40  <sipa> right
3042020-05-03T18:56:44  <elichai2>  #18456 was even worse imo, because it generated sse instructions in the global initialization function
3052020-05-03T18:56:46  <gribble> https://github.com/bitcoin/bitcoin/issues/18456 | bitcoin-qt/bitcoind crashes upon startup of crypto/sha256_shani. Illegal instruction · Issue #18456 · bitcoin/bitcoin · GitHub
3062020-05-03T18:57:00  <sipa> elichai2: which arguably we told it it was fine to do
3072020-05-03T18:57:41  <elichai2> right. but it's also a limitation of C and C++ that you can't always know what will be constantly evaluated and what will be in the global init
3082020-05-03T18:57:55  <sipa> you totally can; constexpr keyword
3092020-05-03T18:58:05  <sipa> only it doesn't work for vector registers :p
3102020-05-03T18:58:24  <vasild> sipa: ok, thanks for the patience with explanations. I catch up slowly, but otoh tend to forget quickly...
3112020-05-03T18:59:16  <sipa> np
3122020-05-03T18:59:24  <elichai2> hehe yeah, we have constexpr now
3132020-05-03T19:00:53  *** whythat has joined #bitcoin-core-dev
3142020-05-03T19:11:31  *** vasild_ has joined #bitcoin-core-dev
3152020-05-03T19:12:03  *** vasild has quit IRC
3162020-05-03T19:12:04  *** vasild_ is now known as vasild
3172020-05-03T19:15:13  *** Talkless has quit IRC
3182020-05-03T19:25:02  *** surja795 has quit IRC
3192020-05-03T19:40:23  *** whythat has quit IRC
3202020-05-03T19:43:35  *** surja795 has joined #bitcoin-core-dev
3212020-05-03T20:15:52  *** surja795 has quit IRC
3222020-05-03T20:16:28  *** whythat has joined #bitcoin-core-dev
3232020-05-03T20:21:38  *** promag has quit IRC
3242020-05-03T20:49:35  *** bitcoin-git has joined #bitcoin-core-dev
3252020-05-03T20:49:35  <bitcoin-git> [bitcoin] practicalswift closed pull request #18859: Remove CCoinsViewCache::GetValueIn(...) (master...GetValueIn) https://github.com/bitcoin/bitcoin/pull/18859
3262020-05-03T20:49:36  *** bitcoin-git has left #bitcoin-core-dev
3272020-05-03T20:50:00  *** surja795 has joined #bitcoin-core-dev
3282020-05-03T20:53:51  *** Guyver2_ has joined #bitcoin-core-dev
3292020-05-03T20:56:50  *** Guyver2 has quit IRC
3302020-05-03T20:57:19  *** promag has joined #bitcoin-core-dev
3312020-05-03T21:00:02  *** paxed1 has quit IRC
3322020-05-03T21:12:31  *** surja795 has quit IRC
3332020-05-03T21:17:01  *** bitcoin-git has joined #bitcoin-core-dev
3342020-05-03T21:17:01  <bitcoin-git> [bitcoin] MarcoFalke reopened pull request #18859: Remove CCoinsViewCache::GetValueIn(...) (master...GetValueIn) https://github.com/bitcoin/bitcoin/pull/18859
3352020-05-03T21:17:02  *** bitcoin-git has left #bitcoin-core-dev
3362020-05-03T21:22:16  *** DarTar has joined #bitcoin-core-dev
3372020-05-03T21:33:23  *** whythat has quit IRC
3382020-05-03T21:39:22  *** davterra has joined #bitcoin-core-dev
3392020-05-03T22:09:59  *** whythat has joined #bitcoin-core-dev
3402020-05-03T22:11:54  *** justanotheruser has quit IRC
3412020-05-03T22:14:23  *** whythat has quit IRC
3422020-05-03T22:28:21  *** justanotheruser has joined #bitcoin-core-dev
3432020-05-03T22:28:39  *** Guyver2_ has quit IRC
3442020-05-03T22:31:36  *** thomasb06 has joined #bitcoin-core-dev
3452020-05-03T22:32:56  <thomasb06> Is it possible to participate writing python test scripts without installing 200Gb?
3462020-05-03T22:36:43  <sipa> sure
3472020-05-03T22:37:03  <sipa> you don't need to run mainnet bitcoind to run or develop tests
3482020-05-03T22:41:14  <thomasb06> sipa: hey.
3492020-05-03T22:42:40  <thomasb06> would you have the link on the website, until now I only found instructions to run a node?
3502020-05-03T22:43:29  <sipa> i don't know what instructions you need
3512020-05-03T22:43:41  <sipa> you can submit pull requests without running anything
3522020-05-03T22:44:28  <thomasb06> well, that would be even better: my disk is 60Gb...
3532020-05-03T22:44:49  <sipa> that's no problem
3542020-05-03T22:44:54  <sipa> just don't run bitcoind
3552020-05-03T22:45:04  *** surja795 has joined #bitcoin-core-dev
3562020-05-03T22:45:11  <sipa> or you can run it in pruning mode which only needs a few GB
3572020-05-03T22:45:24  <sipa> but neither is relevant for the python tests
3582020-05-03T22:45:55  <thomasb06> cool... Promised I won't, even in pruning (I'm a student)
3592020-05-03T22:46:45  <thomasb06> how can I find what tests are needed?
3602020-05-03T22:47:59  <fanquake> thomasb06: If you want to start contributing, look at the "Good first issue" list: https://github.com/bitcoin/bitcoin/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22
3612020-05-03T22:50:17  *** surja795 has quit IRC
3622020-05-03T22:50:51  <thomasb06> fanquake: thanks. Any advice?
3632020-05-03T22:53:02  <fanquake> thomasb06: read https://github.com/bitcoin/bitcoin/blob/master/CONTRIBUTING.md if you haven't already
3642020-05-03T22:55:07  *** surja795_ has joined #bitcoin-core-dev
3652020-05-03T22:56:56  <thomasb06> I just started. Is there an issue or a category among those flaged "Good
3662020-05-03T22:56:56  <thomasb06> 	   first issue" I should privilege, or I can pick on randomly?
3672020-05-03T22:59:22  *** surja795_ has quit IRC
3682020-05-03T23:01:06  *** whythat has joined #bitcoin-core-dev
3692020-05-03T23:05:23  *** whythat has quit IRC
3702020-05-03T23:11:15  *** marcoagner has quit IRC
3712020-05-03T23:19:23  *** Dean_Guss has quit IRC
3722020-05-03T23:20:09  *** brakmic has quit IRC
3732020-05-03T23:28:06  *** Nebraskka has joined #bitcoin-core-dev
3742020-05-03T23:30:40  *** michaelfolkson has joined #bitcoin-core-dev
3752020-05-03T23:37:04  *** sosthene has quit IRC
3762020-05-03T23:38:54  *** Nebraskka has quit IRC
3772020-05-03T23:53:05  *** Nebraskka has joined #bitcoin-core-dev
3782020-05-03T23:58:51  *** thomasb06 has quit IRC