draw/llvm: don't flush in vs_llvm_delete()
authorBrian Paul <brianp@vmware.com>
Fri, 10 Dec 2010 01:37:59 +0000 (18:37 -0700)
committerBrian Paul <brianp@vmware.com>
Fri, 10 Dec 2010 01:41:22 +0000 (18:41 -0700)
commitbecc4bb90c46f754018ac596460e6c6a650d399c
tree5cbf6ee7acf18438d2e93e7cdb0ca6656ccba9cf
parent70ca06445445060c75b55b6c5531973b7b2a1bf0
draw/llvm: don't flush in vs_llvm_delete()

Fixes piglit glx-shader-sharing crash.

When shaders are shared by multiple contexts, the shader's draw context
pointer may point to a previously destroyed context.  Dereferencing the
context pointer will lead to a crash.

In this case, simply removing the flushing code avoids the crash (the
exec and sse shader paths don't flush here either).

There's a deeper issue here, however, that needs examination.  Shaders
should not keep pointers to contexts since contexts might get destroyed
at any time.

NOTE: This is a candidate for the 7.10 branch (after this has been
tested for a while).
src/gallium/auxiliary/draw/draw_vs_llvm.c