Closed wxjstz closed 5 years ago
All three of your changes are merged in the (default) integration branch
We can move them from:
to:
once we have tests (before/after fail/pass) for the changes...
I will help with some open source tests, so that we can put them into regression testing. We also will need to rebase qemu-for-testing with Signed-Off-By tags and links to these PRs and tests (minimal reproducers) so we can post them to upstream QEMU, the changes can be verified. We also need to verify the granularity behavior (as mentioned) due to QEMU's TLB limiting permission checks to 4K granules.
thanks again
This needs to be merged. When reading pmpcfg0 / pmpcfg2 under RV64, this bug will cause the upper 32 bits to be zero.
related code:
C language automatic type promotion to int. The length of the int under RV64 may be small with target_ulong, and the upper 32 bits can't be read.