| Message ID | 1505972355-23903-1-git-send-email-thuth@redhat.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Am 21.09.2017 um 07:39 schrieb Thomas Huth: > If QEMU has been compiled with the flags --enable-tcg-interpreter and > --enable-debug, the guest is running incredibly slow. The prom-env > test is approximately 10 times slower than normal in this case, and > it takes up to 500 seconds until the test with the pseries machine > finishs. While we should still look for ways to speed up the test > on the pseries machine here, let's bump the timeout to 600 seconds to > allow the test to pass with this unusal configuration already now. > Also move the pseries test into the "slow" category - since it is > really a very slow test. > > Signed-off-by: Thomas Huth <thuth@redhat.com> > --- > tests/prom-env-test.c | 14 ++++++++------ > 1 file changed, 8 insertions(+), 6 deletions(-) > > diff --git a/tests/prom-env-test.c b/tests/prom-env-test.c > index eac207b..bc8b616 100644 > --- a/tests/prom-env-test.c > +++ b/tests/prom-env-test.c > @@ -1,7 +1,7 @@ > /* > - * Test OpenBIOS-based machines. > + * Test Open-Firmware-based machines. > * > - * Copyright (c) 2016 Red Hat Inc. > + * Copyright (c) 2016, 2017 Red Hat Inc. > * > * Author: > * Thomas Huth <thuth@redhat.com> > @@ -30,8 +30,8 @@ static void check_guest_memory(void) > uint32_t signature; > int i; > > - /* Poll until code has run and modified memory. Wait at most 120 seconds */ > - for (i = 0; i < 12000; ++i) { > + /* Poll until code has run and modified memory. Wait at most 600 seconds */ > + for (i = 0; i < 60000; ++i) { > signature = readl(ADDRESS); > if (signature == MAGIC) { > break; > @@ -78,7 +78,6 @@ int main(int argc, char *argv[]) > const char *sparc_machines[] = { "SPARCbook", "Voyager", "SS-20", NULL }; > const char *sparc64_machines[] = { "sun4u", NULL }; > const char *ppc_machines[] = { "mac99", "g3beige", NULL }; > - const char *ppc64_machines[] = { "mac99", "g3beige", "pseries", NULL }; > const char *arch = qtest_get_arch(); > > g_test_init(&argc, &argv, NULL); > @@ -86,7 +85,10 @@ int main(int argc, char *argv[]) > if (!strcmp(arch, "ppc")) { > add_tests(ppc_machines); > } else if (!strcmp(arch, "ppc64")) { > - add_tests(ppc64_machines); > + add_tests(ppc_machines); > + if (g_test_slow()) { > + qtest_add_data_func("prom-env/pseries", "pseries", test_machine); > + } > } else if (!strcmp(arch, "sparc")) { > add_tests(sparc_machines); > } else if (!strcmp(arch, "sparc64")) { > Thanks! Reviewed-by: Stefan Weil <sw@weilnetz.de>
diff --git a/tests/prom-env-test.c b/tests/prom-env-test.c index eac207b..bc8b616 100644 --- a/tests/prom-env-test.c +++ b/tests/prom-env-test.c @@ -1,7 +1,7 @@ /* - * Test OpenBIOS-based machines. + * Test Open-Firmware-based machines. * - * Copyright (c) 2016 Red Hat Inc. + * Copyright (c) 2016, 2017 Red Hat Inc. * * Author: * Thomas Huth <thuth@redhat.com> @@ -30,8 +30,8 @@ static void check_guest_memory(void) uint32_t signature; int i; - /* Poll until code has run and modified memory. Wait at most 120 seconds */ - for (i = 0; i < 12000; ++i) { + /* Poll until code has run and modified memory. Wait at most 600 seconds */ + for (i = 0; i < 60000; ++i) { signature = readl(ADDRESS); if (signature == MAGIC) { break; @@ -78,7 +78,6 @@ int main(int argc, char *argv[]) const char *sparc_machines[] = { "SPARCbook", "Voyager", "SS-20", NULL }; const char *sparc64_machines[] = { "sun4u", NULL }; const char *ppc_machines[] = { "mac99", "g3beige", NULL }; - const char *ppc64_machines[] = { "mac99", "g3beige", "pseries", NULL }; const char *arch = qtest_get_arch(); g_test_init(&argc, &argv, NULL); @@ -86,7 +85,10 @@ int main(int argc, char *argv[]) if (!strcmp(arch, "ppc")) { add_tests(ppc_machines); } else if (!strcmp(arch, "ppc64")) { - add_tests(ppc64_machines); + add_tests(ppc_machines); + if (g_test_slow()) { + qtest_add_data_func("prom-env/pseries", "pseries", test_machine); + } } else if (!strcmp(arch, "sparc")) { add_tests(sparc_machines); } else if (!strcmp(arch, "sparc64")) {
If QEMU has been compiled with the flags --enable-tcg-interpreter and --enable-debug, the guest is running incredibly slow. The prom-env test is approximately 10 times slower than normal in this case, and it takes up to 500 seconds until the test with the pseries machine finishs. While we should still look for ways to speed up the test on the pseries machine here, let's bump the timeout to 600 seconds to allow the test to pass with this unusal configuration already now. Also move the pseries test into the "slow" category - since it is really a very slow test. Signed-off-by: Thomas Huth <thuth@redhat.com> --- tests/prom-env-test.c | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-)