On 27/05/2016 05:07, David Gibson wrote: > On Thu, May 26, 2016 at 12:12:41PM +0200, Paolo Bonzini wrote: >> >> >> On 12/05/2016 05:48, Bharata B Rao wrote: >>> CPUState *cpu gets added to the cpus list during cpu_exec_init(). It >>> should be removed from cpu_exec_exit(). >>> >>> cpu_exec_exit() is called from generic CPU::instance_finalize and some >>> archs like PowerPC call it from CPU unrealizefn. So ensure that we >>> dequeue the cpu only once. >> >> I think the better thing would be to call it from CPU::unrealize, but >> this patch is okay too. >> >> Thanks, >> >> Paolo > > Thanks for the review Paolo. > > However, what I'm really unclear on is what is the next step towards > merging these. Will you take them through your tree? Should Bharata > send a formal pull request with the prelim patches? If so, to whom?
Feel free to take them and add an Acked-by for me. The fewer patches I merge, the better. :) Thanks, Paolo
