[PATCH i-g-t 3/5] tests/intel/xe_pm: Also disable display for mmap_system test

[Gupta, Anshuman]

> -----Original Message-----
> From: Vivi, Rodrigo <rodrigo.vivi at intel.com>
> Sent: Tuesday, May 14, 2024 12:25 AM
> To: igt-dev at lists.freedesktop.org
> Cc: intel-xe at lists.freedesktop.org; Vivi, Rodrigo <rodrigo.vivi at intel.com>;
> Nilawar, Badal <badal.nilawar at intel.com>; Gupta, Anshuman
> <anshuman.gupta at intel.com>; Dugast, Francois <francois.dugast at intel.com>
> Subject: [PATCH i-g-t 3/5] tests/intel/xe_pm: Also disable display for
> mmap_system test
> 
> If display is enabled, the always-on components will always keeps runtime pm
> references, unless you put DPMS_OFF.
> 
> Cc: Badal Nilawar <badal.nilawar at intel.com>
> Cc: Anshuman Gupta <anshuman.gupta at intel.com>
> Cc: Francois Dugast <francois.dugast at intel.com>
> Signed-off-by: Rodrigo Vivi <rodrigo.vivi at intel.com>
[Gupta, Anshuman] looks good to me.
Reviewed-by: Anshuman Gupta <anshuman.gupta at intel.com>
> ---
>  tests/intel/xe_pm.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/tests/intel/xe_pm.c b/tests/intel/xe_pm.c index
> 3f963bd9b..72d484163 100644
> --- a/tests/intel/xe_pm.c
> +++ b/tests/intel/xe_pm.c
> @@ -780,7 +780,9 @@ igt_main
>  		igt_describe("Validate mmap memory mappings with system
> region,"
>  			     "when device along with parent bridge in d3");
>  		igt_subtest("d3-mmap-system") {
> +			dpms_on_off(device, DRM_MODE_DPMS_OFF);
>  			test_mmap(device, system_memory(device.fd_xe), 0);
> +			dpms_on_off(device, DRM_MODE_DPMS_ON);
>  		}
> 
>  		igt_describe("Validate mmap memory mappings with vram
> region,"
> --
> 2.44.0