[util] Remove debug prints to speed up secded_gen.py

Now that we're adding ECC check bits to flash images, a significant
fraction of SW build time is spent running gen-flash-img.py. Profiling
a run of:

    ../util/design/gen-flash-img.py \
      sw/device/tests/uart_smoketest_sim_dv.64.vmem \
      sw/device/tests/uart_smoketest_sim_dv.64.scr.vmem

(run this in the build-out directory; choice of test completely
arbitrary!) with py-spy, I realised that we were spending almost half
our time constructing debug messages that we didn't write out.

I'm sure there are cleverer things we can do, but it looks like we
probably don't really need these so let's just drop them for now.

After these changes, here are the big contributors to the total
runtime for the script:

    38%    load up secded_cfg.hjson
    36%    pick an ECC code (we choose it on-the-fly)
   ~15%    actually computing ECC bits

This still seems a little silly, but this patch gets us a factor of 2,
so maybe it's a good start!

Signed-off-by: Rupert Swarbrick <rswarbrick@lowrisc.org>
1 file changed
tree: 869b0c193939a8d89c815b4d9eaa82fc65e6ec76
  1. .github/
  2. ci/
  3. doc/
  4. hw/
  5. rules/
  6. site/
  7. sw/
  8. test/
  9. util/
  10. .bazelignore
  11. .bazelrc
  12. .bazelversion
  13. .clang-format
  14. .dockerignore
  15. .flake8
  16. .gitignore
  17. .style.yapf
  18. .svlint.toml
  19. .svls.toml
  20. _index.md
  21. apt-requirements.txt
  22. azure-pipelines.yml
  23. BUILD
  24. check_tool_requirements.core
  25. CLA
  26. COMMITTERS
  27. CONTRIBUTING.md
  28. LICENSE
  29. meson.build
  30. meson_init.sh
  31. meson_options.txt
  32. python-requirements.txt
  33. README.md
  34. tool_requirements.py
  35. toolchain.txt
  36. topgen-reg-only.core
  37. topgen.core
  38. WORKSPACE
  39. yum-requirements.txt
README.md

OpenTitan

OpenTitan logo

About the project

OpenTitan is an open source silicon Root of Trust (RoT) project. OpenTitan will make the silicon RoT design and implementation more transparent, trustworthy, and secure for enterprises, platform providers, and chip manufacturers. OpenTitan is administered by lowRISC CIC as a collaborative project to produce high quality, open IP for instantiation as a full-featured product. See the OpenTitan site and OpenTitan docs for more information about the project.

About this repository

This repository contains hardware, software and utilities written as part of the OpenTitan project. It is structured as monolithic repository, or “monorepo”, where all components live in one repository. It exists to enable collaboration across partners participating in the OpenTitan project.

Documentation

The project contains comprehensive documentation of all IPs and tools. You can access it online at docs.opentitan.org.

How to contribute

Have a look at CONTRIBUTING and our documentation on project organization and processes for guidelines on how to contribute code to this repository.

Licensing

Unless otherwise noted, everything in this repository is covered by the Apache License, Version 2.0 (see LICENSE for full text).