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

Tip lock handling in tier two. #1656

Closed
wants to merge 4 commits into from

Conversation

furszy
Copy link

@furszy furszy commented May 31, 2020

This is coming from #710 + adapted to our sources.

Can be taken as a good performance improvement for 4.2 over the tier two cs_main lock usage (without make radical changes).
5.0 will need to have major changes in this area.

  • First commit removes an unneeded cs_main lock.
  • Second commit add some guards to chainActive over the tier two sources.
  • Third commit introduces a new interface and a CBlockIndex* currentTip member on each of the tier two objects to not have to re request the tip, locking cs_main, on every validation/action.

-- Needs proper testing --

furszy added 3 commits May 31, 2020 11:55
…o tip updates and trigger updates in payments and budgets. Should significantly reduce number of references to chainActive.Tip().

Update miner_test for mnpayments
@furszy furszy self-assigned this May 31, 2020
…rrentBlockIndex is an overkill here, we should just make sure that cs_main lock is already held for us.
@furszy furszy closed this Jun 1, 2020
@furszy furszy deleted the 2020_MNs_Fix_locks_Pack branch November 29, 2022 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant