[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/3] throttle: add detach/attach test case
From: |
Benoît Canet |
Subject: |
Re: [Qemu-devel] [PATCH 2/3] throttle: add detach/attach test case |
Date: |
Wed, 14 May 2014 17:08:11 +0200 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
The Wednesday 14 May 2014 à 16:22:46 (+0200), Stefan Hajnoczi wrote :
> Add a test case that checks the timer is really removed/added by the
> detach/attach functions.
>
> Cc: Benoît Canet <address@hidden>
> Signed-off-by: Stefan Hajnoczi <address@hidden>
> ---
> tests/test-throttle.c | 24 ++++++++++++++++++++++++
> 1 file changed, 24 insertions(+)
>
> diff --git a/tests/test-throttle.c b/tests/test-throttle.c
> index 5fa5000..3de6ab8 100644
> --- a/tests/test-throttle.c
> +++ b/tests/test-throttle.c
> @@ -338,6 +338,29 @@ static void test_have_timer(void)
> throttle_destroy(&ts);
> }
>
> +static void test_detach_attach(void)
> +{
> + /* zero the structure */
> + memset(&ts, 0, sizeof(ts));
> +
> + /* init the structure */
> + throttle_init(&ts, ctx, QEMU_CLOCK_VIRTUAL,
> + read_timer_cb, write_timer_cb, &ts);
> +
> + /* timer set by init should return true */
> + g_assert(throttle_have_timer(&ts));
> +
> + /* timer should no longer exist after detaching */
> + throttle_detach_aio_context(&ts);
> + g_assert(!throttle_have_timer(&ts));
> +
> + /* timer should exist again after attaching */
> + throttle_attach_aio_context(&ts, ctx);
> + g_assert(throttle_have_timer(&ts));
> +
> + throttle_destroy(&ts);
> +}
> +
> static bool do_test_accounting(bool is_ops, /* are we testing bps or ops */
> int size, /* size of the operation to do */
> double avg, /* io limit */
> @@ -486,6 +509,7 @@ int main(int argc, char **argv)
> g_test_add_func("/throttle/init", test_init);
> g_test_add_func("/throttle/destroy", test_destroy);
> g_test_add_func("/throttle/have_timer", test_have_timer);
> + g_test_add_func("/throttle/detach_attach", test_detach_attach);
> g_test_add_func("/throttle/config/enabled", test_enabled);
> g_test_add_func("/throttle/config/conflicting", test_conflicting_config);
> g_test_add_func("/throttle/config/is_valid", test_is_valid);
> --
> 1.9.0
>
Reviewed-by: Benoit Canet <address@hidden>
- [Qemu-devel] [PATCH 0/3] throttle: use AioContext for dataplane support, Stefan Hajnoczi, 2014/05/14
- [Qemu-devel] [PATCH 1/3] throttle: add throttle_detach/attach_aio_context(), Stefan Hajnoczi, 2014/05/14
- [Qemu-devel] [PATCH 2/3] throttle: add detach/attach test case, Stefan Hajnoczi, 2014/05/14
- Re: [Qemu-devel] [PATCH 2/3] throttle: add detach/attach test case,
Benoît Canet <=
- [Qemu-devel] [PATCH 3/3] blockdev: acquire AioContext in block_set_io_throttle, Stefan Hajnoczi, 2014/05/14
- Re: [Qemu-devel] [PATCH 0/3] throttle: use AioContext for dataplane support, Benoît Canet, 2014/05/14
- Re: [Qemu-devel] [PATCH 0/3] throttle: use AioContext for dataplane support, Benoît Canet, 2014/05/14