[igt-dev] [PATCH i-g-t] tests/kms_cursor_legacy: Fix CRC mismatch for flip-vs-cursor-busy-crc-*

[Mark Yacoub]

Applied. Thanks Vidya and Bhanu.

On Mon, Sep 20, 2021 at 6:51 AM Modem, Bhanuprakash
<bhanuprakash.modem at intel.com> wrote:
>
> > From: Srinivas, Vidya <vidya.srinivas at intel.com>
> > Sent: Thursday, September 16, 2021 7:57 PM
> > To: igt-dev at lists.freedesktop.org
> > Cc: Modem, Bhanuprakash <bhanuprakash.modem at intel.com>; markyacoub at google.com;
> > Srinivas, Vidya <vidya.srinivas at intel.com>
> > Subject: [PATCH i-g-t] tests/kms_cursor_legacy: Fix CRC mismatch for flip-vs-
> > cursor-busy-crc-*
> >
> > Few variants of Intel devices are failing randomly with CRC mismatch for
> > flip-vs-cursor-busy-crc-atomic and flip-vs-cursor-busy-crc-legacy.
> > Patch adds stop of CRC collection to fix this issue.
> >
> > Signed-off-by: Vidya Srinivas <vidya.srinivas at intel.com>
> > ---
> >  tests/kms_cursor_legacy.c | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/tests/kms_cursor_legacy.c b/tests/kms_cursor_legacy.c
> > index 3ddea9c35340..8136280701e6 100644
> > --- a/tests/kms_cursor_legacy.c
> > +++ b/tests/kms_cursor_legacy.c
> > @@ -1401,6 +1401,7 @@ static void flip_vs_cursor_busy_crc(igt_display_t
> > *display, bool atomic)
> >               igt_assert_crc_equal(&crcs[i], &test_crc);
> >       }
> >
> > +     igt_pipe_crc_stop(pipe_crc);
>
> LGTM:
> Reviewed-by: Bhanuprakash Modem <Bhanuprakash.modem at intel.com>
>
> - Bhanu
>
> >       igt_remove_fb(display->drm_fd, &fb_info[1]);
> >       igt_remove_fb(display->drm_fd, &fb_info[0]);
> >       igt_remove_fb(display->drm_fd, &cursor_fb);
> > --
> > 2.33.0
>