diff mbox series

[v5,06/10] target/ppc: Put LPCR[GTSE] in hflags

Message ID 20210323184340.619757-7-richard.henderson@linaro.org
State Superseded
Headers show
Series target/ppc: Fix truncation of env->hflags | expand

Commit Message

Richard Henderson March 23, 2021, 6:43 p.m. UTC
Because this bit was not in hflags, the privilege check
for tlb instructions was essentially random.
Recompute hflags when storing to LPCR.

Reviewed-by: David Gibson <david@gibson.dropbear.id.au>

Signed-off-by: Richard Henderson <richard.henderson@linaro.org>

---
 target/ppc/cpu.h         | 1 +
 target/ppc/helper_regs.c | 3 +++
 target/ppc/mmu-hash64.c  | 3 +++
 target/ppc/translate.c   | 2 +-
 4 files changed, 8 insertions(+), 1 deletion(-)

-- 
2.25.1

Comments

David Gibson March 24, 2021, 12:06 a.m. UTC | #1
On Tue, Mar 23, 2021 at 12:43:36PM -0600, Richard Henderson wrote:
> Because this bit was not in hflags, the privilege check

> for tlb instructions was essentially random.

> Recompute hflags when storing to LPCR.

> 

> Reviewed-by: David Gibson <david@gibson.dropbear.id.au>

> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


Applied to ppc-for-6.0, thanks.

> ---

>  target/ppc/cpu.h         | 1 +

>  target/ppc/helper_regs.c | 3 +++

>  target/ppc/mmu-hash64.c  | 3 +++

>  target/ppc/translate.c   | 2 +-

>  4 files changed, 8 insertions(+), 1 deletion(-)

> 

> diff --git a/target/ppc/cpu.h b/target/ppc/cpu.h

> index d5f362506a..3c28ddb331 100644

> --- a/target/ppc/cpu.h

> +++ b/target/ppc/cpu.h

> @@ -596,6 +596,7 @@ enum {

>      HFLAGS_LE = 0,   /* MSR_LE -- comes from elsewhere on 601 */

>      HFLAGS_HV = 1,   /* computed from MSR_HV and other state */

>      HFLAGS_64 = 2,   /* computed from MSR_CE and MSR_SF */

> +    HFLAGS_GTSE = 3, /* computed from SPR_LPCR[GTSE] */

>      HFLAGS_DR = 4,   /* MSR_DR */

>      HFLAGS_IR = 5,   /* MSR_IR */

>      HFLAGS_SPE = 6,  /* from MSR_SPE if cpu has SPE; avoid overlap w/ MSR_VR */

> diff --git a/target/ppc/helper_regs.c b/target/ppc/helper_regs.c

> index e345966b6b..f85bb14d1d 100644

> --- a/target/ppc/helper_regs.c

> +++ b/target/ppc/helper_regs.c

> @@ -149,6 +149,9 @@ void hreg_compute_hflags(CPUPPCState *env)

>      if ((ppc_flags & POWERPC_FLAG_TM) && (msr & (1ull << MSR_TM))) {

>          hflags |= 1 << HFLAGS_TM;

>      }

> +    if (env->spr[SPR_LPCR] & LPCR_GTSE) {

> +        hflags |= 1 << HFLAGS_GTSE;

> +    }

>  

>  #ifndef CONFIG_USER_ONLY

>      if (!env->has_hv_mode || (msr & (1ull << MSR_HV))) {

> diff --git a/target/ppc/mmu-hash64.c b/target/ppc/mmu-hash64.c

> index 0fabc10302..d517a99832 100644

> --- a/target/ppc/mmu-hash64.c

> +++ b/target/ppc/mmu-hash64.c

> @@ -30,6 +30,7 @@

>  #include "exec/log.h"

>  #include "hw/hw.h"

>  #include "mmu-book3s-v3.h"

> +#include "helper_regs.h"

>  

>  /* #define DEBUG_SLB */

>  

> @@ -1125,6 +1126,8 @@ void ppc_store_lpcr(PowerPCCPU *cpu, target_ulong val)

>      CPUPPCState *env = &cpu->env;

>  

>      env->spr[SPR_LPCR] = val & pcc->lpcr_mask;

> +    /* The gtse bit affects hflags */

> +    hreg_compute_hflags(env);

>  }

>  

>  void helper_store_lpcr(CPUPPCState *env, target_ulong val)

> diff --git a/target/ppc/translate.c b/target/ppc/translate.c

> index d48c554290..5e629291d3 100644

> --- a/target/ppc/translate.c

> +++ b/target/ppc/translate.c

> @@ -7908,7 +7908,7 @@ static void ppc_tr_init_disas_context(DisasContextBase *dcbase, CPUState *cs)

>      ctx->altivec_enabled = (hflags >> HFLAGS_VR) & 1;

>      ctx->vsx_enabled = (hflags >> HFLAGS_VSX) & 1;

>      ctx->tm_enabled = (hflags >> HFLAGS_TM) & 1;

> -    ctx->gtse = !!(env->spr[SPR_LPCR] & LPCR_GTSE);

> +    ctx->gtse = (hflags >> HFLAGS_GTSE) & 1;

>  

>      ctx->singlestep_enabled = 0;

>      if ((hflags >> HFLAGS_SE) & 1) {


-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson
diff mbox series

Patch

diff --git a/target/ppc/cpu.h b/target/ppc/cpu.h
index d5f362506a..3c28ddb331 100644
--- a/target/ppc/cpu.h
+++ b/target/ppc/cpu.h
@@ -596,6 +596,7 @@  enum {
     HFLAGS_LE = 0,   /* MSR_LE -- comes from elsewhere on 601 */
     HFLAGS_HV = 1,   /* computed from MSR_HV and other state */
     HFLAGS_64 = 2,   /* computed from MSR_CE and MSR_SF */
+    HFLAGS_GTSE = 3, /* computed from SPR_LPCR[GTSE] */
     HFLAGS_DR = 4,   /* MSR_DR */
     HFLAGS_IR = 5,   /* MSR_IR */
     HFLAGS_SPE = 6,  /* from MSR_SPE if cpu has SPE; avoid overlap w/ MSR_VR */
diff --git a/target/ppc/helper_regs.c b/target/ppc/helper_regs.c
index e345966b6b..f85bb14d1d 100644
--- a/target/ppc/helper_regs.c
+++ b/target/ppc/helper_regs.c
@@ -149,6 +149,9 @@  void hreg_compute_hflags(CPUPPCState *env)
     if ((ppc_flags & POWERPC_FLAG_TM) && (msr & (1ull << MSR_TM))) {
         hflags |= 1 << HFLAGS_TM;
     }
+    if (env->spr[SPR_LPCR] & LPCR_GTSE) {
+        hflags |= 1 << HFLAGS_GTSE;
+    }
 
 #ifndef CONFIG_USER_ONLY
     if (!env->has_hv_mode || (msr & (1ull << MSR_HV))) {
diff --git a/target/ppc/mmu-hash64.c b/target/ppc/mmu-hash64.c
index 0fabc10302..d517a99832 100644
--- a/target/ppc/mmu-hash64.c
+++ b/target/ppc/mmu-hash64.c
@@ -30,6 +30,7 @@ 
 #include "exec/log.h"
 #include "hw/hw.h"
 #include "mmu-book3s-v3.h"
+#include "helper_regs.h"
 
 /* #define DEBUG_SLB */
 
@@ -1125,6 +1126,8 @@  void ppc_store_lpcr(PowerPCCPU *cpu, target_ulong val)
     CPUPPCState *env = &cpu->env;
 
     env->spr[SPR_LPCR] = val & pcc->lpcr_mask;
+    /* The gtse bit affects hflags */
+    hreg_compute_hflags(env);
 }
 
 void helper_store_lpcr(CPUPPCState *env, target_ulong val)
diff --git a/target/ppc/translate.c b/target/ppc/translate.c
index d48c554290..5e629291d3 100644
--- a/target/ppc/translate.c
+++ b/target/ppc/translate.c
@@ -7908,7 +7908,7 @@  static void ppc_tr_init_disas_context(DisasContextBase *dcbase, CPUState *cs)
     ctx->altivec_enabled = (hflags >> HFLAGS_VR) & 1;
     ctx->vsx_enabled = (hflags >> HFLAGS_VSX) & 1;
     ctx->tm_enabled = (hflags >> HFLAGS_TM) & 1;
-    ctx->gtse = !!(env->spr[SPR_LPCR] & LPCR_GTSE);
+    ctx->gtse = (hflags >> HFLAGS_GTSE) & 1;
 
     ctx->singlestep_enabled = 0;
     if ((hflags >> HFLAGS_SE) & 1) {