[igt-dev] [PATCH i-g-t v11] tests/i915/i915_pm_dc: Add DC9 igt test

[Gupta, Anshuman]

> -----Original Message-----
> From: Bhatt, Jigar <jigar.bhatt at intel.com>
> Sent: Monday, March 22, 2021 1:16 PM
> To: igt-dev at lists.freedesktop.org
> Cc: Bhatt, Jigar <jigar.bhatt at intel.com>; Gupta, Anshuman
> <anshuman.gupta at intel.com>
> Subject: [PATCH i-g-t v11] 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.
Reviewed-by: Anshuman Gupta <anshuman.gupta at intel.com>
> 
> 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 at intel.com>
> ---
>  tests/i915/i915_pm_dc.c | 53
> +++++++++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 53 insertions(+)
> 
> diff --git a/tests/i915/i915_pm_dc.c b/tests/i915/i915_pm_dc.c index
> 30c6024..f5ea278 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);
> --
> 2.8.1