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

Add feature Smcdeleg #1913

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

JJ-Gaisler
Copy link

This PR is based on the following PR : #1896

Please see attached test case, the test checks the exception logic related to the extension and that proxy counters work as intended.

spike -m0x80000000:0x10000000,0x20000000:0x1000,0x4000:0x1000,0xB000:0x1000 --isa rv64gh_svinval_zba_zbb_zbc_zbs_zfh_zfa_zbkb_zbkx_zicsr_zifencei_sscofpmf_smstateen_smepmp_zicbom_zicntr_zihpm_svadu_sstc_zicond_smcdeleg_smcsrind_sscsrind -p1 csr_test.txt
csr_test.txt

JJ-Gaisler and others added 9 commits January 28, 2025 08:45
missing proxy csr and check for csr privilege in sscsring_reg_csr_t
since mireg uses the same class.
Add stateen checks for scontext/hcontext
Fix for issue: 1893
Ensuring No negation on -2^63(int64_t) is performed during mulh/mulhsu

Signed-off-by: Nicolas Brunie <82109999+nibrunieAtSi5@users.noreply.github.com>

Update riscv/arith.h

Signed-off-by: Nicolas Brunie <82109999+nibrunieAtSi5@users.noreply.github.com>
Pointer masking needs to be reapplied after computing the address of the
tail of a misaligned access in case there's a carry-out into the MSBs.

Resolves riscv-software-src#1895
@aswaterman
Copy link
Collaborator

@JJ-Gaisler The commit history looks messed up; several unrelated commits are included in the changelog.

@bcstrongx could I ask you to review this PR at some point (ignoring the aforementioned unrelated commits), since you authored the spec?

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.

6 participants