diff options
author | Marek Olšák <[email protected]> | 2011-12-13 20:27:48 +0100 |
---|---|---|
committer | Marek Olšák <[email protected]> | 2011-12-13 21:34:18 +0100 |
commit | 35d2764f853fa749e36871d11fc22ced2a851702 (patch) | |
tree | 14b8d0ac2bcfadbcabbc660702f9a6312b967f0a | |
parent | 10e931219f0619c01456dd13fea65b8c21f3b87b (diff) |
r300g: take advantage of KEEP_TILING_FLAGS flush flag
This fixes at least two multi-context-related races.
-rw-r--r-- | src/gallium/drivers/r300/r300_flush.c | 4 | ||||
-rw-r--r-- | src/gallium/drivers/r300/r300_state.c | 8 |
2 files changed, 10 insertions, 2 deletions
diff --git a/src/gallium/drivers/r300/r300_flush.c b/src/gallium/drivers/r300/r300_flush.c index 8b710c1a5ee..9459a95cd73 100644 --- a/src/gallium/drivers/r300/r300_flush.c +++ b/src/gallium/drivers/r300/r300_flush.c @@ -72,6 +72,10 @@ void r300_flush(struct pipe_context *pipe, if (r300->draw && !r300->draw_vbo_locked) r300_draw_flush_vbuf(r300); + if (r300->screen->info.drm_minor >= 12) { + flags |= RADEON_FLUSH_KEEP_TILING_FLAGS; + } + if (rfence) { /* Create a fence, which is a dummy BO. */ *rfence = r300->rws->buffer_create(r300->rws, 1, 1, diff --git a/src/gallium/drivers/r300/r300_state.c b/src/gallium/drivers/r300/r300_state.c index b0aae534a6d..d704057e466 100644 --- a/src/gallium/drivers/r300/r300_state.c +++ b/src/gallium/drivers/r300/r300_state.c @@ -872,8 +872,12 @@ r300_set_framebuffer_state(struct pipe_context* pipe, r300_mark_atom_dirty(r300, &r300->dsa_state); } - /* The tiling flags are dependent on the surface miplevel, unfortunately. */ - r300_fb_set_tiling_flags(r300, state); + if (r300->screen->info.drm_minor < 12) { + /* The tiling flags are dependent on the surface miplevel, unfortunately. + * This workarounds a bad design decision in old kernels which were + * rewriting tile fields in registers. */ + r300_fb_set_tiling_flags(r300, state); + } util_copy_framebuffer_state(r300->fb_state.state, state); |