[FFmpeg-devel] [PATCH] vp9: fix show-existing-frames for multi-threading.

[Michael Niedermayer]

On Thu, Apr 30, 2015 at 05:10:35PM -0300, James Almer wrote:
> On 30/04/15 5:03 PM, Ronald S. Bultje wrote:
> > This also fixes intra-only MT failures (it was the same bug), see trac
> > 4526 and 4527.
> > ---
> >  libavcodec/vp9.c | 7 +++++++
> >  1 file changed, 7 insertions(+)
> > 
> > diff --git a/libavcodec/vp9.c b/libavcodec/vp9.c
> > index d5d8030..71ed4e6 100644
> > --- a/libavcodec/vp9.c
> > +++ b/libavcodec/vp9.c
> > @@ -3752,6 +3752,13 @@ static int vp9_decode_frame(AVCodecContext *ctx, void *frame,
> >              return res;
> >          ((AVFrame *)frame)->pkt_pts = pkt->pts;
> >          ((AVFrame *)frame)->pkt_dts = pkt->dts;
> > +        for (i = 0; i < 8; i++) {
> > +            if (s->next_refs[i].f->data[0])
> > +                ff_thread_release_buffer(ctx, &s->next_refs[i]);
> > +            if (s->refs[i].f->data[0] &&
> > +                (res = ff_thread_ref_frame(&s->next_refs[i], &s->refs[i])) < 0)
> > +                return res;
> > +        }
> >          *got_frame = 1;
> >          return pkt->size;
> >      }
> > 
> 
> LGTM. Tested and works.

applied

thanks

[...]
--
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 1
"Used only once"    - "Some unspecified defect prevented a second use"
"In good condition" - "Can be repaird by experienced expert"
"As is" - "You wouldnt want it even if you were payed for it, if you knew ..."
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150430/84baa74c/attachment.asc>