[prev in list] [next in list] [prev in thread] [next in thread]
List: qemu-devel
Subject: Re: [PATCH v2 3/3] target/hppa: Fix diag instructions to set/restore shadow registers
From: Helge Deller <deller () gmx ! de>
Date: 2024-03-26 20:35:07
Message-ID: 7b9ef2af-d137-4db4-a286-9c5ea996bff9 () gmx ! de
[Download RAW message or body]
On 3/26/24 19:10, Richard Henderson wrote:
> The 32-bit 7300LC CPU and the 64-bit PCX-W 8500 CPU use different
I initially should have used the correct naming in my patch.
So this sentence above should read:
The 32-bit PA-7300LC (PCX-L2) CPU and the 64-bit PA8700 (PCX-W2) CPU use different....
> diag instructions to save or restore the CPU registers to/from
> the shadow registers.
>
> Implement those per-CPU architecture diag instructions to fix those
> parts of the HP ODE testcases (L2DIAG and WDIAG, section 1) which test
> the shadow registers.
>
> Signed-off-by: Helge Deller <deller@gmx.de>
> [rth: Use decodetree to distinguish cases]
> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
> ---
> target/hppa/insns.decode | 10 ++++++++++
> target/hppa/translate.c | 34 ++++++++++++++++++++++++++++++++++
> 2 files changed, 44 insertions(+)
>
> diff --git a/target/hppa/insns.decode b/target/hppa/insns.decode
> index 9f6ffd8e2c..c2acb3796c 100644
> --- a/target/hppa/insns.decode
> +++ b/target/hppa/insns.decode
> @@ -65,6 +65,8 @@
> # Argument set definitions
> ####
>
> +&empty
> +
> # All insns that need to form a virtual address should use this set.
> &ldst t b x disp sp m scale size
>
> @@ -638,6 +640,14 @@ xmpyu 001110 ..... ..... 010 .0111 .00 t:5 r1=%ra64 r2=%rb64
> [
> diag_btlb 000101 00 0000 0000 0000 0001 0000 0000
> diag_cout 000101 00 0000 0000 0000 0001 0000 0001
> +
> + # For 32-bit 7300C
For 32-bit PA-7300LC (PCX-L2) CPU:
> + diag_getshadowregs_pa1 000101 00 0000 0000 0001 1010 0000 0000
> + diag_putshadowregs_pa1 000101 00 0000 0000 0001 1010 0100 0000
> +
> + # For 64-bit PCX-W 8500
For 64-bit PA8700 (PCX-W2) CPU:
other than that:
Reviewed-by: Helge Deller <deller@gmx.de>
Tested-by: Helge Deller <deller@gmx.de>
Thanks a lot for the nice rewrite & cleanup!!
Helge
> + diag_getshadowregs_pa2 000101 00 0111 1000 0001 1000 0100 0000
> + diag_putshadowregs_pa2 000101 00 0111 0000 0001 1000 0100 0000
> ]
> diag_unimp 000101 i:26
> }
> diff --git a/target/hppa/translate.c b/target/hppa/translate.c
> index 42dd3f2c8d..143818c2d9 100644
> --- a/target/hppa/translate.c
> +++ b/target/hppa/translate.c
> @@ -2399,6 +2399,20 @@ static bool do_getshadowregs(DisasContext *ctx)
> return nullify_end(ctx);
> }
>
> +static bool do_putshadowregs(DisasContext *ctx)
> +{
> + CHECK_MOST_PRIVILEGED(EXCP_PRIV_OPR);
> + nullify_over(ctx);
> + tcg_gen_st_i64(cpu_gr[1], tcg_env, offsetof(CPUHPPAState, shadow[0]));
> + tcg_gen_st_i64(cpu_gr[8], tcg_env, offsetof(CPUHPPAState, shadow[1]));
> + tcg_gen_st_i64(cpu_gr[9], tcg_env, offsetof(CPUHPPAState, shadow[2]));
> + tcg_gen_st_i64(cpu_gr[16], tcg_env, offsetof(CPUHPPAState, shadow[3]));
> + tcg_gen_st_i64(cpu_gr[17], tcg_env, offsetof(CPUHPPAState, shadow[4]));
> + tcg_gen_st_i64(cpu_gr[24], tcg_env, offsetof(CPUHPPAState, shadow[5]));
> + tcg_gen_st_i64(cpu_gr[25], tcg_env, offsetof(CPUHPPAState, shadow[6]));
> + return nullify_end(ctx);
> +}
> +
> static bool trans_getshadowregs(DisasContext *ctx, arg_getshadowregs *a)
> {
> return do_getshadowregs(ctx);
> @@ -4594,6 +4608,26 @@ static bool trans_diag_cout(DisasContext *ctx, arg_diag_cout *a)
> #endif
> }
>
> +static bool trans_diag_getshadowregs_pa1(DisasContext *ctx, arg_empty *a)
> +{
> + return !ctx->is_pa20 && do_getshadowregs(ctx);
> +}
> +
> +static bool trans_diag_getshadowregs_pa2(DisasContext *ctx, arg_empty *a)
> +{
> + return ctx->is_pa20 && do_getshadowregs(ctx);
> +}
> +
> +static bool trans_diag_putshadowregs_pa1(DisasContext *ctx, arg_empty *a)
> +{
> + return !ctx->is_pa20 && do_putshadowregs(ctx);
> +}
> +
> +static bool trans_diag_putshadowregs_pa2(DisasContext *ctx, arg_empty *a)
> +{
> + return ctx->is_pa20 && do_putshadowregs(ctx);
> +}
> +
> static bool trans_diag_unimp(DisasContext *ctx, arg_diag_unimp *a)
> {
> CHECK_MOST_PRIVILEGED(EXCP_PRIV_OPR);
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic