Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Hypervisor Extension enable issue #414

Open
Allen-StarFive opened this issue Nov 30, 2023 · 5 comments
Open

Hypervisor Extension enable issue #414

Allen-StarFive opened this issue Nov 30, 2023 · 5 comments

Comments

@Allen-StarFive
Copy link

Our DUT hardware supports H extension, so we want to enable MISA.H = 1.
But Reference-sail_c_simulator doesn't support it at present, which causes some test cases to fail.
How to configure yaml files in this case?

Example in attach files:
Dubhe-90.zip

Test case fail cause analysis:
image
In function <Mxcpt_sig_sv> read misa to t1, Our DUT return misa=0x80000000001411AD, which cause branch unsatisfied.
But SAIL always return misa=0x800000000014112D, which doesn't match H extension.

@allenjbaum
Copy link
Collaborator

The short answer is that Hypervisor has not been implemented in Sail yet, so you can't configure it.
The people who understand Sail and the people that understand the H-extension have a very small overlap.
There are some changes being made to the Sail model that should make implementation simpler... but not simple

@Allen-StarFive
Copy link
Author

what's your suggestion for next step? Can we just waive these test failures?

@allenjbaum
Copy link
Collaborator

allenjbaum commented Dec 4, 2023 via email

@Allen-StarFive
Copy link
Author

Allen-StarFive commented Dec 26, 2023

I have replaced SAIL with Spike and run test again. It looks all tests have passed.
And I have submitted the report to riscv-arch-test-reports (riscv-non-isa/riscv-arch-test-reports#6).
Would you please help to take a look?

@allenjbaum
Copy link
Collaborator

Wow, I just reread my original comments, and its full of weird typos.
OK, I can merge this - but note that this didn't actually run any hypervisor tests!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants