Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Duplicate shares #222

Closed
SChernykh opened this issue Jan 7, 2023 · 0 comments
Closed

Duplicate shares #222

SChernykh opened this issue Jan 7, 2023 · 0 comments
Assignees
Labels
hardfork Hardfork is required to fix it

Comments

@SChernykh
Copy link
Owner

A high hashrate miner can potentially find 2 or more shares with the same ID but different nonce/extra_nonce values at exactly the same time. This will result in only one of the shares added to P2Pool's sidechain.

Intended behavior: 2nd, 3rd... shares found at the same time should have different IDs and they should all be added to P2Pool's sidechain as uncle shares.

The fix for this will require a P2Pool hardfork.

Tentative fix

Add miner controlled sidechain_extra_nonce to sidechain data so that every P2Pool node will be able to generate a unique sidechain_id for every connected worker.

Current workaround

Don't connect too much hashrate to a single P2Pool node. It's recommended to have less than 5-10 MH/s per node. If you have more than 5-10 MH/s, run multiple P2Pool nodes and split your hashrate between them equally.

@SChernykh SChernykh added the enhancement New feature or request label Jan 7, 2023
@SChernykh SChernykh self-assigned this Jan 7, 2023
@SChernykh SChernykh added hardfork Hardfork is required to fix it and removed enhancement New feature or request labels Jan 7, 2023
@SChernykh SChernykh changed the title Duplicate shares [hardfork required to fix] Duplicate shares Jan 7, 2023
@SChernykh SChernykh mentioned this issue Jan 7, 2023
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hardfork Hardfork is required to fix it
Projects
None yet
Development

No branches or pull requests

1 participant