Re: [Qemu-devel] [PATCH 0/3] Fix "stop" draining block jobs

2015-04-03 Thread Fam Zheng
On Fri, 04/03 13:10, Paolo Bonzini wrote: > > > On 03/04/2015 12:38, Fam Zheng wrote: > > Stopping the vm will drive the block job all the way to the end, because the > > sleep duration is too short, which means the block_job_sleep_ns in the block > > jobs are unhelpful. That is because the timer

Re: [Qemu-devel] [PATCH 0/3] Fix "stop" draining block jobs

2015-04-03 Thread Paolo Bonzini
On 03/04/2015 12:38, Fam Zheng wrote: > Stopping the vm will drive the block job all the way to the end, because the > sleep duration is too short, which means the block_job_sleep_ns in the block > jobs are unhelpful. That is because the timer will fire too soon, even before > the aio_poll in bdr

[Qemu-devel] [PATCH 0/3] Fix "stop" draining block jobs

2015-04-03 Thread Fam Zheng
Stopping the vm will drive the block job all the way to the end, because the sleep duration is too short, which means the block_job_sleep_ns in the block jobs are unhelpful. That is because the timer will fire too soon, even before the aio_poll in bdrv_drain_all returns. Lengthen the sleep and add