Page MenuHomePhabricator

Remove unknown version warning from UpdateTip
ClosedPublic

Authored by markblundeberg on Apr 9 2019, 01:46.

Details

Summary

Since Antminer S9s now are able to do overt asicboost, there are many unknown
versions on mainnet headers. This is triggering warnings in the debug.log and it
is not apparent that this is meaningful any longer. Additionally, the BIP9 code
has been removed for activating any softforks based on the version bits anyways.

Depends on D3381

Test Plan
make check

Diff Detail

Repository
rABC Bitcoin ABC
Branch
remove-warning2 (branched from master)
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 5420
Build 8902: Bitcoin ABC Buildbot (legacy)
Build 8901: arc lint + arc unit

Event Timeline

Owners added a reviewer: Restricted Owners Package.Apr 9 2019, 01:46

This should in principle work on master now, once rebased...

markblundeberg edited the summary of this revision. (Show Details)

rebased (part of this is now already done, in D3181)

src/validation.cpp
2144 ↗(On Diff #9573)

note that VERSIONBITS_LAST_OLD_BLOCK_VERSION, like other constants in versionbits.h, is unused now.

(only VERSIONBITS_TOP_BITS remains used)

jasonbcox requested changes to this revision.Jun 21 2019, 22:06

Depends on itself?

This revision now requires changes to proceed.Jun 21 2019, 22:06

Note backports are basically caught up on this part of the code, except weak overlap with the incomplete parts of D1972, and some linting-related stuff in #12891 (which it looks like will we cannot take as-is due to code style differences).

This revision is now accepted and ready to land.Jun 24 2019, 18:32