From patchwork Thu Sep 1 01:45:10 2011 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: David Gibson X-Patchwork-Id: 112791 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from lists.gnu.org (lists.gnu.org [140.186.70.17]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id 7AFC9B6F7B for ; Thu, 1 Sep 2011 11:51:22 +1000 (EST) Received: from localhost ([::1]:40779 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QywQz-0003wr-JW for incoming@patchwork.ozlabs.org; Wed, 31 Aug 2011 21:51:13 -0400 Received: from eggs.gnu.org ([140.186.70.92]:59403) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QywQr-0003wc-GF for qemu-devel@nongnu.org; Wed, 31 Aug 2011 21:51:07 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1QywQo-0007D9-47 for qemu-devel@nongnu.org; Wed, 31 Aug 2011 21:51:05 -0400 Received: from ozlabs.org ([203.10.76.45]:33011) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QywQn-0007Cl-G7 for qemu-devel@nongnu.org; Wed, 31 Aug 2011 21:51:02 -0400 Received: by ozlabs.org (Postfix, from userid 1007) id D41E7B6F7C; Thu, 1 Sep 2011 11:50:56 +1000 (EST) Date: Thu, 1 Sep 2011 11:45:10 +1000 From: David Gibson To: Alexander Graf Message-ID: <20110901014510.GH11906@yookeroo.fritz.box> Mail-Followup-To: Alexander Graf , qemu-devel@nongnu.org References: <1312441339-22477-1-git-send-email-david@gibson.dropbear.id.au> <4E42A0D3.1000203@suse.de> <20110811003916.GC6342@yookeroo.fritz.box> <7872027B-E2E0-435F-BE12-65C2E269EA93@suse.de> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <7872027B-E2E0-435F-BE12-65C2E269EA93@suse.de> User-Agent: Mutt/1.5.21 (2010-09-15) X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 203.10.76.45 Cc: qemu-devel@nongnu.org Subject: Re: [Qemu-devel] pseries machine updates X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Sender: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org On Wed, Aug 31, 2011 at 11:17:13AM +0200, Alexander Graf wrote: > > On 11.08.2011, at 02:39, David Gibson wrote: > > > On Wed, Aug 10, 2011 at 05:16:35PM +0200, Alexander Graf wrote: > >> On 08/04/2011 09:02 AM, David Gibson wrote: > >>> Hi Alex, > >>> > >>> Here's another batch of assorted updates for the pseries machine. > >> > >> Looks pretty nice. Please update patch 2/6 with the bug you found > >> and the whitespace problems. I'll put the others into my tree > >> already. > > > > Here's the updated 2/6 > > > > From e5b9ba608d4814a46f256337bbf60b94fdc2c5d9 Mon Sep 17 00:00:00 2001 > > From: Ben Herrenschmidt > > Date: Thu, 4 Aug 2011 16:56:41 +1000 > > Subject: [PATCH] Implement POWER7's CFAR in TCG > > > > This patch implements support for the CFAR SPR on POWER7 (Come From > > Address Register), which snapshots the PC value at the time of a branch or > > an rfid. The latest powerpc-next kernel also catches it and can show it in > > xmon or in the signal frames. > > > > This works well enough to let recent kernels boot (which otherwise oops > > on the CFAR access). It hasn't been tested enough to be confident that the > > CFAR values are actually accurate, but one thing at a time. > > > > Signed-off-by: Ben Herrenschmidt > > Signed-off-by: David Gibson > > agraf@lychee:/home/agraf/release/qemu> git pw am 109480 > ERROR: code indent should never use tabs > #107: FILE: target-ppc/translate.c:162: > +^I^I^I^I offsetof(CPUState, cfar), "cfar");$ > > ERROR: code indent should never use tabs > #174: FILE: target-ppc/translate.c:9289: > +^Icpu_fprintf(f, " CFAR " TARGET_FMT_lx"\n", env->cfar);$ Blah. Fixed now, I'll try to get BenH to use the emacs magic to stop it putting tabs in. > WARNING: space prohibited between function name and open parenthesis '(' > #199: FILE: target-ppc/translate_init.c:134: > +static void spr_read_cfar (void *opaque, int gprn, int sprn) > > WARNING: space prohibited between function name and open parenthesis '(' > #204: FILE: target-ppc/translate_init.c:139: > +static void spr_write_cfar (void *opaque, int sprn, int gprn) Well, these are deliberate, on the grounds that matching the surrounding functions seemed more important than matching the global style guidelines. Revised patch below From b35b94ea867550faf99fc553b661739551c9bb8b Mon Sep 17 00:00:00 2001 From: Ben Herrenschmidt Date: Thu, 4 Aug 2011 16:56:41 +1000 Subject: [PATCH] Implement POWER7's CFAR in TCG This patch implements support for the CFAR SPR on POWER7 (Come From Address Register), which snapshots the PC value at the time of a branch or an rfid. The latest powerpc-next kernel also catches it and can show it in xmon or in the signal frames. This works well enough to let recent kernels boot (which otherwise oops on the CFAR access). It hasn't been tested enough to be confident that the CFAR values are actually accurate, but one thing at a time. Signed-off-by: Ben Herrenschmidt Signed-off-by: David Gibson --- target-ppc/cpu.h | 8 ++++++++ target-ppc/translate.c | 28 ++++++++++++++++++++++++++++ target-ppc/translate_init.c | 23 ++++++++++++++++++++++- 3 files changed, 58 insertions(+), 1 deletions(-) diff --git a/target-ppc/cpu.h b/target-ppc/cpu.h index b8d42e0..145103a 100644 --- a/target-ppc/cpu.h +++ b/target-ppc/cpu.h @@ -555,6 +555,8 @@ enum { /* Decrementer clock: RTC clock (POWER, 601) or bus clock */ POWERPC_FLAG_RTC_CLK = 0x00010000, POWERPC_FLAG_BUS_CLK = 0x00020000, + /* Has CFAR */ + POWERPC_FLAG_CFAR = 0x00040000, }; /*****************************************************************************/ @@ -872,6 +874,10 @@ struct CPUPPCState { target_ulong ctr; /* condition register */ uint32_t crf[8]; +#if defined(TARGET_PPC64) + /* CFAR */ + target_ulong cfar; +#endif /* XER */ target_ulong xer; /* Reservation address */ @@ -1202,6 +1208,7 @@ static inline void cpu_clone_regs(CPUState *env, target_ulong newsp) #define SPR_601_UDECR (0x006) #define SPR_LR (0x008) #define SPR_CTR (0x009) +#define SPR_DSCR (0x011) #define SPR_DSISR (0x012) #define SPR_DAR (0x013) /* DAE for PowerPC 601 */ #define SPR_601_RTCU (0x014) @@ -1210,6 +1217,7 @@ static inline void cpu_clone_regs(CPUState *env, target_ulong newsp) #define SPR_SDR1 (0x019) #define SPR_SRR0 (0x01A) #define SPR_SRR1 (0x01B) +#define SPR_CFAR (0x01C) #define SPR_AMR (0x01D) #define SPR_BOOKE_PID (0x030) #define SPR_BOOKE_DECAR (0x036) diff --git a/target-ppc/translate.c b/target-ppc/translate.c index 4277460..1e362fc 100644 --- a/target-ppc/translate.c +++ b/target-ppc/translate.c @@ -69,6 +69,9 @@ static TCGv cpu_nip; static TCGv cpu_msr; static TCGv cpu_ctr; static TCGv cpu_lr; +#if defined(TARGET_PPC64) +static TCGv cpu_cfar; +#endif static TCGv cpu_xer; static TCGv cpu_reserve; static TCGv_i32 cpu_fpscr; @@ -154,6 +157,11 @@ void ppc_translate_init(void) cpu_lr = tcg_global_mem_new(TCG_AREG0, offsetof(CPUState, lr), "lr"); +#if defined(TARGET_PPC64) + cpu_cfar = tcg_global_mem_new(TCG_AREG0, + offsetof(CPUState, cfar), "cfar"); +#endif + cpu_xer = tcg_global_mem_new(TCG_AREG0, offsetof(CPUState, xer), "xer"); @@ -187,6 +195,7 @@ typedef struct DisasContext { int le_mode; #if defined(TARGET_PPC64) int sf_mode; + int has_cfar; #endif int fpu_enabled; int altivec_enabled; @@ -3345,6 +3354,14 @@ static inline void gen_qemu_st32fiw(DisasContext *ctx, TCGv_i64 arg1, TCGv arg2) /* stfiwx */ GEN_STXF(stfiw, st32fiw, 0x17, 0x1E, PPC_FLOAT_STFIWX); +static inline void gen_update_cfar(DisasContext *ctx, target_ulong nip) +{ +#if defined(TARGET_PPC64) + if (ctx->has_cfar) + tcg_gen_movi_tl(cpu_cfar, nip); +#endif +} + /*** Branch ***/ static inline void gen_goto_tb(DisasContext *ctx, int n, target_ulong dest) { @@ -3407,6 +3424,7 @@ static void gen_b(DisasContext *ctx) target = li; if (LK(ctx->opcode)) gen_setlr(ctx, ctx->nip); + gen_update_cfar(ctx, ctx->nip); gen_goto_tb(ctx, 0, target); } @@ -3469,6 +3487,7 @@ static inline void gen_bcond(DisasContext *ctx, int type) } tcg_temp_free_i32(temp); } + gen_update_cfar(ctx, ctx->nip); if (type == BCOND_IM) { target_ulong li = (target_long)((int16_t)(BD(ctx->opcode))); if (likely(AA(ctx->opcode) == 0)) { @@ -3580,6 +3599,7 @@ static void gen_rfi(DisasContext *ctx) gen_inval_exception(ctx, POWERPC_EXCP_PRIV_OPC); return; } + gen_update_cfar(ctx, ctx->nip); gen_helper_rfi(); gen_sync_exception(ctx); #endif @@ -3596,6 +3616,7 @@ static void gen_rfid(DisasContext *ctx) gen_inval_exception(ctx, POWERPC_EXCP_PRIV_OPC); return; } + gen_update_cfar(ctx, ctx->nip); gen_helper_rfid(); gen_sync_exception(ctx); #endif @@ -9263,6 +9284,12 @@ void cpu_dump_state (CPUState *env, FILE *f, fprintf_function cpu_fprintf, */ } +#if defined(TARGET_PPC64) + if (env->flags & POWERPC_FLAG_CFAR) { + cpu_fprintf(f, " CFAR " TARGET_FMT_lx"\n", env->cfar); + } +#endif + switch (env->mmu_model) { case POWERPC_MMU_32B: case POWERPC_MMU_601: @@ -9371,6 +9398,7 @@ static inline void gen_intermediate_code_internal(CPUState *env, ctx.le_mode = env->hflags & (1 << MSR_LE) ? 1 : 0; #if defined(TARGET_PPC64) ctx.sf_mode = msr_sf; + ctx.has_cfar = !!(env->flags & POWERPC_FLAG_CFAR); #endif ctx.fpu_enabled = msr_fp; if ((env->flags & POWERPC_FLAG_SPE) && msr_spe) diff --git a/target-ppc/translate_init.c b/target-ppc/translate_init.c index 9ea193d..211f3bd 100644 --- a/target-ppc/translate_init.c +++ b/target-ppc/translate_init.c @@ -129,6 +129,19 @@ static void spr_write_lr (void *opaque, int sprn, int gprn) tcg_gen_mov_tl(cpu_lr, cpu_gpr[gprn]); } +/* CFAR */ +#if defined(TARGET_PPC64) && !defined(CONFIG_USER_ONLY) +static void spr_read_cfar (void *opaque, int gprn, int sprn) +{ + tcg_gen_mov_tl(cpu_gpr[gprn], cpu_cfar); +} + +static void spr_write_cfar (void *opaque, int sprn, int gprn) +{ + tcg_gen_mov_tl(cpu_cfar, cpu_gpr[gprn]); +} +#endif /* defined(TARGET_PPC64) && !defined(CONFIG_USER_ONLY) */ + /* CTR */ static void spr_read_ctr (void *opaque, int gprn, int sprn) { @@ -6489,7 +6502,7 @@ static void init_proc_970MP (CPUPPCState *env) #define POWERPC_BFDM_POWER7 (bfd_mach_ppc64) #define POWERPC_FLAG_POWER7 (POWERPC_FLAG_VRE | POWERPC_FLAG_SE | \ POWERPC_FLAG_BE | POWERPC_FLAG_PMM | \ - POWERPC_FLAG_BUS_CLK) + POWERPC_FLAG_BUS_CLK | POWERPC_FLAG_CFAR) #define check_pow_POWER7 check_pow_nocheck static void init_proc_POWER7 (CPUPPCState *env) @@ -6508,6 +6521,14 @@ static void init_proc_POWER7 (CPUPPCState *env) &spr_read_purr, SPR_NOACCESS, &spr_read_purr, SPR_NOACCESS, 0x00000000); + spr_register(env, SPR_CFAR, "SPR_CFAR", + SPR_NOACCESS, SPR_NOACCESS, + &spr_read_cfar, &spr_write_cfar, + 0x00000000); + spr_register(env, SPR_DSCR, "SPR_DSCR", + SPR_NOACCESS, SPR_NOACCESS, + &spr_read_generic, &spr_write_generic, + 0x00000000); #endif /* !CONFIG_USER_ONLY */ /* Memory management */ /* XXX : not implemented */