Re: [PATCH v12 2/5] target/ppc: make power8-pmu.c CONFIG_TCG only

[Richard Henderson]

On 2/16/22 21:10, Daniel Henrique Barboza wrote:
  static void init_tcg_pmu_power8(CPUPPCState *env)
  {
-#if defined(TARGET_PPC64) && !defined(CONFIG_USER_ONLY)
+#if defined(CONFIG_TCG)
      /* Init PMU overflow timers */
      if (!kvm_enabled()) {
          cpu_ppc_pmu_init(env);
@@ -7872,10 +7872,9 @@ static void ppc_cpu_reset(DeviceState *dev)
      if (env->mmu_model != POWERPC_MMU_REAL) {
          ppc_tlb_invalidate_all(env);
      }
+    pmu_update_summaries(env);
  #endif /* CONFIG_TCG */
  #endif
-
-    pmu_update_summaries(env);

It looks like you could remove all of the ifdefs if you simply use tcg_enabled() rather 
than !kvm_enabled().  If !defined(CONFIG_TCG), tcg_enabled() will be constant false, and 
the block will be optimized away.


r~