[Intel-gfx] [PATCH i-g-t] tests/kms_plane_multiple: Fix reference CRC
Kahola, Mika
mika.kahola at intel.com
Mon Jul 31 09:04:50 UTC 2017
> -----Original Message-----
> From: daniel.vetter at ffwll.ch [mailto:daniel.vetter at ffwll.ch] On Behalf Of
> Daniel Vetter
> Sent: Monday, July 31, 2017 11:13 AM
> To: Kahola, Mika <mika.kahola at intel.com>
> Cc: intel-gfx <intel-gfx at lists.freedesktop.org>
> Subject: Re: [Intel-gfx] [PATCH i-g-t] tests/kms_plane_multiple: Fix reference
> CRC
>
> On Fri, Jul 28, 2017 at 2:45 PM, Mika Kahola <mika.kahola at intel.com> wrote:
> > When grabbing reference CRC with igt_pipe_crc_get_crcs() the number of
> > words in igt_crc_t structure was incorrectly collected. The fix here
> > is to switch to igt_pipe_crc_collect_crc() function when collecting
> > CRC for reference frame.
>
> So there's also a bug in the core library that this patch papers over?
> Do you have a patch for that one too?
The core library got updated and that actually revealed a bug in the test itself. igt_crc_t struct member n_words was not correctly copied and the updated core function checks if this is 0 and if so the crc check fails. All we need to do is have a fix on this test.
Cheers,
Mika
> -Daniel
>
> >
> > The problem was caught by CI system and at least affects on HSW platform.
> >
> > Signed-off-by: Mika Kahola <mika.kahola at intel.com>
> > ---
> > tests/kms_plane_multiple.c | 10 ++++++----
> > 1 file changed, 6 insertions(+), 4 deletions(-)
> >
> > diff --git a/tests/kms_plane_multiple.c b/tests/kms_plane_multiple.c
> > index f6c6223..08f184a 100644
> > --- a/tests/kms_plane_multiple.c
> > +++ b/tests/kms_plane_multiple.c
> > @@ -110,7 +110,7 @@ test_grab_crc(data_t *data, igt_output_t *output,
> > enum pipe pipe, bool atomic, {
> > drmModeModeInfo *mode;
> > igt_plane_t *primary;
> > - int ret, n;
> > + int ret;
> >
> > igt_output_set_pipe(output, pipe);
> >
> > @@ -131,9 +131,7 @@ test_grab_crc(data_t *data, igt_output_t *output,
> enum pipe pipe, bool atomic,
> > atomic ? COMMIT_ATOMIC : COMMIT_LEGACY);
> > igt_skip_on(ret != 0);
> >
> > - igt_pipe_crc_start(data->pipe_crc);
> > - n = igt_pipe_crc_get_crcs(data->pipe_crc, 1, &crc);
> > - igt_assert_eq(n, 1);
> > + igt_pipe_crc_collect_crc(data->pipe_crc, crc);
> > }
> >
> > /*
> > @@ -278,6 +276,8 @@ test_atomic_plane_position_with_output(data_t
> *data, enum pipe pipe,
> > test_grab_crc(data, output, pipe, true, &blue, tiling,
> > &test.reference_crc);
> >
> > + igt_pipe_crc_start(data->pipe_crc);
> > +
> > i = 0;
> > while (i < iterations || loop_forever) {
> > prepare_planes(data, pipe, &blue, tiling, n_planes,
> > output); @@ -344,6 +344,8 @@
> test_legacy_plane_position_with_output(data_t *data, enum pipe pipe,
> > test_grab_crc(data, output, pipe, false, &blue, tiling,
> > &test.reference_crc);
> >
> > + igt_pipe_crc_start(data->pipe_crc);
> > +
> > i = 0;
> > while (i < iterations || loop_forever) {
> > prepare_planes(data, pipe, &blue, tiling, n_planes,
> > output);
> > --
> > 2.7.4
> >
> > _______________________________________________
> > Intel-gfx mailing list
> > Intel-gfx at lists.freedesktop.org
> > https://lists.freedesktop.org/mailman/listinfo/intel-gfx
>
>
>
> --
> Daniel Vetter
> Software Engineer, Intel Corporation
> +41 (0) 79 365 57 48 - http://blog.ffwll.ch
More information about the Intel-gfx
mailing list