<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Nov 13, 2017 at 7:33 AM, Emil Velikov <span dir="ltr"><<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Jason,<br>
<br>
I'm not too familiar with the spec in question, so just a minor<br>
question/suggestion below.<br>
<span class=""><br>
On 8 November 2017 at 23:20, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
<br>
> +void<br>
> +intelReleaseTexBuffer(__<wbr>DRIcontext *pDRICtx, GLint target,<br>
> +                      __DRIdrawable *dPriv)<br>
> +{<br>
> +   struct brw_context *brw = pDRICtx->driverPrivate;<br>
> +   struct gl_context *ctx = &brw->ctx;<br>
> +   struct gl_texture_object *tex_obj;<br>
> +   struct intel_texture_object *intel_tex;<br>
> +<br>
> +   tex_obj = _mesa_get_current_tex_object(<wbr>ctx, target);<br>
> +   if (!tex_obj)<br>
> +      return;<br>
> +<br>
> +   _mesa_lock_texture(&brw->ctx, tex_obj);<br>
> +<br>
> +   intel_tex = intel_texture_object(tex_obj);<br>
> +   if (!intel_tex->mt)<br>
> +      return;<br>
</span>Here we should be unlocking the texobj, right?<span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div> </div></div></div><div class="gmail_extra">You are correct, sir.  Fixed locally.</div><div class="gmail_extra"><br></div><div class="gmail_extra">--Jason<br></div></div>