mbox series

[v15,0/7] Pointer Masking update for Zjpm v1.0

Message ID 20250106102346.1100149-1-baturo.alexey@gmail.com (mailing list archive)
Headers show
Series Pointer Masking update for Zjpm v1.0 | expand

Message

Alexey Baturo Jan. 6, 2025, 10:23 a.m. UTC
From: Alexey Baturo <baturo.alexey@gmail.com>

Hi,

Rebased against Alistair's riscv-to-apply.next

Thanks

[v14]:
Rebased and addressed Alistair's comments on code style.


[v13]:
Rebased and addressed Daniel's comments about the return type of the helper.

Thanks

[v12]:
Rebased and addressed Richard's comments about proper masking virtualized accesses.

Thanks

[v11]:
As suggested on the mailing list by Daniel, I'm resubmitting this series and keeping the original versioning number.
So that makes this one v11 and previous - v10.
Also I applied previously issues reviewed-by tags on some of the patches that were present in v9 series, but only for the code, that didn't change much.
For the others I'd really like to have them reviewed as there were a lot of comments on v9 series.
Also rebased on the current upstream.

Thanks

[v10]:
I've rebased this patch series and addressed Richard's and Daniel's comments.
Thanks

[v0]:
As Pointer Masking is finally ratified, these patches intend to update the existing code to the final version.
These patches have been submitted previously and I tried to address all the suggestions, but I'd suggest to review them from the clean slate and then finally push them to the repo.
Thanks.

Alexey Baturo (7):
  target/riscv: Remove obsolete pointer masking extension code.
  target/riscv: Add new CSR fields for S{sn,mn,m}pm extensions as part
    of Zjpm v1.0
  target/riscv: Add helper functions to calculate current number of
    masked bits for pointer masking
  target/riscv: Add pointer masking tb flags
  target/riscv: Update address modify functions to take into account
    pointer masking
  target/riscv: Apply pointer masking for virtualized memory accesses
  target/riscv: Enable updates for pointer masking variables and thus
    enable pointer masking extension

 target/riscv/cpu.c           |  19 +-
 target/riscv/cpu.h           |  50 ++---
 target/riscv/cpu_bits.h      |  91 +--------
 target/riscv/cpu_cfg.h       |   3 +
 target/riscv/cpu_helper.c    | 128 +++++++++----
 target/riscv/csr.c           | 359 +++--------------------------------
 target/riscv/internals.h     |  54 ++++++
 target/riscv/machine.c       |  17 +-
 target/riscv/op_helper.c     |  16 +-
 target/riscv/pmp.c           |  14 +-
 target/riscv/pmp.h           |   1 +
 target/riscv/tcg/tcg-cpu.c   |   5 +-
 target/riscv/translate.c     |  47 ++---
 target/riscv/vector_helper.c |   5 -
 14 files changed, 258 insertions(+), 551 deletions(-)

Comments

Alistair Francis Jan. 7, 2025, 1:36 a.m. UTC | #1
On Mon, Jan 6, 2025 at 8:25 PM <baturo.alexey@gmail.com> wrote:
>
> From: Alexey Baturo <baturo.alexey@gmail.com>
>
> Hi,
>
> Rebased against Alistair's riscv-to-apply.next
>
> Thanks
>
> [v14]:
> Rebased and addressed Alistair's comments on code style.
>
>
> [v13]:
> Rebased and addressed Daniel's comments about the return type of the helper.
>
> Thanks
>
> [v12]:
> Rebased and addressed Richard's comments about proper masking virtualized accesses.
>
> Thanks
>
> [v11]:
> As suggested on the mailing list by Daniel, I'm resubmitting this series and keeping the original versioning number.
> So that makes this one v11 and previous - v10.
> Also I applied previously issues reviewed-by tags on some of the patches that were present in v9 series, but only for the code, that didn't change much.
> For the others I'd really like to have them reviewed as there were a lot of comments on v9 series.
> Also rebased on the current upstream.
>
> Thanks
>
> [v10]:
> I've rebased this patch series and addressed Richard's and Daniel's comments.
> Thanks
>
> [v0]:
> As Pointer Masking is finally ratified, these patches intend to update the existing code to the final version.
> These patches have been submitted previously and I tried to address all the suggestions, but I'd suggest to review them from the clean slate and then finally push them to the repo.
> Thanks.
>
> Alexey Baturo (7):
>   target/riscv: Remove obsolete pointer masking extension code.
>   target/riscv: Add new CSR fields for S{sn,mn,m}pm extensions as part
>     of Zjpm v1.0
>   target/riscv: Add helper functions to calculate current number of
>     masked bits for pointer masking
>   target/riscv: Add pointer masking tb flags
>   target/riscv: Update address modify functions to take into account
>     pointer masking
>   target/riscv: Apply pointer masking for virtualized memory accesses
>   target/riscv: Enable updates for pointer masking variables and thus
>     enable pointer masking extension

Thanks!

Applied to riscv-to-apply.next

Alistair

>
>  target/riscv/cpu.c           |  19 +-
>  target/riscv/cpu.h           |  50 ++---
>  target/riscv/cpu_bits.h      |  91 +--------
>  target/riscv/cpu_cfg.h       |   3 +
>  target/riscv/cpu_helper.c    | 128 +++++++++----
>  target/riscv/csr.c           | 359 +++--------------------------------
>  target/riscv/internals.h     |  54 ++++++
>  target/riscv/machine.c       |  17 +-
>  target/riscv/op_helper.c     |  16 +-
>  target/riscv/pmp.c           |  14 +-
>  target/riscv/pmp.h           |   1 +
>  target/riscv/tcg/tcg-cpu.c   |   5 +-
>  target/riscv/translate.c     |  47 ++---
>  target/riscv/vector_helper.c |   5 -
>  14 files changed, 258 insertions(+), 551 deletions(-)
>
> --
> 2.39.5
>
>