mbox series

[v2,0/7] target/arm: Introduce aarch64_set_svcr

Message ID 20230112102436.1913-1-philmd@linaro.org
Headers show
Series target/arm: Introduce aarch64_set_svcr | expand

Message

Philippe Mathieu-Daudé Jan. 12, 2023, 10:24 a.m. UTC 
This is a respin of Richard's patch
https://lore.kernel.org/qemu-devel/20230112004322.161330-1-richard.henderson@linaro.org/
but split in multiple trivial changes, as I was having hard
time to understand all changes at once while reviewing it.

Richard Henderson (7):
  target/arm/sme: Reorg SME access handling in handle_msr_i()
  target/arm/sme: Rebuild hflags in set_pstate() helpers
  target/arm/sme: Introduce aarch64_set_svcr()
  target/arm/sme: Reset SVE state in aarch64_set_svcr()
  target/arm/sme: Reset ZA state in aarch64_set_svcr()
  target/arm/sme: Rebuild hflags in aarch64_set_svcr()
  target/arm/sme: Unify set_pstate() SM/ZA helpers as set_svcr()

 linux-user/aarch64/cpu_loop.c | 11 ++--------
 linux-user/aarch64/signal.c   | 13 ++---------
 target/arm/cpu.h              |  2 +-
 target/arm/helper-sme.h       |  3 +--
 target/arm/helper.c           | 41 ++++++++++++++++++++++++++++++++---
 target/arm/sme_helper.c       | 37 ++-----------------------------
 target/arm/translate-a64.c    | 19 ++++++----------
 7 files changed, 53 insertions(+), 73 deletions(-)

Comments

Fabiano Rosas Jan. 12, 2023, 2:20 p.m. UTC | #1 
Philippe Mathieu-Daudé <philmd@linaro.org> writes:

> This is a respin of Richard's patch
> https://lore.kernel.org/qemu-devel/20230112004322.161330-1-richard.henderson@linaro.org/
> but split in multiple trivial changes, as I was having hard
> time to understand all changes at once while reviewing it.
>
> Richard Henderson (7):
>   target/arm/sme: Reorg SME access handling in handle_msr_i()
>   target/arm/sme: Rebuild hflags in set_pstate() helpers
>   target/arm/sme: Introduce aarch64_set_svcr()
>   target/arm/sme: Reset SVE state in aarch64_set_svcr()
>   target/arm/sme: Reset ZA state in aarch64_set_svcr()
>   target/arm/sme: Rebuild hflags in aarch64_set_svcr()
>   target/arm/sme: Unify set_pstate() SM/ZA helpers as set_svcr()
>
>  linux-user/aarch64/cpu_loop.c | 11 ++--------
>  linux-user/aarch64/signal.c   | 13 ++---------
>  target/arm/cpu.h              |  2 +-
>  target/arm/helper-sme.h       |  3 +--
>  target/arm/helper.c           | 41 ++++++++++++++++++++++++++++++++---
>  target/arm/sme_helper.c       | 37 ++-----------------------------
>  target/arm/translate-a64.c    | 19 ++++++----------
>  7 files changed, 53 insertions(+), 73 deletions(-)

Reviewed-by: Fabiano Rosas <farosas@suse.de>
Peter Maydell Jan. 17, 2023, 4:33 p.m. UTC | #2 
On Thu, 12 Jan 2023 at 10:24, Philippe Mathieu-Daudé <philmd@linaro.org> wrote:
>
> This is a respin of Richard's patch
> https://lore.kernel.org/qemu-devel/20230112004322.161330-1-richard.henderson@linaro.org/
> but split in multiple trivial changes, as I was having hard
> time to understand all changes at once while reviewing it.
>



Applied to target-arm.next, thanks.

-- PMM