From 8645afce4c19ca78fe7087b325467293c3c04bea Mon Sep 17 00:00:00 2001 From: Boris Brezillon Date: Fri, 2 Aug 2019 19:18:43 +0200 Subject: [PATCH] panfrost: Bail out early when new and current FB states are equal If the current FB matches the new one there's nothing to be done in panfrost_set_framebuffer_state(). By bailing out early in that case we avoid emitting new FB descriptors (the old ones are still valid). Signed-off-by: Boris Brezillon Reviewed-by: Alyssa Rosenzweig --- src/gallium/drivers/panfrost/pan_context.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/gallium/drivers/panfrost/pan_context.c b/src/gallium/drivers/panfrost/pan_context.c index a88e6a4607d..65e6824a9b0 100644 --- a/src/gallium/drivers/panfrost/pan_context.c +++ b/src/gallium/drivers/panfrost/pan_context.c @@ -2365,6 +2365,10 @@ panfrost_set_framebuffer_state(struct pipe_context *pctx, bool is_scanout = panfrost_is_scanout(ctx); bool has_draws = job->last_job.gpu; + /* Bail out early when the current and new states are the same. */ + if (util_framebuffer_state_equal(&ctx->pipe_framebuffer, fb)) + return; + if (!ctx->wallpaper_batch && (!is_scanout || has_draws)) { panfrost_flush(pctx, NULL, PIPE_FLUSH_END_OF_FRAME); } -- 2.30.2