Integration Candidate: 2020-10-07 #612
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
Define an interface to allow an app/PSP to be notified when state changes or other events occur at the OS level. Initially defined events are resource creation/deletion and task startup. The interface is easily extendable with more events as needed. This can be used to add platform-specific/nonstandard functions by putting the code inside the event handler at the PSP level.
Fix #540, add event callback framework
Correct issues involving recent OS_Milli2Ticks change. Note - for OS_TaskDelay it is important that this doesn't degrade into a no-op. So if OS_Milli2Ticks fails here, this returns the error but still also does a non-zero delay.
astrogeco
force-pushed
the
integration-candidate
branch
from
64611c6 to
2722525
Compare
astrogeco
added a commit
to nasa/cFS
that referenced
this pull request
Oct 6, 2020
Combines - nasa/cFE#936 - nasa/osal#612 *** * Add osal#541 * Add cfe#898 * Add cfe#925 * Add cfe#917 * Add cFE#899 * Add nasa/cFE#888 * Add CFE hotfix for documentation link warnings * Add FastTracked nasa/osal#616 * Set submodules to main for IC-2020-10-07 Co-authored-by: Joseph Hickey <joseph.p.hickey@nasa.gov>
jphickey
added a commit
to jphickey/osal
that referenced
this pull request
Aug 10, 2022
OSAL now sets these as UT_COVERAGE_COMPILE_FLAGS, UT_COVERAGE_LINK_FLAGS. Building and linking the UT executable needs a corresponding update.
jphickey
pushed a commit
to jphickey/osal
that referenced
this pull request
Aug 10, 2022
Fix nasa#612, Update coverage compile/link flag options
# for free
to join this conversation on GitHub.
Already have an account?
# 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.
Describe the contribution
Fix #540
Fix #615
Testing performed
Bundle CI: https://github.com/nasa/cFS/pull/145/checks
Expected behavior changes
PR #541 - Adds an event callback mechanism to certain state changes in OSAL. This allows the CFE PSP to be notified at these points, and therefore it can add platform-specific functionality. This can, for instance, set the task name as requested in #532 or set the processor affinity in a multi-core setup.
PR #616 - Correct issues involving recent OS_Milli2Ticks change.
System(s) tested on
Ubuntu on Travis CI
Additional context
Part of nasa/cFS#145
Third party code
None
Contributor Info - All information REQUIRED for consideration of pull request
@jphickey