On 11/08/2015 12:53, Paolo Bonzini wrote:

On 10/08/2015 17:27, fred.kon...@greensocs.com wrote:
@@ -583,5 +587,6 @@ int cpu_exec(CPUState *cpu)
/* fail safe : never use current_cpu outside cpu_exec() */
      current_cpu = NULL;
+    tcg_cpu_allow_execution(cpu);
I don't think this is correct; safe_work_pending() is a much clearer
test.  I'll revert locally to the previous version to play more with the
code.

Paolo

Yes definitely but we might have a race if we just use safe_work_pending().

Fred

      return ret;
  }
diff --git a/include/qom/cpu.h b/include/qom/cpu.h


Reply via email to