Hi Jason, I'm not too familiar with the spec in question, so just a minor question/suggestion below.
On 8 November 2017 at 23:20, Jason Ekstrand <ja...@jlekstrand.net> wrote: > +void > +intelReleaseTexBuffer(__DRIcontext *pDRICtx, GLint target, > + __DRIdrawable *dPriv) > +{ > + struct brw_context *brw = pDRICtx->driverPrivate; > + struct gl_context *ctx = &brw->ctx; > + struct gl_texture_object *tex_obj; > + struct intel_texture_object *intel_tex; > + > + tex_obj = _mesa_get_current_tex_object(ctx, target); > + if (!tex_obj) > + return; > + > + _mesa_lock_texture(&brw->ctx, tex_obj); > + > + intel_tex = intel_texture_object(tex_obj); > + if (!intel_tex->mt) > + return; Here we should be unlocking the texobj, right? Thanks Emil _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev