[otbn,dv] Sample RTL signals on first cycle of instruction

We sample various internal RTL signals in order to do coverage
collection. These are supposed to be "the value of the state just
before the instruction ran". Most of the time, this is the same as
"the value of the state just before the last cycle of running the
instruction", but that's not quite always true.

In particular, consider the following instruction:

  BN.LID x1, 0(x0)

This reads from the call stack (x1) in order to get a destination
address. With the current RTL implementation, the call stack pop
happens on the first cycle which means that the previous code was
sampling too late and saw an empty call stack. We wouldn't have
noticed, except that the checks on "lockl_x1_uflow" in otbn_env_cov
were failing (because we thought we should have seen a call stack
underflow and the RTL disagreed).

Fixes #8587.

Signed-off-by: Rupert Swarbrick <rswarbrick@lowrisc.org>
1 file changed
tree: 47937ce504d5dc4a049623b068ba0c1ccfa5db06
  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).