> On Nov 14, 2019, at 1:25 AM, Ronald S. Bultje <[email protected]> wrote: > > Hi, > > On Wed, Nov 13, 2019 at 2:00 AM <[email protected] > <mailto:[email protected]>> wrote: > >> From: Zhao Zhili <[email protected]> >> >> 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 > [email protected] <mailto:[email protected]> > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > <https://ffmpeg.org/mailman/listinfo/ffmpeg-devel> > > To unsubscribe, visit link above, or email > [email protected] <mailto:[email protected]> with > subject "unsubscribe". _______________________________________________ ffmpeg-devel mailing list [email protected] https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email [email protected] with subject "unsubscribe".
