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

Fix #858, use osal id typedef #868

Merged

Conversation

jphickey
Copy link
Contributor

@jphickey jphickey commented Sep 8, 2020

Describe the contribution
Scrub all CFE references/uses of OSAL IDs to use the proper osal_id_t type.

Any place that an OSAL ID is stored in memory or passed in an API call are changed to the osal_id_t type, rather than uint32. Conversions between this and other types (e.g. bare integer) is done using the OSAL-supplied conversion helpers.

Fixes #858

Testing performed
Build and sanity check CFE
Run all unit tests
Also specifically spot check various CFE commands that save data to file (e.g. perf log, query all tasks, etc) to confirm these files are being properly created - as these previously used the int32 type to store the file handle and necessitated a bigger change.

Expected behavior changes
No impact to behavior. Since the osal_id_t is initially a typedef to uint32, this should be effectively no change except to future proof.

System(s) tested on
Ubuntu 20.04

Additional context
Depends on nasa/osal#568 which provides the typedef and conversion helpers (currently in IC)

Contributor Info - All information REQUIRED for consideration of pull request
Joseph Hickey, Vantage Systems, Inc.

Use the OSAL-supplied typedef `osal_id_t` to store OSAL IDs,
along with OSAL-supplied conversion/cast routines when interfacing
this value with other modules.
@jphickey jphickey added CCB:Ready Ready for discussion at the Configuration Control Board (CCB) dependency labels Sep 8, 2020
@jphickey jphickey linked an issue Sep 9, 2020 that may be closed by this pull request
@astrogeco
Copy link
Contributor

CCB 2020-09-09 APPROVED, probably want to look at using Open/Create in a separate pull request.

@yammajamma yammajamma added IC-20200909 and removed CCB:Ready Ready for discussion at the Configuration Control Board (CCB) labels Sep 10, 2020
@yammajamma yammajamma merged commit bc48dc0 into nasa:integration-candidate Sep 10, 2020
@jphickey jphickey deleted the fix-858-osal-id-type branch September 29, 2020 21:52
@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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Scrub CFE to use correct type for OSAL IDs
4 participants