cv32e40x icon indicating copy to clipboard operation
cv32e40x copied to clipboard

RVFI Tracer supports out-of-date `scontext` CSR and not `mscontext`.

Open MikeOpenHWGroup opened this issue 3 years ago • 2 comments

Component

./bhv/cv32e40x_rvfi.sv and ./bhv/include/cv32e40x_rvfi_pkg.sv reference the scontext CSR. This is from v0.13.2 of the RISC-V External Debug Support specification and has been supplanted by mscontext.

Steps to Reproduce

Visual inspection

Additional Information

This is a low priority issue (for now, anyway). As far as I can tell, the RTL does not yet support mscontext, and the RVFI Tracer merely sets these to idle.

MikeOpenHWGroup avatar Jan 24 '22 21:01 MikeOpenHWGroup

There are more CSRs missing that are already in the user manual. Let's first add the couple of remaining to be implemented CSRs in the RTL and then update the RVFI adapter to be complete for both the 40X and 40S.

Silabs-ArjanB avatar Jan 25 '22 07:01 Silabs-ArjanB

Rename scontext -> mscontext in following files:

  • bhv/include/cv32e40s_wrapper.vh
  • bhv/cv32e40s_rvfi.sv
  • bhv/include/cv32e40s_rvfi_pkg.sv

Silabs-ArjanB avatar Feb 21 '22 07:02 Silabs-ArjanB

This issue seems obsolete as the *context CSRs are not implemented and removed from RVFI, RTL and user manual. Do you agree @MikeOpenHWGroup ?

silabs-oysteink avatar May 08 '23 11:05 silabs-oysteink

Agreed. Closing the issue. Thanks!

MikeOpenHWGroup avatar May 08 '23 12:05 MikeOpenHWGroup