On Mon, Aug 22, 2016 at 09:44:59AM +0200, Christian König wrote:
> Am 21.08.2016 um 23:11 schrieb Eric Engestrom:
> > Fixes: c59628d11b134fc01638 ("st/va: enable dual instances encode by sync
> > surface")
> > Signed-off-by: Eric Engestrom <e...@engestrom.ch>
>
> Reviewed-by: Christian König <christian.koe...@amd.com>
>
> Do you have commit access?
I do not. Feel free to push this and any of my other patches :)
Note to self: I need to go through my outstanding patches and
ping people to review/push them...
Cheers,
Eric
>
> Regards,
> Christian.
>
> > ---
> > src/gallium/state_trackers/va/surface.c | 4 +++-
> > 1 file changed, 3 insertions(+), 1 deletion(-)
> >
> > diff --git a/src/gallium/state_trackers/va/surface.c
> > b/src/gallium/state_trackers/va/surface.c
> > index 012e48e..3ee1cdd 100644
> > --- a/src/gallium/state_trackers/va/surface.c
> > +++ b/src/gallium/state_trackers/va/surface.c
> > @@ -106,8 +106,10 @@ vlVaSyncSurface(VADriverContextP ctx, VASurfaceID
> > render_target)
> > pipe_mutex_lock(drv->mutex);
> > surf = handle_table_get(drv->htab, render_target);
> > - if (!surf || !surf->buffer)
> > + if (!surf || !surf->buffer) {
> > + pipe_mutex_unlock(drv->mutex);
> > return VA_STATUS_ERROR_INVALID_SURFACE;
> > + }
> > context = handle_table_get(drv->htab, surf->ctx);
> > if (!context) {
>
>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
