[FFmpeg-devel] [PATCH] libavcodec/vp8dec: fix the multi-thread HWAccel decode error

[Li, Zhong]

> > >     if (width  != s->avctx->width || ((width+15)/16 != s->mb_width ||
> > > (height+15)/16 != s->mb_height) && s->macroblocks_base ||
> > >         height != s->avctx->height) {
> > >        ...
> > >        ff_set_dimensions(...);
> > > }
> > >
> > > The condition for updating dimensions is simple in VP9:
> >
> > Now I didn't test VP9 decoder in this case
> 
> Yes, this patch is for VP8 decoder only. What I mean is that
> ff_set_dimensions()
> is called too when {width, height} are changed without allocated
> macroblocks_base. So dim_reset should be 1 instead of
> (s->macroblocks_base !=
> NULL) in the patch.

Agree dim-reset should be 1, and this comment should be addressed.