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.

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

Resolve SEM_VALUE_MAX not defined test issue #632

Open
skliper opened this issue Oct 22, 2020 · 0 comments
Open

Resolve SEM_VALUE_MAX not defined test issue #632

skliper opened this issue Oct 22, 2020 · 0 comments

Comments

@skliper
Copy link
Contributor

skliper commented Oct 22, 2020

Is your feature request related to a problem? Please describe.
Was a FIXME:

/*
* FIXME: This test can only be done if the OS defines a specific "SEM_VALUE_MAX"
* The OSAL should define this for itself, but it currently does not.
* (This macro is not currently defined in RTEMS)
*/
#ifdef SEM_VALUE_MAX
res = OS_CountSemCreate(&count_sem_ids[0], "CountSem1", ((uint32)SEM_VALUE_MAX) + 1, 0);
if (res == OS_INVALID_SEM_VALUE)
UT_OS_TEST_RESULT(testDesc, UTASSERT_CASETYPE_PASS);
else
UT_OS_TEST_RESULT(testDesc, UTASSERT_CASETYPE_FAILURE);
#else
UT_OS_TEST_RESULT(testDesc, UTASSERT_CASETYPE_NA);
#endif

Describe the solution you'd like
Address/disposition as an issue.

Describe alternatives you've considered
None

Additional context
None

Requester Info
Jacob Hageman - NASA/GSFC

@skliper skliper added unit-test Tickets related to the OSAL unit testing (functional and/or coverage) enhancement and removed unit-test Tickets related to the OSAL unit testing (functional and/or coverage) labels Oct 22, 2020
@skliper skliper added this to the 6.0.0 milestone Oct 22, 2020
@skliper skliper removed this from the 6.0.0 milestone Dec 9, 2020
jphickey added a commit to jphickey/osal that referenced this issue Aug 10, 2022
Normally the CFE PSP uses/links with an OSAL BSP of the same name.
This removes the need to explicitly specify OSAL_SYSTEM_BSPTYPE
in the toolchain file, as it can be reliably inferred.
jphickey pushed a commit to jphickey/osal that referenced this issue Aug 10, 2022
Fix nasa#632, Infer OSAL_SYSTEM_BSPTYPE from CFE_SYSTEM_PSP_NAME
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant