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

Loom model checking #133

Closed
loyd opened this issue Mar 24, 2024 · 4 comments
Closed

Loom model checking #133

loyd opened this issue Mar 24, 2024 · 4 comments

Comments

@loyd
Copy link

loyd commented Mar 24, 2024

I understand this requires a vast amount of work. However, at least an issue should exist.

Except for using loom's primitives for testing, I think it should use them under --cfg scc_loom (in the same way as crossbeam_loom exists) to support testing in implementation of structures based on scc-ebr implementation.

@loyd
Copy link
Author

loyd commented Mar 24, 2024

@wvwwvwwv, what does "Formally verified EBR" in README mean? Is it about https://github.com/wvwwvwwv/scalable-concurrent-containers/blob/main/src/tests/model.rs?

@wvwwvwwv
Copy link
Owner

I also know that the loom tests in the crate is not ideal. At my previous workplace, I used a generalised atomic interface to avoid any use of compile-time flags, so I may try this approach here as well.

Please don't pay much attention to Formally verified EBR part :-) It just means that I developed ebr along with those loom tests; doesn't mean that I used Coq or the likes to formally prove the algorithm. Note that the use of every memory barrier/fence was justified internally in my previous company, though, I cannot disclose any part of it.

@shamilsan
Copy link

Let me try to implement some loom-based tests as a sponsored work backed by @loyd.

shamilsan added a commit to shamilsan/scalable-concurrent-containers that referenced this issue Mar 26, 2024
shamilsan added a commit to shamilsan/scalable-concurrent-containers that referenced this issue Mar 26, 2024
@loyd
Copy link
Author

loyd commented Apr 21, 2024

I close this issue in favor of wvwwvwwv/scalable-delayed-dealloc#1 due to #131

@loyd loyd closed this as completed Apr 21, 2024
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

No branches or pull requests

3 participants