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

util: Extend simulation and trace utilities #178

Merged
merged 11 commits into from
Aug 12, 2024
Merged

util: Extend simulation and trace utilities #178

merged 11 commits into from
Aug 12, 2024

Conversation

colluca
Copy link
Collaborator

@colluca colluca commented Aug 11, 2024

  • The simulation time would be taken from the first line printing the time, this included error messages. We fix this by taking the last line printing the time.
  • We use the PrettyTable title feature to replace our custom title output for the test summary table.
  • Dry simulation runs do not produce any log file. We should not attempt to read the return code from the log file when the run is dry.
  • Add column field to annotated simulation traces. This is possible only with LLVM toolchains, so we must handle GCC and LLVM toolchains differently.
  • Previously, lines without timestamps, e.g. FPU instructions executed in the same cycle as an integer instruction, were being parsed incorrectly, so they would not be annotated. Now we use a regex to handle these lines properly.
  • Fix Garbled performance metrics at end of annotated traces  #85.
  • In some corner cases, addr2line does not display all information for a certain function. We shift to regular expressions to parse addr2line output and correct this condition.
  • Extend trace visualization utilities to include CVA6 trace in heterogeneous systems including Snitch cluster.
  • Extend tracevis.py to process traces in parallel.
  • Extend gen_trace.py to print the line number on an exception.

@colluca colluca marked this pull request as ready for review August 11, 2024 14:57
@colluca colluca merged commit 5928ffa into main Aug 12, 2024
27 checks passed
@colluca colluca deleted the extend-utils branch August 12, 2024 07:08
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.

Garbled performance metrics at end of annotated traces
1 participant