From dcb1e8fef8ae60877a696a5bca337eba5475085d Mon Sep 17 00:00:00 2001 From: Pierre-Eric Pelloux-Prayer Date: Fri, 10 Apr 2020 14:46:17 +0200 Subject: [PATCH] radeonsi: use thread_context::bytes_mapped_limit MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Limit the amount of "in-flight" mapping to 1/4 of the total RAM. Closes: https://gitlab.freedesktop.org/mesa/mesa/-/issues/2735 Reviewed-by: Marek Olšák Part-of: --- src/gallium/drivers/radeonsi/si_pipe.c | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/src/gallium/drivers/radeonsi/si_pipe.c b/src/gallium/drivers/radeonsi/si_pipe.c index 816015d1f82..e3ba1c2fb93 100644 --- a/src/gallium/drivers/radeonsi/si_pipe.c +++ b/src/gallium/drivers/radeonsi/si_pipe.c @@ -686,6 +686,7 @@ static struct pipe_context *si_pipe_create_context(struct pipe_screen *screen, v { struct si_screen *sscreen = (struct si_screen *)screen; struct pipe_context *ctx; + uint64_t total_ram; if (sscreen->debug_flags & DBG(CHECK_VM)) flags |= PIPE_CONTEXT_DEBUG; @@ -706,9 +707,16 @@ static struct pipe_context *si_pipe_create_context(struct pipe_screen *screen, v /* Use asynchronous flushes only on amdgpu, since the radeon * implementation for fence_server_sync is incomplete. */ - return threaded_context_create(ctx, &sscreen->pool_transfers, si_replace_buffer_storage, - sscreen->info.is_amdgpu ? si_create_fence : NULL, - &((struct si_context *)ctx)->tc); + struct pipe_context * tc = threaded_context_create( + ctx, &sscreen->pool_transfers, si_replace_buffer_storage, + sscreen->info.is_amdgpu ? si_create_fence : NULL, + &((struct si_context *)ctx)->tc); + + if (os_get_total_physical_memory(&total_ram)) { + ((struct threaded_context *) tc)->bytes_mapped_limit = total_ram / 4; + } + + return tc; } /* -- 2.30.2