|
 |
1bdc94 |
From 8c5d6c7bdc1cd5557503764ff8187d39bed3cbe1 Mon Sep 17 00:00:00 2001
|
|
|
1bdc94 |
From: Fam Zheng <famz@redhat.com>
|
|
|
1bdc94 |
Date: Fri, 17 Aug 2018 03:08:35 +0200
|
|
|
1bdc94 |
Subject: [PATCH 4/5] aio-posix: Don't count ctx->notifier as progress when
|
|
|
1bdc94 |
polling
|
|
|
1bdc94 |
|
|
|
1bdc94 |
RH-Author: Fam Zheng <famz@redhat.com>
|
|
|
1bdc94 |
Message-id: <20180817030836.20581-2-famz@redhat.com>
|
|
|
1bdc94 |
Patchwork-id: 81861
|
|
|
1bdc94 |
O-Subject: [RHEL-7.6 qemu-kvm-rhev PATCH 1/2] aio-posix: Don't count ctx->notifier as progress when polling
|
|
|
1bdc94 |
Bugzilla: 1562750
|
|
|
1bdc94 |
RH-Acked-by: Paolo Bonzini <pbonzini@redhat.com>
|
|
|
1bdc94 |
RH-Acked-by: Thomas Huth <thuth@redhat.com>
|
|
|
1bdc94 |
RH-Acked-by: Laszlo Ersek <lersek@redhat.com>
|
|
|
1bdc94 |
|
|
|
1bdc94 |
The same logic exists in fd polling. This change is especially important
|
|
|
1bdc94 |
to avoid busy loop once we limit aio_notify_accept() to blocking
|
|
|
1bdc94 |
aio_poll().
|
|
|
1bdc94 |
|
|
|
1bdc94 |
Cc: qemu-stable@nongnu.org
|
|
|
1bdc94 |
Signed-off-by: Fam Zheng <famz@redhat.com>
|
|
|
1bdc94 |
Message-Id: <20180809132259.18402-2-famz@redhat.com>
|
|
|
1bdc94 |
Signed-off-by: Fam Zheng <famz@redhat.com>
|
|
|
1bdc94 |
(cherry picked from commit 70232b5253a3c4e03ed1ac47ef9246a8ac66c6fa)
|
|
|
1bdc94 |
Signed-off-by: Fam Zheng <famz@redhat.com>
|
|
|
1bdc94 |
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
1bdc94 |
---
|
|
|
1bdc94 |
util/aio-posix.c | 3 ++-
|
|
|
1bdc94 |
1 file changed, 2 insertions(+), 1 deletion(-)
|
|
|
1bdc94 |
|
|
|
1bdc94 |
diff --git a/util/aio-posix.c b/util/aio-posix.c
|
|
|
1bdc94 |
index d8f0cb4..f650c7c 100644
|
|
|
1bdc94 |
--- a/util/aio-posix.c
|
|
|
1bdc94 |
+++ b/util/aio-posix.c
|
|
|
1bdc94 |
@@ -494,7 +494,8 @@ static bool run_poll_handlers_once(AioContext *ctx)
|
|
|
1bdc94 |
QLIST_FOREACH_RCU(node, &ctx->aio_handlers, node) {
|
|
|
1bdc94 |
if (!node->deleted && node->io_poll &&
|
|
|
1bdc94 |
aio_node_check(ctx, node->is_external) &&
|
|
|
1bdc94 |
- node->io_poll(node->opaque)) {
|
|
|
1bdc94 |
+ node->io_poll(node->opaque) &&
|
|
|
1bdc94 |
+ node->opaque != &ctx->notifier) {
|
|
|
1bdc94 |
progress = true;
|
|
|
1bdc94 |
}
|
|
|
1bdc94 |
|
|
|
1bdc94 |
--
|
|
|
1bdc94 |
1.8.3.1
|
|
|
1bdc94 |
|