RTX: Fixed RTXv5 mutex owner list handling. #4646
Merged
Conversation
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
The prev-pointer of the running threads mutex list was not set when multiple mutexes are acquired at a time. This leads to a corrupted list if the mutexes are not freed in reversed order. Original commit for CMSIS_5: commit 729f6ab Author: Jonatan Antoni <[email protected]> Date: Wed Apr 19 15:38:43 2017 +0200
|
/morph test |
0xc0170
approved these changes
Jun 27, 2017
Result: FAILUREYour command has finished executing! Here's what you wrote!
OutputTest failed! |
|
Failures don't seem related /morph test |
|
restart uvisor |
|
retest uvisor |
Result: SUCCESSYour command has finished executing! Here's what you wrote!
OutputAll builds and test passed! |
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.
The prev-pointer of the running threads mutex list was not set when
multiple mutexes are acquired at a time. This leads to a corrupted list
if the mutexes are not freed in reversed order.
Original commit for CMSIS_5:
commit 729f6ab
Author: Jonatan Antoni [email protected]
Date: Wed Apr 19 15:38:43 2017 +0200
Fix ported from CMSIS_5 repo to address #4630
Status
READY
Close #4630
It's a point fix brought over from CMSIS repo, but we should still think about having some process to update CMSIS in regular intervals (eg each CMSIS patch release?)
@0xc0170 @c1728p9