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

trace: Add functionality to parse DMA trace #71

Merged
merged 8 commits into from
Aug 10, 2024
Merged

trace: Add functionality to parse DMA trace #71

merged 8 commits into from
Aug 10, 2024

Conversation

colluca
Copy link
Collaborator

@colluca colluca commented Dec 7, 2023

  • gen_trace.py: output the trace to a file provided on the command-line rather than stdout
  • gen_trace.py: add functionality to parse DMA traces, calculate DMA performance metrics and output to file
  • target: update commands invoking the tracer scripts to align them with the previous changes
  • util/bench: update benchmarking scripts to process both hart and DMA traces. Includes:
    • Major change from CSV to JSON format in intermediate output files
    • Add mkdocstrings documentation for benchmarking scripts
    • Following test-driven development approach, add pytest unit tests to CI
  • Solve issues described in (Not a PR to main): Improve DMA tracer handling of 1d requests after 2d requests and scale units correctly #69
  • Rebase onto main, including iDMA change
  • Update tutorial [POSTPONED]
  • Wait for release of Restore tracing in Snitch DMA iDMA#52 [POSTPONED]: instead temporarily build sources locally, revert this later

@colluca colluca force-pushed the tracer/dma branch 4 times, most recently from d14a448 to 4b42bf3 Compare August 9, 2024 12:12
@colluca colluca force-pushed the tracer/dma branch 2 times, most recently from e37a230 to fd61651 Compare August 9, 2024 15:12
@colluca colluca marked this pull request as ready for review August 10, 2024 05:22
@colluca colluca merged commit f0749fd into main Aug 10, 2024
25 of 27 checks passed
@colluca colluca deleted the tracer/dma branch August 10, 2024 05:24
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