[FFmpeg-devel] [PATCH 4/4] lavc/vvc/refs: Use dpb_max_num_reorder_pics to control output

[Nuo Mi]

Thank you for the clip.
Applied all except the libxevd one

On Sun, Apr 21, 2024 at 10:34 AM mypopy at gmail.com <mypopy at gmail.com> wrote:

> Send it privately via email
>
>
> On Sat, Apr 20, 2024 at 11:55 AM Nuo Mi <nuomi2021 at gmail.com> wrote:
> >
> > Hi Barry and Eliny,
> > LGTM.
> > Thank you for the patch.
> > Is it possible to provide the clip so we can add it to our CI?
> >
> >  ci like https://github.com/ffvvc/FFmpeg/actions
> >
> >
>
> > On Fri, Apr 19, 2024 at 9:48 PM Jun Zhao <mypopy at gmail.com> wrote:
> >
> > > From: Jun Zhao <mypopydev at gmail.com>
> > >
> > > Use dpb_max_num_reorder_pics to control output instead of
> > > dpb_max_dec_pic_buffering, when dpb_max_dec_pic_buffering
> > > is much larger than dpb_max_num_reorder_pics, it may cause
> > > dpb overflow error.
> > >
> > > Signed-off-by: Jun Zhao <barryjzhao at tencent.com>
> > > Signed-off-by: elinyhuang <elinyhuang at tencent.com>
> > > ---
> > >  libavcodec/vvc/refs.c | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/libavcodec/vvc/refs.c b/libavcodec/vvc/refs.c
> > > index a5ee7338d6..6694bc4c51 100644
> > > --- a/libavcodec/vvc/refs.c
> > > +++ b/libavcodec/vvc/refs.c
> > > @@ -226,7 +226,7 @@ int ff_vvc_output_frame(VVCContext *s,
> VVCFrameContext
> > > *fc, AVFrame *out, const
> > >
> > >          /* wait for more frames before output */
> > >          if (!flush && s->seq_output == s->seq_decode && sps &&
> > > -            nb_output <=
> > >
> sps->r->sps_dpb_params.dpb_max_dec_pic_buffering_minus1[sps->r->sps_max_sublayers_minus1]
> > > + 1)
> > > +            nb_output <=
> > >
> sps->r->sps_dpb_params.dpb_max_num_reorder_pics[sps->r->sps_max_sublayers_minus1])
> > >              return 0;
> > >
> > >          if (nb_output) {
> > > --
> > > 2.25.1
> > >
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
>