We should fix st/mesa and move the code to cso_destroy_context, where it belongs.
Marek On Mon, May 18, 2015 at 3:49 AM, Ilia Mirkin <imir...@alum.mit.edu> wrote: > Hm, oops. But I guess my tess patchset will take care of this. I guess > it's a side-effect of adding to the gallium enums. > > Marek, you ended up pushing those gallium patches out... I was going > to wait until the branchpoint was done. Should we just fix up the > drivers as necessary, or fix up st_destroy_context to not call it for > tess shaders when !tess? [But then we have to teach st/mesa about > tess] > > On Sun, May 17, 2015 at 8:56 PM, Tobias Klausmann > <tobias.johannes.klausm...@mni.thm.de> wrote: >> Backtrace: >> ... >> 0x00007ffff57fc392 in __assert_fail () from /lib64/libc.so.6 >> 0x00007ffff0cf5bec in nvc0_shader_stage (pipe=<optimized out>) at >> ./nvc0/nvc0_context.h:204 >> nvc0_set_constant_buffer (pipe=0x631080, shader=<optimized out>, >> index=<optimized out>, cb=0x0) >> at nvc0/nvc0_state.c:771 >> 0x00007ffff0a2cf63 in st_destroy_context (st=0x68a9f0) at >> state_tracker/st_context.c:382 >> ... >> >> Signed-off-by: Tobias Klausmann <tobias.johannes.klausm...@mni.thm.de> >> --- >> src/gallium/drivers/nouveau/nvc0/nvc0_context.h | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_context.h >> b/src/gallium/drivers/nouveau/nvc0/nvc0_context.h >> index 09d08e4..f910541 100644 >> --- a/src/gallium/drivers/nouveau/nvc0/nvc0_context.h >> +++ b/src/gallium/drivers/nouveau/nvc0/nvc0_context.h >> @@ -195,8 +195,8 @@ nvc0_shader_stage(unsigned pipe) >> { >> switch (pipe) { >> case PIPE_SHADER_VERTEX: return 0; >> -/* case PIPE_SHADER_TESSELLATION_CONTROL: return 1; */ >> -/* case PIPE_SHADER_TESSELLATION_EVALUATION: return 2; */ >> + case PIPE_SHADER_TESS_CTRL: return 1; >> + case PIPE_SHADER_TESS_EVAL: return 2; >> case PIPE_SHADER_GEOMETRY: return 3; >> case PIPE_SHADER_FRAGMENT: return 4; >> case PIPE_SHADER_COMPUTE: return 5; >> -- >> 2.4.1 >> >> _______________________________________________ >> Nouveau mailing list >> nouv...@lists.freedesktop.org >> http://lists.freedesktop.org/mailman/listinfo/nouveau _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev