feat: add policy-based worker and spin_lock data structure#3
Open
feat: add policy-based worker and spin_lock data structure#3
Conversation
- Refactor Worker to PolicyThreadWorker<Policy> template for flexible task policies - Add functional_task_policy and pointer_task_policy in types.hpp - Add spin_lock implementation with configurable spin count policy - Add comprehensive unit tests for spin_lock - Rename tests/ to test/ directory
Add three workflow files based on xcmixin's CI configuration: - ci.yml: Build and test on Ubuntu, macOS, Windows with clang/gcc - code-quality.yml: Clang format checking - release.yml: Automated release creation on tag push Also update .gitignore to allow .github directory.
…xamples - Add clang-format installation on all platforms (Linux, macOS, Windows) - Add ninja installation on macOS - Add conditional check for examples directory existence - Format code to pass clang-format check
- Add clang-format to macOS brew install command - Use llvm package on Windows instead of non-existent clang-format package
- Remove wait_until_unlock method from spin_lock, use lock() instead - Change memory_order from acquire to acq_rel for try_lock - Add notify_one() call in unlock() to wake waiters - Move task_t definition to public section in worker - Add type aliases (Worker, task_t, worker_ptr) in scheduler
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
WorkertoPolicyThreadWorker<Policy>template for flexible task policiesfunctional_task_policyandpointer_task_policyin types.hppspin_lockimplementation with configurable spin count policy inxcoroutine/structure/spin_lock.hpptest/spin_lock.ccTest plan
cmake --build . --config Debugctest -C Debug --output-on-failure./build/test/test_spin_lock.exe