1
0
mirror of https://github.com/FFmpeg/FFmpeg.git synced 2025-04-14 00:58:38 +02:00

ffmpeg: switch to new FIFO API

This commit is contained in:
Anton Khirnov 2022-01-10 11:04:36 +01:00 committed by Andreas Rheinhardt
parent bdf9ed41fe
commit e6469e68cc
4 changed files with 37 additions and 56 deletions

View File

@ -527,23 +527,17 @@ static void ffmpeg_cleanup(int ret)
for (j = 0; j < fg->nb_inputs; j++) { for (j = 0; j < fg->nb_inputs; j++) {
InputFilter *ifilter = fg->inputs[j]; InputFilter *ifilter = fg->inputs[j];
struct InputStream *ist = ifilter->ist; struct InputStream *ist = ifilter->ist;
AVFrame *frame;
while (av_fifo_size(ifilter->frame_queue)) { while (av_fifo_read(ifilter->frame_queue, &frame, 1) >= 0)
AVFrame *frame;
av_fifo_generic_read(ifilter->frame_queue, &frame,
sizeof(frame), NULL);
av_frame_free(&frame); av_frame_free(&frame);
} av_fifo_freep2(&ifilter->frame_queue);
av_fifo_freep(&ifilter->frame_queue);
av_freep(&ifilter->displaymatrix); av_freep(&ifilter->displaymatrix);
if (ist->sub2video.sub_queue) { if (ist->sub2video.sub_queue) {
while (av_fifo_size(ist->sub2video.sub_queue)) { AVSubtitle sub;
AVSubtitle sub; while (av_fifo_read(ist->sub2video.sub_queue, &sub, 1) >= 0)
av_fifo_generic_read(ist->sub2video.sub_queue,
&sub, sizeof(sub), NULL);
avsubtitle_free(&sub); avsubtitle_free(&sub);
} av_fifo_freep2(&ist->sub2video.sub_queue);
av_fifo_freep(&ist->sub2video.sub_queue);
} }
av_buffer_unref(&ifilter->hw_frames_ctx); av_buffer_unref(&ifilter->hw_frames_ctx);
av_freep(&ifilter->name); av_freep(&ifilter->name);
@ -608,12 +602,10 @@ static void ffmpeg_cleanup(int ret)
avcodec_parameters_free(&ost->ref_par); avcodec_parameters_free(&ost->ref_par);
if (ost->muxing_queue) { if (ost->muxing_queue) {
while (av_fifo_size(ost->muxing_queue)) { AVPacket *pkt;
AVPacket *pkt; while (av_fifo_read(ost->muxing_queue, &pkt, 1) >= 0)
av_fifo_generic_read(ost->muxing_queue, &pkt, sizeof(pkt), NULL);
av_packet_free(&pkt); av_packet_free(&pkt);
} av_fifo_freep2(&ost->muxing_queue);
av_fifo_freep(&ost->muxing_queue);
} }
av_freep(&output_streams[i]); av_freep(&output_streams[i]);
@ -749,11 +741,11 @@ static void write_packet(OutputFile *of, AVPacket *pkt, OutputStream *ost, int u
if (!of->header_written) { if (!of->header_written) {
AVPacket *tmp_pkt; AVPacket *tmp_pkt;
/* the muxer is not initialized yet, buffer the packet */ /* the muxer is not initialized yet, buffer the packet */
if (!av_fifo_space(ost->muxing_queue)) { if (!av_fifo_can_write(ost->muxing_queue)) {
size_t cur_size = av_fifo_size(ost->muxing_queue); size_t cur_size = av_fifo_can_read(ost->muxing_queue);
unsigned int are_we_over_size = unsigned int are_we_over_size =
(ost->muxing_queue_data_size + pkt->size) > ost->muxing_queue_data_threshold; (ost->muxing_queue_data_size + pkt->size) > ost->muxing_queue_data_threshold;
size_t limit = are_we_over_size ? ost->max_muxing_queue_size : INT_MAX; size_t limit = are_we_over_size ? ost->max_muxing_queue_size : SIZE_MAX;
size_t new_size = FFMIN(2 * cur_size, limit); size_t new_size = FFMIN(2 * cur_size, limit);
if (new_size <= cur_size) { if (new_size <= cur_size) {
@ -762,7 +754,7 @@ static void write_packet(OutputFile *of, AVPacket *pkt, OutputStream *ost, int u
ost->file_index, ost->st->index); ost->file_index, ost->st->index);
exit_program(1); exit_program(1);
} }
ret = av_fifo_realloc2(ost->muxing_queue, new_size); ret = av_fifo_grow2(ost->muxing_queue, new_size - cur_size);
if (ret < 0) if (ret < 0)
exit_program(1); exit_program(1);
} }
@ -774,7 +766,7 @@ static void write_packet(OutputFile *of, AVPacket *pkt, OutputStream *ost, int u
exit_program(1); exit_program(1);
av_packet_move_ref(tmp_pkt, pkt); av_packet_move_ref(tmp_pkt, pkt);
ost->muxing_queue_data_size += tmp_pkt->size; ost->muxing_queue_data_size += tmp_pkt->size;
av_fifo_generic_write(ost->muxing_queue, &tmp_pkt, sizeof(tmp_pkt), NULL); av_fifo_write(ost->muxing_queue, &tmp_pkt, 1);
return; return;
} }
@ -2195,15 +2187,11 @@ static int ifilter_send_frame(InputFilter *ifilter, AVFrame *frame, int keep_ref
if (!tmp) if (!tmp)
return AVERROR(ENOMEM); return AVERROR(ENOMEM);
if (!av_fifo_space(ifilter->frame_queue)) { ret = av_fifo_write(ifilter->frame_queue, &tmp, 1);
ret = av_fifo_realloc2(ifilter->frame_queue, 2 * av_fifo_size(ifilter->frame_queue)); if (ret < 0)
if (ret < 0) { av_frame_free(&tmp);
av_frame_free(&tmp);
return ret; return ret;
}
}
av_fifo_generic_write(ifilter->frame_queue, &tmp, sizeof(tmp), NULL);
return 0;
} }
ret = reap_filters(1); ret = reap_filters(1);
@ -2526,15 +2514,13 @@ static int transcode_subtitles(InputStream *ist, AVPacket *pkt, int *got_output,
sub2video_update(ist, INT64_MIN, &subtitle); sub2video_update(ist, INT64_MIN, &subtitle);
} else if (ist->nb_filters) { } else if (ist->nb_filters) {
if (!ist->sub2video.sub_queue) if (!ist->sub2video.sub_queue)
ist->sub2video.sub_queue = av_fifo_alloc(8 * sizeof(AVSubtitle)); ist->sub2video.sub_queue = av_fifo_alloc2(8, sizeof(AVSubtitle), AV_FIFO_FLAG_AUTO_GROW);
if (!ist->sub2video.sub_queue) if (!ist->sub2video.sub_queue)
exit_program(1); exit_program(1);
if (!av_fifo_space(ist->sub2video.sub_queue)) {
ret = av_fifo_realloc2(ist->sub2video.sub_queue, 2 * av_fifo_size(ist->sub2video.sub_queue)); ret = av_fifo_write(ist->sub2video.sub_queue, &subtitle, 1);
if (ret < 0) if (ret < 0)
exit_program(1); exit_program(1);
}
av_fifo_generic_write(ist->sub2video.sub_queue, &subtitle, sizeof(subtitle), NULL);
free_sub = 0; free_sub = 0;
} }
@ -2976,14 +2962,13 @@ static int check_init_output_file(OutputFile *of, int file_index)
/* flush the muxing queues */ /* flush the muxing queues */
for (i = 0; i < of->ctx->nb_streams; i++) { for (i = 0; i < of->ctx->nb_streams; i++) {
OutputStream *ost = output_streams[of->ost_index + i]; OutputStream *ost = output_streams[of->ost_index + i];
AVPacket *pkt;
/* try to improve muxing time_base (only possible if nothing has been written yet) */ /* try to improve muxing time_base (only possible if nothing has been written yet) */
if (!av_fifo_size(ost->muxing_queue)) if (!av_fifo_can_read(ost->muxing_queue))
ost->mux_timebase = ost->st->time_base; ost->mux_timebase = ost->st->time_base;
while (av_fifo_size(ost->muxing_queue)) { while (av_fifo_read(ost->muxing_queue, &pkt, 1) >= 0) {
AVPacket *pkt;
av_fifo_generic_read(ost->muxing_queue, &pkt, sizeof(pkt), NULL);
ost->muxing_queue_data_size -= pkt->size; ost->muxing_queue_data_size -= pkt->size;
write_packet(of, pkt, ost, 1); write_packet(of, pkt, ost, 1);
av_packet_free(&pkt); av_packet_free(&pkt);

View File

@ -241,7 +241,7 @@ typedef struct InputFilter {
uint8_t *name; uint8_t *name;
enum AVMediaType type; // AVMEDIA_TYPE_SUBTITLE for sub2video enum AVMediaType type; // AVMEDIA_TYPE_SUBTITLE for sub2video
AVFifoBuffer *frame_queue; AVFifo *frame_queue;
// parameters configured for this input // parameters configured for this input
int format; int format;
@ -355,7 +355,7 @@ typedef struct InputStream {
struct sub2video { struct sub2video {
int64_t last_pts; int64_t last_pts;
int64_t end_pts; int64_t end_pts;
AVFifoBuffer *sub_queue; ///< queue of AVSubtitle* before filter init AVFifo *sub_queue; ///< queue of AVSubtitle* before filter init
AVFrame *frame; AVFrame *frame;
int w, h; int w, h;
unsigned int initialize; ///< marks if sub2video_update should force an initialization unsigned int initialize; ///< marks if sub2video_update should force an initialization
@ -555,7 +555,7 @@ typedef struct OutputStream {
int max_muxing_queue_size; int max_muxing_queue_size;
/* the packets are buffered here until the muxer is ready to be initialized */ /* the packets are buffered here until the muxer is ready to be initialized */
AVFifoBuffer *muxing_queue; AVFifo *muxing_queue;
/* /*
* The size of the AVPackets' buffers in queue. * The size of the AVPackets' buffers in queue.

View File

@ -178,7 +178,7 @@ int init_simple_filtergraph(InputStream *ist, OutputStream *ost)
ifilter->graph = fg; ifilter->graph = fg;
ifilter->format = -1; ifilter->format = -1;
ifilter->frame_queue = av_fifo_alloc(8 * sizeof(AVFrame*)); ifilter->frame_queue = av_fifo_alloc2(8, sizeof(AVFrame*), AV_FIFO_FLAG_AUTO_GROW);
if (!ifilter->frame_queue) if (!ifilter->frame_queue)
exit_program(1); exit_program(1);
@ -286,7 +286,7 @@ static void init_input_filter(FilterGraph *fg, AVFilterInOut *in)
ifilter->type = ist->st->codecpar->codec_type; ifilter->type = ist->st->codecpar->codec_type;
ifilter->name = describe_filter_link(fg, in, 1); ifilter->name = describe_filter_link(fg, in, 1);
ifilter->frame_queue = av_fifo_alloc(8 * sizeof(AVFrame*)); ifilter->frame_queue = av_fifo_alloc2(8, sizeof(AVFrame*), AV_FIFO_FLAG_AUTO_GROW);
if (!ifilter->frame_queue) if (!ifilter->frame_queue)
exit_program(1); exit_program(1);
@ -1106,9 +1106,8 @@ int configure_filtergraph(FilterGraph *fg)
} }
for (i = 0; i < fg->nb_inputs; i++) { for (i = 0; i < fg->nb_inputs; i++) {
while (av_fifo_size(fg->inputs[i]->frame_queue)) { AVFrame *tmp;
AVFrame *tmp; while (av_fifo_read(fg->inputs[i]->frame_queue, &tmp, 1) >= 0) {
av_fifo_generic_read(fg->inputs[i]->frame_queue, &tmp, sizeof(tmp), NULL);
ret = av_buffersrc_add_frame(fg->inputs[i]->filter, tmp); ret = av_buffersrc_add_frame(fg->inputs[i]->filter, tmp);
av_frame_free(&tmp); av_frame_free(&tmp);
if (ret < 0) if (ret < 0)
@ -1129,9 +1128,8 @@ int configure_filtergraph(FilterGraph *fg)
for (i = 0; i < fg->nb_inputs; i++) { for (i = 0; i < fg->nb_inputs; i++) {
InputStream *ist = fg->inputs[i]->ist; InputStream *ist = fg->inputs[i]->ist;
if (ist->sub2video.sub_queue && ist->sub2video.frame) { if (ist->sub2video.sub_queue && ist->sub2video.frame) {
while (av_fifo_size(ist->sub2video.sub_queue)) { AVSubtitle tmp;
AVSubtitle tmp; while (av_fifo_read(ist->sub2video.sub_queue, &tmp, 1) >= 0) {
av_fifo_generic_read(ist->sub2video.sub_queue, &tmp, sizeof(tmp), NULL);
sub2video_update(ist, INT64_MIN, &tmp); sub2video_update(ist, INT64_MIN, &tmp);
avsubtitle_free(&tmp); avsubtitle_free(&tmp);
} }

View File

@ -1589,8 +1589,6 @@ static OutputStream *new_output_stream(OptionsContext *o, AVFormatContext *oc, e
ost->max_muxing_queue_size = 128; ost->max_muxing_queue_size = 128;
MATCH_PER_STREAM_OPT(max_muxing_queue_size, i, ost->max_muxing_queue_size, oc, st); MATCH_PER_STREAM_OPT(max_muxing_queue_size, i, ost->max_muxing_queue_size, oc, st);
ost->max_muxing_queue_size = FFMIN(ost->max_muxing_queue_size, INT_MAX / sizeof(ost->pkt));
ost->max_muxing_queue_size *= sizeof(ost->pkt);
ost->muxing_queue_data_size = 0; ost->muxing_queue_data_size = 0;
@ -1617,7 +1615,7 @@ static OutputStream *new_output_stream(OptionsContext *o, AVFormatContext *oc, e
} }
ost->last_mux_dts = AV_NOPTS_VALUE; ost->last_mux_dts = AV_NOPTS_VALUE;
ost->muxing_queue = av_fifo_alloc(8 * sizeof(AVPacket)); ost->muxing_queue = av_fifo_alloc2(8, sizeof(AVPacket*), 0);
if (!ost->muxing_queue) if (!ost->muxing_queue)
exit_program(1); exit_program(1);