[v3] Fix aio_notify_accept()

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

Fam Zheng posted 2 patches 7 years, 2 months ago

Diff against v2
Download series mbox

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

Test checkpatch passed

Test docker-mingw@fedora passed

Test docker-clang@ubuntu passed

Test docker-quick@centos7 passed

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

Expand all Fold all

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

Posted by Fam Zheng 7 years, 2 months ago

v3: Fix commit message's bug description. [Paolo]

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

Re: [Qemu-devel] [PATCH v3 0/2] Fix aio_notify_accept()

Posted by Fam Zheng 7 years, 2 months ago

On Thu, 08/09 21:22, Fam Zheng wrote:
> v3: Fix commit message's bug description. [Paolo]

If there's no objection, I'm queuing this for 3.1.

Fam

Re: [Qemu-devel] [PATCH v3 0/2] Fix aio_notify_accept()

Posted by Paolo Bonzini 7 years, 2 months ago

On 14/08/2018 04:50, Fam Zheng wrote:
> On Thu, 08/09 21:22, Fam Zheng wrote:
>> v3: Fix commit message's bug description. [Paolo]
> 
> If there's no objection, I'm queuing this for 3.1.

Sure, thanks.

Paolo