[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 16/17] tests: drop event_active_cb()
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH v4 16/17] tests: drop event_active_cb() |
Date: |
Thu, 27 Jun 2013 15:20:51 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130514 Thunderbird/17.0.6 |
Il 14/06/2013 13:44, Stefan Hajnoczi ha scritto:
> The .io_flush() handler no longer exists and has no users. Drop the
> io_flush argument to aio_set_fd_handler() and related functions.
Wrong commit message, but otherwise
Reviewed-by: Paolo Bonzini <address@hidden>
> Signed-off-by: Stefan Hajnoczi <address@hidden>
> ---
> tests/test-aio.c | 22 ++++++++--------------
> 1 file changed, 8 insertions(+), 14 deletions(-)
>
> diff --git a/tests/test-aio.c b/tests/test-aio.c
> index 1251952..7b2892a 100644
> --- a/tests/test-aio.c
> +++ b/tests/test-aio.c
> @@ -65,12 +65,6 @@ static void bh_delete_cb(void *opaque)
> }
> }
>
> -static int event_active_cb(EventNotifier *e)
> -{
> - EventNotifierTestData *data = container_of(e, EventNotifierTestData, e);
> - return data->active > 0;
> -}
> -
> static void event_ready_cb(EventNotifier *e)
> {
> EventNotifierTestData *data = container_of(e, EventNotifierTestData, e);
> @@ -239,7 +233,7 @@ static void test_set_event_notifier(void)
> {
> EventNotifierTestData data = { .n = 0, .active = 0 };
> event_notifier_init(&data.e, false);
> - aio_set_event_notifier(ctx, &data.e, event_ready_cb, event_active_cb);
> + aio_set_event_notifier(ctx, &data.e, event_ready_cb, NULL);
> g_assert(!aio_poll(ctx, false));
> g_assert_cmpint(data.n, ==, 0);
>
> @@ -253,7 +247,7 @@ static void test_wait_event_notifier(void)
> {
> EventNotifierTestData data = { .n = 0, .active = 1 };
> event_notifier_init(&data.e, false);
> - aio_set_event_notifier(ctx, &data.e, event_ready_cb, event_active_cb);
> + aio_set_event_notifier(ctx, &data.e, event_ready_cb, NULL);
> g_assert(!aio_poll(ctx, false));
> g_assert_cmpint(data.n, ==, 0);
> g_assert_cmpint(data.active, ==, 1);
> @@ -278,7 +272,7 @@ static void test_flush_event_notifier(void)
> {
> EventNotifierTestData data = { .n = 0, .active = 10, .auto_set = true };
> event_notifier_init(&data.e, false);
> - aio_set_event_notifier(ctx, &data.e, event_ready_cb, event_active_cb);
> + aio_set_event_notifier(ctx, &data.e, event_ready_cb, NULL);
> g_assert(!aio_poll(ctx, false));
> g_assert_cmpint(data.n, ==, 0);
> g_assert_cmpint(data.active, ==, 10);
> @@ -318,7 +312,7 @@ static void test_wait_event_notifier_noflush(void)
>
> /* An active event notifier forces aio_poll to look at EventNotifiers.
> */
> event_notifier_init(&dummy.e, false);
> - aio_set_event_notifier(ctx, &dummy.e, event_ready_cb, event_active_cb);
> + aio_set_event_notifier(ctx, &dummy.e, event_ready_cb, NULL);
>
> event_notifier_set(&data.e);
> g_assert(aio_poll(ctx, false));
> @@ -521,7 +515,7 @@ static void test_source_set_event_notifier(void)
> {
> EventNotifierTestData data = { .n = 0, .active = 0 };
> event_notifier_init(&data.e, false);
> - aio_set_event_notifier(ctx, &data.e, event_ready_cb, event_active_cb);
> + aio_set_event_notifier(ctx, &data.e, event_ready_cb, NULL);
> while (g_main_context_iteration(NULL, false));
> g_assert_cmpint(data.n, ==, 0);
>
> @@ -535,7 +529,7 @@ static void test_source_wait_event_notifier(void)
> {
> EventNotifierTestData data = { .n = 0, .active = 1 };
> event_notifier_init(&data.e, false);
> - aio_set_event_notifier(ctx, &data.e, event_ready_cb, event_active_cb);
> + aio_set_event_notifier(ctx, &data.e, event_ready_cb, NULL);
> g_assert(g_main_context_iteration(NULL, false));
> g_assert_cmpint(data.n, ==, 0);
> g_assert_cmpint(data.active, ==, 1);
> @@ -560,7 +554,7 @@ static void test_source_flush_event_notifier(void)
> {
> EventNotifierTestData data = { .n = 0, .active = 10, .auto_set = true };
> event_notifier_init(&data.e, false);
> - aio_set_event_notifier(ctx, &data.e, event_ready_cb, event_active_cb);
> + aio_set_event_notifier(ctx, &data.e, event_ready_cb, NULL);
> g_assert(g_main_context_iteration(NULL, false));
> g_assert_cmpint(data.n, ==, 0);
> g_assert_cmpint(data.active, ==, 10);
> @@ -600,7 +594,7 @@ static void test_source_wait_event_notifier_noflush(void)
>
> /* An active event notifier forces aio_poll to look at EventNotifiers.
> */
> event_notifier_init(&dummy.e, false);
> - aio_set_event_notifier(ctx, &dummy.e, event_ready_cb, event_active_cb);
> + aio_set_event_notifier(ctx, &dummy.e, event_ready_cb, NULL);
>
> event_notifier_set(&data.e);
> g_assert(g_main_context_iteration(NULL, false));
>
- [Qemu-devel] [PATCH v4 11/17] block/rbd: drop qemu_rbd_aio_flush_cb(), (continued)
- [Qemu-devel] [PATCH v4 11/17] block/rbd: drop qemu_rbd_aio_flush_cb(), Stefan Hajnoczi, 2013/06/14
- [Qemu-devel] [PATCH v4 02/17] dataplane/virtio-blk: check exit conditions before aio_poll(), Stefan Hajnoczi, 2013/06/14
- [Qemu-devel] [PATCH v4 12/17] block/sheepdog: drop have_co_req() and aio_flush_request(), Stefan Hajnoczi, 2013/06/14
- [Qemu-devel] [PATCH v4 03/17] tests: adjust test-aio to new aio_poll() semantics, Stefan Hajnoczi, 2013/06/14
- [Qemu-devel] [PATCH v4 13/17] block/ssh: drop return_true(), Stefan Hajnoczi, 2013/06/14
- [Qemu-devel] [PATCH v4 15/17] thread-pool: drop thread_pool_active(), Stefan Hajnoczi, 2013/06/14
- [Qemu-devel] [PATCH v4 16/17] tests: drop event_active_cb(), Stefan Hajnoczi, 2013/06/14
- Re: [Qemu-devel] [PATCH v4 16/17] tests: drop event_active_cb(),
Paolo Bonzini <=
- [Qemu-devel] [PATCH v4 17/17] aio: drop io_flush argument, Stefan Hajnoczi, 2013/06/14
- [Qemu-devel] [PATCH v4 05/17] aio: stop using .io_flush(), Stefan Hajnoczi, 2013/06/14
- [Qemu-devel] [PATCH v4 14/17] dataplane/virtio-blk: drop flush_true() and flush_io(), Stefan Hajnoczi, 2013/06/14
- Re: [Qemu-devel] [PATCH v4 00/17] aio: drop io_flush(), Stefan Hajnoczi, 2013/06/27
- Re: [Qemu-devel] [PATCH v4 00/17] aio: drop io_flush(), Paolo Bonzini, 2013/06/27