[igt-dev] [PATCH i-g-t] kms_psr2_su: Skip page-flip tests if FB_DAMAGE_CLIPS not supported

[Hogander, Jouni]

On Mon, 2022-04-25 at 13:36 +0000, Souza, Jose wrote:
> On Fri, 2022-04-22 at 09:02 +0300, Jouni Högander wrote:
> > Currently we are seeing failures in ICL platforms due to
> > missing support for IGT_PLANE_FB_DAMAGE_CLIPS. Skip page-flip
> > tests if IGT_PLANE_FB_DAMAGE_CLIPS is not supported.
> 
> It should skip this whole test test in display <= 11
> 

Yes, I considered this one as well. I just thought it would be better
to check possible support for FB_DAMAGE_CLIPS (which doesn't exist for
"<= 11" either) rather than add one more platform check into igt. This
is basically what the problem in 
https://gitlab.freedesktop.org/drm/intel/-/issues/4148 is. We try to
run this test on platform not supporting FB_DAMAGE_CLIPS.

drivers/gpu/drm/i915/display/skl_universal_plane.c:

	if (DISPLAY_VER(dev_priv) >= 12)
		drm_plane_enable_fb_damage_clips(&plane->base);

What do you think?

> > Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/4148
> > 
> > Signed-off-by: Jouni Högander <jouni.hogander at intel.com>
> > Cc: Petri Latvala <petri.latvala at intel.com>
> > Cc: José Roberto de Souza <jose.souza at intel.com>
> > ---
> >  tests/i915/kms_psr2_su.c | 7 +++++++
> >  1 file changed, 7 insertions(+)
> > 
> > diff --git a/tests/i915/kms_psr2_su.c b/tests/i915/kms_psr2_su.c
> > index 474b1dfd..0ce1f924 100644
> > --- a/tests/i915/kms_psr2_su.c
> > +++ b/tests/i915/kms_psr2_su.c
> > @@ -322,6 +322,13 @@ igt_main
> >   */
> >  igt_skip("PSR2 selective fetch is doing full frame fetches for
> > frontbuffer rendering\n");
> >  }
> > +if (data.op == PAGE_FLIP &&
> > +    !igt_plane_has_prop(igt_output_get_plane_type(
> > +data.output,
> > +DRM_PLANE_TYPE_PRIMARY),
> > +IGT_PLANE_FB_DAMAGE_CLIPS)) {
> > +igt_skip("FB_DAMAGE_CLIPS not supported\n");
> > +}
> >  prepare(&data);
> >  run(&data);
> >  cleanup(&data);