[PATCH i-g-t 1/2] tests/intel/kms_fbt: Skip fbcpsr tests on Wa_14016291713
Hogander, Jouni
jouni.hogander at intel.com
Thu Mar 28 10:32:03 UTC 2024
On Wed, 2024-03-27 at 12:26 +0000, Govindapillai, Vinod wrote:
> On Wed, 2024-03-27 at 07:47 +0200, Jouni Högander wrote:
> > Wa_1401629171 is disabling FBC if PSR is enabled. Check this from
> > FBC
> > debugfs interface and skip current test if FBC is disabled due to
> > Wa_14016291713.
> >
> > Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/9525
> > Signed-off-by: Jouni Högander <jouni.hogander at intel.com>
> > ---
> > tests/intel/kms_frontbuffer_tracking.c | 9 +++++++++
> > 1 file changed, 9 insertions(+)
>
> Reviewed-by: Vinod Govindapillai <vinod.govindapillai at intel.com>
Thank you Vinod for reviewing my patches. These are now pushed to
master.
BR,
Jouni Högander
>
> >
> > diff --git a/tests/intel/kms_frontbuffer_tracking.c
> > b/tests/intel/kms_frontbuffer_tracking.c
> > index db2db0ff1..6d5395a0c 100644
> > --- a/tests/intel/kms_frontbuffer_tracking.c
> > +++ b/tests/intel/kms_frontbuffer_tracking.c
> > @@ -2025,6 +2025,14 @@ static bool fbc_mode_too_large(void)
> > return strstr(buf, "FBC disabled: mode too large for
> > compression\n");
> > }
> >
> > +static bool fbc_psr_not_possible(void)
> > +{
> > + char buf[128];
> > +
> > + debugfs_read_crtc("i915_fbc_status", buf);
> > + return strstr(buf, "FBC disabled: PSR1 enabled
> > (Wa_14016291713)");
> > +}
> > +
> > static bool fbc_enable_per_plane(int plane_index, enum pipe pipe)
> > {
> > char buf[PATH_MAX];
> > @@ -2699,6 +2707,7 @@ static void do_status_assertions(int flags)
> > igt_require(!fbc_not_enough_stolen());
> > igt_require(!fbc_stride_not_supported());
> > igt_require(!fbc_mode_too_large());
> > + igt_require(!fbc_psr_not_possible());
> > if (!intel_fbc_wait_until_enabled(drm.fd,
> > prim_mode_params.pipe)) {
> > igt_assert_f(intel_fbc_is_enabled(drm.fd,
> >
> > prim_mode_params.pipe,
>
More information about the igt-dev
mailing list