On Thu, May 26, 2016 at 12:19:05PM +0200, Paolo Bonzini wrote: > > > On 12/05/2016 05:48, Bharata B Rao wrote: > > @@ -1531,6 +1563,9 @@ static void tcg_exec_all(void) > > break; > > } > > } else if (cpu->stop || cpu->stopped) { > > + if (cpu->unplug) { > > This "if" is unnecessary. next_cpu is not used anymore after the > "break", so the store is dead.
next_cpu is a global here. Won't tcg_exec_all() continue with running the next_cpu when it is called the next time ? Regards, Bharata.