2008-09-22 23:17:18 +04:00
|
|
|
/*
|
|
|
|
* QEMU aio implementation
|
|
|
|
*
|
|
|
|
* Copyright IBM, Corp. 2008
|
|
|
|
*
|
|
|
|
* Authors:
|
|
|
|
* Anthony Liguori <aliguori@us.ibm.com>
|
|
|
|
*
|
|
|
|
* This work is licensed under the terms of the GNU GPL, version 2. See
|
|
|
|
* the COPYING file in the top-level directory.
|
|
|
|
*
|
2012-01-13 20:44:23 +04:00
|
|
|
* Contributions after 2012-01-13 are licensed under the terms of the
|
|
|
|
* GNU GPL, version 2 or (at your option) any later version.
|
2008-09-22 23:17:18 +04:00
|
|
|
*/
|
|
|
|
|
2016-01-29 20:50:05 +03:00
|
|
|
#include "qemu/osdep.h"
|
2008-09-22 23:17:18 +04:00
|
|
|
#include "qemu-common.h"
|
2012-12-17 21:19:44 +04:00
|
|
|
#include "block/block.h"
|
2017-01-12 21:07:57 +03:00
|
|
|
#include "qemu/rcu_queue.h"
|
2012-12-17 21:20:00 +04:00
|
|
|
#include "qemu/sockets.h"
|
2016-12-01 22:26:42 +03:00
|
|
|
#include "qemu/cutils.h"
|
2017-02-13 16:52:18 +03:00
|
|
|
#include "trace.h"
|
2016-02-23 18:42:14 +03:00
|
|
|
#ifdef CONFIG_EPOLL_CREATE1
|
2015-10-30 07:06:29 +03:00
|
|
|
#include <sys/epoll.h>
|
|
|
|
#endif
|
2008-09-22 23:17:18 +04:00
|
|
|
|
|
|
|
struct AioHandler
|
|
|
|
{
|
2012-09-24 16:57:22 +04:00
|
|
|
GPollFD pfd;
|
2008-09-22 23:17:18 +04:00
|
|
|
IOHandler *io_read;
|
|
|
|
IOHandler *io_write;
|
2016-12-01 22:26:42 +03:00
|
|
|
AioPollFn *io_poll;
|
2016-12-01 22:26:49 +03:00
|
|
|
IOHandler *io_poll_begin;
|
|
|
|
IOHandler *io_poll_end;
|
2008-09-22 23:17:18 +04:00
|
|
|
int deleted;
|
|
|
|
void *opaque;
|
2015-10-23 06:08:05 +03:00
|
|
|
bool is_external;
|
2009-09-12 11:36:22 +04:00
|
|
|
QLIST_ENTRY(AioHandler) node;
|
2008-09-22 23:17:18 +04:00
|
|
|
};
|
|
|
|
|
2016-02-23 18:42:14 +03:00
|
|
|
#ifdef CONFIG_EPOLL_CREATE1
|
2015-10-30 07:06:29 +03:00
|
|
|
|
|
|
|
/* The fd number threashold to switch to epoll */
|
|
|
|
#define EPOLL_ENABLE_THRESHOLD 64
|
|
|
|
|
|
|
|
static void aio_epoll_disable(AioContext *ctx)
|
|
|
|
{
|
2018-05-17 03:42:43 +03:00
|
|
|
ctx->epoll_enabled = false;
|
|
|
|
if (!ctx->epoll_available) {
|
2015-10-30 07:06:29 +03:00
|
|
|
return;
|
|
|
|
}
|
2018-05-17 03:42:43 +03:00
|
|
|
ctx->epoll_available = false;
|
2015-10-30 07:06:29 +03:00
|
|
|
close(ctx->epollfd);
|
|
|
|
}
|
|
|
|
|
|
|
|
static inline int epoll_events_from_pfd(int pfd_events)
|
|
|
|
{
|
|
|
|
return (pfd_events & G_IO_IN ? EPOLLIN : 0) |
|
|
|
|
(pfd_events & G_IO_OUT ? EPOLLOUT : 0) |
|
|
|
|
(pfd_events & G_IO_HUP ? EPOLLHUP : 0) |
|
|
|
|
(pfd_events & G_IO_ERR ? EPOLLERR : 0);
|
|
|
|
}
|
|
|
|
|
|
|
|
static bool aio_epoll_try_enable(AioContext *ctx)
|
|
|
|
{
|
|
|
|
AioHandler *node;
|
|
|
|
struct epoll_event event;
|
|
|
|
|
2017-01-12 21:07:57 +03:00
|
|
|
QLIST_FOREACH_RCU(node, &ctx->aio_handlers, node) {
|
2015-10-30 07:06:29 +03:00
|
|
|
int r;
|
|
|
|
if (node->deleted || !node->pfd.events) {
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
event.events = epoll_events_from_pfd(node->pfd.events);
|
|
|
|
event.data.ptr = node;
|
|
|
|
r = epoll_ctl(ctx->epollfd, EPOLL_CTL_ADD, node->pfd.fd, &event);
|
|
|
|
if (r) {
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
ctx->epoll_enabled = true;
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
|
|
|
|
static void aio_epoll_update(AioContext *ctx, AioHandler *node, bool is_new)
|
|
|
|
{
|
|
|
|
struct epoll_event event;
|
|
|
|
int r;
|
2016-11-08 16:55:24 +03:00
|
|
|
int ctl;
|
2015-10-30 07:06:29 +03:00
|
|
|
|
|
|
|
if (!ctx->epoll_enabled) {
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
if (!node->pfd.events) {
|
2016-11-08 16:55:24 +03:00
|
|
|
ctl = EPOLL_CTL_DEL;
|
2015-10-30 07:06:29 +03:00
|
|
|
} else {
|
|
|
|
event.data.ptr = node;
|
|
|
|
event.events = epoll_events_from_pfd(node->pfd.events);
|
2016-11-08 16:55:24 +03:00
|
|
|
ctl = is_new ? EPOLL_CTL_ADD : EPOLL_CTL_MOD;
|
|
|
|
}
|
|
|
|
|
|
|
|
r = epoll_ctl(ctx->epollfd, ctl, node->pfd.fd, &event);
|
|
|
|
if (r) {
|
|
|
|
aio_epoll_disable(ctx);
|
2015-10-30 07:06:29 +03:00
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
static int aio_epoll(AioContext *ctx, GPollFD *pfds,
|
|
|
|
unsigned npfd, int64_t timeout)
|
|
|
|
{
|
|
|
|
AioHandler *node;
|
|
|
|
int i, ret = 0;
|
|
|
|
struct epoll_event events[128];
|
|
|
|
|
|
|
|
assert(npfd == 1);
|
|
|
|
assert(pfds[0].fd == ctx->epollfd);
|
|
|
|
if (timeout > 0) {
|
|
|
|
ret = qemu_poll_ns(pfds, npfd, timeout);
|
|
|
|
}
|
|
|
|
if (timeout <= 0 || ret > 0) {
|
|
|
|
ret = epoll_wait(ctx->epollfd, events,
|
2017-07-18 09:09:59 +03:00
|
|
|
ARRAY_SIZE(events),
|
2015-10-30 07:06:29 +03:00
|
|
|
timeout);
|
|
|
|
if (ret <= 0) {
|
|
|
|
goto out;
|
|
|
|
}
|
|
|
|
for (i = 0; i < ret; i++) {
|
|
|
|
int ev = events[i].events;
|
|
|
|
node = events[i].data.ptr;
|
|
|
|
node->pfd.revents = (ev & EPOLLIN ? G_IO_IN : 0) |
|
|
|
|
(ev & EPOLLOUT ? G_IO_OUT : 0) |
|
|
|
|
(ev & EPOLLHUP ? G_IO_HUP : 0) |
|
|
|
|
(ev & EPOLLERR ? G_IO_ERR : 0);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
out:
|
|
|
|
return ret;
|
|
|
|
}
|
|
|
|
|
|
|
|
static bool aio_epoll_enabled(AioContext *ctx)
|
|
|
|
{
|
|
|
|
/* Fall back to ppoll when external clients are disabled. */
|
|
|
|
return !aio_external_disabled(ctx) && ctx->epoll_enabled;
|
|
|
|
}
|
|
|
|
|
|
|
|
static bool aio_epoll_check_poll(AioContext *ctx, GPollFD *pfds,
|
|
|
|
unsigned npfd, int64_t timeout)
|
|
|
|
{
|
|
|
|
if (!ctx->epoll_available) {
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
if (aio_epoll_enabled(ctx)) {
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
if (npfd >= EPOLL_ENABLE_THRESHOLD) {
|
|
|
|
if (aio_epoll_try_enable(ctx)) {
|
|
|
|
return true;
|
|
|
|
} else {
|
|
|
|
aio_epoll_disable(ctx);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
|
|
|
#else
|
|
|
|
|
|
|
|
static void aio_epoll_update(AioContext *ctx, AioHandler *node, bool is_new)
|
|
|
|
{
|
|
|
|
}
|
|
|
|
|
|
|
|
static int aio_epoll(AioContext *ctx, GPollFD *pfds,
|
|
|
|
unsigned npfd, int64_t timeout)
|
|
|
|
{
|
|
|
|
assert(false);
|
|
|
|
}
|
|
|
|
|
|
|
|
static bool aio_epoll_enabled(AioContext *ctx)
|
|
|
|
{
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
|
|
|
static bool aio_epoll_check_poll(AioContext *ctx, GPollFD *pfds,
|
|
|
|
unsigned npfd, int64_t timeout)
|
|
|
|
{
|
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
|
|
|
#endif
|
|
|
|
|
2012-09-13 14:28:51 +04:00
|
|
|
static AioHandler *find_aio_handler(AioContext *ctx, int fd)
|
2008-09-22 23:17:18 +04:00
|
|
|
{
|
|
|
|
AioHandler *node;
|
|
|
|
|
2012-09-13 14:28:51 +04:00
|
|
|
QLIST_FOREACH(node, &ctx->aio_handlers, node) {
|
2012-09-24 16:57:22 +04:00
|
|
|
if (node->pfd.fd == fd)
|
2009-05-06 04:58:48 +04:00
|
|
|
if (!node->deleted)
|
|
|
|
return node;
|
2008-09-22 23:17:18 +04:00
|
|
|
}
|
|
|
|
|
|
|
|
return NULL;
|
|
|
|
}
|
|
|
|
|
2012-09-13 14:28:51 +04:00
|
|
|
void aio_set_fd_handler(AioContext *ctx,
|
|
|
|
int fd,
|
2015-10-23 06:08:05 +03:00
|
|
|
bool is_external,
|
2012-09-13 14:28:51 +04:00
|
|
|
IOHandler *io_read,
|
|
|
|
IOHandler *io_write,
|
2016-12-01 22:26:41 +03:00
|
|
|
AioPollFn *io_poll,
|
2012-09-13 14:28:51 +04:00
|
|
|
void *opaque)
|
2008-09-22 23:17:18 +04:00
|
|
|
{
|
|
|
|
AioHandler *node;
|
2015-10-30 07:06:29 +03:00
|
|
|
bool is_new = false;
|
2015-11-16 09:32:14 +03:00
|
|
|
bool deleted = false;
|
2008-09-22 23:17:18 +04:00
|
|
|
|
2017-01-12 21:07:57 +03:00
|
|
|
qemu_lockcnt_lock(&ctx->list_lock);
|
|
|
|
|
2012-09-13 14:28:51 +04:00
|
|
|
node = find_aio_handler(ctx, fd);
|
2008-09-22 23:17:18 +04:00
|
|
|
|
|
|
|
/* Are we deleting the fd handler? */
|
2016-12-01 22:26:42 +03:00
|
|
|
if (!io_read && !io_write && !io_poll) {
|
2016-11-08 16:55:23 +03:00
|
|
|
if (node == NULL) {
|
2017-01-12 21:07:57 +03:00
|
|
|
qemu_lockcnt_unlock(&ctx->list_lock);
|
2016-11-08 16:55:23 +03:00
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
2017-09-28 05:59:58 +03:00
|
|
|
/* If the GSource is in the process of being destroyed then
|
|
|
|
* g_source_remove_poll() causes an assertion failure. Skip
|
|
|
|
* removal in that case, because glib cleans up its state during
|
|
|
|
* destruction anyway.
|
|
|
|
*/
|
|
|
|
if (!g_source_is_destroyed(&ctx->source)) {
|
|
|
|
g_source_remove_poll(&ctx->source, &node->pfd);
|
|
|
|
}
|
2016-11-08 16:55:23 +03:00
|
|
|
|
2018-08-03 09:39:17 +03:00
|
|
|
/* If a read is in progress, just mark the node as deleted */
|
2017-01-12 21:07:57 +03:00
|
|
|
if (qemu_lockcnt_count(&ctx->list_lock)) {
|
2016-11-08 16:55:23 +03:00
|
|
|
node->deleted = 1;
|
|
|
|
node->pfd.revents = 0;
|
|
|
|
} else {
|
|
|
|
/* Otherwise, delete it for real. We can't just mark it as
|
2017-01-12 21:07:57 +03:00
|
|
|
* deleted because deleted nodes are only cleaned up while
|
|
|
|
* no one is walking the handlers list.
|
2016-11-08 16:55:23 +03:00
|
|
|
*/
|
|
|
|
QLIST_REMOVE(node, node);
|
|
|
|
deleted = true;
|
2008-09-22 23:17:18 +04:00
|
|
|
}
|
2016-12-01 22:26:42 +03:00
|
|
|
|
|
|
|
if (!node->io_poll) {
|
|
|
|
ctx->poll_disable_cnt--;
|
|
|
|
}
|
2008-09-22 23:17:18 +04:00
|
|
|
} else {
|
|
|
|
if (node == NULL) {
|
|
|
|
/* Alloc and insert if it's not already there */
|
2014-12-04 15:55:09 +03:00
|
|
|
node = g_new0(AioHandler, 1);
|
2012-09-24 16:57:22 +04:00
|
|
|
node->pfd.fd = fd;
|
2017-01-12 21:07:57 +03:00
|
|
|
QLIST_INSERT_HEAD_RCU(&ctx->aio_handlers, node, node);
|
2012-09-24 16:57:41 +04:00
|
|
|
|
|
|
|
g_source_add_poll(&ctx->source, &node->pfd);
|
2015-10-30 07:06:29 +03:00
|
|
|
is_new = true;
|
2016-12-01 22:26:42 +03:00
|
|
|
|
|
|
|
ctx->poll_disable_cnt += !io_poll;
|
|
|
|
} else {
|
|
|
|
ctx->poll_disable_cnt += !io_poll - !node->io_poll;
|
2008-09-22 23:17:18 +04:00
|
|
|
}
|
2016-12-01 22:26:42 +03:00
|
|
|
|
2008-09-22 23:17:18 +04:00
|
|
|
/* Update handler with latest information */
|
|
|
|
node->io_read = io_read;
|
|
|
|
node->io_write = io_write;
|
2016-12-01 22:26:42 +03:00
|
|
|
node->io_poll = io_poll;
|
2008-09-22 23:17:18 +04:00
|
|
|
node->opaque = opaque;
|
2015-10-23 06:08:05 +03:00
|
|
|
node->is_external = is_external;
|
2012-09-24 16:57:22 +04:00
|
|
|
|
2013-02-20 14:28:33 +04:00
|
|
|
node->pfd.events = (io_read ? G_IO_IN | G_IO_HUP | G_IO_ERR : 0);
|
|
|
|
node->pfd.events |= (io_write ? G_IO_OUT | G_IO_ERR : 0);
|
2008-09-22 23:17:18 +04:00
|
|
|
}
|
2012-09-25 12:22:39 +04:00
|
|
|
|
2015-10-30 07:06:29 +03:00
|
|
|
aio_epoll_update(ctx, node, is_new);
|
2017-01-12 21:07:57 +03:00
|
|
|
qemu_lockcnt_unlock(&ctx->list_lock);
|
2012-09-25 12:22:39 +04:00
|
|
|
aio_notify(ctx);
|
2016-12-01 22:26:42 +03:00
|
|
|
|
2015-11-16 09:32:14 +03:00
|
|
|
if (deleted) {
|
|
|
|
g_free(node);
|
|
|
|
}
|
2012-06-09 05:44:00 +04:00
|
|
|
}
|
|
|
|
|
2016-12-01 22:26:49 +03:00
|
|
|
void aio_set_fd_poll(AioContext *ctx, int fd,
|
|
|
|
IOHandler *io_poll_begin,
|
|
|
|
IOHandler *io_poll_end)
|
|
|
|
{
|
|
|
|
AioHandler *node = find_aio_handler(ctx, fd);
|
|
|
|
|
|
|
|
if (!node) {
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
node->io_poll_begin = io_poll_begin;
|
|
|
|
node->io_poll_end = io_poll_end;
|
|
|
|
}
|
|
|
|
|
2012-09-13 14:28:51 +04:00
|
|
|
void aio_set_event_notifier(AioContext *ctx,
|
|
|
|
EventNotifier *notifier,
|
2015-10-23 06:08:05 +03:00
|
|
|
bool is_external,
|
2016-12-01 22:26:41 +03:00
|
|
|
EventNotifierHandler *io_read,
|
|
|
|
AioPollFn *io_poll)
|
2008-09-22 23:17:18 +04:00
|
|
|
{
|
2016-12-01 22:26:41 +03:00
|
|
|
aio_set_fd_handler(ctx, event_notifier_get_fd(notifier), is_external,
|
|
|
|
(IOHandler *)io_read, NULL, io_poll, notifier);
|
2008-09-22 23:17:18 +04:00
|
|
|
}
|
|
|
|
|
2016-12-01 22:26:49 +03:00
|
|
|
void aio_set_event_notifier_poll(AioContext *ctx,
|
|
|
|
EventNotifier *notifier,
|
|
|
|
EventNotifierHandler *io_poll_begin,
|
|
|
|
EventNotifierHandler *io_poll_end)
|
|
|
|
{
|
|
|
|
aio_set_fd_poll(ctx, event_notifier_get_fd(notifier),
|
|
|
|
(IOHandler *)io_poll_begin,
|
|
|
|
(IOHandler *)io_poll_end);
|
|
|
|
}
|
|
|
|
|
|
|
|
static void poll_set_started(AioContext *ctx, bool started)
|
|
|
|
{
|
|
|
|
AioHandler *node;
|
|
|
|
|
|
|
|
if (started == ctx->poll_started) {
|
|
|
|
return;
|
|
|
|
}
|
|
|
|
|
|
|
|
ctx->poll_started = started;
|
|
|
|
|
2017-01-12 21:07:57 +03:00
|
|
|
qemu_lockcnt_inc(&ctx->list_lock);
|
|
|
|
QLIST_FOREACH_RCU(node, &ctx->aio_handlers, node) {
|
2016-12-01 22:26:49 +03:00
|
|
|
IOHandler *fn;
|
|
|
|
|
|
|
|
if (node->deleted) {
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (started) {
|
|
|
|
fn = node->io_poll_begin;
|
|
|
|
} else {
|
|
|
|
fn = node->io_poll_end;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (fn) {
|
|
|
|
fn(node->opaque);
|
|
|
|
}
|
|
|
|
}
|
2017-01-12 21:07:57 +03:00
|
|
|
qemu_lockcnt_dec(&ctx->list_lock);
|
2016-12-01 22:26:49 +03:00
|
|
|
}
|
|
|
|
|
|
|
|
|
2014-07-09 13:53:08 +04:00
|
|
|
bool aio_prepare(AioContext *ctx)
|
|
|
|
{
|
2016-12-01 22:26:49 +03:00
|
|
|
/* Poll mode cannot be used with glib's event loop, disable it. */
|
|
|
|
poll_set_started(ctx, false);
|
|
|
|
|
2014-07-09 13:53:08 +04:00
|
|
|
return false;
|
|
|
|
}
|
|
|
|
|
2012-09-24 16:57:22 +04:00
|
|
|
bool aio_pending(AioContext *ctx)
|
|
|
|
{
|
|
|
|
AioHandler *node;
|
2017-01-12 21:07:57 +03:00
|
|
|
bool result = false;
|
2012-09-24 16:57:22 +04:00
|
|
|
|
2017-01-12 21:07:57 +03:00
|
|
|
/*
|
|
|
|
* We have to walk very carefully in case aio_set_fd_handler is
|
|
|
|
* called while we're walking.
|
|
|
|
*/
|
|
|
|
qemu_lockcnt_inc(&ctx->list_lock);
|
|
|
|
|
|
|
|
QLIST_FOREACH_RCU(node, &ctx->aio_handlers, node) {
|
2012-09-24 16:57:22 +04:00
|
|
|
int revents;
|
|
|
|
|
|
|
|
revents = node->pfd.revents & node->pfd.events;
|
2016-04-22 16:53:55 +03:00
|
|
|
if (revents & (G_IO_IN | G_IO_HUP | G_IO_ERR) && node->io_read &&
|
|
|
|
aio_node_check(ctx, node->is_external)) {
|
2017-01-12 21:07:57 +03:00
|
|
|
result = true;
|
|
|
|
break;
|
2012-09-24 16:57:22 +04:00
|
|
|
}
|
2016-04-22 16:53:55 +03:00
|
|
|
if (revents & (G_IO_OUT | G_IO_ERR) && node->io_write &&
|
|
|
|
aio_node_check(ctx, node->is_external)) {
|
2017-01-12 21:07:57 +03:00
|
|
|
result = true;
|
|
|
|
break;
|
2012-09-24 16:57:22 +04:00
|
|
|
}
|
|
|
|
}
|
2017-01-12 21:07:57 +03:00
|
|
|
qemu_lockcnt_dec(&ctx->list_lock);
|
2012-09-24 16:57:22 +04:00
|
|
|
|
2017-01-12 21:07:57 +03:00
|
|
|
return result;
|
2012-09-24 16:57:22 +04:00
|
|
|
}
|
|
|
|
|
2017-01-12 21:07:55 +03:00
|
|
|
static bool aio_dispatch_handlers(AioContext *ctx)
|
2008-09-22 23:17:18 +04:00
|
|
|
{
|
2017-01-12 21:07:56 +03:00
|
|
|
AioHandler *node, *tmp;
|
2013-02-20 14:28:31 +04:00
|
|
|
bool progress = false;
|
2012-09-24 16:37:53 +04:00
|
|
|
|
2017-01-12 21:07:57 +03:00
|
|
|
QLIST_FOREACH_SAFE_RCU(node, &ctx->aio_handlers, node, tmp) {
|
2017-01-12 21:07:56 +03:00
|
|
|
int revents;
|
2012-09-24 16:57:22 +04:00
|
|
|
|
|
|
|
revents = node->pfd.revents & node->pfd.events;
|
|
|
|
node->pfd.revents = 0;
|
|
|
|
|
2013-02-20 14:28:31 +04:00
|
|
|
if (!node->deleted &&
|
|
|
|
(revents & (G_IO_IN | G_IO_HUP | G_IO_ERR)) &&
|
2016-04-22 16:53:55 +03:00
|
|
|
aio_node_check(ctx, node->is_external) &&
|
2013-02-20 14:28:31 +04:00
|
|
|
node->io_read) {
|
2012-09-24 16:57:22 +04:00
|
|
|
node->io_read(node->opaque);
|
2013-04-11 18:56:50 +04:00
|
|
|
|
|
|
|
/* aio_notify() does not count as progress */
|
|
|
|
if (node->opaque != &ctx->notifier) {
|
|
|
|
progress = true;
|
|
|
|
}
|
2012-09-24 16:57:22 +04:00
|
|
|
}
|
2013-02-20 14:28:31 +04:00
|
|
|
if (!node->deleted &&
|
|
|
|
(revents & (G_IO_OUT | G_IO_ERR)) &&
|
2016-04-22 16:53:55 +03:00
|
|
|
aio_node_check(ctx, node->is_external) &&
|
2013-02-20 14:28:31 +04:00
|
|
|
node->io_write) {
|
2012-09-24 16:57:22 +04:00
|
|
|
node->io_write(node->opaque);
|
|
|
|
progress = true;
|
|
|
|
}
|
|
|
|
|
2017-01-12 21:07:56 +03:00
|
|
|
if (node->deleted) {
|
2017-01-12 21:07:57 +03:00
|
|
|
if (qemu_lockcnt_dec_if_lock(&ctx->list_lock)) {
|
2017-01-12 21:07:56 +03:00
|
|
|
QLIST_REMOVE(node, node);
|
|
|
|
g_free(node);
|
2017-01-12 21:07:57 +03:00
|
|
|
qemu_lockcnt_inc_and_unlock(&ctx->list_lock);
|
2017-01-12 21:07:56 +03:00
|
|
|
}
|
2012-09-24 16:57:22 +04:00
|
|
|
}
|
|
|
|
}
|
2013-08-21 19:02:53 +04:00
|
|
|
|
2017-01-12 21:07:55 +03:00
|
|
|
return progress;
|
|
|
|
}
|
|
|
|
|
2017-02-13 16:52:33 +03:00
|
|
|
void aio_dispatch(AioContext *ctx)
|
2017-01-12 21:07:55 +03:00
|
|
|
{
|
2017-02-13 16:52:33 +03:00
|
|
|
qemu_lockcnt_inc(&ctx->list_lock);
|
2017-02-13 16:52:34 +03:00
|
|
|
aio_bh_poll(ctx);
|
2017-02-13 16:52:33 +03:00
|
|
|
aio_dispatch_handlers(ctx);
|
|
|
|
qemu_lockcnt_dec(&ctx->list_lock);
|
2013-08-21 19:02:53 +04:00
|
|
|
|
2017-02-13 16:52:33 +03:00
|
|
|
timerlistgroup_run_timers(&ctx->tlg);
|
2013-02-20 14:28:31 +04:00
|
|
|
}
|
|
|
|
|
2015-02-20 19:26:50 +03:00
|
|
|
/* These thread-local variables are used only in a small part of aio_poll
|
|
|
|
* around the call to the poll() system call. In particular they are not
|
|
|
|
* used while aio_poll is performing callbacks, which makes it much easier
|
|
|
|
* to think about reentrancy!
|
|
|
|
*
|
|
|
|
* Stack-allocated arrays would be perfect but they have size limitations;
|
|
|
|
* heap allocation is expensive enough that we want to reuse arrays across
|
|
|
|
* calls to aio_poll(). And because poll() has to be called without holding
|
|
|
|
* any lock, the arrays cannot be stored in AioContext. Thread-local data
|
|
|
|
* has none of the disadvantages of these three options.
|
|
|
|
*/
|
|
|
|
static __thread GPollFD *pollfds;
|
|
|
|
static __thread AioHandler **nodes;
|
|
|
|
static __thread unsigned npfd, nalloc;
|
|
|
|
static __thread Notifier pollfds_cleanup_notifier;
|
|
|
|
|
|
|
|
static void pollfds_cleanup(Notifier *n, void *unused)
|
|
|
|
{
|
|
|
|
g_assert(npfd == 0);
|
|
|
|
g_free(pollfds);
|
|
|
|
g_free(nodes);
|
|
|
|
nalloc = 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
static void add_pollfd(AioHandler *node)
|
|
|
|
{
|
|
|
|
if (npfd == nalloc) {
|
|
|
|
if (nalloc == 0) {
|
|
|
|
pollfds_cleanup_notifier.notify = pollfds_cleanup;
|
|
|
|
qemu_thread_atexit_add(&pollfds_cleanup_notifier);
|
|
|
|
nalloc = 8;
|
|
|
|
} else {
|
|
|
|
g_assert(nalloc <= INT_MAX);
|
|
|
|
nalloc *= 2;
|
|
|
|
}
|
|
|
|
pollfds = g_renew(GPollFD, pollfds, nalloc);
|
|
|
|
nodes = g_renew(AioHandler *, nodes, nalloc);
|
|
|
|
}
|
|
|
|
nodes[npfd] = node;
|
|
|
|
pollfds[npfd] = (GPollFD) {
|
|
|
|
.fd = node->pfd.fd,
|
|
|
|
.events = node->pfd.events,
|
|
|
|
};
|
|
|
|
npfd++;
|
|
|
|
}
|
|
|
|
|
2016-12-01 22:26:49 +03:00
|
|
|
static bool run_poll_handlers_once(AioContext *ctx)
|
|
|
|
{
|
|
|
|
bool progress = false;
|
|
|
|
AioHandler *node;
|
|
|
|
|
2017-01-12 21:07:57 +03:00
|
|
|
QLIST_FOREACH_RCU(node, &ctx->aio_handlers, node) {
|
2016-12-01 22:26:49 +03:00
|
|
|
if (!node->deleted && node->io_poll &&
|
2017-01-24 12:53:50 +03:00
|
|
|
aio_node_check(ctx, node->is_external) &&
|
2018-08-09 16:22:58 +03:00
|
|
|
node->io_poll(node->opaque) &&
|
|
|
|
node->opaque != &ctx->notifier) {
|
2016-12-01 22:26:49 +03:00
|
|
|
progress = true;
|
|
|
|
}
|
|
|
|
|
|
|
|
/* Caller handles freeing deleted nodes. Don't do it here. */
|
|
|
|
}
|
|
|
|
|
|
|
|
return progress;
|
|
|
|
}
|
|
|
|
|
2016-12-01 22:26:42 +03:00
|
|
|
/* run_poll_handlers:
|
|
|
|
* @ctx: the AioContext
|
|
|
|
* @max_ns: maximum time to poll for, in nanoseconds
|
|
|
|
*
|
|
|
|
* Polls for a given time.
|
|
|
|
*
|
|
|
|
* Note that ctx->notify_me must be non-zero so this function can detect
|
|
|
|
* aio_notify().
|
|
|
|
*
|
2017-01-12 21:07:57 +03:00
|
|
|
* Note that the caller must have incremented ctx->list_lock.
|
2016-12-01 22:26:42 +03:00
|
|
|
*
|
|
|
|
* Returns: true if progress was made, false otherwise
|
|
|
|
*/
|
|
|
|
static bool run_poll_handlers(AioContext *ctx, int64_t max_ns)
|
|
|
|
{
|
2016-12-01 22:26:49 +03:00
|
|
|
bool progress;
|
2016-12-01 22:26:42 +03:00
|
|
|
int64_t end_time;
|
|
|
|
|
|
|
|
assert(ctx->notify_me);
|
2017-01-12 21:07:57 +03:00
|
|
|
assert(qemu_lockcnt_count(&ctx->list_lock) > 0);
|
2016-12-01 22:26:42 +03:00
|
|
|
assert(ctx->poll_disable_cnt == 0);
|
|
|
|
|
|
|
|
trace_run_poll_handlers_begin(ctx, max_ns);
|
|
|
|
|
|
|
|
end_time = qemu_clock_get_ns(QEMU_CLOCK_REALTIME) + max_ns;
|
|
|
|
|
|
|
|
do {
|
2016-12-01 22:26:49 +03:00
|
|
|
progress = run_poll_handlers_once(ctx);
|
2016-12-01 22:26:42 +03:00
|
|
|
} while (!progress && qemu_clock_get_ns(QEMU_CLOCK_REALTIME) < end_time);
|
|
|
|
|
|
|
|
trace_run_poll_handlers_end(ctx, progress);
|
|
|
|
|
|
|
|
return progress;
|
|
|
|
}
|
|
|
|
|
|
|
|
/* try_poll_mode:
|
|
|
|
* @ctx: the AioContext
|
2016-12-01 22:26:49 +03:00
|
|
|
* @blocking: busy polling is only attempted when blocking is true
|
2016-12-01 22:26:42 +03:00
|
|
|
*
|
2016-12-01 22:26:49 +03:00
|
|
|
* ctx->notify_me must be non-zero so this function can detect aio_notify().
|
2016-12-01 22:26:42 +03:00
|
|
|
*
|
2017-01-12 21:07:57 +03:00
|
|
|
* Note that the caller must have incremented ctx->list_lock.
|
2016-12-01 22:26:42 +03:00
|
|
|
*
|
|
|
|
* Returns: true if progress was made, false otherwise
|
|
|
|
*/
|
|
|
|
static bool try_poll_mode(AioContext *ctx, bool blocking)
|
|
|
|
{
|
|
|
|
if (blocking && ctx->poll_max_ns && ctx->poll_disable_cnt == 0) {
|
|
|
|
/* See qemu_soonest_timeout() uint64_t hack */
|
|
|
|
int64_t max_ns = MIN((uint64_t)aio_compute_timeout(ctx),
|
2016-12-01 22:26:51 +03:00
|
|
|
(uint64_t)ctx->poll_ns);
|
2016-12-01 22:26:42 +03:00
|
|
|
|
|
|
|
if (max_ns) {
|
2016-12-01 22:26:49 +03:00
|
|
|
poll_set_started(ctx, true);
|
|
|
|
|
2016-12-01 22:26:42 +03:00
|
|
|
if (run_poll_handlers(ctx, max_ns)) {
|
|
|
|
return true;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2016-12-01 22:26:49 +03:00
|
|
|
poll_set_started(ctx, false);
|
|
|
|
|
|
|
|
/* Even if we don't run busy polling, try polling once in case it can make
|
|
|
|
* progress and the caller will be able to avoid ppoll(2)/epoll_wait(2).
|
|
|
|
*/
|
|
|
|
return run_poll_handlers_once(ctx);
|
2016-12-01 22:26:42 +03:00
|
|
|
}
|
|
|
|
|
2013-02-20 14:28:31 +04:00
|
|
|
bool aio_poll(AioContext *ctx, bool blocking)
|
|
|
|
{
|
|
|
|
AioHandler *node;
|
2016-12-01 22:26:42 +03:00
|
|
|
int i;
|
|
|
|
int ret = 0;
|
2013-04-11 18:56:50 +04:00
|
|
|
bool progress;
|
2015-02-20 19:26:50 +03:00
|
|
|
int64_t timeout;
|
2016-12-01 22:26:51 +03:00
|
|
|
int64_t start = 0;
|
2013-02-20 14:28:31 +04:00
|
|
|
|
2014-07-07 17:18:04 +04:00
|
|
|
/* aio_notify can avoid the expensive event_notifier_set if
|
|
|
|
* everything (file descriptors, bottom halves, timers) will
|
2014-07-09 13:53:05 +04:00
|
|
|
* be re-evaluated before the next blocking poll(). This is
|
|
|
|
* already true when aio_poll is called with blocking == false;
|
AioContext: fix broken ctx->dispatching optimization
This patch rewrites the ctx->dispatching optimization, which was the cause
of some mysterious hangs that could be reproduced on aarch64 KVM only.
The hangs were indirectly caused by aio_poll() and in particular by
flash memory updates's call to blk_write(), which invokes aio_poll().
Fun stuff: they had an extremely short race window, so much that
adding all kind of tracing to either the kernel or QEMU made it
go away (a single printf made it half as reproducible).
On the plus side, the failure mode (a hang until the next keypress)
made it very easy to examine the state of the process with a debugger.
And there was a very nice reproducer from Laszlo, which failed pretty
often (more than half of the time) on any version of QEMU with a non-debug
kernel; it also failed fast, while still in the firmware. So, it could
have been worse.
For some unknown reason they happened only with virtio-scsi, but
that's not important. It's more interesting that they disappeared with
io=native, making thread-pool.c a likely suspect for where the bug arose.
thread-pool.c is also one of the few places which use bottom halves
across threads, by the way.
I hope that no other similar bugs exist, but just in case :) I am
going to describe how the successful debugging went... Since the
likely culprit was the ctx->dispatching optimization, which mostly
affects bottom halves, the first observation was that there are two
qemu_bh_schedule() invocations in the thread pool: the one in the aio
worker and the one in thread_pool_completion_bh. The latter always
causes the optimization to trigger, the former may or may not. In
order to restrict the possibilities, I introduced new functions
qemu_bh_schedule_slow() and qemu_bh_schedule_fast():
/* qemu_bh_schedule_slow: */
ctx = bh->ctx;
bh->idle = 0;
if (atomic_xchg(&bh->scheduled, 1) == 0) {
event_notifier_set(&ctx->notifier);
}
/* qemu_bh_schedule_fast: */
ctx = bh->ctx;
bh->idle = 0;
assert(ctx->dispatching);
atomic_xchg(&bh->scheduled, 1);
Notice how the atomic_xchg is still in qemu_bh_schedule_slow(). This
was already debated a few months ago, so I assumed it to be correct.
In retrospect this was a very good idea, as you'll see later.
Changing thread_pool_completion_bh() to qemu_bh_schedule_fast() didn't
trigger the assertion (as expected). Changing the worker's invocation
to qemu_bh_schedule_slow() didn't hide the bug (another assumption
which luckily held). This already limited heavily the amount of
interaction between the threads, hinting that the problematic events
must have triggered around thread_pool_completion_bh().
As mentioned early, invoking a debugger to examine the state of a
hung process was pretty easy; the iothread was always waiting on a
poll(..., -1) system call. Infinite timeouts are much rarer on x86,
and this could be the reason why the bug was never observed there.
With the buggy sequence more or less resolved to an interaction between
thread_pool_completion_bh() and poll(..., -1), my "tracing" strategy was
to just add a few qemu_clock_get_ns(QEMU_CLOCK_REALTIME) calls, hoping
that the ordering of aio_ctx_prepare(), aio_ctx_dispatch, poll() and
qemu_bh_schedule_fast() would provide some hint. The output was:
(gdb) p last_prepare
$3 = 103885451
(gdb) p last_dispatch
$4 = 103876492
(gdb) p last_poll
$5 = 115909333
(gdb) p last_schedule
$6 = 115925212
Notice how the last call to qemu_poll_ns() came after aio_ctx_dispatch().
This makes little sense unless there is an aio_poll() call involved,
and indeed with a slightly different instrumentation you can see that
there is one:
(gdb) p last_prepare
$3 = 107569679
(gdb) p last_dispatch
$4 = 107561600
(gdb) p last_aio_poll
$5 = 110671400
(gdb) p last_schedule
$6 = 110698917
So the scenario becomes clearer:
iothread VCPU thread
--------------------------------------------------------------------------
aio_ctx_prepare
aio_ctx_check
qemu_poll_ns(timeout=-1)
aio_poll
aio_dispatch
thread_pool_completion_bh
qemu_bh_schedule()
At this point bh->scheduled = 1 and the iothread has not been woken up.
The solution must be close, but this alone should not be a problem,
because the bottom half is only rescheduled to account for rare situations
(see commit 3c80ca1, thread-pool: avoid deadlock in nested aio_poll()
calls, 2014-07-15).
Introducing a third thread---a thread pool worker thread, which
also does qemu_bh_schedule()---does bring out the problematic case.
The third thread must be awakened *after* the callback is complete and
thread_pool_completion_bh has redone the whole loop, explaining the
short race window. And then this is what happens:
thread pool worker
--------------------------------------------------------------------------
<I/O completes>
qemu_bh_schedule()
Tada, bh->scheduled is already 1, so qemu_bh_schedule() does nothing
and the iothread is never woken up. This is where the bh->scheduled
optimization comes into play---it is correct, but removing it would
have masked the bug.
So, what is the bug?
Well, the question asked by the ctx->dispatching optimization ("is any
active aio_poll dispatching?") was wrong. The right question to ask
instead is "is any active aio_poll *not* dispatching", i.e. in the prepare
or poll phases? In that case, the aio_poll is sleeping or might go to
sleep anytime soon, and the EventNotifier must be invoked to wake
it up.
In any other case (including if there is *no* active aio_poll at all!)
we can just wait for the next prepare phase to pick up the event (e.g. a
bottom half); the prepare phase will avoid the blocking and service the
bottom half.
Expressing the invariant with a logic formula, the broken one looked like:
!(exists(thread): in_dispatching(thread)) => !optimize
or equivalently:
!(exists(thread):
in_aio_poll(thread) && in_dispatching(thread)) => !optimize
In the correct one, the negation is in a slightly different place:
(exists(thread):
in_aio_poll(thread) && !in_dispatching(thread)) => !optimize
or equivalently:
(exists(thread): in_prepare_or_poll(thread)) => !optimize
Even if the difference boils down to moving an exclamation mark :)
the implementation is quite different. However, I think the new
one is simpler to understand.
In the old implementation, the "exists" was implemented with a boolean
value. This didn't really support well the case of multiple concurrent
event loops, but I thought that this was okay: aio_poll holds the
AioContext lock so there cannot be concurrent aio_poll invocations, and
I was just considering nested event loops. However, aio_poll _could_
indeed be concurrent with the GSource. This is why I came up with the
wrong invariant.
In the new implementation, "exists" is computed simply by counting how many
threads are in the prepare or poll phases. There are some interesting
points to consider, but the gist of the idea remains:
1) AioContext can be used through GSource as well; as mentioned in the
patch, bit 0 of the counter is reserved for the GSource.
2) the counter need not be updated for a non-blocking aio_poll, because
it won't sleep forever anyway. This is just a matter of checking
the "blocking" variable. This requires some changes to the win32
implementation, but is otherwise not too complicated.
3) as mentioned above, the new implementation will not call aio_notify
when there is *no* active aio_poll at all. The tests have to be
adjusted for this change. The calls to aio_notify in async.c are fine;
they only want to kick aio_poll out of a blocking wait, but need not
do anything if aio_poll is not running.
4) nested aio_poll: these just work with the new implementation; when
a nested event loop is invoked, the outer event loop is never in the
prepare or poll phases. The outer event loop thus has already decremented
the counter.
Reported-by: Richard W. M. Jones <rjones@redhat.com>
Reported-by: Laszlo Ersek <lersek@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Reviewed-by: Fam Zheng <famz@redhat.com>
Tested-by: Richard W.M. Jones <rjones@redhat.com>
Message-id: 1437487673-23740-5-git-send-email-pbonzini@redhat.com
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
2015-07-21 17:07:51 +03:00
|
|
|
* if blocking == true, it is only true after poll() returns,
|
|
|
|
* so disable the optimization now.
|
2014-07-07 17:18:04 +04:00
|
|
|
*/
|
AioContext: fix broken ctx->dispatching optimization
This patch rewrites the ctx->dispatching optimization, which was the cause
of some mysterious hangs that could be reproduced on aarch64 KVM only.
The hangs were indirectly caused by aio_poll() and in particular by
flash memory updates's call to blk_write(), which invokes aio_poll().
Fun stuff: they had an extremely short race window, so much that
adding all kind of tracing to either the kernel or QEMU made it
go away (a single printf made it half as reproducible).
On the plus side, the failure mode (a hang until the next keypress)
made it very easy to examine the state of the process with a debugger.
And there was a very nice reproducer from Laszlo, which failed pretty
often (more than half of the time) on any version of QEMU with a non-debug
kernel; it also failed fast, while still in the firmware. So, it could
have been worse.
For some unknown reason they happened only with virtio-scsi, but
that's not important. It's more interesting that they disappeared with
io=native, making thread-pool.c a likely suspect for where the bug arose.
thread-pool.c is also one of the few places which use bottom halves
across threads, by the way.
I hope that no other similar bugs exist, but just in case :) I am
going to describe how the successful debugging went... Since the
likely culprit was the ctx->dispatching optimization, which mostly
affects bottom halves, the first observation was that there are two
qemu_bh_schedule() invocations in the thread pool: the one in the aio
worker and the one in thread_pool_completion_bh. The latter always
causes the optimization to trigger, the former may or may not. In
order to restrict the possibilities, I introduced new functions
qemu_bh_schedule_slow() and qemu_bh_schedule_fast():
/* qemu_bh_schedule_slow: */
ctx = bh->ctx;
bh->idle = 0;
if (atomic_xchg(&bh->scheduled, 1) == 0) {
event_notifier_set(&ctx->notifier);
}
/* qemu_bh_schedule_fast: */
ctx = bh->ctx;
bh->idle = 0;
assert(ctx->dispatching);
atomic_xchg(&bh->scheduled, 1);
Notice how the atomic_xchg is still in qemu_bh_schedule_slow(). This
was already debated a few months ago, so I assumed it to be correct.
In retrospect this was a very good idea, as you'll see later.
Changing thread_pool_completion_bh() to qemu_bh_schedule_fast() didn't
trigger the assertion (as expected). Changing the worker's invocation
to qemu_bh_schedule_slow() didn't hide the bug (another assumption
which luckily held). This already limited heavily the amount of
interaction between the threads, hinting that the problematic events
must have triggered around thread_pool_completion_bh().
As mentioned early, invoking a debugger to examine the state of a
hung process was pretty easy; the iothread was always waiting on a
poll(..., -1) system call. Infinite timeouts are much rarer on x86,
and this could be the reason why the bug was never observed there.
With the buggy sequence more or less resolved to an interaction between
thread_pool_completion_bh() and poll(..., -1), my "tracing" strategy was
to just add a few qemu_clock_get_ns(QEMU_CLOCK_REALTIME) calls, hoping
that the ordering of aio_ctx_prepare(), aio_ctx_dispatch, poll() and
qemu_bh_schedule_fast() would provide some hint. The output was:
(gdb) p last_prepare
$3 = 103885451
(gdb) p last_dispatch
$4 = 103876492
(gdb) p last_poll
$5 = 115909333
(gdb) p last_schedule
$6 = 115925212
Notice how the last call to qemu_poll_ns() came after aio_ctx_dispatch().
This makes little sense unless there is an aio_poll() call involved,
and indeed with a slightly different instrumentation you can see that
there is one:
(gdb) p last_prepare
$3 = 107569679
(gdb) p last_dispatch
$4 = 107561600
(gdb) p last_aio_poll
$5 = 110671400
(gdb) p last_schedule
$6 = 110698917
So the scenario becomes clearer:
iothread VCPU thread
--------------------------------------------------------------------------
aio_ctx_prepare
aio_ctx_check
qemu_poll_ns(timeout=-1)
aio_poll
aio_dispatch
thread_pool_completion_bh
qemu_bh_schedule()
At this point bh->scheduled = 1 and the iothread has not been woken up.
The solution must be close, but this alone should not be a problem,
because the bottom half is only rescheduled to account for rare situations
(see commit 3c80ca1, thread-pool: avoid deadlock in nested aio_poll()
calls, 2014-07-15).
Introducing a third thread---a thread pool worker thread, which
also does qemu_bh_schedule()---does bring out the problematic case.
The third thread must be awakened *after* the callback is complete and
thread_pool_completion_bh has redone the whole loop, explaining the
short race window. And then this is what happens:
thread pool worker
--------------------------------------------------------------------------
<I/O completes>
qemu_bh_schedule()
Tada, bh->scheduled is already 1, so qemu_bh_schedule() does nothing
and the iothread is never woken up. This is where the bh->scheduled
optimization comes into play---it is correct, but removing it would
have masked the bug.
So, what is the bug?
Well, the question asked by the ctx->dispatching optimization ("is any
active aio_poll dispatching?") was wrong. The right question to ask
instead is "is any active aio_poll *not* dispatching", i.e. in the prepare
or poll phases? In that case, the aio_poll is sleeping or might go to
sleep anytime soon, and the EventNotifier must be invoked to wake
it up.
In any other case (including if there is *no* active aio_poll at all!)
we can just wait for the next prepare phase to pick up the event (e.g. a
bottom half); the prepare phase will avoid the blocking and service the
bottom half.
Expressing the invariant with a logic formula, the broken one looked like:
!(exists(thread): in_dispatching(thread)) => !optimize
or equivalently:
!(exists(thread):
in_aio_poll(thread) && in_dispatching(thread)) => !optimize
In the correct one, the negation is in a slightly different place:
(exists(thread):
in_aio_poll(thread) && !in_dispatching(thread)) => !optimize
or equivalently:
(exists(thread): in_prepare_or_poll(thread)) => !optimize
Even if the difference boils down to moving an exclamation mark :)
the implementation is quite different. However, I think the new
one is simpler to understand.
In the old implementation, the "exists" was implemented with a boolean
value. This didn't really support well the case of multiple concurrent
event loops, but I thought that this was okay: aio_poll holds the
AioContext lock so there cannot be concurrent aio_poll invocations, and
I was just considering nested event loops. However, aio_poll _could_
indeed be concurrent with the GSource. This is why I came up with the
wrong invariant.
In the new implementation, "exists" is computed simply by counting how many
threads are in the prepare or poll phases. There are some interesting
points to consider, but the gist of the idea remains:
1) AioContext can be used through GSource as well; as mentioned in the
patch, bit 0 of the counter is reserved for the GSource.
2) the counter need not be updated for a non-blocking aio_poll, because
it won't sleep forever anyway. This is just a matter of checking
the "blocking" variable. This requires some changes to the win32
implementation, but is otherwise not too complicated.
3) as mentioned above, the new implementation will not call aio_notify
when there is *no* active aio_poll at all. The tests have to be
adjusted for this change. The calls to aio_notify in async.c are fine;
they only want to kick aio_poll out of a blocking wait, but need not
do anything if aio_poll is not running.
4) nested aio_poll: these just work with the new implementation; when
a nested event loop is invoked, the outer event loop is never in the
prepare or poll phases. The outer event loop thus has already decremented
the counter.
Reported-by: Richard W. M. Jones <rjones@redhat.com>
Reported-by: Laszlo Ersek <lersek@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Reviewed-by: Fam Zheng <famz@redhat.com>
Tested-by: Richard W.M. Jones <rjones@redhat.com>
Message-id: 1437487673-23740-5-git-send-email-pbonzini@redhat.com
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
2015-07-21 17:07:51 +03:00
|
|
|
if (blocking) {
|
aio: Do aio_notify_accept only during blocking aio_poll
An aio_notify() pairs with an aio_notify_accept(). The former should
happen in the main thread or a vCPU thread, and the latter should be
done in the IOThread.
There is one rare case that the main thread or vCPU thread may "steal"
the aio_notify() event just raised by itself, in bdrv_set_aio_context()
[1]. The sequence is like this:
main thread IO Thread
===============================================================
bdrv_drained_begin()
aio_disable_external(ctx)
aio_poll(ctx, true)
ctx->notify_me += 2
...
bdrv_drained_end()
...
aio_notify()
...
bdrv_set_aio_context()
aio_poll(ctx, false)
[1] aio_notify_accept(ctx)
ppoll() /* Hang! */
[1] is problematic. It will clear the ctx->notifier event so that
the blocked ppoll() will not return.
(For the curious, this bug was noticed when booting a number of VMs
simultaneously in RHV. One or two of the VMs will hit this race
condition, making the VIRTIO device unresponsive to I/O commands. When
it hangs, Seabios is busy waiting for a read request to complete (read
MBR), right after initializing the virtio-blk-pci device, using 100%
guest CPU. See also https://bugzilla.redhat.com/show_bug.cgi?id=1562750
for the original bug analysis.)
aio_notify() only injects an event when ctx->notify_me is set,
correspondingly aio_notify_accept() is only useful when ctx->notify_me
_was_ set. Move the call to it into the "blocking" branch. This will
effectively skip [1] and fix the hang.
Furthermore, blocking aio_poll is only allowed on home thread
(in_aio_context_home_thread), because otherwise two blocking
aio_poll()'s can steal each other's ctx->notifier event and cause
hanging just like described above.
Cc: qemu-stable@nongnu.org
Suggested-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Fam Zheng <famz@redhat.com>
Message-Id: <20180809132259.18402-3-famz@redhat.com>
Signed-off-by: Fam Zheng <famz@redhat.com>
2018-08-09 16:22:59 +03:00
|
|
|
assert(in_aio_context_home_thread(ctx));
|
AioContext: fix broken ctx->dispatching optimization
This patch rewrites the ctx->dispatching optimization, which was the cause
of some mysterious hangs that could be reproduced on aarch64 KVM only.
The hangs were indirectly caused by aio_poll() and in particular by
flash memory updates's call to blk_write(), which invokes aio_poll().
Fun stuff: they had an extremely short race window, so much that
adding all kind of tracing to either the kernel or QEMU made it
go away (a single printf made it half as reproducible).
On the plus side, the failure mode (a hang until the next keypress)
made it very easy to examine the state of the process with a debugger.
And there was a very nice reproducer from Laszlo, which failed pretty
often (more than half of the time) on any version of QEMU with a non-debug
kernel; it also failed fast, while still in the firmware. So, it could
have been worse.
For some unknown reason they happened only with virtio-scsi, but
that's not important. It's more interesting that they disappeared with
io=native, making thread-pool.c a likely suspect for where the bug arose.
thread-pool.c is also one of the few places which use bottom halves
across threads, by the way.
I hope that no other similar bugs exist, but just in case :) I am
going to describe how the successful debugging went... Since the
likely culprit was the ctx->dispatching optimization, which mostly
affects bottom halves, the first observation was that there are two
qemu_bh_schedule() invocations in the thread pool: the one in the aio
worker and the one in thread_pool_completion_bh. The latter always
causes the optimization to trigger, the former may or may not. In
order to restrict the possibilities, I introduced new functions
qemu_bh_schedule_slow() and qemu_bh_schedule_fast():
/* qemu_bh_schedule_slow: */
ctx = bh->ctx;
bh->idle = 0;
if (atomic_xchg(&bh->scheduled, 1) == 0) {
event_notifier_set(&ctx->notifier);
}
/* qemu_bh_schedule_fast: */
ctx = bh->ctx;
bh->idle = 0;
assert(ctx->dispatching);
atomic_xchg(&bh->scheduled, 1);
Notice how the atomic_xchg is still in qemu_bh_schedule_slow(). This
was already debated a few months ago, so I assumed it to be correct.
In retrospect this was a very good idea, as you'll see later.
Changing thread_pool_completion_bh() to qemu_bh_schedule_fast() didn't
trigger the assertion (as expected). Changing the worker's invocation
to qemu_bh_schedule_slow() didn't hide the bug (another assumption
which luckily held). This already limited heavily the amount of
interaction between the threads, hinting that the problematic events
must have triggered around thread_pool_completion_bh().
As mentioned early, invoking a debugger to examine the state of a
hung process was pretty easy; the iothread was always waiting on a
poll(..., -1) system call. Infinite timeouts are much rarer on x86,
and this could be the reason why the bug was never observed there.
With the buggy sequence more or less resolved to an interaction between
thread_pool_completion_bh() and poll(..., -1), my "tracing" strategy was
to just add a few qemu_clock_get_ns(QEMU_CLOCK_REALTIME) calls, hoping
that the ordering of aio_ctx_prepare(), aio_ctx_dispatch, poll() and
qemu_bh_schedule_fast() would provide some hint. The output was:
(gdb) p last_prepare
$3 = 103885451
(gdb) p last_dispatch
$4 = 103876492
(gdb) p last_poll
$5 = 115909333
(gdb) p last_schedule
$6 = 115925212
Notice how the last call to qemu_poll_ns() came after aio_ctx_dispatch().
This makes little sense unless there is an aio_poll() call involved,
and indeed with a slightly different instrumentation you can see that
there is one:
(gdb) p last_prepare
$3 = 107569679
(gdb) p last_dispatch
$4 = 107561600
(gdb) p last_aio_poll
$5 = 110671400
(gdb) p last_schedule
$6 = 110698917
So the scenario becomes clearer:
iothread VCPU thread
--------------------------------------------------------------------------
aio_ctx_prepare
aio_ctx_check
qemu_poll_ns(timeout=-1)
aio_poll
aio_dispatch
thread_pool_completion_bh
qemu_bh_schedule()
At this point bh->scheduled = 1 and the iothread has not been woken up.
The solution must be close, but this alone should not be a problem,
because the bottom half is only rescheduled to account for rare situations
(see commit 3c80ca1, thread-pool: avoid deadlock in nested aio_poll()
calls, 2014-07-15).
Introducing a third thread---a thread pool worker thread, which
also does qemu_bh_schedule()---does bring out the problematic case.
The third thread must be awakened *after* the callback is complete and
thread_pool_completion_bh has redone the whole loop, explaining the
short race window. And then this is what happens:
thread pool worker
--------------------------------------------------------------------------
<I/O completes>
qemu_bh_schedule()
Tada, bh->scheduled is already 1, so qemu_bh_schedule() does nothing
and the iothread is never woken up. This is where the bh->scheduled
optimization comes into play---it is correct, but removing it would
have masked the bug.
So, what is the bug?
Well, the question asked by the ctx->dispatching optimization ("is any
active aio_poll dispatching?") was wrong. The right question to ask
instead is "is any active aio_poll *not* dispatching", i.e. in the prepare
or poll phases? In that case, the aio_poll is sleeping or might go to
sleep anytime soon, and the EventNotifier must be invoked to wake
it up.
In any other case (including if there is *no* active aio_poll at all!)
we can just wait for the next prepare phase to pick up the event (e.g. a
bottom half); the prepare phase will avoid the blocking and service the
bottom half.
Expressing the invariant with a logic formula, the broken one looked like:
!(exists(thread): in_dispatching(thread)) => !optimize
or equivalently:
!(exists(thread):
in_aio_poll(thread) && in_dispatching(thread)) => !optimize
In the correct one, the negation is in a slightly different place:
(exists(thread):
in_aio_poll(thread) && !in_dispatching(thread)) => !optimize
or equivalently:
(exists(thread): in_prepare_or_poll(thread)) => !optimize
Even if the difference boils down to moving an exclamation mark :)
the implementation is quite different. However, I think the new
one is simpler to understand.
In the old implementation, the "exists" was implemented with a boolean
value. This didn't really support well the case of multiple concurrent
event loops, but I thought that this was okay: aio_poll holds the
AioContext lock so there cannot be concurrent aio_poll invocations, and
I was just considering nested event loops. However, aio_poll _could_
indeed be concurrent with the GSource. This is why I came up with the
wrong invariant.
In the new implementation, "exists" is computed simply by counting how many
threads are in the prepare or poll phases. There are some interesting
points to consider, but the gist of the idea remains:
1) AioContext can be used through GSource as well; as mentioned in the
patch, bit 0 of the counter is reserved for the GSource.
2) the counter need not be updated for a non-blocking aio_poll, because
it won't sleep forever anyway. This is just a matter of checking
the "blocking" variable. This requires some changes to the win32
implementation, but is otherwise not too complicated.
3) as mentioned above, the new implementation will not call aio_notify
when there is *no* active aio_poll at all. The tests have to be
adjusted for this change. The calls to aio_notify in async.c are fine;
they only want to kick aio_poll out of a blocking wait, but need not
do anything if aio_poll is not running.
4) nested aio_poll: these just work with the new implementation; when
a nested event loop is invoked, the outer event loop is never in the
prepare or poll phases. The outer event loop thus has already decremented
the counter.
Reported-by: Richard W. M. Jones <rjones@redhat.com>
Reported-by: Laszlo Ersek <lersek@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Reviewed-by: Fam Zheng <famz@redhat.com>
Tested-by: Richard W.M. Jones <rjones@redhat.com>
Message-id: 1437487673-23740-5-git-send-email-pbonzini@redhat.com
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
2015-07-21 17:07:51 +03:00
|
|
|
atomic_add(&ctx->notify_me, 2);
|
|
|
|
}
|
2014-07-07 17:18:04 +04:00
|
|
|
|
2017-01-12 21:07:57 +03:00
|
|
|
qemu_lockcnt_inc(&ctx->list_lock);
|
2008-09-22 23:17:18 +04:00
|
|
|
|
2016-12-01 22:26:51 +03:00
|
|
|
if (ctx->poll_max_ns) {
|
|
|
|
start = qemu_clock_get_ns(QEMU_CLOCK_REALTIME);
|
|
|
|
}
|
|
|
|
|
2017-02-13 16:52:28 +03:00
|
|
|
progress = try_poll_mode(ctx, blocking);
|
|
|
|
if (!progress) {
|
2016-12-01 22:26:42 +03:00
|
|
|
assert(npfd == 0);
|
2008-09-22 23:17:18 +04:00
|
|
|
|
2016-12-01 22:26:42 +03:00
|
|
|
/* fill pollfds */
|
2016-09-14 14:03:39 +03:00
|
|
|
|
2016-12-01 22:26:42 +03:00
|
|
|
if (!aio_epoll_enabled(ctx)) {
|
2017-01-12 21:07:57 +03:00
|
|
|
QLIST_FOREACH_RCU(node, &ctx->aio_handlers, node) {
|
2016-12-01 22:26:42 +03:00
|
|
|
if (!node->deleted && node->pfd.events
|
|
|
|
&& aio_node_check(ctx, node->is_external)) {
|
|
|
|
add_pollfd(node);
|
|
|
|
}
|
2016-09-14 14:03:39 +03:00
|
|
|
}
|
2012-04-12 16:00:56 +04:00
|
|
|
}
|
2008-09-22 23:17:18 +04:00
|
|
|
|
2016-12-01 22:26:42 +03:00
|
|
|
timeout = blocking ? aio_compute_timeout(ctx) : 0;
|
2008-09-22 23:17:18 +04:00
|
|
|
|
2016-12-01 22:26:42 +03:00
|
|
|
/* wait until next event */
|
|
|
|
if (aio_epoll_check_poll(ctx, pollfds, npfd, timeout)) {
|
|
|
|
AioHandler epoll_handler;
|
|
|
|
|
|
|
|
epoll_handler.pfd.fd = ctx->epollfd;
|
|
|
|
epoll_handler.pfd.events = G_IO_IN | G_IO_OUT | G_IO_HUP | G_IO_ERR;
|
|
|
|
npfd = 0;
|
|
|
|
add_pollfd(&epoll_handler);
|
|
|
|
ret = aio_epoll(ctx, pollfds, npfd, timeout);
|
|
|
|
} else {
|
|
|
|
ret = qemu_poll_ns(pollfds, npfd, timeout);
|
|
|
|
}
|
2015-10-30 07:06:29 +03:00
|
|
|
}
|
2016-12-01 22:26:42 +03:00
|
|
|
|
AioContext: fix broken ctx->dispatching optimization
This patch rewrites the ctx->dispatching optimization, which was the cause
of some mysterious hangs that could be reproduced on aarch64 KVM only.
The hangs were indirectly caused by aio_poll() and in particular by
flash memory updates's call to blk_write(), which invokes aio_poll().
Fun stuff: they had an extremely short race window, so much that
adding all kind of tracing to either the kernel or QEMU made it
go away (a single printf made it half as reproducible).
On the plus side, the failure mode (a hang until the next keypress)
made it very easy to examine the state of the process with a debugger.
And there was a very nice reproducer from Laszlo, which failed pretty
often (more than half of the time) on any version of QEMU with a non-debug
kernel; it also failed fast, while still in the firmware. So, it could
have been worse.
For some unknown reason they happened only with virtio-scsi, but
that's not important. It's more interesting that they disappeared with
io=native, making thread-pool.c a likely suspect for where the bug arose.
thread-pool.c is also one of the few places which use bottom halves
across threads, by the way.
I hope that no other similar bugs exist, but just in case :) I am
going to describe how the successful debugging went... Since the
likely culprit was the ctx->dispatching optimization, which mostly
affects bottom halves, the first observation was that there are two
qemu_bh_schedule() invocations in the thread pool: the one in the aio
worker and the one in thread_pool_completion_bh. The latter always
causes the optimization to trigger, the former may or may not. In
order to restrict the possibilities, I introduced new functions
qemu_bh_schedule_slow() and qemu_bh_schedule_fast():
/* qemu_bh_schedule_slow: */
ctx = bh->ctx;
bh->idle = 0;
if (atomic_xchg(&bh->scheduled, 1) == 0) {
event_notifier_set(&ctx->notifier);
}
/* qemu_bh_schedule_fast: */
ctx = bh->ctx;
bh->idle = 0;
assert(ctx->dispatching);
atomic_xchg(&bh->scheduled, 1);
Notice how the atomic_xchg is still in qemu_bh_schedule_slow(). This
was already debated a few months ago, so I assumed it to be correct.
In retrospect this was a very good idea, as you'll see later.
Changing thread_pool_completion_bh() to qemu_bh_schedule_fast() didn't
trigger the assertion (as expected). Changing the worker's invocation
to qemu_bh_schedule_slow() didn't hide the bug (another assumption
which luckily held). This already limited heavily the amount of
interaction between the threads, hinting that the problematic events
must have triggered around thread_pool_completion_bh().
As mentioned early, invoking a debugger to examine the state of a
hung process was pretty easy; the iothread was always waiting on a
poll(..., -1) system call. Infinite timeouts are much rarer on x86,
and this could be the reason why the bug was never observed there.
With the buggy sequence more or less resolved to an interaction between
thread_pool_completion_bh() and poll(..., -1), my "tracing" strategy was
to just add a few qemu_clock_get_ns(QEMU_CLOCK_REALTIME) calls, hoping
that the ordering of aio_ctx_prepare(), aio_ctx_dispatch, poll() and
qemu_bh_schedule_fast() would provide some hint. The output was:
(gdb) p last_prepare
$3 = 103885451
(gdb) p last_dispatch
$4 = 103876492
(gdb) p last_poll
$5 = 115909333
(gdb) p last_schedule
$6 = 115925212
Notice how the last call to qemu_poll_ns() came after aio_ctx_dispatch().
This makes little sense unless there is an aio_poll() call involved,
and indeed with a slightly different instrumentation you can see that
there is one:
(gdb) p last_prepare
$3 = 107569679
(gdb) p last_dispatch
$4 = 107561600
(gdb) p last_aio_poll
$5 = 110671400
(gdb) p last_schedule
$6 = 110698917
So the scenario becomes clearer:
iothread VCPU thread
--------------------------------------------------------------------------
aio_ctx_prepare
aio_ctx_check
qemu_poll_ns(timeout=-1)
aio_poll
aio_dispatch
thread_pool_completion_bh
qemu_bh_schedule()
At this point bh->scheduled = 1 and the iothread has not been woken up.
The solution must be close, but this alone should not be a problem,
because the bottom half is only rescheduled to account for rare situations
(see commit 3c80ca1, thread-pool: avoid deadlock in nested aio_poll()
calls, 2014-07-15).
Introducing a third thread---a thread pool worker thread, which
also does qemu_bh_schedule()---does bring out the problematic case.
The third thread must be awakened *after* the callback is complete and
thread_pool_completion_bh has redone the whole loop, explaining the
short race window. And then this is what happens:
thread pool worker
--------------------------------------------------------------------------
<I/O completes>
qemu_bh_schedule()
Tada, bh->scheduled is already 1, so qemu_bh_schedule() does nothing
and the iothread is never woken up. This is where the bh->scheduled
optimization comes into play---it is correct, but removing it would
have masked the bug.
So, what is the bug?
Well, the question asked by the ctx->dispatching optimization ("is any
active aio_poll dispatching?") was wrong. The right question to ask
instead is "is any active aio_poll *not* dispatching", i.e. in the prepare
or poll phases? In that case, the aio_poll is sleeping or might go to
sleep anytime soon, and the EventNotifier must be invoked to wake
it up.
In any other case (including if there is *no* active aio_poll at all!)
we can just wait for the next prepare phase to pick up the event (e.g. a
bottom half); the prepare phase will avoid the blocking and service the
bottom half.
Expressing the invariant with a logic formula, the broken one looked like:
!(exists(thread): in_dispatching(thread)) => !optimize
or equivalently:
!(exists(thread):
in_aio_poll(thread) && in_dispatching(thread)) => !optimize
In the correct one, the negation is in a slightly different place:
(exists(thread):
in_aio_poll(thread) && !in_dispatching(thread)) => !optimize
or equivalently:
(exists(thread): in_prepare_or_poll(thread)) => !optimize
Even if the difference boils down to moving an exclamation mark :)
the implementation is quite different. However, I think the new
one is simpler to understand.
In the old implementation, the "exists" was implemented with a boolean
value. This didn't really support well the case of multiple concurrent
event loops, but I thought that this was okay: aio_poll holds the
AioContext lock so there cannot be concurrent aio_poll invocations, and
I was just considering nested event loops. However, aio_poll _could_
indeed be concurrent with the GSource. This is why I came up with the
wrong invariant.
In the new implementation, "exists" is computed simply by counting how many
threads are in the prepare or poll phases. There are some interesting
points to consider, but the gist of the idea remains:
1) AioContext can be used through GSource as well; as mentioned in the
patch, bit 0 of the counter is reserved for the GSource.
2) the counter need not be updated for a non-blocking aio_poll, because
it won't sleep forever anyway. This is just a matter of checking
the "blocking" variable. This requires some changes to the win32
implementation, but is otherwise not too complicated.
3) as mentioned above, the new implementation will not call aio_notify
when there is *no* active aio_poll at all. The tests have to be
adjusted for this change. The calls to aio_notify in async.c are fine;
they only want to kick aio_poll out of a blocking wait, but need not
do anything if aio_poll is not running.
4) nested aio_poll: these just work with the new implementation; when
a nested event loop is invoked, the outer event loop is never in the
prepare or poll phases. The outer event loop thus has already decremented
the counter.
Reported-by: Richard W. M. Jones <rjones@redhat.com>
Reported-by: Laszlo Ersek <lersek@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Reviewed-by: Fam Zheng <famz@redhat.com>
Tested-by: Richard W.M. Jones <rjones@redhat.com>
Message-id: 1437487673-23740-5-git-send-email-pbonzini@redhat.com
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
2015-07-21 17:07:51 +03:00
|
|
|
if (blocking) {
|
|
|
|
atomic_sub(&ctx->notify_me, 2);
|
aio: Do aio_notify_accept only during blocking aio_poll
An aio_notify() pairs with an aio_notify_accept(). The former should
happen in the main thread or a vCPU thread, and the latter should be
done in the IOThread.
There is one rare case that the main thread or vCPU thread may "steal"
the aio_notify() event just raised by itself, in bdrv_set_aio_context()
[1]. The sequence is like this:
main thread IO Thread
===============================================================
bdrv_drained_begin()
aio_disable_external(ctx)
aio_poll(ctx, true)
ctx->notify_me += 2
...
bdrv_drained_end()
...
aio_notify()
...
bdrv_set_aio_context()
aio_poll(ctx, false)
[1] aio_notify_accept(ctx)
ppoll() /* Hang! */
[1] is problematic. It will clear the ctx->notifier event so that
the blocked ppoll() will not return.
(For the curious, this bug was noticed when booting a number of VMs
simultaneously in RHV. One or two of the VMs will hit this race
condition, making the VIRTIO device unresponsive to I/O commands. When
it hangs, Seabios is busy waiting for a read request to complete (read
MBR), right after initializing the virtio-blk-pci device, using 100%
guest CPU. See also https://bugzilla.redhat.com/show_bug.cgi?id=1562750
for the original bug analysis.)
aio_notify() only injects an event when ctx->notify_me is set,
correspondingly aio_notify_accept() is only useful when ctx->notify_me
_was_ set. Move the call to it into the "blocking" branch. This will
effectively skip [1] and fix the hang.
Furthermore, blocking aio_poll is only allowed on home thread
(in_aio_context_home_thread), because otherwise two blocking
aio_poll()'s can steal each other's ctx->notifier event and cause
hanging just like described above.
Cc: qemu-stable@nongnu.org
Suggested-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Fam Zheng <famz@redhat.com>
Message-Id: <20180809132259.18402-3-famz@redhat.com>
Signed-off-by: Fam Zheng <famz@redhat.com>
2018-08-09 16:22:59 +03:00
|
|
|
aio_notify_accept(ctx);
|
AioContext: fix broken ctx->dispatching optimization
This patch rewrites the ctx->dispatching optimization, which was the cause
of some mysterious hangs that could be reproduced on aarch64 KVM only.
The hangs were indirectly caused by aio_poll() and in particular by
flash memory updates's call to blk_write(), which invokes aio_poll().
Fun stuff: they had an extremely short race window, so much that
adding all kind of tracing to either the kernel or QEMU made it
go away (a single printf made it half as reproducible).
On the plus side, the failure mode (a hang until the next keypress)
made it very easy to examine the state of the process with a debugger.
And there was a very nice reproducer from Laszlo, which failed pretty
often (more than half of the time) on any version of QEMU with a non-debug
kernel; it also failed fast, while still in the firmware. So, it could
have been worse.
For some unknown reason they happened only with virtio-scsi, but
that's not important. It's more interesting that they disappeared with
io=native, making thread-pool.c a likely suspect for where the bug arose.
thread-pool.c is also one of the few places which use bottom halves
across threads, by the way.
I hope that no other similar bugs exist, but just in case :) I am
going to describe how the successful debugging went... Since the
likely culprit was the ctx->dispatching optimization, which mostly
affects bottom halves, the first observation was that there are two
qemu_bh_schedule() invocations in the thread pool: the one in the aio
worker and the one in thread_pool_completion_bh. The latter always
causes the optimization to trigger, the former may or may not. In
order to restrict the possibilities, I introduced new functions
qemu_bh_schedule_slow() and qemu_bh_schedule_fast():
/* qemu_bh_schedule_slow: */
ctx = bh->ctx;
bh->idle = 0;
if (atomic_xchg(&bh->scheduled, 1) == 0) {
event_notifier_set(&ctx->notifier);
}
/* qemu_bh_schedule_fast: */
ctx = bh->ctx;
bh->idle = 0;
assert(ctx->dispatching);
atomic_xchg(&bh->scheduled, 1);
Notice how the atomic_xchg is still in qemu_bh_schedule_slow(). This
was already debated a few months ago, so I assumed it to be correct.
In retrospect this was a very good idea, as you'll see later.
Changing thread_pool_completion_bh() to qemu_bh_schedule_fast() didn't
trigger the assertion (as expected). Changing the worker's invocation
to qemu_bh_schedule_slow() didn't hide the bug (another assumption
which luckily held). This already limited heavily the amount of
interaction between the threads, hinting that the problematic events
must have triggered around thread_pool_completion_bh().
As mentioned early, invoking a debugger to examine the state of a
hung process was pretty easy; the iothread was always waiting on a
poll(..., -1) system call. Infinite timeouts are much rarer on x86,
and this could be the reason why the bug was never observed there.
With the buggy sequence more or less resolved to an interaction between
thread_pool_completion_bh() and poll(..., -1), my "tracing" strategy was
to just add a few qemu_clock_get_ns(QEMU_CLOCK_REALTIME) calls, hoping
that the ordering of aio_ctx_prepare(), aio_ctx_dispatch, poll() and
qemu_bh_schedule_fast() would provide some hint. The output was:
(gdb) p last_prepare
$3 = 103885451
(gdb) p last_dispatch
$4 = 103876492
(gdb) p last_poll
$5 = 115909333
(gdb) p last_schedule
$6 = 115925212
Notice how the last call to qemu_poll_ns() came after aio_ctx_dispatch().
This makes little sense unless there is an aio_poll() call involved,
and indeed with a slightly different instrumentation you can see that
there is one:
(gdb) p last_prepare
$3 = 107569679
(gdb) p last_dispatch
$4 = 107561600
(gdb) p last_aio_poll
$5 = 110671400
(gdb) p last_schedule
$6 = 110698917
So the scenario becomes clearer:
iothread VCPU thread
--------------------------------------------------------------------------
aio_ctx_prepare
aio_ctx_check
qemu_poll_ns(timeout=-1)
aio_poll
aio_dispatch
thread_pool_completion_bh
qemu_bh_schedule()
At this point bh->scheduled = 1 and the iothread has not been woken up.
The solution must be close, but this alone should not be a problem,
because the bottom half is only rescheduled to account for rare situations
(see commit 3c80ca1, thread-pool: avoid deadlock in nested aio_poll()
calls, 2014-07-15).
Introducing a third thread---a thread pool worker thread, which
also does qemu_bh_schedule()---does bring out the problematic case.
The third thread must be awakened *after* the callback is complete and
thread_pool_completion_bh has redone the whole loop, explaining the
short race window. And then this is what happens:
thread pool worker
--------------------------------------------------------------------------
<I/O completes>
qemu_bh_schedule()
Tada, bh->scheduled is already 1, so qemu_bh_schedule() does nothing
and the iothread is never woken up. This is where the bh->scheduled
optimization comes into play---it is correct, but removing it would
have masked the bug.
So, what is the bug?
Well, the question asked by the ctx->dispatching optimization ("is any
active aio_poll dispatching?") was wrong. The right question to ask
instead is "is any active aio_poll *not* dispatching", i.e. in the prepare
or poll phases? In that case, the aio_poll is sleeping or might go to
sleep anytime soon, and the EventNotifier must be invoked to wake
it up.
In any other case (including if there is *no* active aio_poll at all!)
we can just wait for the next prepare phase to pick up the event (e.g. a
bottom half); the prepare phase will avoid the blocking and service the
bottom half.
Expressing the invariant with a logic formula, the broken one looked like:
!(exists(thread): in_dispatching(thread)) => !optimize
or equivalently:
!(exists(thread):
in_aio_poll(thread) && in_dispatching(thread)) => !optimize
In the correct one, the negation is in a slightly different place:
(exists(thread):
in_aio_poll(thread) && !in_dispatching(thread)) => !optimize
or equivalently:
(exists(thread): in_prepare_or_poll(thread)) => !optimize
Even if the difference boils down to moving an exclamation mark :)
the implementation is quite different. However, I think the new
one is simpler to understand.
In the old implementation, the "exists" was implemented with a boolean
value. This didn't really support well the case of multiple concurrent
event loops, but I thought that this was okay: aio_poll holds the
AioContext lock so there cannot be concurrent aio_poll invocations, and
I was just considering nested event loops. However, aio_poll _could_
indeed be concurrent with the GSource. This is why I came up with the
wrong invariant.
In the new implementation, "exists" is computed simply by counting how many
threads are in the prepare or poll phases. There are some interesting
points to consider, but the gist of the idea remains:
1) AioContext can be used through GSource as well; as mentioned in the
patch, bit 0 of the counter is reserved for the GSource.
2) the counter need not be updated for a non-blocking aio_poll, because
it won't sleep forever anyway. This is just a matter of checking
the "blocking" variable. This requires some changes to the win32
implementation, but is otherwise not too complicated.
3) as mentioned above, the new implementation will not call aio_notify
when there is *no* active aio_poll at all. The tests have to be
adjusted for this change. The calls to aio_notify in async.c are fine;
they only want to kick aio_poll out of a blocking wait, but need not
do anything if aio_poll is not running.
4) nested aio_poll: these just work with the new implementation; when
a nested event loop is invoked, the outer event loop is never in the
prepare or poll phases. The outer event loop thus has already decremented
the counter.
Reported-by: Richard W. M. Jones <rjones@redhat.com>
Reported-by: Laszlo Ersek <lersek@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Reviewed-by: Fam Zheng <famz@redhat.com>
Tested-by: Richard W.M. Jones <rjones@redhat.com>
Message-id: 1437487673-23740-5-git-send-email-pbonzini@redhat.com
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
2015-07-21 17:07:51 +03:00
|
|
|
}
|
2012-04-12 16:00:56 +04:00
|
|
|
|
2016-12-01 22:26:51 +03:00
|
|
|
/* Adjust polling time */
|
|
|
|
if (ctx->poll_max_ns) {
|
|
|
|
int64_t block_ns = qemu_clock_get_ns(QEMU_CLOCK_REALTIME) - start;
|
|
|
|
|
|
|
|
if (block_ns <= ctx->poll_ns) {
|
|
|
|
/* This is the sweet spot, no adjustment needed */
|
|
|
|
} else if (block_ns > ctx->poll_max_ns) {
|
|
|
|
/* We'd have to poll for too long, poll less */
|
|
|
|
int64_t old = ctx->poll_ns;
|
|
|
|
|
|
|
|
if (ctx->poll_shrink) {
|
|
|
|
ctx->poll_ns /= ctx->poll_shrink;
|
|
|
|
} else {
|
|
|
|
ctx->poll_ns = 0;
|
|
|
|
}
|
|
|
|
|
|
|
|
trace_poll_shrink(ctx, old, ctx->poll_ns);
|
|
|
|
} else if (ctx->poll_ns < ctx->poll_max_ns &&
|
|
|
|
block_ns < ctx->poll_max_ns) {
|
|
|
|
/* There is room to grow, poll longer */
|
|
|
|
int64_t old = ctx->poll_ns;
|
|
|
|
int64_t grow = ctx->poll_grow;
|
|
|
|
|
|
|
|
if (grow == 0) {
|
|
|
|
grow = 2;
|
|
|
|
}
|
|
|
|
|
|
|
|
if (ctx->poll_ns) {
|
|
|
|
ctx->poll_ns *= grow;
|
|
|
|
} else {
|
|
|
|
ctx->poll_ns = 4000; /* start polling at 4 microseconds */
|
|
|
|
}
|
|
|
|
|
|
|
|
if (ctx->poll_ns > ctx->poll_max_ns) {
|
|
|
|
ctx->poll_ns = ctx->poll_max_ns;
|
|
|
|
}
|
|
|
|
|
|
|
|
trace_poll_grow(ctx, old, ctx->poll_ns);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2012-04-12 16:00:56 +04:00
|
|
|
/* if we have any readable fds, dispatch event */
|
|
|
|
if (ret > 0) {
|
2015-02-20 19:26:50 +03:00
|
|
|
for (i = 0; i < npfd; i++) {
|
|
|
|
nodes[i]->pfd.revents = pollfds[i].revents;
|
2008-09-22 23:17:18 +04:00
|
|
|
}
|
2013-08-21 19:02:53 +04:00
|
|
|
}
|
|
|
|
|
2015-02-20 19:26:50 +03:00
|
|
|
npfd = 0;
|
|
|
|
|
2017-02-13 16:52:33 +03:00
|
|
|
progress |= aio_bh_poll(ctx);
|
|
|
|
|
|
|
|
if (ret > 0) {
|
|
|
|
progress |= aio_dispatch_handlers(ctx);
|
2012-04-12 16:00:56 +04:00
|
|
|
}
|
2012-04-12 16:00:55 +04:00
|
|
|
|
2017-02-13 16:52:34 +03:00
|
|
|
qemu_lockcnt_dec(&ctx->list_lock);
|
|
|
|
|
2017-02-13 16:52:33 +03:00
|
|
|
progress |= timerlistgroup_run_timers(&ctx->tlg);
|
|
|
|
|
2013-04-11 18:56:50 +04:00
|
|
|
return progress;
|
2008-09-22 23:17:18 +04:00
|
|
|
}
|
2015-10-30 07:06:28 +03:00
|
|
|
|
2016-07-15 13:28:44 +03:00
|
|
|
void aio_context_setup(AioContext *ctx)
|
2015-10-30 07:06:28 +03:00
|
|
|
{
|
2016-02-23 18:42:14 +03:00
|
|
|
#ifdef CONFIG_EPOLL_CREATE1
|
2015-10-30 07:06:29 +03:00
|
|
|
assert(!ctx->epollfd);
|
|
|
|
ctx->epollfd = epoll_create1(EPOLL_CLOEXEC);
|
|
|
|
if (ctx->epollfd == -1) {
|
2016-07-15 13:28:44 +03:00
|
|
|
fprintf(stderr, "Failed to create epoll instance: %s", strerror(errno));
|
2015-10-30 07:06:29 +03:00
|
|
|
ctx->epoll_available = false;
|
|
|
|
} else {
|
|
|
|
ctx->epoll_available = true;
|
|
|
|
}
|
|
|
|
#endif
|
2015-10-30 07:06:28 +03:00
|
|
|
}
|
2016-12-01 22:26:42 +03:00
|
|
|
|
2018-05-17 03:42:43 +03:00
|
|
|
void aio_context_destroy(AioContext *ctx)
|
|
|
|
{
|
|
|
|
#ifdef CONFIG_EPOLL_CREATE1
|
|
|
|
aio_epoll_disable(ctx);
|
|
|
|
#endif
|
|
|
|
}
|
|
|
|
|
2016-12-01 22:26:51 +03:00
|
|
|
void aio_context_set_poll_params(AioContext *ctx, int64_t max_ns,
|
|
|
|
int64_t grow, int64_t shrink, Error **errp)
|
2016-12-01 22:26:42 +03:00
|
|
|
{
|
2016-12-01 22:26:51 +03:00
|
|
|
/* No thread synchronization here, it doesn't matter if an incorrect value
|
|
|
|
* is used once.
|
2016-12-01 22:26:42 +03:00
|
|
|
*/
|
|
|
|
ctx->poll_max_ns = max_ns;
|
2016-12-01 22:26:51 +03:00
|
|
|
ctx->poll_ns = 0;
|
|
|
|
ctx->poll_grow = grow;
|
|
|
|
ctx->poll_shrink = shrink;
|
2016-12-01 22:26:42 +03:00
|
|
|
|
|
|
|
aio_notify(ctx);
|
|
|
|
}
|