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

Tests of ePMP #301

Closed
wants to merge 8 commits into from
Closed

Tests of ePMP #301

wants to merge 8 commits into from

Conversation

HamzaKh01
Copy link

Description

Written tests of ePMP. Tests adhere to the latest TestFormatSpec. Tests have the same format as of pmp tests that are already written. In the branch test here test lies riscv-arch-test/riscv-test-suite/rv32i_m/privilege/src/ePMP32. Test plan which covers the cases is attached to each test file, In addition, the test file also contains the test motive and expected output. For symmetry test plan is divided into M mode test and S/U mode tests as Mmode has one behaviour and S/U mode has one behaviour for ePMP but despite having same behaviour for a region S/U mode tests are seprated too due to the dependency of a region to be writeable and executable for mMode and we were not able to use macro Go_to_Mmode to switch to Mmode and the goto Umode to execute the same behaviour for a region that is why tests are separated. Tests are verified on the first-hand thoroughly by running on the spike. As SAIL lacks the support of ePMP, working on adding support of ePMP in SAIL it will be made live by the 31st of December Insha'Allah.

Related Issues

NA

Ratified/Unratified Extensions

  • Ratified
  • Unratified

List Extensions

NA

Reference Model Used

  • SAIL
  • Spike
  • Other - < SPECIFY HERE >

Mandatory Checklist:

  • All tests are compliant with the test-format spec present in this repo ?
  • Ran the new tests on RISCOF with SAIL/Spike as reference model successfully ?
  • Ran the new tests on RISCOF in coverage mode
  • Link to Google-Drive folder containing the new coverage reports (See this for more info): < SPECIFY HERE >
  • Link to PR in RISCV-ISAC from which the reports were generated : < SPECIFY HERE >
  • Changelog entry created with a minor patch

Optional Checklist:

  • RISCV-V CTG PR link if tests were generated using it : < SPECIFY HERE >
  • Were the tests hand-written/modified ?
  • Have you run these on any hard DUT model ? Please specify name and provide link if possible in the description
  • If you have modified arch_test.h Please provide a detailed description of the changes in the Description section above.

Hamza Khan added 7 commits November 8, 2022 12:27
Kindly review these files in each file test motive
and expected output is written in addition testplan
is also provided which is being followed
Kindly review these files in each file test motive
and expected output is written in addition link of the testplan
is also provided which is being followed
From file ePMP09.S to ePMP_46.S from the test plan attach in each file
From file ePMP47.S to ePMP_54.S from the test plan attach in each file
From file ePMP_51.S to ePMP_54.S from the test plan attach in each file
In each file test motive and expected output is written, in addition
the link of the testplan is also provided which is being followed
@HamzaKh01
Copy link
Author

Hi Hamza here,

Sail support for Smepmp is live here.

@jjscheel jjscheel mentioned this pull request Mar 16, 2023
13 tasks
@jamesbeyond
Copy link
Collaborator

Pending for the PMP test merged first
cover point (cgf files) need to be generated

@jamesbeyond jamesbeyond changed the base branch from main to dev May 28, 2024 16:59
@UmerShahidengr
Copy link
Collaborator

This PR is quite old and have run on the older version of Sail. It will be done again after testing against the updated Sail model and integrating the changes in the latest trap handler.

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

Successfully merging this pull request may close these issues.

3 participants