From 1401fc055c5496933318e87503144392ad3b6811 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Marek=20Ol=C5=A1=C3=A1k?= Date: Mon, 22 Jun 2020 23:54:59 -0400 Subject: [PATCH] radeonsi: don't flush in fence_server_sync This reverts commit 50b06cbc10dbca1dfee89b529ba9b564cc4ea6f6 and fixes an Android performance regression. Fixes: 50b06cbc10dbca1dfee89b529ba9b564cc4ea6f6 "radeonsi: fix fence_server_sync() holding up extra work v2" Acked-by: Pierre-Eric Pelloux-Prayer Part-of: --- src/gallium/drivers/radeonsi/si_fence.c | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_fence.c b/src/gallium/drivers/radeonsi/si_fence.c index f6505ec..0ab28c4 100644 --- a/src/gallium/drivers/radeonsi/si_fence.c +++ b/src/gallium/drivers/radeonsi/si_fence.c @@ -589,13 +589,16 @@ static void si_fence_server_sync(struct pipe_context *ctx, struct pipe_fence_han if (sfence->gfx_unflushed.ctx && sfence->gfx_unflushed.ctx == sctx) return; - /* All unflushed commands will not start execution before - * this fence dependency is signalled. + /* All unflushed commands will not start execution before this fence + * dependency is signalled. That's fine. Flushing is very expensive + * if we get fence_server_sync after every draw call. (which happens + * with Android/SurfaceFlinger) * - * Therefore we must flush before inserting the dependency + * In a nutshell, when CPU overhead is greater than GPU overhead, + * or when the time it takes to execute an IB on the GPU is less than + * the time it takes to create and submit that IB, flushing decreases + * performance. Therefore, DO NOT FLUSH. */ - si_flush_from_st(ctx, NULL, PIPE_FLUSH_ASYNC); - if (sfence->sdma) si_add_fence_dependency(sctx, sfence->sdma); if (sfence->gfx) -- 2.7.4