Skip to content

Conversation

@mafoste3
Copy link

@mafoste3 mafoste3 commented Sep 29, 2025

Checklist (Please check before submitting)

Describe the contribution
If the critical data store (CDS) memory is not used to recover the memory dwell tables and the tables are loaded directly from the filesystem, the MD app will pass initialization with table verification however will fail to process the tables when memory dwell is started with the MD_START_DWELL_CC command.

This fix adds obtaining the table address and copying the table(s) to the MD App Global Data structure if the tables are not restored from the CDS.

Testing performed
Steps taken to test the contribution:

  1. Add 2-3 valid memory dwell entries for MD Dwell table md_dw01.c
  2. Add MD app to targets.cmake dynamic app list
  3. Add MD app to CFE startup script
  4. Add MD app MD_HK_TLM_MID to to_lab app for telemetry output
  5. Add MD app MD_SEND_HK_MID @1Hz to sch_lab table to schedule MD app HK telemetry
  6. Add MD app MD_WAKEUP_MID @1Hz to sch_lab table to schedule MD app memory dwell wakeup
  7. Build cFS with native BSP
  8. Run cFS
  9. Send MD_START_DWELL_CC Command (TableMask=0x1, Padding=0x0)

Expected behavior changes

  • The MD app will now process the non CDS loaded table entries vs erroring that no processing will occur

System(s) tested on

  • Hardware: VirtualBox
  • OS: Ubuntu 24.04, GCC 13.3
  • Versions: cFS equulues-rc1+dev31

Additional context

  • This fix does remove the automatic setting of MD_DWELL_STREAM_DISABLED for a non CDS recovered table. This allows a user to utilize the Enabled flag that would otherwise be overwritten, allowing for a dwell stream to automatically start on a POWER_ON_RESET condition.

Third party code

  • None

Contributor Info - All information REQUIRED for consideration of pull request

  • Mark Foster
  • NASA / Johnson Space Center / ER6

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.

1 participant