Re: [Qemu-devel] [PATCH] mirror: Fix coroutine reentrance

2015-08-14 Thread Jeff Cody
On Thu, Aug 13, 2015 at 10:41:50AM +0200, Kevin Wolf wrote: > This fixes a regression introduced by commit dcfb3beb ("mirror: Do zero > write on target if sectors not allocated"), which was reported to cause > aborts with the message "Co-routine re-entered recursively". > > The cause for this bug

Re: [Qemu-devel] [PATCH] mirror: Fix coroutine reentrance

2015-08-14 Thread Jeff Cody
On Thu, Aug 13, 2015 at 10:41:50AM +0200, Kevin Wolf wrote: > This fixes a regression introduced by commit dcfb3beb ("mirror: Do zero > write on target if sectors not allocated"), which was reported to cause > aborts with the message "Co-routine re-entered recursively". > > The cause for this bug

Re: [Qemu-devel] [PATCH] mirror: Fix coroutine reentrance

2015-08-14 Thread Stefan Hajnoczi
On Thu, Aug 13, 2015 at 10:41:50AM +0200, Kevin Wolf wrote: > This fixes a regression introduced by commit dcfb3beb ("mirror: Do zero > write on target if sectors not allocated"), which was reported to cause > aborts with the message "Co-routine re-entered recursively". > > The cause for this bug

Re: [Qemu-devel] [PATCH] mirror: Fix coroutine reentrance

2015-08-13 Thread Paolo Bonzini
On 13/08/2015 10:41, Kevin Wolf wrote: > This fixes a regression introduced by commit dcfb3beb ("mirror: Do zero > write on target if sectors not allocated"), which was reported to cause > aborts with the message "Co-routine re-entered recursively". > > The cause for this bug is the following co

[Qemu-devel] [PATCH] mirror: Fix coroutine reentrance

2015-08-13 Thread Kevin Wolf
This fixes a regression introduced by commit dcfb3beb ("mirror: Do zero write on target if sectors not allocated"), which was reported to cause aborts with the message "Co-routine re-entered recursively". The cause for this bug is the following code in mirror_iteration_done(): if (s->common.b