-
Notifications
You must be signed in to change notification settings - Fork 57
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 #111, do not assume a specific core name #207
Fix #111, do not assume a specific core name #207
Conversation
Improve the module ID lookup when getting the CFE core text segment info. - Ideally get the ID directly from what was loaded by startCfeCore - As a fallback use the actual CFE core name from the configdata Do not use a hardcoded name.
CCB: 2020-10-14: Approved (pending test). Requesting ticket to add a functional test (not implemented is OK on platforms that don't support it). |
Yes - I was able to run this on the MCP750 and confirm that I'm getting a valid result from @astrogeco this should be OK to merge. |
Note - the part I am not able to test is the one where |
Copy, fine by me. |
* Add nasa/PSP#207 * Add nasa/cFE#946 * Update CFE to fix startup issue * Add nasa/osal#626 Co-authored-by: Joseph Hickey <[email protected]>
Describe the contribution
Improve the module ID lookup when getting the CFE core text segment info.
Do not use a hardcoded name.
Fixes #111
Testing performed
Build and run unit tests
Expected behavior changes
VxWorks PSP should use the real module name, not assume
cfe-core.o
when getting text segment infoSystem(s) tested on
Ubuntu 20.04
Contributor Info - All information REQUIRED for consideration of pull request
Joseph Hickey, Vantage Systems, Inc.