Re: [Qemu-devel] [PATCH 05/11] blockjob: separate monitor and blockjob APIs

2017-05-10 Thread Paolo Bonzini
On 09/05/2017 19:06, Jeff Cody wrote: >> Keep the two APIs separate in the blockjob.c file too. This will >> be useful when transitioning away from the AioContext lock, because >> there will be locking policies for the two categories, too---the >> monitor will have to call new block_job_lock/unl

Re: [Qemu-devel] [PATCH 05/11] blockjob: separate monitor and blockjob APIs

2017-05-09 Thread Jeff Cody
On Mon, May 08, 2017 at 04:13:04PM +0200, Paolo Bonzini wrote: > We have two different headers for block job operations, blockjob.h > and blockjob_int.h. The former contains APIs called by the monitor, > the latter contains APIs called by the block job drivers and the > block layer itself. > > Ke

[Qemu-devel] [PATCH 05/11] blockjob: separate monitor and blockjob APIs

2017-05-08 Thread Paolo Bonzini
We have two different headers for block job operations, blockjob.h and blockjob_int.h. The former contains APIs called by the monitor, the latter contains APIs called by the block job drivers and the block layer itself. Keep the two APIs separate in the blockjob.c file too. This will be useful w

Re: [Qemu-devel] [PATCH 05/11] blockjob: separate monitor and blockjob APIs

2017-05-04 Thread Stefan Hajnoczi
On Wed, Apr 19, 2017 at 04:42:13PM +0200, Paolo Bonzini wrote: > We have two different headers for block job operations, blockjob.h > and blockjob_int.h. The former contains APIs called by the monitor, > the latter contains APIs called by the block job drivers and the > block layer itself. > > Ke

Re: [Qemu-devel] [PATCH 05/11] blockjob: separate monitor and blockjob APIs

2017-04-24 Thread John Snow
On 04/19/2017 10:42 AM, Paolo Bonzini wrote: > We have two different headers for block job operations, blockjob.h > and blockjob_int.h. The former contains APIs called by the monitor, > the latter contains APIs called by the block job drivers and the > block layer itself. > > Keep the two APIs

[Qemu-devel] [PATCH 05/11] blockjob: separate monitor and blockjob APIs

2017-04-19 Thread Paolo Bonzini
We have two different headers for block job operations, blockjob.h and blockjob_int.h. The former contains APIs called by the monitor, the latter contains APIs called by the block job drivers and the block layer itself. Keep the two APIs separate in the blockjob.c file too. This will be useful w