-
Notifications
You must be signed in to change notification settings - Fork 215
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
Fix #1432 #1411, RTEMS semaphore return bugs and functional test update #1438
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
skliper
added
the
CCB:Ready
Pull request is ready for discussion at the Configuration Control Board (CCB)
label
Dec 29, 2023
skliper
force-pushed
the
fix1432-rtems_semaphore_returns
branch
from
December 29, 2023 21:13
4a51d37
to
46c65e6
Compare
skliper
changed the title
Fix #1432, RTEMS semaphore return bugs and functional test update
Fix #1432 #1411, RTEMS semaphore return bugs and functional test update
Dec 29, 2023
Format check fails on commit message due to solving multiple issues (check too restrictive) |
dzbaker
added
CCB:Approved
Indicates code review and approval by community CCB
and removed
CCB:Ready
Pull request is ready for discussion at the Configuration Control Board (CCB)
labels
Jan 4, 2024
2 tasks
dzbaker
added a commit
to nasa/cFS
that referenced
this pull request
Jan 5, 2024
*Combines:* sch_lab v2.5.0-rc4+dev83 osal v6.0.0-rc4+dev251 PSP v1.6.0-rc4+dev115 **Includes:** *cFS* - #725 *sch_lab* - nasa/sch_lab#164 - nasa/sch_lab#163 *osal* - nasa/osal#1438 *PSP* - nasa/PSP#380 Co-authored by: Joseph Hickey <jphickey@users.noreply.github.com> Co-authored by: Avi Weiss <thnkslprpt@users.noreply.github.com> Co-authored by: Jacob Hageman <skliper@users.noreply.github.com>
dzbaker
added a commit
to nasa/cFS
that referenced
this pull request
Jan 8, 2024
*Combines:* sch_lab v2.5.0-rc4+dev83 osal v6.0.0-rc4+dev251 PSP v1.6.0-rc4+dev115 **Includes:** *cFS* - #725 *sch_lab* - nasa/sch_lab#164 - nasa/sch_lab#163 *osal* - nasa/osal#1438 *PSP* - nasa/PSP#380 Co-authored by: Joseph Hickey <jphickey@users.noreply.github.com> Co-authored by: Avi Weiss <thnkslprpt@users.noreply.github.com> Co-authored by: Jacob Hageman <skliper@users.noreply.github.com>
skliper
force-pushed
the
fix1432-rtems_semaphore_returns
branch
from
January 10, 2024 17:33
1ebcf42
to
0f54ccc
Compare
dzbaker
added a commit
to nasa/cFS
that referenced
this pull request
Jan 10, 2024
*Combines:* sch_lab v2.5.0-rc4+dev83 osal v6.0.0-rc4+dev251 PSP v1.6.0-rc4+dev115 **Includes:** *cFS* - #725 *sch_lab* - nasa/sch_lab#164 - nasa/sch_lab#163 *osal* - nasa/osal#1438 *PSP* - nasa/PSP#380 Co-authored by: Joseph Hickey <jphickey@users.noreply.github.com> Co-authored by: Avi Weiss <thnkslprpt@users.noreply.github.com> Co-authored by: Jacob Hageman <skliper@users.noreply.github.com>
dzbaker
added a commit
to nasa/cFS
that referenced
this pull request
Jan 10, 2024
*Combines:* sch_lab v2.5.0-rc4+dev83 osal v6.0.0-rc4+dev251 PSP v1.6.0-rc4+dev115 **Includes:** *cFS* - #725 *sch_lab* - nasa/sch_lab#164 - nasa/sch_lab#163 *osal* - nasa/osal#1438 *PSP* - nasa/PSP#380 Co-authored by: Joseph Hickey <jphickey@users.noreply.github.com> Co-authored by: Avi Weiss <thnkslprpt@users.noreply.github.com> Co-authored by: Jacob Hageman <skliper@users.noreply.github.com>
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
OS_BinSemTimedWait_Impl
andOS_CountSemTimedWait_Impl
return incorrect status #1432Testing performed
Ran the updated functional tests on pclinux and leon3/RTEMS 5, passed
Expected behavior changes
Semaphore returns now match expected behavior (see #1432)
System(s) tested on
Additional context
None
Third party code
None
Contributor Info - All information REQUIRED for consideration of pull request
Ping @pepepr08