[Intel-gfx] [PATCH i-g-t] lib/igt_kms: Only print changed mode objects during atomic commit.

Maarten Lankhorst maarten.lankhorst at linux.intel.com
Fri Oct 20 13:23:57 UTC 2017


When we only print mode objects that have changed properties, we
reduce a lot of the spam. Fortuantely we have a single bitfield
now that gets printed when something is changed. Use that to decrease
the amount of spam.

Signed-off-by: Maarten Lankhorst <maarten.lankhorst at linux.intel.com>
---
 lib/igt_kms.c | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/lib/igt_kms.c b/lib/igt_kms.c
index cb2bc2b8df98..e3f2305667bd 100644
--- a/lib/igt_kms.c
+++ b/lib/igt_kms.c
@@ -2653,10 +2653,12 @@ static int igt_atomic_commit(igt_display_t *display, uint32_t flags, void *user_
 		/*
 		 * Add CRTC Properties to the property set
 		 */
-		igt_atomic_prepare_crtc_commit(pipe_obj, req);
+		if (pipe_obj->changed)
+			igt_atomic_prepare_crtc_commit(pipe_obj, req);
 
 		for_each_plane_on_pipe(display, pipe, plane) {
-			igt_atomic_prepare_plane_commit(plane, pipe_obj, req);
+			if (plane->changed)
+				igt_atomic_prepare_plane_commit(plane, pipe_obj, req);
 		}
 
 	}
@@ -2664,7 +2666,7 @@ static int igt_atomic_commit(igt_display_t *display, uint32_t flags, void *user_
 	for (i = 0; i < display->n_outputs; i++) {
 		output = &display->outputs[i];
 
-		if (!output->config.connector)
+		if (!output->config.connector || !output->changed)
 			continue;
 
 		LOG(display, "%s: preparing atomic, pipe: %s\n",
-- 
2.14.1



More information about the Intel-gfx mailing list