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.

Sign up for GitHub

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

Jump to bottom

Implement PaddedSpinlock #190

Open
wants to merge 6 commits into
base: v1.10.2+RAI
Choose a base branch
from
Open

Implement PaddedSpinlock #190

wants to merge 6 commits into from

Conversation

kuszmaul
Copy link

@kuszmaul kuszmaul commented Sep 30, 2024 •
edited
Loading

PR Description

Create PaddedSpinLock that avoids false sharing. We have a new AbstractSpinLock with SpinLock <: AbstractSpinLock and PaddedSpinLock <: AbstractSpinLock.

Checklist

Requirements for merging:

  • I have opened an issue or PR upstream on JuliaLang/julia: SpinLocks suffer from false sharing
  • I have removed the port-to-* labels that don't apply.
  • I have opened a PR on raicode to test these changes:

@github-actions github-actions bot added port-to-v1.10 This change should apply to Julia v1.10 builds port-to-master This change should apply to all future Julia builds port-to-v1.9 This change should apply to Julia v1.9 builds labels Sep 30, 2024
@kuszmaul kuszmaul changed the title Bck padded spinlock Imlement PaddedSpinlock Sep 30, 2024
@kuszmaul kuszmaul changed the title Imlement PaddedSpinlock Implement PaddedSpinlock Sep 30, 2024
@kuszmaul kuszmaul changed the base branch from master to v1.10.2+RAI September 30, 2024 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
port-to-master This change should apply to all future Julia builds port-to-v1.9 This change should apply to Julia v1.9 builds port-to-v1.10 This change should apply to Julia v1.10 builds
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@kuszmaul