diff options
author | Paulo Zanoni <paulo.r.zanoni@intel.com> | 2014-10-14 14:42:23 -0300 |
---|---|---|
committer | Paulo Zanoni <paulo.r.zanoni@intel.com> | 2014-10-14 15:17:32 -0300 |
commit | 7308e3a92be0529eaff26bc488f86924db09b688 (patch) | |
tree | 783c3e0fee35265f48beb4034430b9938c15b078 /tests | |
parent | 8aa7fdaee8afe569544ffd33f5c3ebac27d2da5b (diff) |
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 <paulo.r.zanoni@intel.com>
Diffstat (limited to 'tests')
-rw-r--r-- | tests/pm_rpm.c | 5 |
1 files changed, 4 insertions, 1 deletions
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) |