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

Support atomic CAS on BPF targets #29

Open
taiki-e opened this issue Aug 9, 2022 · 3 comments · May be fixed by #78
Open

Support atomic CAS on BPF targets #29

taiki-e opened this issue Aug 9, 2022 · 3 comments · May be fixed by #78
Labels
C-enhancement Category: A new feature or an improvement for an existing one O-bpf Target: BPF

Comments

@taiki-e
Copy link
Owner

taiki-e commented Aug 9, 2022

Another target that doesn't support atomic CAS in the standard library.

bpf*) ;; # TODO

This LLVM test indicates that many of the 64-bit and 32-bit atomic operations are already supported.

Refs: https://www.kernel.org/doc/html/latest/bpf/instruction-set.html#atomic-operations

@taiki-e taiki-e added C-enhancement Category: A new feature or an improvement for an existing one help wanted Extra attention is needed O-bpf Target: BPF labels Aug 9, 2022
@taiki-e
Copy link
Owner Author

taiki-e commented Dec 18, 2022

This will be resolved by rust-lang/rust#105708.

@taiki-e taiki-e removed the help wanted Extra attention is needed label Dec 21, 2022
@taiki-e
Copy link
Owner Author

taiki-e commented Jan 10, 2023

rust-lang/rust#105708 merged.

@taiki-e taiki-e closed this as completed Jan 10, 2023
@taiki-e
Copy link
Owner Author

taiki-e commented Jan 22, 2023

rust-lang/rust#105708 is not actually working properly.

@taiki-e taiki-e reopened this Jan 22, 2023
@taiki-e taiki-e linked a pull request Feb 23, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-enhancement Category: A new feature or an improvement for an existing one O-bpf Target: BPF
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant