Igt-dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [igt-dev] [PATCH i-g-t] tests/i915/kms_big_fb: take out test only flag for try commit in inner loop
@ 2022-04-06 13:00 Juha-Pekka Heikkila
  2022-04-06 13:53 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: Juha-Pekka Heikkila @ 2022-04-06 13:00 UTC (permalink / raw)
  To: igt-dev

This will cause try commit to do modeset which could cause crc buffer
overflow in later commit. I changed small fb not to be cleared with
green color so there will be no unnecessary green flashes, this seem
to also take portion of execution time away.

Signed-off-by: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com>
---
 tests/i915/kms_big_fb.c | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/tests/i915/kms_big_fb.c b/tests/i915/kms_big_fb.c
index 8724d6069..7c9bb5d2a 100644
--- a/tests/i915/kms_big_fb.c
+++ b/tests/i915/kms_big_fb.c
@@ -348,8 +348,7 @@ static bool test_plane(data_t *data)
 		 */
 		if (i == 0 && data->display.is_atomic &&
 		    igt_display_try_commit_atomic(&data->display,
-						  DRM_MODE_ATOMIC_ALLOW_MODESET |
-						  DRM_MODE_ATOMIC_TEST_ONLY,
+						  DRM_MODE_ATOMIC_ALLOW_MODESET,
 						  NULL) != 0) {
 			if (igt_plane_has_prop(plane, IGT_PLANE_ROTATION))
 				igt_plane_set_rotation(plane, IGT_ROTATION_0);
@@ -420,9 +419,8 @@ static bool test_pipe(data_t *data)
 	if (igt_rotation_90_or_270(data->rotation))
 		igt_swap(width, height);
 
-	igt_create_color_fb(data->drm_fd, width, height,
-			    data->format, data->modifier,
-			    0, 1, 0, &data->small_fb);
+	igt_create_fb(data->drm_fd, width, height,
+		      data->format, data->modifier, &data->small_fb);
 
 	igt_output_set_pipe(data->output, data->pipe);
 
-- 
2.28.0

^ permalink raw reply related	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2022-04-06 19:09 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-04-06 13:00 [igt-dev] [PATCH i-g-t] tests/i915/kms_big_fb: take out test only flag for try commit in inner loop Juha-Pekka Heikkila
2022-04-06 13:53 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
2022-04-06 13:53 ` [igt-dev] [PATCH i-g-t] " Ville Syrjälä
2022-04-06 19:09   ` Juha-Pekka Heikkilä
2022-04-06 17:06 ` [igt-dev] ✗ Fi.CI.IGT: failure for " Patchwork

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox