-
Notifications
You must be signed in to change notification settings - Fork 220
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
Virtual Memory SV-32 Address Translation Scheme Tests and Covergroups #516
Virtual Memory SV-32 Address Translation Scheme Tests and Covergroups #516
Conversation
Signed-off-by: Muhammad Hammad Bashir <139617104+MuhammadHammad001@users.noreply.github.com>
Signed-off-by: Muhammad Hammad Bashir <139617104+MuhammadHammad001@users.noreply.github.com>
@allenjbaum CI is passing for these tests too, please review these tests. |
@allenjbaum @UmerShahidengr Covergroups have been updated and now are written using the translator format. |
Signed-off-by: Muhammad Hammad Bashir <139617104+MuhammadHammad001@users.noreply.github.com>
Hi @allenjbaum @UmerShahidengr few more tests and covergroups have been added. Please review this Test Plan as well and please let me know the cases that I have marked as NOT Possible can be achieved or not. Else these tests are complete and need your kind review |
Signed-off-by: Muhammad Hammad Bashir <139617104+MuhammadHammad001@users.noreply.github.com>
LGTM |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are likely more coverpoints and tests needed to account for some architectural options, but this is an excellent start
Description
This PR contains the tests and covergroups for Virtual Memory SV32 address translation scheme. The tests and covergroups are written using the following Test Plan
Related Issues
This PR has a dependency on the PR# 563.
Ratified/Unratified Extensions
List Extensions
NA
Reference Model Used
Mandatory Checklist:
Optional Checklist: