diff options
author | Chad Versace <[email protected]> | 2012-01-25 19:38:10 -0800 |
---|---|---|
committer | Chad Versace <[email protected]> | 2012-01-27 13:34:26 -0800 |
commit | 1c0f1dd42a50464eeb81de4aad8eecf24b3d6c89 (patch) | |
tree | 356acdc359c537d38265763a84552f712c45bc53 /src/mesa/swrast/s_fragprog.h | |
parent | 5665b5cc31da70e833f80e7a17bfa034d2f7ba44 (diff) |
swrast: Fix fixed-function fragment processing
On i965, _mesa_ir_link_shader is never called. As a consequence, the
current fragment program (ctx->FragmentProgram->_Current) exists but is
invalid because it has no instructions. Yet swrast continued to attempt to
use the empty program.
To avoid using the empty program, this patch 1) defines a new function,
_swrast_use_fragment_program, which checks if the current fragment program
exists and differs from the fixed function fragment program, and, when
appropriate, 2) replaces checks of the form
if (ctx->FragmentProgram->_Current == NULL)
with
if (_swrast_use_fragment_program(ctx))
Fixes the following oglconform regressions on i965/gen6:
api-fogcoord(basic.allCases.log)
api-mtexcoord(basic.allCases.log)
api-seccolor(basic.allCases.log)
api-texcoord(basic.allCases.log)
blend-separate(basic.allCases)
colorsum(basic.allCases.log)
The tests were ran with the GLXFBConfig:
visual x bf lv rg d st colorbuffer sr ax dp st accumbuffer ms cav
id dep cl sp sz l ci b ro r g b a F gb bf th cl r g b a ns b eat
----------------------------------------------------------------------------
0x021 24 tc 0 32 0 r y . 8 8 8 8 . . 0 24 8 0 0 0 0 0 0 None
(Note: I originally believed that the hunk in
_swrast_update_fragment_program was unnecessary. But it is required to fix
blend-separate.)
Note: This is a candidate for the 8.0 branch.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=43327
Reveiwed-by: Eric Anholt <[email protected]>
Reviewed-by: Ian Romanick <[email protected]>
Signed-off-by: Chad Versace <[email protected]>
Diffstat (limited to 'src/mesa/swrast/s_fragprog.h')
-rw-r--r-- | src/mesa/swrast/s_fragprog.h | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/mesa/swrast/s_fragprog.h b/src/mesa/swrast/s_fragprog.h index 62a68361769..ac1f5ff4cf4 100644 --- a/src/mesa/swrast/s_fragprog.h +++ b/src/mesa/swrast/s_fragprog.h @@ -31,6 +31,8 @@ struct gl_context; +GLboolean +_swrast_use_fragment_program(struct gl_context *ctx); extern void _swrast_exec_fragment_program(struct gl_context *ctx, SWspan *span); |