[Libguestfs] [PATCH libnbd] lib: Remove cookie parameter from completion callbacks.
Richard W.M. Jones
rjones at redhat.com
Tue Jul 30 16:28:24 UTC 2019
On Tue, Jul 30, 2019 at 11:21:25AM -0500, Eric Blake wrote:
> On 7/30/19 10:36 AM, Richard W.M. Jones wrote:
> > +When the command completes, C<callback>
> > will be invoked as described in L<libnbd(3)/Completion callbacks>.
> > +
>
> Do we need wording anywhere (probably in the .pod, so we only state it
> once) that mentions that the callback routine is not invoked if
> nbd_aio_FOO_callback returns -1?
I guess it's implicit from the fact that returning -1 indicates an
error.
In fact the relationship ought to be stronger than that - the command
should run if and only if nbd_aio_FOO_callback returns >= 0. (That's
not actually true at the moment because an error can happen after
we've enqueued the command.)
Rich.
--
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Read my programming and virtualization blog: http://rwmj.wordpress.com
virt-df lists disk usage of guests without needing to install any
software inside the virtual machine. Supports Linux and Windows.
http://people.redhat.com/~rjones/virt-df/
More information about the Libguestfs
mailing list