Re: [Mesa-dev] [PATCH 3/3] mesa/main: Free ctx->DrawIndirectBuffer during teardown

2014-01-10 Thread Aaron Watry
On Jan 10, 2014 8:47 PM, Chris Forbes wrote: > > Oops. Thanks for fixing this. > > I don't think you want this in 10.0 though -- ARB_draw_indirect only > landed in 10.1/master. In that case I guess back porting this to 10.0 would probably not work so well.  Thanks for the review. --Aaron >

Re: [Mesa-dev] [PATCH 3/3] mesa/main: Free ctx->DrawIndirectBuffer during teardown

2014-01-10 Thread Chris Forbes
Oops. Thanks for fixing this. I don't think you want this in 10.0 though -- ARB_draw_indirect only landed in 10.1/master. Reviewed-by: Chris Forbes ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/m

[Mesa-dev] [PATCH 3/3] mesa/main: Free ctx->DrawIndirectBuffer during teardown

2014-01-10 Thread Aaron Watry
ctx->DrawIndirectBuffer wasn't being free'd in _mesa_free_buffer_objects With this patch, "valgrind --leak-check=full glxgears" on evergreen (CEDAR) now shows: LEAK SUMMARY: definitely lost: 0 bytes in 0 blocks indirectly lost: 0 bytes in 0 blocks possibly lost: 0 bytes in 0 blocks