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 #2627, update check, docstr for max table sz #2632

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

irowebbn
Copy link
Contributor

Checklist (Please check before submitting)

Describe the contribution
A clear and concise description of what the contribution is.

Testing performed
Steps taken to test the contribution:

  1. Normal build and compile steps

Expected behavior changes
A clear and concise description of how this contribution will change behavior and level of impact.

  • Allows a wider range of combinations for table size and table memory.

System(s) tested on

  • Versions: cFE trunk

Contributor Info - All information REQUIRED for consideration of pull request
Isaac Rowe NASA JSC, JETS II contract

@irowebbn
Copy link
Contributor Author

I understand that there is a new method to configure cFE that obviates the need for the headers. I was unable to find documentation on how to setup and run that. This PR does not address any necessary changes that would be needed for that method.

@irowebbn
Copy link
Contributor Author

Also, this change implies that MAX_SIMULTANEOUS_LOADS can be set to 1. However, doing so breaks some old unit tests. I can address that in a follow-up PR.

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 this pull request may close these issues.

TBL verify check for CFE_PLATFORM_TBL_MAX_SNGL_TABLE_SIZE does not match documented description
1 participant