diff options
author | Jigar Bhatt <jigar.bhatt@intel.com> | 2021-03-22 13:16:01 +0530 |
---|---|---|
committer | Swati Sharma <swati2.sharma@intel.com> | 2021-03-23 17:28:00 +0530 |
commit | 529e182e30117d083ac0693011f1af04357d0115 (patch) | |
tree | ad0d6eac2c07e821f7ddb5e974538f7965b22a6a /tests/i915/i915_pm_dc.c | |
parent | 24c95fb6f5e1d18609391dffe33f247b5a39a895 (diff) |
tests/i915/i915_pm_dc: Add DC9 igt test
DC9 igt test validation depends on DC{5,6} counters reset.
When Display Engine enters to DC9, it resets DMC f/w and DC5/DC6 counter.
We don't have a DC9 counter unlike DC{5,6},
therefore this tests uses DC{5,6} counter reset method to validate DC9.
v11 :
- Changing title.
v10: [Anshuman]
- Revert the function name with check_dc9() only.
v9: [Anshuman]
- Change function name check_dc9() to dc9_failed().
v8: [Anshuman]
- Add support_dc6() to check whether a platform supports dc6.
v7: [Anshuman]
- Skip if no support of DC6.
v6: [Anshuman]
- assert for DC6 counter in setup_dc9_dpms.
- Move dpms_off() to test_dc9_dpms().
Signed-off-by: Jigar Bhatt <jigar.bhatt@intel.com>
Reviewed-by: Anshuman Gupta <anshuman.gupta@intel.com>
Diffstat (limited to 'tests/i915/i915_pm_dc.c')
-rw-r--r-- | tests/i915/i915_pm_dc.c | 53 |
1 files changed, 53 insertions, 0 deletions
diff --git a/tests/i915/i915_pm_dc.c b/tests/i915/i915_pm_dc.c index 30c60243..17ae6622 100644 --- a/tests/i915/i915_pm_dc.c +++ b/tests/i915/i915_pm_dc.c @@ -384,6 +384,52 @@ static void test_dc_state_dpms(data_t *data, int dc_flag) cleanup_dc_dpms(data); } +static bool support_dc6(int debugfs_fd) +{ + char buf[4096]; + + igt_debugfs_simple_read(debugfs_fd, "i915_dmc_info", + buf, sizeof(buf)); + return strstr(buf, "DC5 -> DC6 count"); +} + +static bool check_dc9(uint32_t debugfs_fd, int prev_dc, bool dc6_supported, int seconds) +{ +/* + * Since we do not have DC9 Counter, + * So we rely on dc5/dc6 counter reset to check if Display Engine was in DC9. + */ + return igt_wait(dc6_supported ? read_dc_counter(debugfs_fd, CHECK_DC6) < + prev_dc : read_dc_counter(debugfs_fd, CHECK_DC5) < + prev_dc, seconds, 100); +} + +static void setup_dc9_dpms(data_t *data, int prev_dc, bool dc6_supported) +{ + setup_dc_dpms(data); + dpms_off(data); + igt_skip_on_f(!(igt_wait(dc6_supported ? read_dc_counter(data->debugfs_fd, CHECK_DC6) > + prev_dc : read_dc_counter(data->debugfs_fd, CHECK_DC5) > + prev_dc, 3000, 100)), "Unable to enters shallow DC states\n"); + dpms_on(data); + cleanup_dc_dpms(data); +} + +static void test_dc9_dpms(data_t *data) +{ + require_dc_counter(data->debugfs_fd, CHECK_DC5); + bool dc6_supported = support_dc6(data->debugfs_fd); + + setup_dc9_dpms(data, dc6_supported ? read_dc_counter(data->debugfs_fd, CHECK_DC6) : + read_dc_counter(data->debugfs_fd, CHECK_DC5), dc6_supported); + dpms_off(data); + igt_assert_f(check_dc9(data->debugfs_fd, dc6_supported ? + read_dc_counter(data->debugfs_fd, CHECK_DC6) : + read_dc_counter(data->debugfs_fd, CHECK_DC5), + dc6_supported, 3000), "Not in DC9\n"); + dpms_on(data); +} + IGT_TEST_DESCRIPTION("These tests validate Display Power DC states"); int main(int argc, char *argv[]) { @@ -449,6 +495,13 @@ int main(int argc, char *argv[]) test_dc_state_dpms(&data, CHECK_DC6); } + igt_describe("This test validates display engine entry to DC9 state"); + igt_subtest("dc9-dpms") { + igt_require_f(igt_pm_pc8_plus_residencies_enabled(data.msr_fd), + "PC8+ residencies not supported\n"); + test_dc9_dpms(&data); + } + igt_fixture { free(data.pwr_dmn_info); close(data.debugfs_fd); |