Skip to content

Conversation

@mergify
Copy link

@mergify mergify bot commented Mar 18, 2024

Problem

Usage of sync locks on the quic server can cause thrashing when under lock contention

Summary of Changes

Use async lock

Fixes #


This is an automatic backport of pull request #293 done by [Mergify](https://mergify.com).

…ashing (#293)

Make the quic server connection table use an async lock, reducing lock contention

(cherry picked from commit 36e9765)
Copy link

@t-nelson t-nelson left a comment

Choose a reason for hiding this comment

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

:shipit:

@t-nelson t-nelson requested a review from lijunwangs March 22, 2024 16:56
Copy link

@lijunwangs lijunwangs left a comment

Choose a reason for hiding this comment

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

lgtm

@lijunwangs lijunwangs merged commit b01b0d0 into v1.18 Mar 22, 2024
@lijunwangs lijunwangs deleted the mergify/bp/v1.18/pr-293 branch March 22, 2024 23:32
anwayde pushed a commit to firedancer-io/agave that referenced this pull request Jul 23, 2024
…ing thrashing (backport of anza-xyz#293) (anza-xyz#300)

Make the quic server connection table use an async lock, reducing thrashing (anza-xyz#293)
OliverNChalk pushed a commit to OliverNChalk/agave that referenced this pull request Nov 11, 2025
* add rand xfer w/ memo attack

* fix verify, unit test
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.

4 participants