[FFmpeg-devel] [PATCH 2/5 v3] avcodec: add an AV1 frame merge BSF
James Almer
jamrial at gmail.com
Tue Nov 12 17:00:40 EET 2019
Signed-off-by: James Almer <jamrial at gmail.com>
---
Assorted improvements to avoid creating new references, copying packet
properties, etc.
configure | 1 +
libavcodec/Makefile | 1 +
libavcodec/av1_frame_merge_bsf.c | 173 +++++++++++++++++++++++++++++++
libavcodec/bitstream_filters.c | 1 +
4 files changed, 176 insertions(+)
create mode 100644 libavcodec/av1_frame_merge_bsf.c
diff --git a/configure b/configure
index 1de90e93fd..70f60997c1 100755
--- a/configure
+++ b/configure
@@ -3115,6 +3115,7 @@ vc1_parser_select="vc1dsp"
# bitstream_filters
aac_adtstoasc_bsf_select="adts_header"
+av1_frame_merge_bsf_select="cbs_av1"
av1_frame_split_bsf_select="cbs_av1"
av1_metadata_bsf_select="cbs_av1"
eac3_core_bsf_select="ac3_parser"
diff --git a/libavcodec/Makefile b/libavcodec/Makefile
index b990c6ba87..006a472a6d 100644
--- a/libavcodec/Makefile
+++ b/libavcodec/Makefile
@@ -1075,6 +1075,7 @@ OBJS-$(CONFIG_XMA_PARSER) += xma_parser.o
# bitstream filters
OBJS-$(CONFIG_AAC_ADTSTOASC_BSF) += aac_adtstoasc_bsf.o mpeg4audio.o
OBJS-$(CONFIG_AV1_METADATA_BSF) += av1_metadata_bsf.o
+OBJS-$(CONFIG_AV1_FRAME_MERGE_BSF) += av1_frame_merge_bsf.o
OBJS-$(CONFIG_AV1_FRAME_SPLIT_BSF) += av1_frame_split_bsf.o
OBJS-$(CONFIG_CHOMP_BSF) += chomp_bsf.o
OBJS-$(CONFIG_DUMP_EXTRADATA_BSF) += dump_extradata_bsf.o
diff --git a/libavcodec/av1_frame_merge_bsf.c b/libavcodec/av1_frame_merge_bsf.c
new file mode 100644
index 0000000000..a2cb09b55e
--- /dev/null
+++ b/libavcodec/av1_frame_merge_bsf.c
@@ -0,0 +1,173 @@
+/*
+ * Copyright (c) 2019 James Almer <jamrial at gmail.com>
+ *
+ * This file is part of FFmpeg.
+ *
+ * FFmpeg is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2.1 of the License, or (at your option) any later version.
+ *
+ * FFmpeg is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with FFmpeg; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
+ */
+
+#include "avcodec.h"
+#include "bsf.h"
+#include "cbs.h"
+#include "cbs_av1.h"
+
+typedef struct AV1FMergeContext {
+ CodedBitstreamContext *cbc;
+ CodedBitstreamFragment *temporal_unit;
+ CodedBitstreamFragment *frag;
+ AVPacket *buffer_pkt;
+ AVPacket *in;
+} AV1FMergeContext;
+
+static int av1_frame_merge_filter(AVBSFContext *bsf, AVPacket *out)
+{
+ AV1FMergeContext *ctx = bsf->priv_data;
+ CodedBitstreamFragment *frag = ctx->frag, *tu = ctx->temporal_unit;
+ int err, i;
+
+ err = ff_bsf_get_packet_ref(bsf, ctx->in);
+ if (err < 0) {
+ if (err == AVERROR_EOF && tu->nb_units > 0)
+ goto eof;
+ return err;
+ }
+
+ err = ff_cbs_read_packet(ctx->cbc, frag, ctx->in);
+ if (err < 0) {
+ av_log(bsf, AV_LOG_ERROR, "Failed to read packet.\n");
+ goto fail;
+ }
+
+ if (frag->nb_units == 0) {
+ av_log(bsf, AV_LOG_ERROR, "No OBU in packet.\n");
+ err = AVERROR_INVALIDDATA;
+ goto fail;
+ }
+
+ if (tu->nb_units == 0) {
+ if (frag->units[0].type != AV1_OBU_TEMPORAL_DELIMITER) {
+ av_log(bsf, AV_LOG_ERROR, "Missing Temporal Delimiter.\n");
+ err = AVERROR_INVALIDDATA;
+ goto fail;
+ }
+ av_packet_move_ref(ctx->buffer_pkt, ctx->in);
+ }
+
+ if (tu->nb_units > 0 && frag->units[0].type == AV1_OBU_TEMPORAL_DELIMITER) {
+eof:
+ err = ff_cbs_write_packet(ctx->cbc, ctx->buffer_pkt, tu);
+ if (err < 0) {
+ av_log(bsf, AV_LOG_ERROR, "Failed to write packet.\n");
+ goto fail;
+ }
+ av_packet_move_ref(out, ctx->buffer_pkt);
+
+ ff_cbs_fragment_reset(ctx->cbc, tu);
+
+ for (i = 1; i < frag->nb_units; i++) {
+ if (frag->units[i].type == AV1_OBU_TEMPORAL_DELIMITER) {
+ av_log(bsf, AV_LOG_ERROR, "Temporal Delimiter in the middle of a packet.\n");
+ err = AVERROR_INVALIDDATA;
+ goto fail;
+ }
+ }
+ // Swap pointers to avoid copying unit content.
+ ctx->temporal_unit = frag;
+ ctx->frag = tu;
+
+ av_packet_move_ref(ctx->buffer_pkt, ctx->in);
+
+ return 0;
+ }
+
+ for (i = 0; i < frag->nb_units; i++) {
+ if (i && frag->units[i].type == AV1_OBU_TEMPORAL_DELIMITER) {
+ av_log(bsf, AV_LOG_ERROR, "Temporal Delimiter in the middle of a packet.\n");
+ err = AVERROR_INVALIDDATA;
+ goto fail;
+ }
+ err = ff_cbs_insert_unit_content(ctx->cbc, tu, -1, frag->units[i].type,
+ frag->units[i].content, frag->units[i].content_ref);
+ if (err < 0)
+ goto fail;
+ }
+ ff_cbs_fragment_reset(ctx->cbc, frag);
+ av_packet_unref(ctx->in);
+
+ return AVERROR(EAGAIN);
+
+fail:
+ ff_cbs_fragment_reset(ctx->cbc, tu);
+ ff_cbs_fragment_reset(ctx->cbc, frag);
+ av_packet_unref(ctx->buffer_pkt);
+ av_packet_unref(ctx->in);
+ av_packet_unref(out);
+
+ return err;
+}
+
+static int av1_frame_merge_init(AVBSFContext *bsf)
+{
+ AV1FMergeContext *ctx = bsf->priv_data;
+
+ ctx->temporal_unit = av_mallocz(sizeof(*ctx->temporal_unit));
+ ctx->frag = av_mallocz(sizeof(*ctx->frag));
+ ctx->buffer_pkt = av_packet_alloc();
+ ctx->in = av_packet_alloc();
+ if (!ctx->temporal_unit || !ctx->frag ||
+ !ctx->buffer_pkt || !ctx->in)
+ return AVERROR(ENOMEM);
+
+ return ff_cbs_init(&ctx->cbc, AV_CODEC_ID_AV1, bsf);
+}
+
+static void av1_frame_merge_flush(AVBSFContext *bsf)
+{
+ AV1FMergeContext *ctx = bsf->priv_data;
+
+ ff_cbs_fragment_reset(ctx->cbc, &ctx->temporal_unit);
+ ff_cbs_fragment_reset(ctx->cbc, &ctx->frag);
+ av_packet_unref(ctx->buffer_pkt);
+ av_packet_unref(ctx->in);
+}
+
+static void av1_frame_merge_close(AVBSFContext *bsf)
+{
+ AV1FMergeContext *ctx = bsf->priv_data;
+
+ if (ctx->temporal_unit)
+ ff_cbs_fragment_free(ctx->cbc, ctx->temporal_unit);
+ if (ctx->frag)
+ ff_cbs_fragment_free(ctx->cbc, ctx->frag);
+ av_freep(&ctx->temporal_unit);
+ av_freep(&ctx->frag);
+ av_packet_free(&ctx->buffer_pkt);
+ av_packet_free(&ctx->in);
+ ff_cbs_close(&ctx->cbc);
+}
+
+static const enum AVCodecID av1_frame_merge_codec_ids[] = {
+ AV_CODEC_ID_AV1, AV_CODEC_ID_NONE,
+};
+
+const AVBitStreamFilter ff_av1_frame_merge_bsf = {
+ .name = "av1_frame_merge",
+ .priv_data_size = sizeof(AV1FMergeContext),
+ .init = av1_frame_merge_init,
+ .flush = av1_frame_merge_flush,
+ .close = av1_frame_merge_close,
+ .filter = av1_frame_merge_filter,
+ .codec_ids = av1_frame_merge_codec_ids,
+};
diff --git a/libavcodec/bitstream_filters.c b/libavcodec/bitstream_filters.c
index 463003966a..6b5ffe4d70 100644
--- a/libavcodec/bitstream_filters.c
+++ b/libavcodec/bitstream_filters.c
@@ -25,6 +25,7 @@
#include "bsf.h"
extern const AVBitStreamFilter ff_aac_adtstoasc_bsf;
+extern const AVBitStreamFilter ff_av1_frame_merge_bsf;
extern const AVBitStreamFilter ff_av1_frame_split_bsf;
extern const AVBitStreamFilter ff_av1_metadata_bsf;
extern const AVBitStreamFilter ff_chomp_bsf;
--
2.23.0
More information about the ffmpeg-devel
mailing list