Re: [Qemu-block] [PATCH v10 01/16] block: Pause all jobs during bdrv_reopen_multiple()

2016-10-11 Thread Kevin Wolf
Am 11.10.2016 um 11:54 hat Paolo Bonzini geschrieben: > On 11/10/2016 11:39, Kevin Wolf wrote: > > Am 10.10.2016 um 18:41 hat Paolo Bonzini geschrieben: > >> On 10/10/2016 17:37, Kevin Wolf wrote: > +while ((job = block_job_next(job))) { > +AioContext *aio_context =

Re: [Qemu-block] [PATCH v10 01/16] block: Pause all jobs during bdrv_reopen_multiple()

2016-10-11 Thread Paolo Bonzini
On 11/10/2016 11:39, Kevin Wolf wrote: > Am 10.10.2016 um 18:41 hat Paolo Bonzini geschrieben: >> On 10/10/2016 17:37, Kevin Wolf wrote: +while ((job = block_job_next(job))) { +AioContext *aio_context = blk_get_aio_context(job->blk); + +

Re: [Qemu-block] [PATCH v10 01/16] block: Pause all jobs during bdrv_reopen_multiple()

2016-10-11 Thread Kevin Wolf
Am 10.10.2016 um 18:41 hat Paolo Bonzini geschrieben: > On 10/10/2016 17:37, Kevin Wolf wrote: > > > +while ((job = block_job_next(job))) { > > > +AioContext *aio_context = blk_get_aio_context(job->blk); > > > + > > > +aio_context_acquire(aio_context); > > > +

Re: [Qemu-block] [PATCH v10 01/16] block: Pause all jobs during bdrv_reopen_multiple()

2016-10-10 Thread Paolo Bonzini
On 10/10/2016 17:37, Kevin Wolf wrote: > > +while ((job = block_job_next(job))) { > > +AioContext *aio_context = blk_get_aio_context(job->blk); > > + > > +aio_context_acquire(aio_context); > > +block_job_pause(job); > > +aio_context_release(aio_context); > > +

Re: [Qemu-block] [PATCH v10 01/16] block: Pause all jobs during bdrv_reopen_multiple()

2016-10-10 Thread Kevin Wolf
Am 06.10.2016 um 15:02 hat Alberto Garcia geschrieben: > When a BlockDriverState is about to be reopened it can trigger certain > operations that need to write to disk. During this process a different > block job can be woken up. If that block job completes and also needs > to call bdrv_reopen()