[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v6 04/18] tests: adjust test-aio to new aio_poll
From: |
Wenchao Xia |
Subject: |
Re: [Qemu-devel] [PATCH v6 04/18] tests: adjust test-aio to new aio_poll() semantics |
Date: |
Mon, 29 Jul 2013 16:24:20 +0800 |
User-agent: |
Mozilla/5.0 (Windows NT 5.1; rv:17.0) Gecko/20130620 Thunderbird/17.0.7 |
Reviewed-by: Wenchao Xia <address@hidden>
> aio_poll(ctx, true) will soon block if any fd handlers have been set.
> Previously it would only block when .io_flush() returned true.
>
> This means that callers must check their wait condition *before*
> aio_poll() to avoid deadlock.
>
> Reviewed-by: Paolo Bonzini <address@hidden>
> Signed-off-by: Stefan Hajnoczi <address@hidden>
> ---
> tests/test-aio.c | 26 +++++++++++++++++---------
> 1 file changed, 17 insertions(+), 9 deletions(-)
>
> diff --git a/tests/test-aio.c b/tests/test-aio.c
> index c173870..20bf5e6 100644
> --- a/tests/test-aio.c
> +++ b/tests/test-aio.c
> @@ -15,6 +15,13 @@
>
> AioContext *ctx;
>
> +typedef struct {
> + EventNotifier e;
> + int n;
> + int active;
> + bool auto_set;
> +} EventNotifierTestData;
> +
> /* Wait until there are no more BHs or AIO requests */
> static void wait_for_aio(void)
> {
> @@ -23,6 +30,14 @@ static void wait_for_aio(void)
> }
> }
>
> +/* Wait until event notifier becomes inactive */
> +static void wait_until_inactive(EventNotifierTestData *data)
> +{
> + while (data->active > 0) {
> + aio_poll(ctx, true);
> + }
> +}
> +
> /* Simple callbacks for testing. */
>
> typedef struct {
> @@ -50,13 +65,6 @@ static void bh_delete_cb(void *opaque)
> }
> }
>
> -typedef struct {
> - EventNotifier e;
> - int n;
> - int active;
> - bool auto_set;
> -} EventNotifierTestData;
> -
> static int event_active_cb(EventNotifier *e)
> {
> EventNotifierTestData *data = container_of(e, EventNotifierTestData, e);
> @@ -281,7 +289,7 @@ static void test_flush_event_notifier(void)
> g_assert_cmpint(data.active, ==, 9);
> g_assert(aio_poll(ctx, false));
>
> - wait_for_aio();
> + wait_until_inactive(&data);
> g_assert_cmpint(data.n, ==, 10);
> g_assert_cmpint(data.active, ==, 0);
> g_assert(!aio_poll(ctx, false));
> @@ -325,7 +333,7 @@ static void test_wait_event_notifier_noflush(void)
> g_assert_cmpint(data.n, ==, 2);
>
> event_notifier_set(&dummy.e);
> - wait_for_aio();
> + wait_until_inactive(&dummy);
> g_assert_cmpint(data.n, ==, 2);
> g_assert_cmpint(dummy.n, ==, 1);
> g_assert_cmpint(dummy.active, ==, 0);
>
--
Best Regards
Wenchao Xia
- [Qemu-devel] [PATCH v6 15/18] dataplane/virtio-blk: drop flush_true() and flush_io(), (continued)
- [Qemu-devel] [PATCH v6 15/18] dataplane/virtio-blk: drop flush_true() and flush_io(), Stefan Hajnoczi, 2013/07/25
- [Qemu-devel] [PATCH v6 17/18] tests: drop event_active_cb(), Stefan Hajnoczi, 2013/07/25
- [Qemu-devel] [PATCH v6 18/18] aio: drop io_flush argument, Stefan Hajnoczi, 2013/07/25
- [Qemu-devel] [PATCH v6 11/18] block/nbd: drop nbd_have_request(), Stefan Hajnoczi, 2013/07/25
- [Qemu-devel] [PATCH v6 04/18] tests: adjust test-aio to new aio_poll() semantics, Stefan Hajnoczi, 2013/07/25
- [Qemu-devel] [PATCH v6 16/18] thread-pool: drop thread_pool_active(), Stefan Hajnoczi, 2013/07/25
- [Qemu-devel] [PATCH v6 12/18] block/rbd: drop qemu_rbd_aio_flush_cb(), Stefan Hajnoczi, 2013/07/25
- [Qemu-devel] [PATCH v6 14/18] block/ssh: drop return_true(), Stefan Hajnoczi, 2013/07/25
- [Qemu-devel] [PATCH v6 10/18] block/linux-aio: drop qemu_laio_completion_cb(), Stefan Hajnoczi, 2013/07/25
- [Qemu-devel] [PATCH v6 13/18] block/sheepdog: drop have_co_req() and aio_flush_request(), Stefan Hajnoczi, 2013/07/25