BIP323: 24 nVersion bits for general purpose use#2116
Conversation
|
Addressed feedback. |
7a68443 to
c2f8667
Compare
murchandamus
left a comment
There was a problem hiding this comment.
Concept ACK
Just got that one outstanding comment that should be incorporated.
| Non-upgraded nodes will interpret the reserved bits of this proposal as signals for soft forks, and | ||
| may additionally activate the warning system for unknown soft forks. | ||
|
|
||
| At the time of writing no known soft forks are pending using any of 24 bits reserved in this BIP, |
There was a problem hiding this comment.
I saw AJ point out that there was a CTV deployment attempt that uses bit 5. I don’t think knowledge of that is widespread, but that should probably be addressed in the backwards compatibility section here.
There was a problem hiding this comment.
I don't think it makes sense to include a troll deployment in the backwards compatibility section of a serious BIP. As far as i know the activation parameters have not even been specified in a BIP in the first place.
There was a problem hiding this comment.
Still, the wording as it was phrased was at least technically inaccurate. I updated the phrasing to say that no soft forks "which are being actively signaled for" are using these bits. Which I believe is true, the CTV deployment attempt is not being signaled for.
|
ACK from Sv2 contributor here Sv2 introduces the notion of Header-only Mining (HOM) via Standard Channels, which freezes the HOM incurs in a few benefits for mining operations, namely:
in other words with HOM:
and these can become meaningful optimizations when applied at scale. the main challenge however, is the fact that currently, HOM is limited to a 280 TH/s ceiling (assuming
and the mining industry is already surpassing this limit on state-of-the-art devices that would relegate Sv2 Standard Channels to a second tier, throwing away the potential optimizations that HOM can bring when applied at scale on mining farms therefore, lifting the number of rollabe |
murchandamus
left a comment
There was a problem hiding this comment.
Thanks for the update, LGTM
573ea84 to
70253d2
Compare
|
Squashed without further changes. |
|
Let’s call this BIP323! You know the drill: could you please rename the file, update the preamble, add a README table entry, and put the number in the Proposed-Replacement header of BIP320? |
70253d2 to
042f96c
Compare
042f96c to
09c6179
Compare
|
Addressed feedback. |
murchandamus
left a comment
There was a problem hiding this comment.
Gave this another read. LGTM. @TheBlueMatt: Unless you’re waiting for something or have more planned work, I propose that this can be published later this week. WDYT?
|
I see no reason to hold this up on anything else. I believe @darosior was planning to "update" Bitcoin Core (which AFAIU just means changing some logging and RPC warnings). |
This is bitcoin/bitcoin#34779 |
Mail list discussion: https://groups.google.com/g/bitcoindev/c/fCfbi8hy-AE/m/g85UXI4qAAAJ