Re: [PATCH spice-server 26/33] test-stat: Adjust delay checks

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Dec 21, 2018 at 4:04 PM Frediano Ziglio <fziglio@xxxxxxxxxx> wrote:
>
> usleep under Windows does not seem to have the required precision.
> Use milliseconds and adjust check times according.
>

Have you given g_usleep() a try? It has some specific windows handling.


> Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
> ---
>  server/tests/stat-test.c | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/server/tests/stat-test.c b/server/tests/stat-test.c
> index e4a83f4f..444ff7e3 100644
> --- a/server/tests/stat-test.c
> +++ b/server/tests/stat-test.c
> @@ -57,13 +57,13 @@ void TEST_NAME(void)
>
>      stat_init(&info, "test", CLOCK_MONOTONIC);
>      stat_start_time_init(&start_time, &info);
> -    usleep(2);
> +    usleep(2000);
>      stat_add(&info, start_time);
>
>  #ifdef RED_WORKER_STAT
>      g_assert_cmpuint(info.count, ==, 1);
>      g_assert_cmpuint(info.min, ==, info.max);
> -    g_assert_cmpuint(info.min, >=, 2000);
> +    g_assert_cmpuint(info.min, >=, 2000000);
>      g_assert_cmpuint(info.min, <, 100000000);
>  #endif
>
> @@ -71,17 +71,17 @@ void TEST_NAME(void)
>
>      stat_compress_init(&info, "test", CLOCK_MONOTONIC);
>      stat_start_time_init(&start_time, &info);
> -    usleep(2);
> +    usleep(2000);
>      stat_compress_add(&info, start_time, 100, 50);
> -    usleep(1);
> +    usleep(1000);
>      stat_compress_add(&info, start_time, 1000, 500);
>
>  #ifdef COMPRESS_STAT
>      g_assert_cmpuint(info.count, ==, 2);
>      g_assert_cmpuint(info.min, !=, info.max);
> -    g_assert_cmpuint(info.min, >=, 2000);
> +    g_assert_cmpuint(info.min, >=, 2000000);
>      g_assert_cmpuint(info.min, <, 100000000);
> -    g_assert_cmpuint(info.total, >=, 5000);
> +    g_assert_cmpuint(info.total, >=, 5000000);
>      g_assert_cmpuint(info.orig_size, ==, 1100);
>      g_assert_cmpuint(info.comp_size, ==, 550);
>  #endif
> --
> 2.17.2
>
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/spice-devel



--
Marc-André Lureau
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux Virtualization]     [Linux Virtualization]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]