diff --git a/.pick_status.json b/.pick_status.json index 2d4d95f0873..13473537726 100644 --- a/.pick_status.json +++ b/.pick_status.json @@ -1754,7 +1754,7 @@ "description": "aux/trace: move fence_server calls outside the locked area", "nominated": true, "nomination_type": 1, - "resolution": 0, + "resolution": 1, "main_sha": null, "because_sha": null, "notes": null diff --git a/src/gallium/auxiliary/driver_trace/tr_context.c b/src/gallium/auxiliary/driver_trace/tr_context.c index 49b5c4c8a02..d3386c4f8eb 100644 --- a/src/gallium/auxiliary/driver_trace/tr_context.c +++ b/src/gallium/auxiliary/driver_trace/tr_context.c @@ -1659,13 +1659,13 @@ trace_context_fence_server_sync(struct pipe_context *_pipe, struct trace_context *tr_ctx = trace_context(_pipe); struct pipe_context *pipe = tr_ctx->pipe; + pipe->fence_server_sync(pipe, fence); + trace_dump_call_begin("pipe_context", "fence_server_sync"); trace_dump_arg(ptr, pipe); trace_dump_arg(ptr, fence); - pipe->fence_server_sync(pipe, fence); - trace_dump_call_end(); } @@ -1677,13 +1677,13 @@ trace_context_fence_server_signal(struct pipe_context *_pipe, struct trace_context *tr_ctx = trace_context(_pipe); struct pipe_context *pipe = tr_ctx->pipe; + pipe->fence_server_signal(pipe, fence); + trace_dump_call_begin("pipe_context", "fence_server_signal"); trace_dump_arg(ptr, pipe); trace_dump_arg(ptr, fence); - pipe->fence_server_signal(pipe, fence); - trace_dump_call_end(); } diff --git a/src/gallium/auxiliary/driver_trace/tr_screen.c b/src/gallium/auxiliary/driver_trace/tr_screen.c index e643e054340..9bdc244e29d 100644 --- a/src/gallium/auxiliary/driver_trace/tr_screen.c +++ b/src/gallium/auxiliary/driver_trace/tr_screen.c @@ -949,15 +949,12 @@ trace_screen_fence_get_fd(struct pipe_screen *_screen, { struct trace_screen *tr_scr = trace_screen(_screen); struct pipe_screen *screen = tr_scr->screen; - int result; + int result = screen->fence_get_fd(screen, fence); trace_dump_call_begin("pipe_screen", "fence_get_fd"); trace_dump_arg(ptr, screen); trace_dump_arg(ptr, fence); - - result = screen->fence_get_fd(screen, fence); - trace_dump_ret(int, result); trace_dump_call_end();