diff options
author | Anshuman Gupta <anshuman.gupta@intel.com> | 2022-03-10 19:05:51 +0530 |
---|---|---|
committer | Ashutosh Dixit <ashutosh.dixit@intel.com> | 2022-03-10 13:13:39 -0800 |
commit | aa6eb64bac510b7d617436997171bfe388943d89 (patch) | |
tree | 3c754b920996a9abd4db13f6ffd41a10cf3b0535 /tests/i915/i915_pm_rpm.c | |
parent | 68efa99c62e22e25abd08ce0f044308575a882e3 (diff) |
i915_pm_rpm: Add placement to gem_exec_stress
This add memory region placement to gem_exec_stress group of test.
gem-execbuf-stress-pc8 is odd one test as PC8 is applicable to igfx
platform. dgfx soc has its own PkgG sates therefore memory region
placement is irrelevant for gem-execbuf-stress-pc8 test.
v2:
- Deleted gem_memory_topology.[ch] and moved it's content
to intel_memory_region.[ch]. [Ashutosh]
v3:
- Added igt_describe().
Cc: Ashutosh Dixit <ashutosh.dixit@intel.com>
Signed-off-by: Anshuman Gupta <anshuman.gupta@intel.com>
Reviewed-by: Ashutosh Dixit <ashutosh.dixit@intel.com>
Diffstat (limited to 'tests/i915/i915_pm_rpm.c')
-rw-r--r-- | tests/i915/i915_pm_rpm.c | 33 |
1 files changed, 25 insertions, 8 deletions
diff --git a/tests/i915/i915_pm_rpm.c b/tests/i915/i915_pm_rpm.c index 9afe7427..8ae407e5 100644 --- a/tests/i915/i915_pm_rpm.c +++ b/tests/i915/i915_pm_rpm.c @@ -1384,7 +1384,9 @@ static void gem_execbuf_subtest(void) /* Assuming execbuf already works, let's see what happens when we force many * suspend/resume cycles with commands. */ -static void gem_execbuf_stress_subtest(int rounds, int wait_flags) +static void +gem_execbuf_stress_subtest(int rounds, int wait_flags, + struct drm_i915_gem_memory_class_instance *mem_regions) { int i; int batch_size = 4 * sizeof(uint32_t); @@ -1407,7 +1409,12 @@ static void gem_execbuf_stress_subtest(int rounds, int wait_flags) disable_all_screens_and_wait(&ms_data); - handle = gem_create(drm_fd, batch_size); + /* PC8 test is only applicable to igfx */ + if (wait_flags & WAIT_PC8_RES) + handle = gem_create(drm_fd, batch_size); + else + handle = gem_create_in_memory_region_list(drm_fd, batch_size, mem_regions, 1); + gem_write(drm_fd, handle, 0, batch_buf, batch_size); objs[0].handle = handle; @@ -2049,8 +2056,9 @@ igt_main_args("", long_options, help_str, opt_handler, NULL) /* Skip instead of failing in case the machine is not prepared to reach * PC8+. We don't want bug reports from cases where the machine is just * not properly configured. */ - igt_fixture + igt_fixture { igt_require(setup_environment(false)); + } if (stay) igt_subtest("stay") @@ -2162,12 +2170,21 @@ igt_main_args("", long_options, help_str, opt_handler, NULL) system_suspend_subtest(SUSPEND_STATE_DISK, SUSPEND_TEST_NONE); /* GEM stress */ - igt_subtest("gem-execbuf-stress") - gem_execbuf_stress_subtest(rounds, WAIT_STATUS); + igt_subtest_with_dynamic("gem-execbuf-stress") { + for_each_memory_region(r, drm_fd) { + igt_describe("Validate execbuf submission while exercising rpm " + "suspend/resume cycles."); + igt_dynamic_f("%s", r->name) + gem_execbuf_stress_subtest(rounds, WAIT_STATUS, &r->ci); + igt_describe("Validate execbuf submission while exercising rpm " + "suspend/resume cycles with extra wait."); + igt_dynamic_f("%s-%s", "extra-wait", r->name) + gem_execbuf_stress_subtest(rounds, WAIT_STATUS | WAIT_EXTRA, &r->ci); + } + } + igt_subtest("gem-execbuf-stress-pc8") - gem_execbuf_stress_subtest(rounds, WAIT_PC8_RES); - igt_subtest("gem-execbuf-stress-extra-wait") - gem_execbuf_stress_subtest(rounds, WAIT_STATUS | WAIT_EXTRA); + gem_execbuf_stress_subtest(rounds, WAIT_PC8_RES, 0); /* power-wake reference tests */ igt_subtest("pm-tiling") { |