KonstantinSeurer
/
mesa
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

swr: call swr_update_derived unconditionally when drawing/clearing 

Currently a sequence like draw/map/draw/map will cause the second map to
not wait for the second draw. This is because the first map will clear
the resource business bit, and the second draw won't reset it since no
state has changed.

swr_update_derived does a tiny bit of extra work, including updating the
SWR_BACKEND_STATE as well as waiting for prending fences. If that's a
problem, we could call swr_update_resource_status directly from
draw/clear handlers.

Fixes clearbuffer-stencil, clearbuffer-depth, clearbuffer-depth-stencil,
and clearbuffer-display-lists.

Signed-off-by: Ilia Mirkin <imirkin@alum.mit.edu>
Reviewed-by: Bruce Cherniak <bruce.cherniak@intel.com>
This commit is contained in:
Ilia Mirkin 2016-11-20 00:04:42 -05:00
parent ee0b6597a9
commit 16d42f2f3d
2 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/gallium/drivers/swr/swr_clear.cpp
View File

@ -39,8 +39,7 @@ swr_clear(struct pipe_context *pipe,
if (!swr_check_render_cond(pipe))
return;
if (ctx->dirty)
swr_update_derived(pipe);
swr_update_derived(pipe);
if (buffers & PIPE_CLEAR_COLOR && fb->nr_cbufs) {
for (unsigned i = 0; i < fb->nr_cbufs; ++i)

3
src/gallium/drivers/swr/swr_draw.cpp
View File

@ -90,8 +90,7 @@ swr_draw_vbo(struct pipe_context *pipe, const struct pipe_draw_info *info)
}
/* Update derived state, pass draw info to update function */
if (ctx->dirty)
swr_update_derived(pipe, info);
swr_update_derived(pipe, info);
swr_update_draw_context(ctx);