Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

adds a generic implementation of Gossip{Read,Write}Lock #18559

Merged
merged 1 commit into from
Jul 10, 2021

Conversation

behzadnouri
Copy link
Contributor

Problem

Need to improve lock contention in gossip.
But current hard-coded Gossip{Read,Write}Lock structs makes it difficult to
experiment with other sync constructs (e.g. parking_lot).

Summary of Changes

  • added a generic implementation of Gossip{Read,Write}Lock.

@behzadnouri behzadnouri requested review from sakridge and carllin July 9, 2021 17:44
@codecov
Copy link

codecov bot commented Jul 9, 2021

Codecov Report

Merging #18559 (478eb34) into master (761de8b) will increase coverage by 0.0%.
The diff coverage is 94.1%.

@@           Coverage Diff           @@
##           master   #18559   +/-   ##
=======================================
  Coverage    82.7%    82.7%           
=======================================
  Files         442      442           
  Lines      123987   123975   -12     
=======================================
- Hits       102563   102560    -3     
+ Misses      21424    21415    -9     

@behzadnouri behzadnouri merged commit fd9c10c into solana-labs:master Jul 10, 2021
@behzadnouri behzadnouri deleted the gossip-timed-lock branch July 10, 2021 14:13
mergify bot pushed a commit that referenced this pull request Jul 10, 2021
mergify bot added a commit that referenced this pull request Jul 10, 2021
@behzadnouri behzadnouri linked an issue Jul 15, 2021 that may be closed by this pull request
@brooksprumo brooksprumo mentioned this pull request Aug 23, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

slow push_vote in ReplayStage
2 participants