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

Maximum Frame length wrt SDLS added fields #62

Closed
dccutrig opened this issue Jan 20, 2022 · 5 comments · Fixed by #87
Closed

Maximum Frame length wrt SDLS added fields #62

dccutrig opened this issue Jan 20, 2022 · 5 comments · Fixed by #87

Comments

@dccutrig
Copy link
Contributor

Open as a reminder:

Maximum frame length is 1024 octets for a TC Frame. Should it be our responsibility to ensure that we do not violate this CCSDS Spec by adding SDLS to a frame?

e.g. if a frame is 1020 bytes and we add 6 bytes by adding SDLS headers, we've violated maximum TC length.

Thought: Could be checked in the update TF length function, if nothing else

@IbraheemYSaleh
Copy link
Contributor

IbraheemYSaleh commented Feb 5, 2022

If we do add this, I think the proper place it should be added is in the managed parameters for each GVCID. We need to allow users to specify variable maximums per GVCID (which can sometimes be less than 1024 bytes).

Some channels may allow larger frames, while some might be better off with short frames (EG large file transfers to deep space missions) -- it's really not a project-wide configuration that can be hardcoded to 1024 bytes.

@dccutrig
Copy link
Contributor Author

For reference:

image

@dccutrig
Copy link
Contributor Author

dccutrig commented Mar 9, 2022

To follow-up on this, do we want to impose framing restrictions outside of the absolute TC maximum, or leave this to the underlying ground station to generally take care of? (e.g. if it is operating on some assumption that the max length should be less than 1024?)

@IbraheemYSaleh
Copy link
Contributor

image

@IbraheemYSaleh
Copy link
Contributor

Maximum Transfer Frame Length should be a managed parameter ... It needs to be added to the Crypto_Config_Add_Gvcid_Managed_Parameter function. https://github.jpl.nasa.gov/ASEC/AMMOS-CryptoLib/issues/21

@dccutrig dccutrig linked a pull request Mar 14, 2022 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants