Skip to content

[ctrace] Align trace-run metadata with specification - #2581

Open
thorstendb-ARM wants to merge 2 commits into
mainfrom
ctrace-spec-updates
Open

[ctrace] Align trace-run metadata with specification#2581
thorstendb-ARM wants to merge 2 commits into
mainfrom
ctrace-spec-updates

Conversation

@thorstendb-ARM

Copy link
Copy Markdown
Contributor

Fixes

Changes

  • Read DWT metadata from the current address, size, and data-type fields.
  • Support the specified unsigned, signed, and float data types.
  • Use referenced ctrace-setup.data.size when reference-level size is absent.
  • Distinguish DWT data#<index> references from instruction-control references so that only data references create decoded data sources.
  • Ignore unknown and unconsumed YAML fields while retaining validation for fields used by ctrace.
  • Update output planning, CTF metadata, fixtures, and tests to use the current field names.
  • Remove compatibility handling for the previous field names because ctrace has not been released yet.

Checklist

  • 🤖 This change is covered by unit tests (if applicable).
  • 🤹 Manual testing has been performed (if necessary).
  • 🛡️ Security impacts have been considered (if relevant).
  • 📖 Documentation updates are complete (if required).
  • 🧠 Third-party dependencies and TPIP updated (if required).

@github-actions

github-actions Bot commented Aug 31, 2026

Copy link
Copy Markdown

Test Results

  4 files    4 suites   1s ⏱️
183 tests 183 ✅ 0 💤 0 ❌
366 runs  359 ✅ 7 💤 0 ❌

Results for commit 78a5c4f.

♻️ This comment has been updated with latest results.

@codecov

codecov Bot commented Aug 31, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.19%. Comparing base (fd7e0e9) to head (78a5c4f).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2581      +/-   ##
==========================================
- Coverage   72.31%   72.19%   -0.13%     
==========================================
  Files         185      185              
  Lines       30257    30284      +27     
  Branches    17849    17866      +17     
==========================================
- Hits        21880    21863      -17     
- Misses       6072     6109      +37     
- Partials     2305     2312       +7     
Flag Coverage Δ
ctrace-cov 99.80% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
tools/ctrace/src/output/OutputRequirements.cpp 98.13% <100.00%> (+0.07%) ⬆️
tools/ctrace/src/output/TraceOutputConfig.h 100.00% <ø> (ø)
tools/ctrace/src/output/ctf/CtfEncoder.cpp 100.00% <100.00%> (ø)
tools/ctrace/src/output/ctf/CtfMetadataWriter.cpp 100.00% <100.00%> (ø)
tools/ctrace/src/output/ctf/CtfSchema.h 100.00% <100.00%> (ø)
tools/ctrace/src/tracerun/CtraceRunMeta.cpp 99.40% <100.00%> (+0.01%) ⬆️
tools/ctrace/src/tracerun/TraceRunConfig.h 100.00% <100.00%> (ø)
...ls/ctrace/src/tracerun/YmlTraceRunConfigReader.cpp 100.00% <100.00%> (ø)

... and 7 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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