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

Update for suggested alignment enforcement pattern (nasa/cFE#1009) #56

Closed
skliper opened this issue Nov 17, 2020 · 0 comments · Fixed by #57 or #59
Closed

Update for suggested alignment enforcement pattern (nasa/cFE#1009) #56

skliper opened this issue Nov 17, 2020 · 0 comments · Fixed by #57 or #59
Labels
dependency enhancement New feature or request
Milestone
Caelum

Comments

@skliper
Copy link
Contributor

skliper commented Nov 17, 2020

Is your feature request related to a problem? Please describe.
See nasa/cFE#1009, inconsistent pattern

Describe the solution you'd like
Match suggestion in nasa/cFE#1009, use the "raw" message cmd/tlm types in definition.

Describe alternatives you've considered
None

Additional context
nasa/cFE#1009

Requester Info
Jacob Hageman - NASA/GSFC
This was referenced Nov 17, 2020
Merged
Closed
skliper added a commit to skliper/sch_lab that referenced this issue Nov 30, 2020
@skliper
Fix nasa#56, Apply message alignment pattern
d28ea42 
- Use CFE_MSG_CommandHeader_t and CFE_MSG_TelemetryHeader_t in
  command and telemetry type definitions
- Use CFE_SB_TransmitMsg_t to copy the command and telemetry
  into a CFE_SB_Buffer_t and send it where needed
- Avoids need to create send buffers within the app (or union
  the packet types with CFE_SB_Buffer_t)
- Eliminates references to CFE_SB_CmdHdr_t and CFE_SB_TlmHdr_t
  that formerly enforced alignment since these had potential
  to change the actual packet sizes
skliper added a commit to skliper/sch_lab that referenced this issue Nov 30, 2020
@skliper
Fix nasa#56, Apply message alignment pattern
70538ae 
- Use CFE_MSG_CommandHeader_t and CFE_MSG_TelemetryHeader_t in
  command and telemetry type definitions
- Use CFE_SB_TransmitMsg_t to copy the command and telemetry
  into a CFE_SB_Buffer_t and send it where needed
- Avoids need to create send buffers within the app (or union
  the packet types with CFE_SB_Buffer_t)
- Eliminates references to CFE_SB_CmdHdr_t and CFE_SB_TlmHdr_t
  that formerly enforced alignment since these had potential
  to change the actual packet sizes
skliper added a commit to skliper/sch_lab that referenced this issue Nov 30, 2020
@skliper
Fix nasa#56, Apply message alignment pattern
6bdb606 
- Use CFE_MSG_CommandHeader_t and CFE_MSG_TelemetryHeader_t in
  command and telemetry type definitions
- Use CFE_SB_TransmitMsg to copy the command and telemetry
  into a CFE_SB_Buffer_t and send it where needed
- Avoids need to create send buffers within the app (or union
  the packet types with CFE_SB_Buffer_t)
- Eliminates references to CFE_SB_CmdHdr_t and CFE_SB_TlmHdr_t
  that formerly enforced alignment since these had potential
  to change the actual packet sizes
@skliper skliper mentioned this issue Nov 30, 2020
Merged
skliper added a commit to skliper/sch_lab that referenced this issue Dec 8, 2020
@skliper
Fix nasa#56, Apply message alignment pattern
1b5ed8a 
- Use CFE_MSG_CommandHeader_t and CFE_MSG_TelemetryHeader_t in
  command and telemetry type definitions
- Use CFE_SB_TransmitMsg to copy the command and telemetry
  into a CFE_SB_Buffer_t and send it where needed
- Avoids need to create send buffers within the app (or union
  the packet types with CFE_SB_Buffer_t)
- Eliminates references to CFE_SB_CmdHdr_t and CFE_SB_TlmHdr_t
  that formerly enforced alignment since these had potential
  to change the actual packet sizes
@astrogeco astrogeco added this to the 2.5.0 milestone Dec 8, 2020
@astrogeco astrogeco added bug Something isn't working enhancement New feature or request dependency and removed bug Something isn't working labels Dec 8, 2020
astrogeco added a commit that referenced this issue Dec 8, 2020
@astrogeco
Merge pull request #57 from skliper/fix56-alignment-pattern
80fcb3a 
Fix #56, Apply alignment pattern
@astrogeco astrogeco mentioned this issue Dec 9, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
dependency enhancement New feature or request
Projects
None yet
Milestone
Caelum
2 participants
@skliper @astrogeco