Re: [FFmpeg-devel] [PATCH] lavc/vp9: Fix regression introduced in 0ba05857
On Di, 2024-04-23 at 14:14 +0200, Andreas Rheinhardt wrote: > Xiang, Haihao: > > From: Haihao Xiang > > > > It is possible that ff_progress_frame_await() is called but > > ff_progress_frame_report() isn't called when a hardware acceleration > > method is used, so a thread for vp9 decoding might get stuck. > > > > Signed-off-by: Haihao Xiang > > --- > > libavcodec/vp9.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/libavcodec/vp9.c b/libavcodec/vp9.c > > index 3adfb98f2d..6e2d18bf95 100644 > > --- a/libavcodec/vp9.c > > +++ b/libavcodec/vp9.c > > @@ -1735,9 +1735,9 @@ static int vp9_decode_frame(AVCodecContext *avctx, > > AVFrame *frame, > > if (ret < 0) > > goto fail; > > } > > - ff_progress_frame_report(>s.frames[CUR_FRAME].tf, INT_MAX); > > > > finish: > > + ff_progress_frame_report(>s.frames[CUR_FRAME].tf, INT_MAX); > > // ref frame setup > > for (int i = 0; i < 8; i++) > > ff_progress_frame_replace(>s.refs[i], >next_refs[i]); > > LGTM. Sorry for the breakage. > Thanks for reviewing the patch, I pushed the patch. BRs Haihao ___ 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".
Re: [FFmpeg-devel] [PATCH] lavc/vp9: Fix regression introduced in 0ba05857
Xiang, Haihao: > From: Haihao Xiang > > It is possible that ff_progress_frame_await() is called but > ff_progress_frame_report() isn't called when a hardware acceleration > method is used, so a thread for vp9 decoding might get stuck. > > Signed-off-by: Haihao Xiang > --- > libavcodec/vp9.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/vp9.c b/libavcodec/vp9.c > index 3adfb98f2d..6e2d18bf95 100644 > --- a/libavcodec/vp9.c > +++ b/libavcodec/vp9.c > @@ -1735,9 +1735,9 @@ static int vp9_decode_frame(AVCodecContext *avctx, > AVFrame *frame, > if (ret < 0) > goto fail; > } > -ff_progress_frame_report(>s.frames[CUR_FRAME].tf, INT_MAX); > > finish: > +ff_progress_frame_report(>s.frames[CUR_FRAME].tf, INT_MAX); > // ref frame setup > for (int i = 0; i < 8; i++) > ff_progress_frame_replace(>s.refs[i], >next_refs[i]); LGTM. Sorry for the breakage. - Andreas ___ 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".
[FFmpeg-devel] [PATCH] lavc/vp9: Fix regression introduced in 0ba05857
From: Haihao Xiang It is possible that ff_progress_frame_await() is called but ff_progress_frame_report() isn't called when a hardware acceleration method is used, so a thread for vp9 decoding might get stuck. Signed-off-by: Haihao Xiang --- libavcodec/vp9.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/vp9.c b/libavcodec/vp9.c index 3adfb98f2d..6e2d18bf95 100644 --- a/libavcodec/vp9.c +++ b/libavcodec/vp9.c @@ -1735,9 +1735,9 @@ static int vp9_decode_frame(AVCodecContext *avctx, AVFrame *frame, if (ret < 0) goto fail; } -ff_progress_frame_report(>s.frames[CUR_FRAME].tf, INT_MAX); finish: +ff_progress_frame_report(>s.frames[CUR_FRAME].tf, INT_MAX); // ref frame setup for (int i = 0; i < 8; i++) ff_progress_frame_replace(>s.refs[i], >next_refs[i]); -- 2.34.1 ___ 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".