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

Use fixed size types for addresses and address offsets in tables/tlm/cmd #26

Open
3 tasks done
astrogeco opened this issue May 2, 2022 · 1 comment · May be fixed by #86
Open
3 tasks done

Use fixed size types for addresses and address offsets in tables/tlm/cmd #26

astrogeco opened this issue May 2, 2022 · 1 comment · May be fixed by #86

Comments

@astrogeco
Copy link
Contributor

astrogeco commented May 2, 2022

Checklist (Please check before submitting)

  • I reviewed the Contributing Guide.
  • I reviewed the README file to see if the feature is in the major future work.
  • I performed a cursory search to see if the feature request is relevant, not redundant, nor in conflict with other tickets.

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

Changing telemetry and table sizes depending on platform (32 vs 64 bit) when using the cpuaddr type in structures.

Describe the solution you'd like

Use cfe_es_memaddress and cfe_es_offset types for tables and telemetry and define those types as 64 bits. If space is an issue, users can redefine these types as 32-bit.

From CCB:2022-04-27

EDIT: CFE_ES_MemAddress_t and CFE_ES_MemOffset_t are the actual types that should be used

Describe alternatives you've considered

Leave as is, redefine telemetry sizes based on platform in the "ground system"

Additional context
None

Requester Info
Gerardo E. Cruz-Ortiz, NASA

@skliper
Copy link
Contributor

skliper commented May 2, 2022

See also nasa/cFE#2093

@skliper skliper changed the title Use cfe_es_memaddress and cfe_es_offset types for tables and telemetry Use cfe_es_memaddress and cfe_es_offset types for tables/tlm/cmd Jun 6, 2022
@skliper skliper changed the title Use cfe_es_memaddress and cfe_es_offset types for tables/tlm/cmd Use fixed size types for addresses and address offsets in tables/tlm/cmd Jun 6, 2022
skliper pushed a commit to skliper/CS that referenced this issue Sep 9, 2022
thnkslprpt added a commit to thnkslprpt/CS that referenced this issue Jun 11, 2023
thnkslprpt added a commit to thnkslprpt/CS that referenced this issue Sep 5, 2023
thnkslprpt added a commit to thnkslprpt/CS that referenced this issue Sep 5, 2023
thnkslprpt added a commit to thnkslprpt/CS that referenced this issue Sep 5, 2023
thnkslprpt added a commit to thnkslprpt/CS that referenced this issue Mar 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants