1. Patchew
  2. QEMU
  3. [PATCH v5 0/9] Misc clean ups to target/ppc exception handling
  4. View patch

[PATCH v5 6/9] target/ppc: Clean up ifdefs in excp_helper.c, part 1

BALATON Zoltan posted 9 patches 10 months, 1 week ago
Maintainers: Nicholas Piggin <npiggin@gmail.com>, Daniel Henrique Barboza <danielhb413@gmail.com>, "Cédric Le Goater" <clg@kaod.org>
There is a newer version of this series
[PATCH v5 6/9] target/ppc: Clean up ifdefs in excp_helper.c, part 1
Posted by BALATON Zoltan 10 months, 1 week ago 
Use #ifdef, #ifndef for brevity and add comments to #endif that are
more than a few lines apart for clarity.

Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu>
---
 target/ppc/excp_helper.c | 49 ++++++++++++++++++++--------------------
 1 file changed, 24 insertions(+), 25 deletions(-)

diff --git a/target/ppc/excp_helper.c b/target/ppc/excp_helper.c
index 035a9fd968..d8eab4ff6c 100644
--- a/target/ppc/excp_helper.c
+++ b/target/ppc/excp_helper.c
@@ -35,7 +35,7 @@
 
 /*****************************************************************************/
 /* Exception processing */
-#if !defined(CONFIG_USER_ONLY)
+#ifndef CONFIG_USER_ONLY
 
 static const char *powerpc_excp_name(int excp)
 {
@@ -186,7 +186,7 @@ static void ppc_excp_debug_sw_tlb(CPUPPCState *env, int excp)
              env->error_code);
 }
 
-#if defined(TARGET_PPC64)
+#ifdef TARGET_PPC64
 static int powerpc_reset_wakeup(CPUPPCState *env, int excp, target_ulong *msr)
 {
     /* We no longer are in a PM state */
@@ -380,7 +380,7 @@ static void ppc_excp_apply_ail(PowerPCCPU *cpu, int excp, target_ulong msr,
         }
     }
 }
-#endif
+#endif /* TARGET_PPC64 */
 
 static void powerpc_reset_excp_state(PowerPCCPU *cpu)
 {
@@ -1123,7 +1123,7 @@ static void powerpc_excp_booke(PowerPCCPU *cpu, int excp)
         break;
     }
 
-#if defined(TARGET_PPC64)
+#ifdef TARGET_PPC64
     if (env->spr[SPR_BOOKE_EPCR] & EPCR_ICM) {
         /* Cat.64-bit: EPCR.ICM is copied to MSR.CM */
         new_msr |= (target_ulong)1 << MSR_CM;
@@ -1537,7 +1537,7 @@ static inline void powerpc_excp_books(PowerPCCPU *cpu, int excp)
 {
     g_assert_not_reached();
 }
-#endif
+#endif /* TARGET_PPC64 */
 
 static void powerpc_excp(PowerPCCPU *cpu, int excp)
 {
@@ -1588,7 +1588,7 @@ void ppc_cpu_do_interrupt(CPUState *cs)
     powerpc_excp(cpu, cs->exception_index);
 }
 
-#if defined(TARGET_PPC64)
+#ifdef TARGET_PPC64
 #define P7_UNUSED_INTERRUPTS \
     (PPC_INTERRUPT_RESET | PPC_INTERRUPT_HVIRT | PPC_INTERRUPT_CEXT |       \
      PPC_INTERRUPT_WDT | PPC_INTERRUPT_CDOORBELL | PPC_INTERRUPT_FIT |      \
@@ -1919,7 +1919,7 @@ static int p9_next_unmasked_interrupt(CPUPPCState *env)
 
     return 0;
 }
-#endif
+#endif /* TARGET_PPC64 */
 
 static int ppc_next_unmasked_interrupt_generic(CPUPPCState *env)
 {
@@ -2036,7 +2036,7 @@ static int ppc_next_unmasked_interrupt_generic(CPUPPCState *env)
 static int ppc_next_unmasked_interrupt(CPUPPCState *env)
 {
     switch (env->excp_model) {
-#if defined(TARGET_PPC64)
+#ifdef TARGET_PPC64
     case POWERPC_EXCP_POWER7:
         return p7_next_unmasked_interrupt(env);
     case POWERPC_EXCP_POWER8:
@@ -2075,7 +2075,7 @@ void ppc_maybe_interrupt(CPUPPCState *env)
     }
 }
 
-#if defined(TARGET_PPC64)
+#ifdef TARGET_PPC64
 static void p7_deliver_interrupt(CPUPPCState *env, int interrupt)
 {
     PowerPCCPU *cpu = env_archcpu(env);
@@ -2277,7 +2277,7 @@ static void p9_deliver_interrupt(CPUPPCState *env, int interrupt)
                   interrupt);
     }
 }
-#endif
+#endif /* TARGET_PPC64 */
 
 static void ppc_deliver_interrupt_generic(CPUPPCState *env, int interrupt)
 {
@@ -2386,7 +2386,7 @@ static void ppc_deliver_interrupt_generic(CPUPPCState *env, int interrupt)
 static void ppc_deliver_interrupt(CPUPPCState *env, int interrupt)
 {
     switch (env->excp_model) {
-#if defined(TARGET_PPC64)
+#ifdef TARGET_PPC64
     case POWERPC_EXCP_POWER7:
         p7_deliver_interrupt(env, interrupt);
         break;
@@ -2496,9 +2496,9 @@ void helper_raise_exception(CPUPPCState *env, uint32_t exception)
 {
     raise_exception_err_ra(env, exception, 0, 0);
 }
-#endif
+#endif /* CONFIG_TCG */
 
-#if !defined(CONFIG_USER_ONLY)
+#ifndef CONFIG_USER_ONLY
 #ifdef CONFIG_TCG
 void helper_store_msr(CPUPPCState *env, target_ulong val)
 {
@@ -2515,7 +2515,7 @@ void helper_ppc_maybe_interrupt(CPUPPCState *env)
     ppc_maybe_interrupt(env);
 }
 
-#if defined(TARGET_PPC64)
+#ifdef TARGET_PPC64
 void helper_scv(CPUPPCState *env, uint32_t lev)
 {
     if (env->spr[SPR_FSCR] & (1ull << FSCR_SCV)) {
@@ -2544,7 +2544,7 @@ void helper_pminsn(CPUPPCState *env, uint32_t insn)
 
     ppc_maybe_interrupt(env);
 }
-#endif /* defined(TARGET_PPC64) */
+#endif /* TARGET_PPC64 */
 
 static void do_rfi(CPUPPCState *env, target_ulong nip, target_ulong msr)
 {
@@ -2555,7 +2555,7 @@ static void do_rfi(CPUPPCState *env, target_ulong nip, target_ulong msr)
     if (env->flags & POWERPC_FLAG_TGPR)
         msr &= ~(1ULL << MSR_TGPR);
 
-#if defined(TARGET_PPC64)
+#ifdef TARGET_PPC64
     /* Switching to 32-bit ? Crop the nip */
     if (!msr_is_64bit(env, msr)) {
         nip = (uint32_t)nip;
@@ -2584,7 +2584,7 @@ void helper_rfi(CPUPPCState *env)
     do_rfi(env, env->spr[SPR_SRR0], env->spr[SPR_SRR1] & 0xfffffffful);
 }
 
-#if defined(TARGET_PPC64)
+#ifdef TARGET_PPC64
 void helper_rfid(CPUPPCState *env)
 {
     /*
@@ -2605,7 +2605,7 @@ void helper_hrfid(CPUPPCState *env)
 {
     do_rfi(env, env->spr[SPR_HSRR0], env->spr[SPR_HSRR1]);
 }
-#endif
+#endif /* TARGET_PPC64 */
 
 #if defined(TARGET_PPC64) && !defined(CONFIG_USER_ONLY)
 void helper_rfebb(CPUPPCState *env, target_ulong s)
@@ -2682,7 +2682,7 @@ void raise_ebb_perfm_exception(CPUPPCState *env)
 
     do_ebb(env, POWERPC_EXCP_PERFM_EBB);
 }
-#endif
+#endif /* defined(TARGET_PPC64) && !defined(CONFIG_USER_ONLY) */
 
 /*****************************************************************************/
 /* Embedded PowerPC specific helpers */
@@ -2724,7 +2724,7 @@ void helper_tw(CPUPPCState *env, target_ulong arg1, target_ulong arg2,
     }
 }
 
-#if defined(TARGET_PPC64)
+#ifdef TARGET_PPC64
 void helper_td(CPUPPCState *env, target_ulong arg1, target_ulong arg2,
                uint32_t flags)
 {
@@ -2737,8 +2737,8 @@ void helper_td(CPUPPCState *env, target_ulong arg1, target_ulong arg2,
                                POWERPC_EXCP_TRAP, GETPC());
     }
 }
-#endif
-#endif
+#endif /* TARGET_PPC64 */
+#endif /* CONFIG_TCG */
 
 #ifdef CONFIG_TCG
 static uint32_t helper_SIMON_LIKE_32_64(uint32_t x, uint64_t key, uint32_t lane)
@@ -2853,8 +2853,7 @@ HELPER_HASH(HASHSTP, env->spr[SPR_HASHPKEYR], true, PHIE)
 HELPER_HASH(HASHCHKP, env->spr[SPR_HASHPKEYR], false, PHIE)
 #endif /* CONFIG_TCG */
 
-#if !defined(CONFIG_USER_ONLY)
-
+#ifndef CONFIG_USER_ONLY
 #ifdef CONFIG_TCG
 
 /* Embedded.Processor Control */
@@ -2963,7 +2962,7 @@ void helper_book3s_msgsnd(target_ulong rb)
     book3s_msgsnd_common(pir, PPC_INTERRUPT_HDOORBELL);
 }
 
-#if defined(TARGET_PPC64)
+#ifdef TARGET_PPC64
 void helper_book3s_msgclrp(CPUPPCState *env, target_ulong rb)
 {
     helper_hfscr_facility_check(env, HFSCR_MSGP, "msgclrp", HFSCR_IC_MSGP);
-- 
2.30.9
Re: [PATCH v5 6/9] target/ppc: Clean up ifdefs in excp_helper.c, part 1
Posted by Harsh Prateek Bora 9 months, 1 week ago 

On 1/19/24 03:31, BALATON Zoltan wrote:
> Use #ifdef, #ifndef for brevity and add comments to #endif that are
> more than a few lines apart for clarity.
> 
> Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu>

Reviewed-by: Harsh Prateek Bora <harshpb@linux.ibm.com>

> ---
>   target/ppc/excp_helper.c | 49 ++++++++++++++++++++--------------------
>   1 file changed, 24 insertions(+), 25 deletions(-)
> 
> diff --git a/target/ppc/excp_helper.c b/target/ppc/excp_helper.c
> index 035a9fd968..d8eab4ff6c 100644
> --- a/target/ppc/excp_helper.c
> +++ b/target/ppc/excp_helper.c
> @@ -35,7 +35,7 @@
>   
>   /*****************************************************************************/
>   /* Exception processing */
> -#if !defined(CONFIG_USER_ONLY)
> +#ifndef CONFIG_USER_ONLY
>   
>   static const char *powerpc_excp_name(int excp)
>   {
> @@ -186,7 +186,7 @@ static void ppc_excp_debug_sw_tlb(CPUPPCState *env, int excp)
>                env->error_code);
>   }
>   
> -#if defined(TARGET_PPC64)
> +#ifdef TARGET_PPC64
>   static int powerpc_reset_wakeup(CPUPPCState *env, int excp, target_ulong *msr)
>   {
>       /* We no longer are in a PM state */
> @@ -380,7 +380,7 @@ static void ppc_excp_apply_ail(PowerPCCPU *cpu, int excp, target_ulong msr,
>           }
>       }
>   }
> -#endif
> +#endif /* TARGET_PPC64 */
>   
>   static void powerpc_reset_excp_state(PowerPCCPU *cpu)
>   {
> @@ -1123,7 +1123,7 @@ static void powerpc_excp_booke(PowerPCCPU *cpu, int excp)
>           break;
>       }
>   
> -#if defined(TARGET_PPC64)
> +#ifdef TARGET_PPC64
>       if (env->spr[SPR_BOOKE_EPCR] & EPCR_ICM) {
>           /* Cat.64-bit: EPCR.ICM is copied to MSR.CM */
>           new_msr |= (target_ulong)1 << MSR_CM;
> @@ -1537,7 +1537,7 @@ static inline void powerpc_excp_books(PowerPCCPU *cpu, int excp)
>   {
>       g_assert_not_reached();
>   }
> -#endif
> +#endif /* TARGET_PPC64 */
>   
>   static void powerpc_excp(PowerPCCPU *cpu, int excp)
>   {
> @@ -1588,7 +1588,7 @@ void ppc_cpu_do_interrupt(CPUState *cs)
>       powerpc_excp(cpu, cs->exception_index);
>   }
>   
> -#if defined(TARGET_PPC64)
> +#ifdef TARGET_PPC64
>   #define P7_UNUSED_INTERRUPTS \
>       (PPC_INTERRUPT_RESET | PPC_INTERRUPT_HVIRT | PPC_INTERRUPT_CEXT |       \
>        PPC_INTERRUPT_WDT | PPC_INTERRUPT_CDOORBELL | PPC_INTERRUPT_FIT |      \
> @@ -1919,7 +1919,7 @@ static int p9_next_unmasked_interrupt(CPUPPCState *env)
>   
>       return 0;
>   }
> -#endif
> +#endif /* TARGET_PPC64 */
>   
>   static int ppc_next_unmasked_interrupt_generic(CPUPPCState *env)
>   {
> @@ -2036,7 +2036,7 @@ static int ppc_next_unmasked_interrupt_generic(CPUPPCState *env)
>   static int ppc_next_unmasked_interrupt(CPUPPCState *env)
>   {
>       switch (env->excp_model) {
> -#if defined(TARGET_PPC64)
> +#ifdef TARGET_PPC64
>       case POWERPC_EXCP_POWER7:
>           return p7_next_unmasked_interrupt(env);
>       case POWERPC_EXCP_POWER8:
> @@ -2075,7 +2075,7 @@ void ppc_maybe_interrupt(CPUPPCState *env)
>       }
>   }
>   
> -#if defined(TARGET_PPC64)
> +#ifdef TARGET_PPC64
>   static void p7_deliver_interrupt(CPUPPCState *env, int interrupt)
>   {
>       PowerPCCPU *cpu = env_archcpu(env);
> @@ -2277,7 +2277,7 @@ static void p9_deliver_interrupt(CPUPPCState *env, int interrupt)
>                     interrupt);
>       }
>   }
> -#endif
> +#endif /* TARGET_PPC64 */
>   
>   static void ppc_deliver_interrupt_generic(CPUPPCState *env, int interrupt)
>   {
> @@ -2386,7 +2386,7 @@ static void ppc_deliver_interrupt_generic(CPUPPCState *env, int interrupt)
>   static void ppc_deliver_interrupt(CPUPPCState *env, int interrupt)
>   {
>       switch (env->excp_model) {
> -#if defined(TARGET_PPC64)
> +#ifdef TARGET_PPC64
>       case POWERPC_EXCP_POWER7:
>           p7_deliver_interrupt(env, interrupt);
>           break;
> @@ -2496,9 +2496,9 @@ void helper_raise_exception(CPUPPCState *env, uint32_t exception)
>   {
>       raise_exception_err_ra(env, exception, 0, 0);
>   }
> -#endif
> +#endif /* CONFIG_TCG */
>   
> -#if !defined(CONFIG_USER_ONLY)
> +#ifndef CONFIG_USER_ONLY
>   #ifdef CONFIG_TCG
>   void helper_store_msr(CPUPPCState *env, target_ulong val)
>   {
> @@ -2515,7 +2515,7 @@ void helper_ppc_maybe_interrupt(CPUPPCState *env)
>       ppc_maybe_interrupt(env);
>   }
>   
> -#if defined(TARGET_PPC64)
> +#ifdef TARGET_PPC64
>   void helper_scv(CPUPPCState *env, uint32_t lev)
>   {
>       if (env->spr[SPR_FSCR] & (1ull << FSCR_SCV)) {
> @@ -2544,7 +2544,7 @@ void helper_pminsn(CPUPPCState *env, uint32_t insn)
>   
>       ppc_maybe_interrupt(env);
>   }
> -#endif /* defined(TARGET_PPC64) */
> +#endif /* TARGET_PPC64 */
>   
>   static void do_rfi(CPUPPCState *env, target_ulong nip, target_ulong msr)
>   {
> @@ -2555,7 +2555,7 @@ static void do_rfi(CPUPPCState *env, target_ulong nip, target_ulong msr)
>       if (env->flags & POWERPC_FLAG_TGPR)
>           msr &= ~(1ULL << MSR_TGPR);
>   
> -#if defined(TARGET_PPC64)
> +#ifdef TARGET_PPC64
>       /* Switching to 32-bit ? Crop the nip */
>       if (!msr_is_64bit(env, msr)) {
>           nip = (uint32_t)nip;
> @@ -2584,7 +2584,7 @@ void helper_rfi(CPUPPCState *env)
>       do_rfi(env, env->spr[SPR_SRR0], env->spr[SPR_SRR1] & 0xfffffffful);
>   }
>   
> -#if defined(TARGET_PPC64)
> +#ifdef TARGET_PPC64
>   void helper_rfid(CPUPPCState *env)
>   {
>       /*
> @@ -2605,7 +2605,7 @@ void helper_hrfid(CPUPPCState *env)
>   {
>       do_rfi(env, env->spr[SPR_HSRR0], env->spr[SPR_HSRR1]);
>   }
> -#endif
> +#endif /* TARGET_PPC64 */
>   
>   #if defined(TARGET_PPC64) && !defined(CONFIG_USER_ONLY)
>   void helper_rfebb(CPUPPCState *env, target_ulong s)
> @@ -2682,7 +2682,7 @@ void raise_ebb_perfm_exception(CPUPPCState *env)
>   
>       do_ebb(env, POWERPC_EXCP_PERFM_EBB);
>   }
> -#endif
> +#endif /* defined(TARGET_PPC64) && !defined(CONFIG_USER_ONLY) */
>   
>   /*****************************************************************************/
>   /* Embedded PowerPC specific helpers */
> @@ -2724,7 +2724,7 @@ void helper_tw(CPUPPCState *env, target_ulong arg1, target_ulong arg2,
>       }
>   }
>   
> -#if defined(TARGET_PPC64)
> +#ifdef TARGET_PPC64
>   void helper_td(CPUPPCState *env, target_ulong arg1, target_ulong arg2,
>                  uint32_t flags)
>   {
> @@ -2737,8 +2737,8 @@ void helper_td(CPUPPCState *env, target_ulong arg1, target_ulong arg2,
>                                  POWERPC_EXCP_TRAP, GETPC());
>       }
>   }
> -#endif
> -#endif
> +#endif /* TARGET_PPC64 */
> +#endif /* CONFIG_TCG */
>   
>   #ifdef CONFIG_TCG
>   static uint32_t helper_SIMON_LIKE_32_64(uint32_t x, uint64_t key, uint32_t lane)
> @@ -2853,8 +2853,7 @@ HELPER_HASH(HASHSTP, env->spr[SPR_HASHPKEYR], true, PHIE)
>   HELPER_HASH(HASHCHKP, env->spr[SPR_HASHPKEYR], false, PHIE)
>   #endif /* CONFIG_TCG */
>   
> -#if !defined(CONFIG_USER_ONLY)
> -
> +#ifndef CONFIG_USER_ONLY
>   #ifdef CONFIG_TCG
>   
>   /* Embedded.Processor Control */
> @@ -2963,7 +2962,7 @@ void helper_book3s_msgsnd(target_ulong rb)
>       book3s_msgsnd_common(pir, PPC_INTERRUPT_HDOORBELL);
>   }
>   
> -#if defined(TARGET_PPC64)
> +#ifdef TARGET_PPC64
>   void helper_book3s_msgclrp(CPUPPCState *env, target_ulong rb)
>   {
>       helper_hfscr_facility_check(env, HFSCR_MSGP, "msgclrp", HFSCR_IC_MSGP);

Patchew 2.1-devel-b67e4c2

© 2016 - 2024 Red Hat, Inc.