summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRiana Tauro <riana.tauro@intel.com>2022-06-21 19:15:25 +0530
committerAshutosh Dixit <ashutosh.dixit@intel.com>2022-06-27 12:15:53 -0700
commit56e9d4afb2402ce1031f20ff07cbe9f9749378db (patch)
tree0df2704019cd6ead2dc8a8684c1739e6b6713a09
parentd38a476ee4b9f9a95d8f452de0d66cc52f7f079b (diff)
tests/i915/pm_rc6_residency: Refactoring intel_ctx_t to igt_fixture
Refactor code to move intel_ctx_create_all_physical() and intel_ctx_destroy() to igt_fixture to be used by rc6_fence and other tests. No functional changes v2 : refactor code (Anshuman) Signed-off-by: Riana Tauro <riana.tauro@intel.com> Reviewed-by: Anshuman Gupta <anshuman.gupta@intel.com>
-rw-r--r--tests/i915/i915_pm_rc6_residency.c14
1 files changed, 7 insertions, 7 deletions
diff --git a/tests/i915/i915_pm_rc6_residency.c b/tests/i915/i915_pm_rc6_residency.c
index cf9eae90..7c1ff32b 100644
--- a/tests/i915/i915_pm_rc6_residency.c
+++ b/tests/i915/i915_pm_rc6_residency.c
@@ -449,13 +449,12 @@ static void rc6_idle(int i915)
}
}
-static void rc6_fence(int i915)
+static void rc6_fence(int i915, const intel_ctx_t *ctx)
{
const int64_t duration_ns = SLEEP_DURATION * (int64_t)NSEC_PER_SEC;
const int tolerance = 20; /* Some RC6 is better than none! */
const unsigned int gen = intel_gen(intel_get_drm_devid(i915));
const struct intel_execution_engine2 *e;
- const intel_ctx_t *ctx;
struct power_sample sample[2];
unsigned long slept;
uint64_t rc6, ts[2], ahnd;
@@ -485,7 +484,6 @@ static void rc6_fence(int i915)
assert_within_epsilon(rc6, ts[1] - ts[0], 5);
/* Submit but delay execution, we should be idle and conserving power */
- ctx = intel_ctx_create_all_physical(i915);
ahnd = get_reloc_ahnd(i915, ctx->id);
for_each_ctx_engine(i915, ctx, e) {
igt_spin_t *spin;
@@ -524,7 +522,6 @@ static void rc6_fence(int i915)
assert_within_epsilon(rc6, ts[1] - ts[0], tolerance);
gem_quiescent_gpu(i915);
}
- intel_ctx_destroy(i915, ctx);
put_ahnd(ahnd);
rapl_close(&rapl);
@@ -534,10 +531,12 @@ static void rc6_fence(int i915)
igt_main
{
int i915 = -1;
+ const intel_ctx_t *ctx;
/* Use drm_open_driver to verify device existence */
igt_fixture {
i915 = drm_open_driver(DRIVER_INTEL);
+ ctx = intel_ctx_create_all_physical(i915);
}
igt_subtest("rc6-idle") {
@@ -551,7 +550,7 @@ igt_main
igt_require_gem(i915);
gem_quiescent_gpu(i915);
- rc6_fence(i915);
+ rc6_fence(i915, ctx);
}
igt_subtest_group {
@@ -592,7 +591,8 @@ igt_main
close(sysfs);
}
- igt_fixture
+ igt_fixture {
+ intel_ctx_destroy(i915, ctx);
close(i915);
-
+ }
}