[v2] Fix aio_notify_accept()

[Qemu-devel] [PATCH v2 0/2] Fix aio_notify_accept()

Fam Zheng posted 2 patches 7 years, 2 months ago

Diff against v3
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20180807091655.32472-1-famz@redhat.com

Test checkpatch passed

Test docker-mingw@fedora passed

Test docker-clang@ubuntu passed

Test docker-quick@centos7 passed

There is a newer version of this series

util/aio-posix.c | 7 ++++---
util/aio-win32.c | 3 ++-
2 files changed, 6 insertions(+), 4 deletions(-)

Expand all Fold all

[Qemu-devel] [PATCH v2 0/2] Fix aio_notify_accept()

Posted by Fam Zheng 7 years, 2 months ago

v2: Implement the fix following Paolo's idea.
    Testing is still in progress.

Calling aio_notify_accept(iothread->ctx) from main loop when it does
aio_poll(iothread->ctx, false) is a bug because it may steal the event needed
by aio_poll(iothread->ctx, true) in the IOThread. This can cause IOThread
hanging.

Fam Zheng (2):
  aio-posix: Don't count ctx->notifier as progress when polling
  aio: Do aio_notify_accept only during blocking aio_poll

 util/aio-posix.c | 7 ++++---
 util/aio-win32.c | 3 ++-
 2 files changed, 6 insertions(+), 4 deletions(-)

-- 
2.17.1