Ping On Thu, Mar 13, 2025 at 8:30 PM Loïc Lefort <l...@rivosinc.com> wrote:
> These patches fix Smepmp implementation to make it compliant with the spec. > > First patch limits RLB to CSR changes since RLB should not affect privilege > evaluation. Patch 2 extracts some common code into a function (to be used > in > patch 3). Patch 3 fixes validation of pmpcfg CSR writes in order to match > Smepmp > specification. Patch 4 is a small optimization and last patch is just > removing > redundant code. > > --- > Changes in v2: > - rebased to latest riscv-to-apply.next > - addressed Daniel comments on patch 1 > > Link to v1: > > https://lore.kernel.org/qemu-riscv/20250225160052.39564-1-l...@rivosinc.com/ > > Loïc Lefort (5): > target/riscv: pmp: don't allow RLB to bypass rule privileges > target/riscv: pmp: move Smepmp operation conversion into a function > target/riscv: pmp: fix checks on writes to pmpcfg in Smepmp MML mode > target/riscv: pmp: exit csr writes early if value was not changed > target/riscv: pmp: remove redundant check in pmp_is_locked > > target/riscv/pmp.c | 151 +++++++++++++++++++++++++-------------------- > 1 file changed, 83 insertions(+), 68 deletions(-) > > -- > 2.47.2 > >
