Re: [Qemu-devel] [PATCH 08/10] blockjob: introduce block_job_cancel_async

2017-04-10 Thread Stefan Hajnoczi
On Thu, Mar 23, 2017 at 06:39:26PM +0100, Paolo Bonzini wrote: > Signed-off-by: Paolo Bonzini > --- > blockjob.c | 11 --- > 1 file changed, 8 insertions(+), 3 deletions(-) Reviewed-by: Stefan Hajnoczi signature.asc Description: PGP signature

Re: [Qemu-devel] [PATCH 08/10] blockjob: introduce block_job_cancel_async

2017-04-08 Thread Paolo Bonzini
On 08/04/2017 09:13, John Snow wrote: > > > On 03/23/2017 01:39 PM, Paolo Bonzini wrote: >> Signed-off-by: Paolo Bonzini > > What was the bad design that required you to fix the previous test? :) It was actually patch 9 that had the bug, I'll reorder to match the commit message. Paolo >> -

Re: [Qemu-devel] [PATCH 08/10] blockjob: introduce block_job_cancel_async

2017-04-07 Thread John Snow
On 03/23/2017 01:39 PM, Paolo Bonzini wrote: > Signed-off-by: Paolo Bonzini What was the bad design that required you to fix the previous test? :) > --- > blockjob.c | 11 --- > 1 file changed, 8 insertions(+), 3 deletions(-) > > diff --git a/blockjob.c b/blockjob.c > index c9cb5b1..

[Qemu-devel] [PATCH 08/10] blockjob: introduce block_job_cancel_async

2017-03-23 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- blockjob.c | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/blockjob.c b/blockjob.c index c9cb5b1..093962b 100644 --- a/blockjob.c +++ b/blockjob.c @@ -373,6 +373,12 @@ static void block_job_completed_single(BlockJob *job) block