From 7308e3a92be0529eaff26bc488f86924db09b688 Mon Sep 17 00:00:00 2001 From: Paulo Zanoni Date: Tue, 14 Oct 2014 14:42:23 -0300 Subject: [PATCH] tests/pm_rpm: reduce the number of rounds on the stress tests With the current Kernel, it takes more than 10 minutes to run modeset-stress-extra-wait on BYT with eDP, and we don't really want IGT subtests that take more than 10 minutes to run: QA reports them as bugs. Since, as far as I remember, any of these RPM race condition bugs we had could always be triggered with just 4-5 rounds of testing, let's tune down the number of rounds we test: Before: rounds = (--quick option used) ? 10 : 50; After: rounds = (--quick option used) ? 10 : 40; rounds = (wait_flags & WAIT_EXTRA) ? rounds / 2 : rounds; This should make all subtests be way above the 10m limit. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=84897 Signed-off-by: Paulo Zanoni --- tests/pm_rpm.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/tests/pm_rpm.c b/tests/pm_rpm.c index 31459cbb..1e864372 100644 --- a/tests/pm_rpm.c +++ b/tests/pm_rpm.c @@ -768,6 +768,9 @@ static void modeset_subtest(enum screen_type type, int rounds, int wait_flags) if (wait_flags & WAIT_PC8_RES) igt_require(has_pc8); + if (wait_flags & WAIT_EXTRA) + rounds /= 2; + for (i = 0; i < rounds; i++) { if (wait_flags & USE_DPMS) disable_all_screens_dpms(&ms_data); @@ -1719,7 +1722,7 @@ static void fences_subtest(bool dpms) igt_assert(munmap(buf_ptr, params.fb.size) == 0); } -int rounds = 50; +int rounds = 40; bool stay = false; static int opt_handler(int opt, int opt_index)