From: "Modem, Bhanuprakash" <bhanuprakash.modem@intel.com> To: "Srinivas, Vidya" <vidya.srinivas@intel.com>, "intel-gfx@lists.freedesktop.org" <intel-gfx@lists.freedesktop.org>, "igt-dev@lists.freedesktop.org" <igt-dev@lists.freedesktop.org> Cc: "markyacoub@chromium.org" <markyacoub@chromium.org>, "Lin, Charlton" <charlton.lin@intel.com> Subject: Re: [Intel-gfx] [PATCH i-g-t] [RFC] tests/kms_big_fb: Wait for vblank before collecting CRC Date: Tue, 8 Jun 2021 07:01:58 +0000 [thread overview] Message-ID: <CO6PR11MB55693230CBD2262DDCF5A0C18D379@CO6PR11MB5569.namprd11.prod.outlook.com> (raw) In-Reply-To: <1622176025-12499-1-git-send-email-vidya.srinivas@intel.com> > From: Intel-gfx <intel-gfx-bounces@lists.freedesktop.org> On Behalf Of Vidya > Srinivas > Sent: Friday, May 28, 2021 9:57 AM > To: intel-gfx@lists.freedesktop.org; igt-dev@lists.freedesktop.org > Cc: markyacoub@chromium.org; Lin, Charlton <charlton.lin@intel.com> > Subject: [Intel-gfx] [PATCH i-g-t] [RFC] tests/kms_big_fb: Wait for vblank > before collecting CRC > > Without wait for vblank, CRC mismatch is seen > between big and small CRC on few Gen11 systems. > > Signed-off-by: Vidya Srinivas <vidya.srinivas@intel.com> > --- > tests/kms_big_fb.c | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/tests/kms_big_fb.c b/tests/kms_big_fb.c > index b35727a09bd0..f90363c3beb2 100644 > --- a/tests/kms_big_fb.c > +++ b/tests/kms_big_fb.c > @@ -254,6 +254,7 @@ static void unset_lut(data_t *data) > static bool test_plane(data_t *data) > { > igt_plane_t *plane = data->plane; > + igt_display_t *display = &data->display; For code readability purpose, I think we need to update to use this variable wherever we are using "&data->display" in this function. s/"&data->display"/"display"/ With above change, this patch LGTM Reviewed-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com> > struct igt_fb *small_fb = &data->small_fb; > struct igt_fb *big_fb = &data->big_fb; > int w = data->big_fb_width - small_fb->width; > @@ -337,16 +338,17 @@ static bool test_plane(data_t *data) > igt_display_commit2(&data->display, data->display.is_atomic ? > COMMIT_ATOMIC : COMMIT_UNIVERSAL); > > - > + igt_wait_for_vblank(data->drm_fd, display->pipes[data->pipe].crtc_offset); > igt_pipe_crc_collect_crc(data->pipe_crc, &small_crc); > > igt_plane_set_fb(plane, big_fb); > igt_fb_set_position(big_fb, plane, x, y); > igt_fb_set_size(big_fb, plane, small_fb->width, small_fb->height); > + > igt_plane_set_size(plane, data->width, data->height); > igt_display_commit2(&data->display, data->display.is_atomic ? > COMMIT_ATOMIC : COMMIT_UNIVERSAL); > - > + igt_wait_for_vblank(data->drm_fd, display->pipes[data->pipe].crtc_offset); > igt_pipe_crc_collect_crc(data->pipe_crc, &big_crc); > > igt_plane_set_fb(plane, NULL); > -- > 2.7.4 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/intel-gfx _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
WARNING: multiple messages have this Message-ID (diff)
From: "Modem, Bhanuprakash" <bhanuprakash.modem@intel.com> To: "Srinivas, Vidya" <vidya.srinivas@intel.com>, "intel-gfx@lists.freedesktop.org" <intel-gfx@lists.freedesktop.org>, "igt-dev@lists.freedesktop.org" <igt-dev@lists.freedesktop.org> Cc: "Lin, Charlton" <charlton.lin@intel.com> Subject: Re: [igt-dev] [Intel-gfx] [PATCH i-g-t] [RFC] tests/kms_big_fb: Wait for vblank before collecting CRC Date: Tue, 8 Jun 2021 07:01:58 +0000 [thread overview] Message-ID: <CO6PR11MB55693230CBD2262DDCF5A0C18D379@CO6PR11MB5569.namprd11.prod.outlook.com> (raw) In-Reply-To: <1622176025-12499-1-git-send-email-vidya.srinivas@intel.com> > From: Intel-gfx <intel-gfx-bounces@lists.freedesktop.org> On Behalf Of Vidya > Srinivas > Sent: Friday, May 28, 2021 9:57 AM > To: intel-gfx@lists.freedesktop.org; igt-dev@lists.freedesktop.org > Cc: markyacoub@chromium.org; Lin, Charlton <charlton.lin@intel.com> > Subject: [Intel-gfx] [PATCH i-g-t] [RFC] tests/kms_big_fb: Wait for vblank > before collecting CRC > > Without wait for vblank, CRC mismatch is seen > between big and small CRC on few Gen11 systems. > > Signed-off-by: Vidya Srinivas <vidya.srinivas@intel.com> > --- > tests/kms_big_fb.c | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/tests/kms_big_fb.c b/tests/kms_big_fb.c > index b35727a09bd0..f90363c3beb2 100644 > --- a/tests/kms_big_fb.c > +++ b/tests/kms_big_fb.c > @@ -254,6 +254,7 @@ static void unset_lut(data_t *data) > static bool test_plane(data_t *data) > { > igt_plane_t *plane = data->plane; > + igt_display_t *display = &data->display; For code readability purpose, I think we need to update to use this variable wherever we are using "&data->display" in this function. s/"&data->display"/"display"/ With above change, this patch LGTM Reviewed-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com> > struct igt_fb *small_fb = &data->small_fb; > struct igt_fb *big_fb = &data->big_fb; > int w = data->big_fb_width - small_fb->width; > @@ -337,16 +338,17 @@ static bool test_plane(data_t *data) > igt_display_commit2(&data->display, data->display.is_atomic ? > COMMIT_ATOMIC : COMMIT_UNIVERSAL); > > - > + igt_wait_for_vblank(data->drm_fd, display->pipes[data->pipe].crtc_offset); > igt_pipe_crc_collect_crc(data->pipe_crc, &small_crc); > > igt_plane_set_fb(plane, big_fb); > igt_fb_set_position(big_fb, plane, x, y); > igt_fb_set_size(big_fb, plane, small_fb->width, small_fb->height); > + > igt_plane_set_size(plane, data->width, data->height); > igt_display_commit2(&data->display, data->display.is_atomic ? > COMMIT_ATOMIC : COMMIT_UNIVERSAL); > - > + igt_wait_for_vblank(data->drm_fd, display->pipes[data->pipe].crtc_offset); > igt_pipe_crc_collect_crc(data->pipe_crc, &big_crc); > > igt_plane_set_fb(plane, NULL); > -- > 2.7.4 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/intel-gfx _______________________________________________ igt-dev mailing list igt-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/igt-dev
next prev parent reply other threads:[~2021-06-08 7:02 UTC|newest] Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-05-28 4:27 [Intel-gfx] [PATCH i-g-t] [RFC] tests/kms_big_fb: Wait for vblank before collecting CRC Vidya Srinivas 2021-05-28 4:27 ` [igt-dev] " Vidya Srinivas 2021-05-28 13:25 ` [igt-dev] ✓ Fi.CI.BAT: success for tests/kms_big_fb: Wait for vblank before collecting CRC (rev5) Patchwork 2021-05-28 22:58 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork 2021-06-04 18:50 ` [Intel-gfx] [PATCH i-g-t] [RFC] tests/kms_big_fb: Wait for vblank before collecting CRC Mark Yacoub 2021-06-04 18:50 ` [igt-dev] " Mark Yacoub 2021-06-05 5:46 ` [Intel-gfx] " Srinivas, Vidya 2021-06-08 7:01 ` Modem, Bhanuprakash [this message] 2021-06-08 7:01 ` [igt-dev] " Modem, Bhanuprakash 2021-06-08 7:34 ` [Intel-gfx] [igt-dev] " Juha-Pekka Heikkila 2021-06-08 7:48 ` Srinivas, Vidya 2021-06-08 9:19 ` Srinivas, Vidya 2021-06-08 9:19 ` [igt-dev] [Intel-gfx] " Srinivas, Vidya 2021-06-08 11:50 ` [Intel-gfx] [igt-dev] " Juha-Pekka Heikkila 2021-06-08 11:50 ` [igt-dev] [Intel-gfx] " Juha-Pekka Heikkila 2021-06-08 11:54 ` [Intel-gfx] [igt-dev] " Srinivas, Vidya 2021-06-10 8:38 ` Srinivas, Vidya 2021-06-10 8:38 ` [igt-dev] [Intel-gfx] " Srinivas, Vidya 2021-06-10 8:51 ` [Intel-gfx] [igt-dev] " Petri Latvala 2021-06-10 8:51 ` [igt-dev] [Intel-gfx] " Petri Latvala 2021-06-10 8:52 ` [Intel-gfx] [igt-dev] " Srinivas, Vidya 2021-06-08 9:05 ` [igt-dev] [PATCH i-g-t] tests/kms_big_fb: Use igt_pipe_crc_get_current instead of igt_pipe_crc_collect_crc Vidya Srinivas 2021-06-10 8:52 ` Juha-Pekka Heikkila 2021-06-10 8:53 ` Srinivas, Vidya 2021-06-15 8:28 ` Srinivas, Vidya 2021-06-15 8:50 ` Petri Latvala 2021-06-15 8:50 ` Srinivas, Vidya 2021-06-08 10:54 ` [igt-dev] ✓ Fi.CI.BAT: success for tests/kms_big_fb: Wait for vblank before collecting CRC (rev6) Patchwork 2021-06-08 15:28 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork 2021-06-10 5:01 ` [igt-dev] ✓ Fi.CI.BAT: success for tests/kms_big_fb: Wait for vblank before collecting CRC (rev7) Patchwork 2021-06-10 6:27 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork -- strict thread matches above, loose matches on Subject: below -- 2021-05-27 15:40 [Intel-gfx] [PATCH i-g-t] [RFC] tests/kms_big_fb: Wait for vblank before collecting CRC Vidya Srinivas 2021-05-27 15:09 [Intel-gfx] [PATCH] " Vidya Srinivas 2021-05-27 15:23 ` [Intel-gfx] [PATCH i-g-t] " Vidya Srinivas
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=CO6PR11MB55693230CBD2262DDCF5A0C18D379@CO6PR11MB5569.namprd11.prod.outlook.com \ --to=bhanuprakash.modem@intel.com \ --cc=charlton.lin@intel.com \ --cc=igt-dev@lists.freedesktop.org \ --cc=intel-gfx@lists.freedesktop.org \ --cc=markyacoub@chromium.org \ --cc=vidya.srinivas@intel.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.