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) #62

Closed
skliper opened this issue Nov 13, 2020 · 0 comments · Fixed by #63 or #65
Closed

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

skliper opened this issue Nov 13, 2020 · 0 comments · Fixed by #63 or #65
Labels
enhancement New feature or request
Milestone
Caelum

Comments

@skliper
Copy link
Contributor

skliper commented Nov 13, 2020 •
edited
Loading

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
skliper added a commit to skliper/ci_lab that referenced this issue Nov 16, 2020
@skliper
WIP nasa#62 - alignment updates started
06a4587
@skliper skliper changed the title Updated for suggested alignment enforcement pattern (nasa/cFE#1009) Update for suggested alignment enforcement pattern (nasa/cFE#1009) Nov 17, 2020
skliper added a commit to skliper/ci_lab that referenced this issue Nov 17, 2020
@skliper
WIP nasa#62 - alignment updates started
b00e0d5
skliper added a commit to skliper/ci_lab that referenced this issue Nov 18, 2020
@skliper
Fix nasa#62, Apply message alignment pattern
e008e3b 
- Use CFE_SB_Buffer_t for receiving and casting to command types
- Use CFE_MSG_CommandHeader_t and CFE_MSG_TelemetryHeader_t in
  comamnd 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
- No need to cast to CFE_MSG_Message_t anywhere since it's
  available in the CFE_SB_Buffer_t union
skliper added a commit to skliper/ci_lab that referenced this issue Nov 18, 2020
@skliper
Fix nasa#62, Apply message alignment pattern
7ae5e4d 
- Use CFE_SB_Buffer_t for receiving and casting to command types
- Use CFE_MSG_CommandHeader_t and CFE_MSG_TelemetryHeader_t in
  comamnd 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
- No need to cast to CFE_MSG_Message_t anywhere since it's
  available in the CFE_SB_Buffer_t union
@skliper skliper mentioned this issue Nov 18, 2020
Merged
skliper added a commit to skliper/ci_lab that referenced this issue Nov 18, 2020
@skliper
Fix nasa#62, Apply message alignment pattern
29c21e1 
- Use CFE_SB_Buffer_t for receiving and casting to command types
- 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
- No need to cast to CFE_MSG_Message_t anywhere since it's
  available in the CFE_SB_Buffer_t union
skliper added a commit to skliper/ci_lab that referenced this issue Nov 30, 2020
@skliper
Fix nasa#62, Removed hwords and replaced CFE_SB_RcvMsg
2416c2b
skliper added a commit to skliper/ci_lab that referenced this issue Nov 30, 2020
@skliper
Fix nasa#62, Removed hwords and replaced CFE_SB_RcvMsg
42b969b 
 - Replaced CFE_SB_RcvMsg with CFE_SB_ReceiveBuffer
skliper added a commit to skliper/ci_lab that referenced this issue Nov 30, 2020
@skliper
Fix nasa#62, Removed hwords and replaced deprecated elements
05c1424 
 - Replaced CFE_SB_RcvMsg with CFE_SB_ReceiveBuffer
 - Replaced CFE_MSG_Size_t with size_t
skliper added a commit to skliper/ci_lab that referenced this issue Nov 30, 2020
@skliper
Fix nasa#62, Apply message alignment pattern
e6d740a 
- Use CFE_SB_Buffer_t for receiving and casting to command types
- 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
- No need to cast to CFE_MSG_Message_t anywhere since it's
  available in the CFE_SB_Buffer_t union
skliper added a commit to skliper/ci_lab that referenced this issue Nov 30, 2020
@skliper
Fix nasa#62, Removed hwords and replaced deprecated elements
3ed636d 
 - Replaced CFE_SB_RcvMsg with CFE_SB_ReceiveBuffer
 - Replaced CFE_MSG_Size_t with size_t
@astrogeco astrogeco added this to the 2.5.0 milestone Dec 1, 2020
@astrogeco astrogeco added bug Something isn't working enhancement New feature or request and removed bug Something isn't working labels Dec 1, 2020
astrogeco added a commit that referenced this issue Dec 1, 2020
@astrogeco
Merge pull request #63 from skliper/fix62-alignment-pattern
dee470d 
Fix #62, Apply message alignment pattern
@astrogeco astrogeco mentioned this issue Dec 1, 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
enhancement New feature or request
Projects
None yet
Milestone
Caelum
2 participants
@skliper @astrogeco