Message ID | 20220107142134.821-2-jamrial@gmail.com |
---|---|
State | New |
Headers | show |
Series | [FFmpeg-devel,1/2] avcodec/av1: honor the requested skip_frame level | expand |
Context | Check | Description |
---|---|---|
andriy/make_x86 | success | Make finished |
andriy/make_fate_x86 | success | Make fate finished |
andriy/make_ppc | success | Make finished |
andriy/make_fate_ppc | success | Make fate finished |
On Fri, Jan 7, 2022 at 3:22 PM James Almer <jamrial@gmail.com> wrote: > > This supports dropping non-intra, non-key, or all frames. > > Signed-off-by: James Almer <jamrial@gmail.com> > --- > libavcodec/libdav1d.c | 9 +++++++++ > 1 file changed, 9 insertions(+) > > diff --git a/libavcodec/libdav1d.c b/libavcodec/libdav1d.c > index 8e45f533cb..9252a2572d 100644 > --- a/libavcodec/libdav1d.c > +++ b/libavcodec/libdav1d.c > @@ -299,6 +299,7 @@ static int libdav1d_receive_frame(AVCodecContext *c, AVFrame *frame) > #endif > int res; > > +redo: > if (!data->sz) { > AVPacket *const pkt = dav1d->pkt; > > @@ -363,6 +364,14 @@ static int libdav1d_receive_frame(AVCodecContext *c, AVFrame *frame) > > av_assert0(p->data[0] && p->allocator_data); > > + if ((c->skip_frame >= AVDISCARD_NONINTRA && (p->frame_hdr->frame_type != DAV1D_FRAME_TYPE_KEY && > + p->frame_hdr->frame_type != DAV1D_FRAME_TYPE_INTRA)) || > + (c->skip_frame >= AVDISCARD_NONKEY && p->frame_hdr->frame_type != DAV1D_FRAME_TYPE_KEY) || > + c->skip_frame >= AVDISCARD_ALL) { > + dav1d_picture_unref(p); > + goto redo; > + } Is skipping still meaningful if it doesn't actually speed up decoding? - Hendrik
On 1/7/2022 1:14 PM, Hendrik Leppkes wrote: > On Fri, Jan 7, 2022 at 3:22 PM James Almer <jamrial@gmail.com> wrote: >> >> This supports dropping non-intra, non-key, or all frames. >> >> Signed-off-by: James Almer <jamrial@gmail.com> >> --- >> libavcodec/libdav1d.c | 9 +++++++++ >> 1 file changed, 9 insertions(+) >> >> diff --git a/libavcodec/libdav1d.c b/libavcodec/libdav1d.c >> index 8e45f533cb..9252a2572d 100644 >> --- a/libavcodec/libdav1d.c >> +++ b/libavcodec/libdav1d.c >> @@ -299,6 +299,7 @@ static int libdav1d_receive_frame(AVCodecContext *c, AVFrame *frame) >> #endif >> int res; >> >> +redo: >> if (!data->sz) { >> AVPacket *const pkt = dav1d->pkt; >> >> @@ -363,6 +364,14 @@ static int libdav1d_receive_frame(AVCodecContext *c, AVFrame *frame) >> >> av_assert0(p->data[0] && p->allocator_data); >> >> + if ((c->skip_frame >= AVDISCARD_NONINTRA && (p->frame_hdr->frame_type != DAV1D_FRAME_TYPE_KEY && >> + p->frame_hdr->frame_type != DAV1D_FRAME_TYPE_INTRA)) || >> + (c->skip_frame >= AVDISCARD_NONKEY && p->frame_hdr->frame_type != DAV1D_FRAME_TYPE_KEY) || >> + c->skip_frame >= AVDISCARD_ALL) { >> + dav1d_picture_unref(p); >> + goto redo; >> + } > > Is skipping still meaningful if it doesn't actually speed up decoding? Outputting only I frames, but that's also possible using a filter, i guess. In any case, no alternative for libdav1d unless the actual library adds support for this, so I'll drop this patch. > > - Hendrik > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
diff --git a/libavcodec/libdav1d.c b/libavcodec/libdav1d.c index 8e45f533cb..9252a2572d 100644 --- a/libavcodec/libdav1d.c +++ b/libavcodec/libdav1d.c @@ -299,6 +299,7 @@ static int libdav1d_receive_frame(AVCodecContext *c, AVFrame *frame) #endif int res; +redo: if (!data->sz) { AVPacket *const pkt = dav1d->pkt; @@ -363,6 +364,14 @@ static int libdav1d_receive_frame(AVCodecContext *c, AVFrame *frame) av_assert0(p->data[0] && p->allocator_data); + if ((c->skip_frame >= AVDISCARD_NONINTRA && (p->frame_hdr->frame_type != DAV1D_FRAME_TYPE_KEY && + p->frame_hdr->frame_type != DAV1D_FRAME_TYPE_INTRA)) || + (c->skip_frame >= AVDISCARD_NONKEY && p->frame_hdr->frame_type != DAV1D_FRAME_TYPE_KEY) || + c->skip_frame >= AVDISCARD_ALL) { + dav1d_picture_unref(p); + goto redo; + } + // This requires the custom allocator above frame->buf[0] = av_buffer_ref(p->allocator_data); if (!frame->buf[0]) {
This supports dropping non-intra, non-key, or all frames. Signed-off-by: James Almer <jamrial@gmail.com> --- libavcodec/libdav1d.c | 9 +++++++++ 1 file changed, 9 insertions(+)