target-ppc: add flag in check_tlb_flush()
We flush the qemu TLB lazily. check_tlb_flush is called whenever we hit a context synchronizing event or instruction that requires a pending flush to be performed. However, we fail to handle broadcast TLB flush operations. In order to fix that efficiently, we want to differentiate whether check_tlb_flush() needs to only apply pending local flushes (isync instructions, interrupts, ...) or also global pending flush operations. The latter is only needed when executing instructions that are defined architecturally as synchronizing global TLB flush operations. This in our case is ptesync on BookS and tlbsync on BookE along with the paravirtualized hypervisor calls. Signed-off-by: Nikunj A Dadhania <nikunj@linux.vnet.ibm.com> [dwg: Changed gen_check_tlb_flush() to also take a bool, and fixed some spelling errors in commit message] Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
This commit is contained in:
parent
a8a6d53e36
commit
e3cffe6fad
|
@ -201,7 +201,7 @@ static target_ulong h_remove(PowerPCCPU *cpu, sPAPRMachineState *spapr,
|
|||
|
||||
switch (ret) {
|
||||
case REMOVE_SUCCESS:
|
||||
check_tlb_flush(env);
|
||||
check_tlb_flush(env, true);
|
||||
return H_SUCCESS;
|
||||
|
||||
case REMOVE_NOT_FOUND:
|
||||
|
@ -282,7 +282,7 @@ static target_ulong h_bulk_remove(PowerPCCPU *cpu, sPAPRMachineState *spapr,
|
|||
}
|
||||
}
|
||||
exit:
|
||||
check_tlb_flush(env);
|
||||
check_tlb_flush(env, true);
|
||||
|
||||
return rc;
|
||||
}
|
||||
|
|
|
@ -711,7 +711,7 @@ static inline void powerpc_excp(PowerPCCPU *cpu, int excp_model, int excp)
|
|||
/* Any interrupt is context synchronizing, check if TCG TLB
|
||||
* needs a delayed flush on ppc64
|
||||
*/
|
||||
check_tlb_flush(env);
|
||||
check_tlb_flush(env, false);
|
||||
}
|
||||
|
||||
void ppc_cpu_do_interrupt(CPUState *cs)
|
||||
|
@ -973,7 +973,7 @@ static inline void do_rfi(CPUPPCState *env, target_ulong nip, target_ulong msr)
|
|||
cs->interrupt_request |= CPU_INTERRUPT_EXITTB;
|
||||
|
||||
/* Context synchronizing: check if TCG TLB needs flush */
|
||||
check_tlb_flush(env);
|
||||
check_tlb_flush(env, false);
|
||||
}
|
||||
|
||||
void helper_rfi(CPUPPCState *env)
|
||||
|
|
|
@ -18,7 +18,8 @@ DEF_HELPER_1(rfid, void, env)
|
|||
DEF_HELPER_1(hrfid, void, env)
|
||||
DEF_HELPER_2(store_lpcr, void, env, tl)
|
||||
#endif
|
||||
DEF_HELPER_1(check_tlb_flush, void, env)
|
||||
DEF_HELPER_1(check_tlb_flush_local, void, env)
|
||||
DEF_HELPER_1(check_tlb_flush_global, void, env)
|
||||
#endif
|
||||
|
||||
DEF_HELPER_3(lmw, void, env, tl, i32)
|
||||
|
|
|
@ -154,7 +154,7 @@ static inline int hreg_store_msr(CPUPPCState *env, target_ulong value,
|
|||
}
|
||||
|
||||
#if !defined(CONFIG_USER_ONLY)
|
||||
static inline void check_tlb_flush(CPUPPCState *env)
|
||||
static inline void check_tlb_flush(CPUPPCState *env, bool global)
|
||||
{
|
||||
CPUState *cs = CPU(ppc_env_get_cpu(env));
|
||||
if (env->tlb_need_flush & TLB_NEED_LOCAL_FLUSH) {
|
||||
|
@ -163,7 +163,7 @@ static inline void check_tlb_flush(CPUPPCState *env)
|
|||
}
|
||||
}
|
||||
#else
|
||||
static inline void check_tlb_flush(CPUPPCState *env) { }
|
||||
static inline void check_tlb_flush(CPUPPCState *env, bool global) { }
|
||||
#endif
|
||||
|
||||
#endif /* HELPER_REGS_H */
|
||||
|
|
|
@ -2867,9 +2867,14 @@ void helper_booke206_tlbflush(CPUPPCState *env, target_ulong type)
|
|||
}
|
||||
|
||||
|
||||
void helper_check_tlb_flush(CPUPPCState *env)
|
||||
void helper_check_tlb_flush_local(CPUPPCState *env)
|
||||
{
|
||||
check_tlb_flush(env);
|
||||
check_tlb_flush(env, false);
|
||||
}
|
||||
|
||||
void helper_check_tlb_flush_global(CPUPPCState *env)
|
||||
{
|
||||
check_tlb_flush(env, true);
|
||||
}
|
||||
|
||||
/*****************************************************************************/
|
||||
|
|
|
@ -3041,7 +3041,7 @@ static void gen_eieio(DisasContext *ctx)
|
|||
}
|
||||
|
||||
#if !defined(CONFIG_USER_ONLY)
|
||||
static inline void gen_check_tlb_flush(DisasContext *ctx)
|
||||
static inline void gen_check_tlb_flush(DisasContext *ctx, bool global)
|
||||
{
|
||||
TCGv_i32 t;
|
||||
TCGLabel *l;
|
||||
|
@ -3053,12 +3053,16 @@ static inline void gen_check_tlb_flush(DisasContext *ctx)
|
|||
t = tcg_temp_new_i32();
|
||||
tcg_gen_ld_i32(t, cpu_env, offsetof(CPUPPCState, tlb_need_flush));
|
||||
tcg_gen_brcondi_i32(TCG_COND_EQ, t, 0, l);
|
||||
gen_helper_check_tlb_flush(cpu_env);
|
||||
if (global) {
|
||||
gen_helper_check_tlb_flush_global(cpu_env);
|
||||
} else {
|
||||
gen_helper_check_tlb_flush_local(cpu_env);
|
||||
}
|
||||
gen_set_label(l);
|
||||
tcg_temp_free_i32(t);
|
||||
}
|
||||
#else
|
||||
static inline void gen_check_tlb_flush(DisasContext *ctx) { }
|
||||
static inline void gen_check_tlb_flush(DisasContext *ctx, bool global) { }
|
||||
#endif
|
||||
|
||||
/* isync */
|
||||
|
@ -3069,7 +3073,7 @@ static void gen_isync(DisasContext *ctx)
|
|||
* kernel mode however so check MSR_PR
|
||||
*/
|
||||
if (!ctx->pr) {
|
||||
gen_check_tlb_flush(ctx);
|
||||
gen_check_tlb_flush(ctx, false);
|
||||
}
|
||||
gen_stop_exception(ctx);
|
||||
}
|
||||
|
@ -3249,7 +3253,7 @@ static void gen_sync(DisasContext *ctx)
|
|||
* check MSR_PR as well.
|
||||
*/
|
||||
if (((l == 2) || !(ctx->insns_flags & PPC_64B)) && !ctx->pr) {
|
||||
gen_check_tlb_flush(ctx);
|
||||
gen_check_tlb_flush(ctx, true);
|
||||
}
|
||||
}
|
||||
|
||||
|
@ -4458,11 +4462,10 @@ static void gen_tlbsync(DisasContext *ctx)
|
|||
#else
|
||||
CHK_HV;
|
||||
|
||||
/* tlbsync is a nop for server, ptesync handles delayed tlb flush,
|
||||
* embedded however needs to deal with tlbsync. We don't try to be
|
||||
* fancy and swallow the overhead of checking for both.
|
||||
*/
|
||||
gen_check_tlb_flush(ctx);
|
||||
/* BookS does both ptesync and tlbsync make tlbsync a nop for server */
|
||||
if (ctx->insns_flags & PPC_BOOKE) {
|
||||
gen_check_tlb_flush(ctx, true);
|
||||
}
|
||||
#endif /* defined(CONFIG_USER_ONLY) */
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in a new issue