radeonsi: don't check PIPE_BARRIER_MAPPED_BUFFER
authorMarek Olšák <marek.olsak@amd.com>
Fri, 30 Sep 2016 22:46:39 +0000 (00:46 +0200)
committerMarek Olšák <marek.olsak@amd.com>
Tue, 4 Oct 2016 14:11:51 +0000 (16:11 +0200)
Caches are always flushed at IB boundary.

Reviewed-by: Nicolai Hähnle <nicolai.haehnle@amd.com>
Reviewed-by: Edward O'Callaghan <funfunctor@folklore1984.net>
src/gallium/drivers/radeonsi/si_state.c

index 443dc37078b28dcdea828e0ca294d7a24ff78222..04b57dc5e030342f088f04737f20db4840b8df0b 100644 (file)
@@ -3414,14 +3414,13 @@ static void si_memory_barrier(struct pipe_context *ctx, unsigned flags)
        if (flags & PIPE_BARRIER_FRAMEBUFFER)
                sctx->b.flags |= SI_CONTEXT_FLUSH_AND_INV_FRAMEBUFFER;
 
-       if (flags & (PIPE_BARRIER_MAPPED_BUFFER |
-                    PIPE_BARRIER_FRAMEBUFFER |
+       if (flags & (PIPE_BARRIER_FRAMEBUFFER |
                     PIPE_BARRIER_INDIRECT_BUFFER)) {
                /* Not sure if INV_GLOBAL_L2 is the best thing here.
                 *
                 * We need to make sure that TC L1 & L2 are written back to
-                * memory, because neither CPU accesses nor CB fetches consider
-                * TC, but there's no need to invalidate any TC cache lines. */
+                * memory, because CB fetches don't consider TC, but there's
+                * no need to invalidate any TC cache lines. */
                sctx->b.flags |= SI_CONTEXT_INV_GLOBAL_L2;
        }
 }