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

Reply via email to