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.

Sign up for GitHub

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

Jump to bottom

[ISAC] - PTE Permissions Check Function and prev. instr. csr track func Updated #563

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from
Open

[ISAC] - PTE Permissions Check Function and prev. instr. csr track func Updated #563

wants to merge 2 commits into from

Conversation

MuhammadHammad001
Copy link
Contributor

Description

Function to check permissions of a PTE has been updated to take lower and upper case characters for optimization.

@jamesbeyond jamesbeyond changed the title [RISC-V ISAC] - PTE Permissions Check Function Updated [ISAC] - PTE Permissions Check Function Updated Dec 2, 2024
@MuhammadHammad001
rename the old function to old_csr_val for better readability and als…
9ea9d9f 
…o update this to be used in the val comb for better cross coverpoints
@MuhammadHammad001 MuhammadHammad001 changed the title [ISAC] - PTE Permissions Check Function Updated [ISAC] - PTE Permissions Check Function and prev. instr. csr track func Updated Dec 4, 2024
@MuhammadHammad001 MuhammadHammad001 mentioned this pull request Dec 4, 2024
Open
15 tasks
@MuhammadHammad001
Copy link
Contributor Author

This commit adds the support for the old function to be renamed to old_csr_val since it just tracks the previous state CSR values. Additionally, I have also made this function global so it can be used in val_comb and op_comb in addition to csr_comb which will enable us to write coverpoints which are cross with the other instructions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@MuhammadHammad001