| Message ID | 20180719163756.15463-1-tvrtko.ursulin@linux.intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Quoting Tvrtko Ursulin (2018-07-19 17:37:56) > From: Tvrtko Ursulin <tvrtko.ursulin@intel.com> > > Restore runtime PM state (via a newly added library function) when the > test which sets it up exit. This was we avoid running all subsequent sub- > tests in the aggressive runtime PM mode. > > Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com> > --- > lib/igt_pm.c | 13 ++++++++++++- > lib/igt_pm.h | 1 + > tests/perf_pmu.c | 3 +++ > 3 files changed, 16 insertions(+), 1 deletion(-) > > diff --git a/lib/igt_pm.c b/lib/igt_pm.c > index 8ac132269d79..2e2eea5d3b2d 100644 > --- a/lib/igt_pm.c > +++ b/lib/igt_pm.c > @@ -297,7 +297,13 @@ int pm_status_fd = -1; > static char __igt_pm_runtime_autosuspend[64]; > static char __igt_pm_runtime_control[64]; > > -static void __igt_pm_runtime_exit_handler(int sig) > +/** > + * igt_restore_runtime_pm: > + * > + * Restores the runtime PM configuration as it was before the call to > + * igt_setup_runtime_pm. > + */ > +void igt_restore_runtime_pm(void) > { > int fd; Should we eat __igt_pm_runtime_autosuspend so we only apply it once for each igt_setup_runtime_pm? -Chris
diff --git a/lib/igt_pm.c b/lib/igt_pm.c index 8ac132269d79..2e2eea5d3b2d 100644 --- a/lib/igt_pm.c +++ b/lib/igt_pm.c @@ -297,7 +297,13 @@ int pm_status_fd = -1; static char __igt_pm_runtime_autosuspend[64]; static char __igt_pm_runtime_control[64]; -static void __igt_pm_runtime_exit_handler(int sig) +/** + * igt_restore_runtime_pm: + * + * Restores the runtime PM configuration as it was before the call to + * igt_setup_runtime_pm. + */ +void igt_restore_runtime_pm(void) { int fd; @@ -326,6 +332,11 @@ static void __igt_pm_runtime_exit_handler(int sig) close(fd); } +static void __igt_pm_runtime_exit_handler(int sig) +{ + igt_restore_runtime_pm(); +} + /** * igt_setup_runtime_pm: * diff --git a/lib/igt_pm.h b/lib/igt_pm.h index eced39f8801a..10cc6794e4e7 100644 --- a/lib/igt_pm.h +++ b/lib/igt_pm.h @@ -47,6 +47,7 @@ enum igt_runtime_pm_status { }; bool igt_setup_runtime_pm(void); +void igt_restore_runtime_pm(void); enum igt_runtime_pm_status igt_get_runtime_pm_status(void); bool igt_wait_for_pm_status(enum igt_runtime_pm_status status); diff --git a/tests/perf_pmu.c b/tests/perf_pmu.c index a1d36ac4fa9d..9a20abb6b95c 100644 --- a/tests/perf_pmu.c +++ b/tests/perf_pmu.c @@ -1441,6 +1441,9 @@ test_rc6(int gem_fd, unsigned int flags) close(fw); close(fd); + if (flags & TEST_RUNTIME_PM) + igt_restore_runtime_pm(); + assert_within_epsilon(busy - prev, 0.0, tolerance); }