Revert "radeon/r300: no need to flush the cmdbuf when changing scissors state in...
authorDave Airlie <airlied@redhat.com>
Wed, 2 Dec 2009 01:01:19 +0000 (11:01 +1000)
committerDave Airlie <airlied@redhat.com>
Wed, 2 Dec 2009 01:03:22 +0000 (11:03 +1000)
This reverts commit 286bf89e5a1fc931dbf523ded861b809859485e2.

This doesn't appear to be correct, regression so revert it.
http://bugs.freedesktop.org/show_bug.cgi?id=25193

src/mesa/drivers/dri/r300/r300_state.c
src/mesa/drivers/dri/radeon/radeon_common.c

index 1fd32d4..ac20c08 100644 (file)
@@ -1741,8 +1741,7 @@ static void r300Enable(GLcontext * ctx, GLenum cap, GLboolean state)
                r300SetPolygonOffsetState(ctx, state);
                break;
        case GL_SCISSOR_TEST:
-               if (!rmesa->radeon.radeonScreen->kernel_mm)
-                       radeon_firevertices(&rmesa->radeon);
+               radeon_firevertices(&rmesa->radeon);
                rmesa->radeon.state.scissor.enabled = state;
                radeonUpdateScissor( ctx );
                break;
index 3b4366a..184287a 100644 (file)
@@ -257,9 +257,7 @@ void radeonScissor(GLcontext* ctx, GLint x, GLint y, GLsizei w, GLsizei h)
        radeonContextPtr radeon = RADEON_CONTEXT(ctx);
        if (ctx->Scissor.Enabled) {
                /* We don't pipeline cliprect changes */
-               if (!radeon->radeonScreen->kernel_mm) {
-                       radeon_firevertices(radeon);
-               }
+               radeon_firevertices(radeon);
                radeonUpdateScissor(ctx);
        }
 }