Commit f5894178 authored by Mark Thompson's avatar Mark Thompson

cbs_av1: Support redundant frame headers

parent 6bdb7712
...@@ -996,7 +996,10 @@ static int cbs_av1_read_unit(CodedBitstreamContext *ctx, ...@@ -996,7 +996,10 @@ static int cbs_av1_read_unit(CodedBitstreamContext *ctx,
case AV1_OBU_REDUNDANT_FRAME_HEADER: case AV1_OBU_REDUNDANT_FRAME_HEADER:
{ {
err = cbs_av1_read_frame_header_obu(ctx, &gbc, err = cbs_av1_read_frame_header_obu(ctx, &gbc,
&obu->obu.frame_header); &obu->obu.frame_header,
obu->header.obu_type ==
AV1_OBU_REDUNDANT_FRAME_HEADER,
unit->data_ref);
if (err < 0) if (err < 0)
return err; return err;
} }
...@@ -1016,7 +1019,8 @@ static int cbs_av1_read_unit(CodedBitstreamContext *ctx, ...@@ -1016,7 +1019,8 @@ static int cbs_av1_read_unit(CodedBitstreamContext *ctx,
break; break;
case AV1_OBU_FRAME: case AV1_OBU_FRAME:
{ {
err = cbs_av1_read_frame_obu(ctx, &gbc, &obu->obu.frame); err = cbs_av1_read_frame_obu(ctx, &gbc, &obu->obu.frame,
unit->data_ref);
if (err < 0) if (err < 0)
return err; return err;
...@@ -1124,7 +1128,10 @@ static int cbs_av1_write_obu(CodedBitstreamContext *ctx, ...@@ -1124,7 +1128,10 @@ static int cbs_av1_write_obu(CodedBitstreamContext *ctx,
case AV1_OBU_REDUNDANT_FRAME_HEADER: case AV1_OBU_REDUNDANT_FRAME_HEADER:
{ {
err = cbs_av1_write_frame_header_obu(ctx, pbc, err = cbs_av1_write_frame_header_obu(ctx, pbc,
&obu->obu.frame_header); &obu->obu.frame_header,
obu->header.obu_type ==
AV1_OBU_REDUNDANT_FRAME_HEADER,
NULL);
if (err < 0) if (err < 0)
return err; return err;
} }
...@@ -1141,7 +1148,7 @@ static int cbs_av1_write_obu(CodedBitstreamContext *ctx, ...@@ -1141,7 +1148,7 @@ static int cbs_av1_write_obu(CodedBitstreamContext *ctx,
break; break;
case AV1_OBU_FRAME: case AV1_OBU_FRAME:
{ {
err = cbs_av1_write_frame_obu(ctx, pbc, &obu->obu.frame); err = cbs_av1_write_frame_obu(ctx, pbc, &obu->obu.frame, NULL);
if (err < 0) if (err < 0)
return err; return err;
...@@ -1302,6 +1309,7 @@ static void cbs_av1_close(CodedBitstreamContext *ctx) ...@@ -1302,6 +1309,7 @@ static void cbs_av1_close(CodedBitstreamContext *ctx)
CodedBitstreamAV1Context *priv = ctx->priv_data; CodedBitstreamAV1Context *priv = ctx->priv_data;
av_buffer_unref(&priv->sequence_header_ref); av_buffer_unref(&priv->sequence_header_ref);
av_buffer_unref(&priv->frame_header_ref);
av_freep(&priv->write_buffer); av_freep(&priv->write_buffer);
} }
......
...@@ -400,6 +400,9 @@ typedef struct CodedBitstreamAV1Context { ...@@ -400,6 +400,9 @@ typedef struct CodedBitstreamAV1Context {
AVBufferRef *sequence_header_ref; AVBufferRef *sequence_header_ref;
int seen_frame_header; int seen_frame_header;
AVBufferRef *frame_header_ref;
uint8_t *frame_header;
size_t frame_header_size;
int temporal_id; int temporal_id;
int spatial_id; int spatial_id;
......
...@@ -1463,24 +1463,90 @@ static int FUNC(uncompressed_header)(CodedBitstreamContext *ctx, RWContext *rw, ...@@ -1463,24 +1463,90 @@ static int FUNC(uncompressed_header)(CodedBitstreamContext *ctx, RWContext *rw,
} }
static int FUNC(frame_header_obu)(CodedBitstreamContext *ctx, RWContext *rw, static int FUNC(frame_header_obu)(CodedBitstreamContext *ctx, RWContext *rw,
AV1RawFrameHeader *current) AV1RawFrameHeader *current, int redundant,
AVBufferRef *rw_buffer_ref)
{ {
CodedBitstreamAV1Context *priv = ctx->priv_data; CodedBitstreamAV1Context *priv = ctx->priv_data;
int err; int start_pos, fh_bits, fh_bytes, err;
uint8_t *fh_start;
HEADER("Frame Header");
if (priv->seen_frame_header) { if (priv->seen_frame_header) {
// Nothing to do. if (!redundant) {
av_log(ctx->log_ctx, AV_LOG_ERROR, "Invalid repeated "
"frame header OBU.\n");
return AVERROR_INVALIDDATA;
} else { } else {
GetBitContext fh;
size_t i, b;
uint32_t val;
HEADER("Redundant Frame Header");
av_assert0(priv->frame_header_ref && priv->frame_header);
init_get_bits(&fh, priv->frame_header,
priv->frame_header_size);
for (i = 0; i < priv->frame_header_size; i += 8) {
b = FFMIN(priv->frame_header_size - i, 8);
val = get_bits(&fh, b);
xf(b, frame_header_copy[i],
val, val, val, 1, i / 8);
}
}
} else {
if (redundant)
HEADER("Redundant Frame Header (used as Frame Header)");
else
HEADER("Frame Header");
priv->seen_frame_header = 1; priv->seen_frame_header = 1;
#ifdef READ
start_pos = get_bits_count(rw);
#else
start_pos = put_bits_count(rw);
#endif
CHECK(FUNC(uncompressed_header)(ctx, rw, current)); CHECK(FUNC(uncompressed_header)(ctx, rw, current));
if (current->show_existing_frame) { if (current->show_existing_frame) {
priv->seen_frame_header = 0; priv->seen_frame_header = 0;
} else { } else {
priv->seen_frame_header = 1; priv->seen_frame_header = 1;
av_buffer_unref(&priv->frame_header_ref);
#ifdef READ
fh_bits = get_bits_count(rw) - start_pos;
fh_start = (uint8_t*)rw->buffer + start_pos / 8;
#else
// Need to flush the bitwriter so that we can copy its output,
// but use a copy so we don't affect the caller's structure.
{
PutBitContext tmp = *rw;
flush_put_bits(&tmp);
}
fh_bits = put_bits_count(rw) - start_pos;
fh_start = rw->buf + start_pos / 8;
#endif
fh_bytes = (fh_bits + 7) / 8;
priv->frame_header_size = fh_bits;
if (rw_buffer_ref) {
priv->frame_header_ref = av_buffer_ref(rw_buffer_ref);
if (!priv->frame_header_ref)
return AVERROR(ENOMEM);
priv->frame_header = fh_start;
} else {
priv->frame_header_ref =
av_buffer_alloc(fh_bytes + AV_INPUT_BUFFER_PADDING_SIZE);
if (!priv->frame_header_ref)
return AVERROR(ENOMEM);
priv->frame_header = priv->frame_header_ref->data;
memcpy(priv->frame_header, fh_start, fh_bytes);
}
} }
} }
...@@ -1524,11 +1590,13 @@ static int FUNC(tile_group_obu)(CodedBitstreamContext *ctx, RWContext *rw, ...@@ -1524,11 +1590,13 @@ static int FUNC(tile_group_obu)(CodedBitstreamContext *ctx, RWContext *rw,
} }
static int FUNC(frame_obu)(CodedBitstreamContext *ctx, RWContext *rw, static int FUNC(frame_obu)(CodedBitstreamContext *ctx, RWContext *rw,
AV1RawFrame *current) AV1RawFrame *current,
AVBufferRef *rw_buffer_ref)
{ {
int err; int err;
CHECK(FUNC(frame_header_obu)(ctx, rw, &current->header)); CHECK(FUNC(frame_header_obu)(ctx, rw, &current->header,
0, rw_buffer_ref));
CHECK(FUNC(byte_alignment)(ctx, rw)); CHECK(FUNC(byte_alignment)(ctx, rw));
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment