Page MenuHomePhabricator
Differential D16730

Return the RTT parameters in the block template
AcceptedPublic
Actions

Authored by Fabien on Fri, Sep 6, 14:04.

Details

Reviewers
roqqit
Group Reviewers
Restricted Project
Summary

The parameters are chosen so that pool can either:

  • Directly read the real time target value from the rtt.nexttarget field;
  • Or compute it locally by using the other rtt.* parameters.

Depends on D16729.

Test Plan
ninja all check-all

Diff Detail

Event Timeline

Fabien created this revision.Fri, Sep 6, 14:04
Herald added a reviewer: Restricted Project. · View Herald TranscriptFri, Sep 6, 14:04
Fabien requested review of this revision.Fri, Sep 6, 14:04
Harbormaster completed remote builds in B30208: Diff 49532.Fri, Sep 6, 14:27
roqqit accepted this revision.Fri, Sep 6, 17:20
This revision is now accepted and ready to land.Fri, Sep 6, 17:20
Fabien planned changes to this revision.Mon, Sep 9, 07:19
Fabien edited the summary of this revision. (Show Details)Wed, Sep 11, 14:26
Fabien updated this revision to Diff 49593.Wed, Sep 11, 14:26

Rebase, return all 4 header times in gbt, return the RTT as compact since the previous rationale no longer holds

This revision is now accepted and ready to land.Wed, Sep 11, 14:26
Harbormaster completed remote builds in B30256: Diff 49593.Wed, Sep 11, 15:16
roqqit accepted this revision.Wed, Sep 11, 21:22
roqqit added inline comments.
src/rpc/mining.cpp
1203–1206 ↗(On Diff #49593)

nit

bytesofman added a subscriber: bytesofman.Wed, Sep 11, 23:33
bytesofman added inline comments.
test/functional/abc_mining_rtt.py
187 ↗(On Diff #49593)
Fabien updated this revision to Diff 49608.Thu, Sep 12, 07:32

Feedback

Harbormaster completed remote builds in B30269: Diff 49608.Thu, Sep 12, 07:54
PiRK added a subscriber: PiRK.Thu, Sep 12, 13:50

lgtm with two nits

test/functional/abc_mining_rtt.py
115 ↗(On Diff #49608)

unnecessary list access

164 ↗(On Diff #49608)

It takes some digging to figure out that instantiating an AvaP2PInterface causes a block to be mined via gen_proof, if the reader does not already know it. Maybe worth a short comment in get_quorum

Fabien updated this revision to Diff 49610.Thu, Sep 12, 14:32

Remove unnecessary list access and comment on get_ava_p2p_interface mining a block

Harbormaster completed remote builds in B30271: Diff 49610.Thu, Sep 12, 15:07

Revision Contents
Changeset List

PathSize
src/
rpc/
72 lines
test/
functional/
198 lines

Diff 49610

View Options

src/rpc/mining.cpp

Loading...
View Options

test/functional/abc_mining_rtt.py

Loading...