fix(dbltrp): fix sdt/dte interaction logic #3955
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* spike commit: ad9bf364c2067493a6e1c7fdd48db934be9dd81c
* spike config: CPU=XIANGSHAN
Including:
fix(dbltrp): align logic of sdt with xiangshan
* NEMU commit: 0b75a4555a4cac16247a5fe3659f52ea900729e8
* NEMU configs:
* riscv64-xs-ref_defconfig
* riscv64-dual-xs-ref_defconfig
Including:
* fix(dbltrp): fix sdt/dte interation logic
* fix(henvcfg): fix read value for CSR henvcfg.
* fix(hvip): fix the MASK of HVIP for LCOFI bit.
* submodule(ready-to-run): bump spike ref (#686)
* fix(dbltrp): fix difftest read logic of xstatus.SDT (#685)
* fix(watchpoint): support set watchpoint on every 64 bits for vreg.
* feat(watchpoint): add watchpoint of vector and floatponit
* submodule(ready-to-run): bump spike ref in ready-to-run
* config: enable Zimop, Zcmop and Zcb.
* menvcfg.DTE only control Smode dbltrp. Thus mstatus.sdt will not
control by DTE.
* as sstatus is alias of mstatus, when menvcfg.DTE close write
sstatus.sdt cannot lead to shadow write of mstatus.sdt. As a result,
we add wmask of sdt, when write source is from alias write.
While vsstatus is not alias of any other CSR fields, so origin logic
is correct.
4 tasks
|
[Generated by IPC robot]
master branch:
|
huxuan0307
approved these changes
Nov 28, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
menvcfg.DTE only control Smode dbltrp. Thus mstatus.sdt will not
control by DTE.
as sstatus is alias of mstatus, when menvcfg.DTE close write
sstatus.sdt cannot lead to shadow write of mstatus.sdt. As a result,
we add wmask of sdt, when write source is from alias write.
While vsstatus is not alias of any other CSR fields, so origin logic
is correct.
align spike/nemu logic to xiangshan. see spike pr60, NEMU pr688 for detail.