> On Nov 14, 2019, at 1:25 AM, Ronald S. Bultje <rsbul...@gmail.com> wrote: > > Hi, > > On Wed, Nov 13, 2019 at 2:00 AM <quinkbl...@foxmail.com > <mailto:quinkbl...@foxmail.com>> wrote: > >> From: Zhao Zhili <zhiliz...@tencent.com> >> >> vp7 decoder doesn't set update_thread_context field >> --- >> libavcodec/vp8.c | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) >> >> diff --git a/libavcodec/vp8.c b/libavcodec/vp8.c >> index eb51d1f3c9..b4deb3ed67 100644 >> --- a/libavcodec/vp8.c >> +++ b/libavcodec/vp8.c >> @@ -2715,7 +2715,8 @@ int vp78_decode_frame(AVCodecContext *avctx, void >> *data, int *got_frame, >> >> s->next_framep[VP56_FRAME_CURRENT] = curframe; >> >> - ff_thread_finish_setup(avctx); >> + if (avctx->codec->update_thread_context) >> + ff_thread_finish_setup(avctx); >> > > OK, I guess. What's the actual problem solved by this patch (e.g. some > commandline invocation with particular settings)? Does it cause some > assertion with frame-multithreaded VP7 decoding? Maybe add that to the > commit message so we know how to test this change.
Sorry, the commit message is incorrect, I send another patch. It’s actually webp decoder has “multiple ff_thread_finish_setup" issue. The issue can be reproduced with webp samples in fate-suite. Since the vp7 decoder doesn’t have multi-thread capabilities, it is not affected. > > Ronald > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org <mailto:ffmpeg-devel@ffmpeg.org> > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > <https://ffmpeg.org/mailman/listinfo/ffmpeg-devel> > > To unsubscribe, visit link above, or email > ffmpeg-devel-requ...@ffmpeg.org <mailto:ffmpeg-devel-requ...@ffmpeg.org> with > subject "unsubscribe". _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
