Fix #1252, condition variable API for OSAL #1278
Merged
Conversation
This file contains 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
This initial commit defines the conceptual condition variable API. This is not implemented yet for any OS, but should be implementable on POSIX, RTEMS, and VxWorks 7.x. Older VxWorks releases may not have the underlying support to implement this.
Adds a complete implementation of condition variables for POSIX. Includes coverage and unit tests. RTEMS and VxWorks will temporarily use a "no-condvar" placeholder until an implementation for those systems is added.
e69cd92
to
3cb1cf8
Compare
|
CCB 8 September 2022: @jphickey will write issues for extending to additional platforms. |
2 tasks
dzbaker
added a commit
to nasa/cFS
that referenced
this pull request
Sep 8, 2022
*Combines:* osal v7.0.0-rc4+dev108 **Includes:** - nasa/osal#1278, condition variable API for OSAL Co-authored by: Joseph Hickey <[email protected]>
dzbaker
added a commit
to nasa/cFS
that referenced
this pull request
Sep 8, 2022
*Combines:* osal v7.0.0-rc4+dev108 **Includes:** - nasa/osal#1278, condition variable API for OSAL Co-authored by: Joseph Hickey <[email protected]>
This was referenced Sep 12, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Checklist (Please check before submitting)
Describe the contribution
Implementation of condition variable API for OSAL
Currently implemented only for POSIX, RTEMS can be added with future commit.
Testing performed
Coverage and functional test of condvar API is included
Expected behavior changes
Adds condition variable API
System(s) tested on
Ubuntu 22.04
Additional context
The BP/DTN implementation (https://github.com/nasa/bplib) makes use of condition variables to signal between worker tasks. In order to support running this library natively, OSAL would need to provide this set of functionality.
Contributor Info - All information REQUIRED for consideration of pull request
Joseph Hickey, Vantage Systems, Inc.