Hi On Tue, Nov 7, 2017 at 11:37 PM, Eric Blake <ebl...@redhat.com> wrote: > co_sleep_ns() was removed in commit 0b9caf9b, leaving behind a > stale comment. Update the documentation to match the current > usage of this function. > > Suggested-by: Paolo Bonzini <pbonz...@redhat.com> > Signed-off-by: Eric Blake <ebl...@redhat.com> > --- > include/qemu/coroutine.h | 7 +++++-- > 1 file changed, 5 insertions(+), 2 deletions(-) > > diff --git a/include/qemu/coroutine.h b/include/qemu/coroutine.h > index 9aff9a735e..01ae415767 100644 > --- a/include/qemu/coroutine.h > +++ b/include/qemu/coroutine.h > @@ -262,8 +262,11 @@ void qemu_co_rwlock_unlock(CoRwlock *lock); > /** > * Yield the coroutine for a given duration > * > - * Behaves similarly to co_sleep_ns(), but the sleeping coroutine will be > - * resumed when using aio_poll(). > + * This function uses timers and hence needs to know the event loop > + * (#AioContext) to place the timer on. In any case, co_aio_sleep_ns() > + * does not affect the #AioContext where the current coroutine is running, > + * as the coroutine will restart on the same #AioContext that it is > + * running on. > */
comment seems correct to me, Reviewed-by: Marc-André Lureau <marcandre.lur...@redhat.com> > void coroutine_fn co_aio_sleep_ns(AioContext *ctx, QEMUClockType type, > int64_t ns); > -- > 2.13.6 > > -- Marc-André Lureau