On Thu, Jul 25, 2013 at 03:53:55PM +0100, Alex Bligh wrote: > Stefan, > > --On 25 July 2013 11:33:43 +0200 Stefan Hajnoczi <stefa...@gmail.com> wrote: > > >> assert(progress || busy); > >>- return true; > >>+ return progress; > > > >Now aio_poll() can return false when it used to return true? > > I don't think that's a bug. > > Firstly, this is the same thing you fixed and we discussed on another > thread.
I'm fine with the change itself but it needs to be explained in the commit message or a comment. In the patch where I changed the semantics of aio_poll() the change is explained in detail in the commit message: http://patchwork.ozlabs.org/patch/261786/ Stefan
