[PATCH i-g-t] tests/kms_async_flip: Reduce the clobbering area used for CRC test
Juha-Pekka Heikkila
juhapekka.heikkila at gmail.com
Thu Feb 8 14:08:55 UTC 2024
Hi Vandita,
On 8.2.2024 14.10, Vandita Kulkarni wrote:
> On some platforms wc_mmap write takes longer to paint the fb,
> which we are using to clobber the already scanned buffer for
> CRC. Hence reducing the clobbered area.
>
> Signed-off-by: Vandita Kulkarni <vandita.kulkarni at intel.com>
> ---
> tests/kms_async_flips.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tests/kms_async_flips.c b/tests/kms_async_flips.c
> index a0349fa03..2e5960943 100644
> --- a/tests/kms_async_flips.c
> +++ b/tests/kms_async_flips.c
> @@ -589,7 +589,7 @@ static void test_crc(data_t *data)
>
> while (clock_ms() - start < 2000) {
> /* fill the next fb with the expected color */
> - paint_fb(data, &data->bufs[frame], 1, height, 0xff0000ff);
> + paint_fb(data, &data->bufs[frame], 10, 10, 0xff0000ff);
Here really is needed to be painted full height of the framebuffer and
that width can be kept at 1 as any number higher will just increase the
work.
For async flip part of fb which will become visible is not known here so
the full height of the fb need to be painted on to catch if there was
something going wrong.
>
> data->flip_pending = true;
> ret = drmModePageFlip(data->drm_fd, data->crtc_id, data->bufs[frame].fb_id,
> @@ -600,7 +600,7 @@ static void test_crc(data_t *data)
>
> /* clobber the previous fb which should no longer be scanned out */
> frame = !frame;
> - paint_fb(data, &data->bufs[frame], 1, height, rand());
> + paint_fb(data, &data->bufs[frame], 10, 10, rand());
> }
>
> igt_pipe_crc_stop(data->pipe_crc);
More information about the igt-dev
mailing list