Skip to content

Ctrace dev1 - #2548

Open
thorstendb-ARM wants to merge 11 commits into
mainfrom
ctrace-dev1
Open

Ctrace dev1#2548
thorstendb-ARM wants to merge 11 commits into
mainfrom
ctrace-dev1

Conversation

@thorstendb-ARM

Copy link
Copy Markdown
Contributor

Implements

  • Adds the initial ctrace implementation for decoding SWO streams containing ITM and DWT data.
  • Provides CSV and CTF/Trace Compass output.
  • Includes unit and integration tests, documentation, and cross-platform build and release support.

Limitations

  • Trace Bus input, PMU events, event counters, and PC sampling are not supported in the initial release.

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 Jul 30, 2026

Copy link
Copy Markdown

Test Results

  5 files   -    36   39 suites   - 86   3m 23s ⏱️ - 16m 43s
185 tests  -   746  168 ✅  -   746  17 💤 ±0  0 ❌ ±0 
509 runs   - 1 492  458 ✅  - 1 492  51 💤 ±0  0 ❌ ±0 

Results for commit 0f6532a. ± Comparison against base commit 37a8a8e.

This pull request removes 746 tests.
CodeGenerator ‑ Check
CrossPlatformUnitTests ‑ CanExecute
CrossPlatformUnitTests ‑ ExecCommand
CrossPlatformUnitTests ‑ GetEnv_Empty
CrossPlatformUnitTests ‑ GetEnv_Existing
CrossPlatformUnitTests ‑ GetEnv_NoExising
CrossPlatformUnitTests ‑ GetExecutablePath
CrossPlatformUnitTests ‑ GetLongPathRegStatus
CrossPlatformUnitTests ‑ GetPackRootDir_Default
CrossPlatformUnitTests ‑ GetPackRootDir_NoEnvSet
…

♻️ This comment has been updated with latest results.

Comment thread tools/ctrace/src/decode/OpenCsdErrorController.cpp Fixed
Comment thread tools/ctrace/src/output/OutputRequirements.cpp Fixed
Comment thread tools/ctrace/src/output/ctf/CtfMetadataWriter.cpp Fixed
Comment thread tools/ctrace/src/output/ctf/TraceCompassXmlWriter.cpp Fixed
Comment thread tools/ctrace/src/tracerun/YmlTraceRunConfigReader.cpp Fixed
Comment thread tools/ctrace/test/unit/src/cli/CliParserTests.cpp Fixed
Comment thread tools/ctrace/test/unit/src/output/OutputRequirementsTests.cpp Fixed
Comment thread tools/ctrace/test/unit/src/decode/OpenCsdPacketCollectorTests.cpp Fixed

@thorstendb-ARM thorstendb-ARM left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added unit tests

@codecov

codecov Bot commented Jul 31, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.13%. Comparing base (37a8a8e) to head (0f6532a).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2548      +/-   ##
==========================================
+ Coverage   68.34%   72.13%   +3.79%     
==========================================
  Files         141      183      +42     
  Lines       26511    30120    +3609     
  Branches    16022    17630    +1608     
==========================================
+ Hits        18118    21727    +3609     
+ Misses       6090     6081       -9     
- Partials     2303     2312       +9     
Flag Coverage Δ
buildmgr-cov 79.87% <ø> (ø)
ctrace-cov 100.00% <100.00%> (ø)
packchk-cov 69.81% <ø> (ø)
packgen-cov 82.02% <ø> (ø)
projmgr-cov 88.05% <ø> (ø)
svdconv-cov 46.82% <ø> (ø)

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

Files with missing lines Coverage Δ
tools/ctrace/src/CtraceMain.cpp 100.00% <100.00%> (ø)
tools/ctrace/src/cli/CliParser.cpp 100.00% <100.00%> (ø)
tools/ctrace/src/control/DecodeConsumers.cpp 100.00% <100.00%> (ø)
tools/ctrace/src/control/FileDecodeJob.cpp 100.00% <100.00%> (ø)
tools/ctrace/src/control/TraceDirectoryJob.cpp 100.00% <100.00%> (ø)
tools/ctrace/src/decode/CortexMPostDecoder.cpp 100.00% <100.00%> (ø)
tools/ctrace/src/decode/CortexMStreamDecoder.cpp 100.00% <100.00%> (ø)
tools/ctrace/src/decode/DecodePipeline.cpp 100.00% <100.00%> (ø)
tools/ctrace/src/decode/DecodePipeline.hpp 100.00% <100.00%> (ø)
tools/ctrace/src/decode/DwtPacketDecoder.cpp 100.00% <100.00%> (ø)
... and 33 more

... and 3 files with indirect coverage changes

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

@jreineckearm jreineckearm left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just had a look at the first few files in this PR (the GH workflows). You may want to review those first. I think some of them will impact more than ctrace.
Also, why do you make workflows sensitive to changes in subfolders of external? I don't think GitHub actions are clever enough to detect changes in git submodules. But I might be wrong.

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.

3 participants