[Qemu-devel] [PATCH 0/3] aio-posix: polling mode bug fixes

Paolo Bonzini posted 3 patches 7 years, 1 month ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20180912171040.1732-1-pbonzini@redhat.com
Test docker-clang@ubuntu failed
Test checkpatch passed
util/aio-posix.c  | 88 +++++++++++++++++++++++++++--------------------
util/trace-events |  4 +--
2 files changed, 53 insertions(+), 39 deletions(-)
[Qemu-devel] [PATCH 0/3] aio-posix: polling mode bug fixes
Posted by Paolo Bonzini 7 years, 1 month ago
Patch 1 fixes a too-strict assertion that could fire when aio_poll
is called in parallel with aio_set_fd_handler.

Patch 2 and 3 reinstate the performance benefits of polling, which were
essentially disabled by commit 70232b5253 ("aio-posix: Don't count
ctx->notifier as progress when polling", 2018-08-15).

Please review carefully! :)

Thanks,

Paolo

Paolo Bonzini (3):
  aio-posix: fix concurrent access to poll_disable_cnt
  aio-posix: compute timeout before polling
  aio-posix: do skip system call if ctx->notifier polling succeeds

 util/aio-posix.c  | 88 +++++++++++++++++++++++++++--------------------
 util/trace-events |  4 +--
 2 files changed, 53 insertions(+), 39 deletions(-)

-- 
2.17.1


Re: [Qemu-devel] [PATCH 0/3] aio-posix: polling mode bug fixes
Posted by Fam Zheng 7 years, 1 month ago
On Wed, 09/12 19:10, Paolo Bonzini wrote:
> Patch 1 fixes a too-strict assertion that could fire when aio_poll
> is called in parallel with aio_set_fd_handler.
> 
> Patch 2 and 3 reinstate the performance benefits of polling, which were
> essentially disabled by commit 70232b5253 ("aio-posix: Don't count
> ctx->notifier as progress when polling", 2018-08-15).
> 
> Please review carefully! :)

Queued, thanks!

Fam