Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Provide UT_BSP_Lock/Unlock in CFE assert app #1595

Closed
jphickey opened this issue Jun 1, 2021 · 0 comments · Fixed by #1596 or #1619
Closed

Provide UT_BSP_Lock/Unlock in CFE assert app #1595

jphickey opened this issue Jun 1, 2021 · 0 comments · Fixed by #1596 or #1619
Assignees
@jphickey
Labels
enhancement
Milestone
Caelum

Comments

@jphickey
Copy link
Contributor

jphickey commented Jun 1, 2021

Is your feature request related to a problem? Please describe.
In nasa/osal#1065 it adds a UT_BSP_Lock / UT_BSP_Unlock routine to protect the UtAssert global data structures. However for cfe_assert app the utbsp.c file is not used, it provides its own routines. Therefore these routines need to be added.

Describe the solution you'd like
Add UT_BSP_Lock/Unlock wrappers in CFE assert (this already has a mutex).

Requester Info
Joseph Hickey, Vantage Systems, Inc.
@jphickey jphickey self-assigned this Jun 1, 2021
jphickey added a commit to jphickey/cFE that referenced this issue Jun 1, 2021
@jphickey
Fix nasa#1595, provide CFE assert lock/unlock
445e922 
Provide the UT_BSP_Lock/Unlock function to be compatible with
nasa/osal#1065.  The library no longer needs to be locked the
entire time a test runs. This also allows test programs to be
multi threaded.
@jphickey jphickey mentioned this issue Jun 1, 2021
Merged
astrogeco added a commit that referenced this issue Jun 10, 2021
@astrogeco
Merge pull request #1596 from jphickey/fix-1595-assert-lock
a2e1f3c 
Fix #1595, provide CFE assert lock/unlock
@astrogeco astrogeco mentioned this issue Jun 15, 2021
Merged
@skliper skliper added this to the 7.0.0 milestone Sep 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jphickey jphickey

Labels
enhancement
Projects
None yet
Milestone
Caelum
2 participants
@jphickey @skliper