[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] tests/boot-sector: Increase timeout to 600 seco
From: |
Stefan Weil |
Subject: |
Re: [Qemu-devel] [PATCH] tests/boot-sector: Increase timeout to 600 seconds |
Date: |
Fri, 22 Sep 2017 07:17:26 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 |
Am 22.09.2017 um 05:06 schrieb Thomas Huth:
> If QEMU has been compiled with the flags --enable-tcg-interpreter and
> --enable-debug, the guest is running incredibly slow. The pxe boot test
> can take up to 400 seconds when testing the pseries ppc64 machine. While
> we should still look for ways to speed up the test on the pseries machine,
> it's better to increase the timeout in this test to 600 seconds anyway to
> allow the test to pass successfully now with this unusal configuration
> already.
>
> Signed-off-by: Thomas Huth <address@hidden>
> ---
> tests/boot-sector.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tests/boot-sector.c b/tests/boot-sector.c
> index 9ee8537..be29d5b 100644
> --- a/tests/boot-sector.c
> +++ b/tests/boot-sector.c
> @@ -137,9 +137,9 @@ void boot_sector_test(void)
> uint16_t signature;
> int i;
>
> - /* Wait at most 90 seconds */
> + /* Wait at most 600 seconds (test is slow with TCI and --enable-debug) */
> #define TEST_DELAY (1 * G_USEC_PER_SEC / 10)
> -#define TEST_CYCLES MAX((90 * G_USEC_PER_SEC / TEST_DELAY), 1)
> +#define TEST_CYCLES MAX((600 * G_USEC_PER_SEC / TEST_DELAY), 1)
>
> /* Poll until code has run and modified memory. Once it has we know BIOS
> * initialization is done. TODO: check that IP reached the halt
>
Reviewed-by: Stefan Weil <address@hidden>
cc'ing qemu-trivial
Thanks,
Stefan
Re: [Qemu-devel] [PATCH] tests/boot-sector: Increase timeout to 600 seconds, Michael S. Tsirkin, 2017/09/26