[Intel-gfx] [PATCH i-g-t] tests/kms_plane_multiple: Fix reference CRC

[Lofstedt, Marta]

FYI, bug 101907


> -----Original Message-----
> From: Intel-gfx [mailto:intel-gfx-bounces at lists.freedesktop.org] On Behalf
> Of Daniel Vetter
> Sent: Wednesday, August 2, 2017 2:37 PM
> 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 Mon, Jul 31, 2017 at 09:04:50AM +0000, Kahola, Mika wrote:
> > > -----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.
> 
> Yeah, but there's no n_words in kms_plane_multiple.c. How exactly does
> switching from igt_pipe_crc_get_crcs to igt_pipe_crc_collect_crc fix this?
> And if it does, why do we expose a broken function to testcases?
> 
> aka pls explain more, I don't get what's going on here.
> -Daniel
> 
> >
> > 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
> 
> --
> Daniel Vetter
> Software Engineer, Intel Corporation
> http://blog.ffwll.ch
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx