Skip to content

Exclude svnapot svpbmt#919

Closed
chihminchao wants to merge 2 commits intoriscv:devfrom
chihminchao:exclude-svnapot-svpbmt
Closed

Exclude svnapot svpbmt#919
chihminchao wants to merge 2 commits intoriscv:devfrom
chihminchao:exclude-svnapot-svpbmt

Conversation

@chihminchao
Copy link

Description

There are testcase in sv39/sv48/sv57 to test the reserved field defined by Svnapot and Svpbmt. They should be excluded when the model support one of the features, turned on by ISA string.
The patch adds negative matching rule to exclude the testcases.

Related Issues

Please list all the issues related to this PR. Use NA if no issues exist

Ratified/Unratified Extensions

  • Ratified
  • Unratified

List Extensions

List the extensions that your PR affects. In case of unratified extensions, please provide a link to the spec draft that was referred to make this PR.

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 >

Optional Checklist:

  • 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.

To use negative match to exclude related pattern in sv39/48/57

Signed-off-by: Chih-Min Chao <chihmin.chao@sifive.com>
To use negative match to exclude related pattern in sv39/48/57

Signed-off-by: Chih-Min Chao <chihmin.chao@sifive.com>
@jamesbeyond
Copy link
Collaborator

ACT repository have been updated act4 as the default branch, in act4 we have #908 PR covers the same virtual memory test cases. please check out this PR. We'll close this PR for now.

@chihminchao
Copy link
Author

Thanks for the information that I miss and are written in readme

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

Successfully merging this pull request may close these issues.

2 participants