diff options
| author | José Roberto de Souza <jose.souza@intel.com> | 2019-11-05 10:17:37 -0800 |
|---|---|---|
| committer | Arkadiusz Hiler <arkadiusz.hiler@intel.com> | 2019-11-08 09:49:45 +0200 |
| commit | 5ce339549818d301348c75c58a11cfc6e932bb88 (patch) | |
| tree | c89164e20cd227bf0d06aee21299f93a86a940ea /tests/i915/i915_pm_dc.c | |
| parent | 60a67653613c87a69ebecf12cf00aa362ac87597 (diff) | |
tests/i915_pm_dc: Fix the requirement check
igt_debugfs_simple_read() requires the debugfs file descriptor not
the drm device file descriptor, so the requirement check was complete
broken.
Cc: Jeevan B <jeevan.b@intel.com>
Cc: Anshuman Gupta <anshuman.gupta@intel.com>
Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
Signed-off-by: José Roberto de Souza <jose.souza@intel.com>
Reviewed-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
Diffstat (limited to 'tests/i915/i915_pm_dc.c')
| -rw-r--r-- | tests/i915/i915_pm_dc.c | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/tests/i915/i915_pm_dc.c b/tests/i915/i915_pm_dc.c index 90d81cfe..cce70a57 100644 --- a/tests/i915/i915_pm_dc.c +++ b/tests/i915/i915_pm_dc.c @@ -278,24 +278,24 @@ static void check_dc3co_with_videoplayback_like_load(data_t *data) check_dc_counter(data->drm_fd, CHECK_DC3CO, dc3co_prev_cnt); } -static void require_dc_counter(int drm_fd, int dc_flag) +static void require_dc_counter(int debugfs_fd, int dc_flag) { char buf[4096]; - igt_debugfs_simple_read(drm_fd, "i915_dmc_info", + igt_debugfs_simple_read(debugfs_fd, "i915_dmc_info", buf, sizeof(buf)); switch (dc_flag) { case CHECK_DC3CO: - igt_skip_on_f(strstr(buf, "DC3CO count"), + igt_skip_on_f(!strstr(buf, "DC3CO count"), "DC3CO counter is not available\n"); break; case CHECK_DC5: - igt_skip_on_f(strstr(buf, "DC3 -> DC5 count"), + igt_skip_on_f(!strstr(buf, "DC3 -> DC5 count"), "DC5 counter is not available\n"); break; case CHECK_DC6: - igt_skip_on_f(strstr(buf, "DC5 -> DC6 count"), + igt_skip_on_f(!strstr(buf, "DC5 -> DC6 count"), "DC6 counter is not available\n"); break; default: @@ -313,7 +313,7 @@ static void setup_dc3co(data_t *data) static void test_dc3co_vpb_simulation(data_t *data) { - require_dc_counter(data->drm_fd, CHECK_DC3CO); + require_dc_counter(data->debugfs_fd, CHECK_DC3CO); setup_output(data); setup_dc3co(data); setup_videoplayback(data); @@ -325,7 +325,7 @@ static void test_dc_state_psr(data_t *data, int dc_flag) { uint32_t dc_counter_before_psr; - require_dc_counter(data->drm_fd, dc_flag); + require_dc_counter(data->debugfs_fd, dc_flag); dc_counter_before_psr = read_dc_counter(data->drm_fd, dc_flag); setup_output(data); setup_primary(data); @@ -387,7 +387,7 @@ static void test_dc_state_dpms(data_t *data, int dc_flag) { uint32_t dc_counter; - require_dc_counter(data->drm_fd, dc_flag); + require_dc_counter(data->debugfs_fd, dc_flag); setup_dc_dpms(data); dc_counter = read_dc_counter(data->drm_fd, dc_flag); dpms_off(data); |