Greg Chadwick

Results 151 comments of Greg Chadwick

Additional reads of these should be stimulated in DV and coverage added

I have asked the riscv-privileged list if my understanding here is correct but it looks pretty clear from the spec and the fact the compliance tests specifically test accessing these...

After some further digging there's an argument you don't need to implement these going by the latest spec draft, if you don't implement Zicntr (which we don't currently). I'm going...

Thanks for your contribution @leesum1. The RTL change looks good but before we can merge this we need to add DV. We need to add tests to the DV environment...

Thanks for the report @tzwaenn. If you have a fix feel free to submit a PR otherwise we will take a look though this isn't a priority for us right...

Are you referring to the https://github.com/lowRISC/ibex/tree/pulpissimo branch? The co-simulation system is only supported on the master branch.

Moving to V3 as it's a cleanup/enhancement not required functionality for V2S.

You can initialise those addresses by giving them values in the .elf you load into the verilator simulation. Precise details of how to do this depend upon your toolchain. Alternatively...

This may be a genuine issue with our DV code, just one that only causes issues under VCS. So I will leave this open as a reminder for us to...

As I can't see the column headings for those screenshots I cannot comment as to the accuracy of those numbers. Our latest nightly regression (sadly results not currently publicly available...