Skip to content

Feat/fakepow delaysealer#180

Merged
meowsbits merged 7 commits into
masterfrom
feat/fakepow-delaysealer
Dec 1, 2020
Merged

Feat/fakepow delaysealer#180
meowsbits merged 7 commits into
masterfrom
feat/fakepow-delaysealer

Conversation

@meowsbits
Copy link
Copy Markdown
Contributor

@meowsbits meowsbits commented Sep 12, 2020

Installs a configurable fake delay for the PoW ModeFake sealer.

Sealing time determined by a Poisson distribution and governed by ethash.threads=<mean seconds (lambda)>.

@meowsbits meowsbits requested a review from iquidus September 12, 2020 12:57
@meowsbits meowsbits force-pushed the feat/fakepow-delaysealer branch from 5e060d4 to 9a62f37 Compare September 12, 2020 12:58
@meowsbits
Copy link
Copy Markdown
Contributor Author

meowsbits commented Sep 12, 2020 •
edited
Loading 

TestSealFakeDelay: sealer_test.go:49: ethash.threads,lamda= 3 [1 2 2 2 2 2 2 3 3 3 3 3 3 4 4 4 4 4 5 6]
TestSealFakeDelay: sealer_test.go:49: ethash.threads,lamda=10 [5 6 6 7 8 8 8 9 9 10 10 10 10 11 11 11 12 13 14 16]
TestSealFakeDelay: sealer_test.go:49: ethash.threads,lamda=19 [12 13 13 14 14 14 15 16 17 19 20 20 21 22 23 25 25 25 26 26]
TestSealFakeDelay: sealer_test.go:49: ethash.threads,lamda=33 [19 22 24 26 27 27 28 28 28 31 34 36 36 37 38 39 40 40 40 47]

0177c08

@meowsbits meowsbits mentioned this pull request Sep 13, 2020
Closed
@meowsbits
Copy link
Copy Markdown
Contributor Author

meowsbits commented Oct 14, 2020

Closing this for now since we have no immediate intention of merging the feature to master.

Rel #188, though, so if pursuing that feature, keep this feature in mind too.

@meowsbits meowsbits closed this Oct 14, 2020
meowsbits
meowsbits commented Nov 24, 2020
View reviewed changes
Comment thread consensus/ethash/sealer.go
@meowsbits meowsbits mentioned this pull request Nov 24, 2020
Closed
@ziogaschr ziogaschr mentioned this pull request Nov 27, 2020
Merged
meowsbits and others added 4 commits November 27, 2020 21:52
@meowsbits @ziogaschr
ethash,go.mod,go.sum: init implement fakedelay for ethash pow faker s…
8dcbf4d 
…ealer

Signed-off-by: meows <b5c6@protonmail.com>
@meowsbits @ziogaschr
utils: set ethash config faker mode if FakePoW flag is set
7525b58 
Without this, the only fake part of --fakepow
is the VERIFICATION of blocks, not the sealing.

This makes the sealer fake with --fakepow as well.

Signed-off-by: meows <b5c6@protonmail.com>
@meowsbits @ziogaschr
ethash: improve test cases
7fa737e 
Signed-off-by: meows <b5c6@protonmail.com>
@ziogaschr
consensus/ethash: extract delaysealer to its own mode ModePoissonFake
40564ff
@ziogaschr ziogaschr reopened this Dec 1, 2020
@ziogaschr
Copy link
Copy Markdown
Contributor

ziogaschr commented Dec 1, 2020

This PR is going to be used for #243, so we are re-opening it

@ziogaschr ziogaschr force-pushed the feat/fakepow-delaysealer branch from 0177c08 to 40564ff Compare December 1, 2020 16:32
@meowsbits
ethash,go.mod: use pseudo-random source for fakePoissonDelay
e695a47 
Date: 2020-12-01 12:50:36-06:00
Signed-off-by: meows <b5c6@protonmail.com>

Conflicts:
      go.mod
@meowsbits
go.mod,go.sum: run 'go mod tidy'
af13eb2 
Date: 2020-12-01 12:52:44-06:00
Signed-off-by: meows <b5c6@protonmail.com>
meowsbits
meowsbits commented Dec 1, 2020
View reviewed changes
Comment thread cmd/utils/flags.go Outdated
}

func setEthash(ctx *cli.Context, cfg *eth.Config) {
if ctx.GlobalBool(FakePoWFlag.Name) {
Copy link
Copy Markdown
Contributor Author

@meowsbits meowsbits Dec 1, 2020 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ziogaschr What do you think about including this in this PR?

Although it does indeed seem like something worth doing, I'm inclined to think it should be it's own PR. Maybe we can also think about extending through other possible variations, like maybe --fakepow.poisson...

Thoughts?

Copy link
Copy Markdown
Contributor

@ziogaschr ziogaschr Dec 1, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indeed I would prefer this removed from this PR. It might fit well into #243.
I will remove it now

Copy link
Copy Markdown
Contributor

@ziogaschr ziogaschr Dec 1, 2020 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

About the --fakepow.poisson, I was thinking something like --ethash.mode or --ethash.powmode enum string flag with values fake, fullFake, poissonFake.

Also, if we add it as a generic cli command and not under the --dev.* namespace, I would suggest adding something more fancy like --ethash.[pow]modedangerously.

Not sure how the fakepow will be used, outside of the geth [importChain, exportChain, copyDb, dump, inspect] cmds context, but for now it might make sense to only activating under the --dev flag and not on real chains.

@meowsbits meowsbits requested a review from ziogaschr December 1, 2020 19:02
@meowsbits meowsbits merged commit 774bf52 into master Dec 1, 2020
@meowsbits meowsbits deleted the feat/fakepow-delaysealer branch December 1, 2020 22:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@iquidus iquidus Awaiting requested review from iquidus

1 more reviewer

@ziogaschr ziogaschr ziogaschr approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@meowsbits @ziogaschr