chipsalliance / dromajo

RISC-V RV64GC emulator designed for RTL co-simulation
Apache License 2.0
210 stars 63 forks source link

Meaning of MIE_MCIP field in mie #51

Open scanakci opened 2 years ago

scanakci commented 2 years ago

Hi,

While I was using dromajo, I noticed that mie has a MIE_MCIP field.

https://github.com/chipsalliance/dromajo/blob/master/src/riscv_cpu.cpp#L1215

I could not see this field in SPEC (it is reserved). Can you please let me know if this field is intentionally left there for any reason?

Thanks

et-tommythorn commented 2 years ago

It's complicated :) I think it reflects the hardware it was tested against, but clearly this fall under things that should be configurable. We don't currently have a way to configure this though and I'm debating the future of the configuration file.

Also, so of this dates back to pre-1.11 version of the privileged spec where some fields were different (and named differently).

Please let me know if this is a problem for you and you suggestions to better handling different machine configurations.

scanakci commented 2 years ago

I see, thank you! It is not a problem on my end at all :) I just wanted to inform you in case that field is left there unintentionally. Please feel free to close the issue if you think that no action is required at this point.