-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Patch crossbeam-epoch to avoid overhead #26555
Conversation
650666b
to
597809a
Compare
@@ -968,8 +968,7 @@ dependencies = [ | |||
[[package]] | |||
name = "crossbeam-epoch" | |||
version = "0.9.5" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
$ cat /home/ryoqun/.cargo/registry/src/github.com-1ecc6299db9ec823/crossbeam-epoch-0.9.5/.cargo_vcs_info.json
{
"git": {
"sha1": "6d4cdd4daf9a897deef6cde9569f2fbf12c29bc5"
}
}
...
ryoqun@ubuqun:~/work/crossbeam/crossbeam-epoch$ git diff --patch-with-stat --no-index . /home/ryoqun/.cargo/registry/src/github.com-1ecc6299db9ec823/crossbeam-epoch-0.9.5/
...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you can verify that there is barely minimum modifications by comparing this published commit rev with the parent commit rev of my patch commit: https://github.com/solana-labs/solana/pull/26555/files#r918444566
Cargo.toml
Outdated
@@ -91,5 +91,6 @@ exclude = [ | |||
"programs/bpf", | |||
] | |||
|
|||
# This prevents a Travis CI error when building for Windows. | |||
resolver = "2" | |||
# for details, see https://github.com/ryoqun/crossbeam/commit/fd279d707025f0e60951e429bf778b4813d1b6bf |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
for convenience: anza-xyz/crossbeam@fd279d7
I've run a validator on MNB for multiple days w/o any issues (w/ this change on top of v1.10.32) |
b2fb4e5
to
58aa506
Compare
a few hygiene are done:
|
58aa506
to
6f6636d
Compare
96322d7
to
17369f3
Compare
(cherry picked from commit ad3e10f)
(cherry picked from commit ad3e10f)
Patch crossbeam-epoch to avoid overhead (#26555) (cherry picked from commit ad3e10f) Co-authored-by: Ryo Onodera <[email protected]>
Problem
fixes #22603
our rayon usage might be peculiar enough to warrant patching...:
crossbeam-rs/crossbeam#852 (comment)
Summary of Changes
Fixes #