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

["Nilawar, Badal" <badal.nilawar@intel.com>]

On 14-05-2024 00:25, Rodrigo Vivi wrote:
> If display is enabled, the always-on components will always
> keeps runtime pm references, unless you put DPMS_OFF.
> 
> Cc: Badal Nilawar <badal.nilawar@intel.com>
> Cc: Anshuman Gupta <anshuman.gupta@intel.com>
> Cc: Francois Dugast <francois.dugast@intel.com>
> Signed-off-by: Rodrigo Vivi <rodrigo.vivi@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);
Rahul, please consider this change for mocs tests you added recently in 
xe_pm.c.

Meanwhile this is
Reviewed-by: Badal Nilawar <badal.nilawar@intel.com>

Regards,
Badal Nilawar
>   		}
>   
>   		igt_describe("Validate mmap memory mappings with vram region,"