i965/gen6: Fix crash with VS+TF after rendering with GS
authorIago Toral Quiroga <itoral@igalia.com>
Wed, 7 Jan 2015 09:08:57 +0000 (10:08 +0100)
committerIago Toral Quiroga <itoral@igalia.com>
Fri, 16 Jan 2015 13:16:59 +0000 (14:16 +0100)
commit6367ca8b41ce99de21a27953713413d678e5d30d
treeef8d146f276ffb32ffe2c8399e48840a1bfb5390
parentbc6e57e019bd2399a70acabcad21217aadf2944c
i965/gen6: Fix crash with VS+TF after rendering with GS

Rendering with a GS and then using transform feedback with a program that does
not have a GS can crash in gen6. The reason for this is that
brw_begin_transform_feedback checks brw->geometry_program to decide if there
is a GS program, but this is not correct: brw->geometry_program is updated when
issuing drawing commands, so after rendering with a GS it will be non-NULL
until we draw again with a program that does not have a GS. If the next
program uses TF, we will call glBegintransformFeedback before issuing
the drawing command and hence brw->geometry_program will be non-NULL if
the previous rendering used a GS. The right thing to do here is to check
ctx->_Shader->CurrentProgram[MESA_SHADER_GEOMETRY] instead. This is what the
gen7 code path does too.

Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=87694
Reviewed-by: Tapani Pälli <tapani.palli@intel.com>
src/mesa/drivers/dri/i965/gen6_sol.c