diff options
author | zhilizhao <zhilizhao@tencent.com> | 2021-04-27 13:09:56 +0800 |
---|---|---|
committer | Steven Liu <lq@chinaffmpeg.org> | 2021-06-10 19:54:03 +0800 |
commit | cf1746d77989c2742c8c364f57a626942cccd8d1 (patch) | |
tree | 1a36bc79e148425168256a57092fbf682a64651e | |
parent | 08d933bf6169a93631fd76354214f837c973c18f (diff) |
ffmpeg_videotoolbox: skip memory copy if hwaccel_output_format match
Simple test results:
Command:
./ffmpeg -y -hwaccel videotoolbox -hwaccel_output_format videotoolbox_vld \
-i test.mp4 -an -c:v h264_videotoolbox -benchmark out.mp4
Before:
frame= 1221 fps= 66 q=-0.0 Lsize= 3144kB time=00:00:20.33 bitrate=1266.6kbits/s dup=4 drop=0 speed= 1.1x
bench: utime=2.714s stime=1.218s rtime=18.574s
After:
frame= 1221 fps=137 q=-0.0 Lsize= 3144kB time=00:00:20.33 bitrate=1266.4kbits/s dup=4 drop=0 speed=2.28x
bench: utime=1.450s stime=1.440s rtime=8.924s
It has limited usecase since there is no video filter support, so
a log message is added to notify the user.
Reviewed-by: Steven Liu <liuqi05@kuaishou.com>
-rw-r--r-- | fftools/ffmpeg_videotoolbox.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/fftools/ffmpeg_videotoolbox.c b/fftools/ffmpeg_videotoolbox.c index a6b78d0f7d..4ba8618539 100644 --- a/fftools/ffmpeg_videotoolbox.c +++ b/fftools/ffmpeg_videotoolbox.c @@ -29,6 +29,7 @@ typedef struct VTContext { AVFrame *tmp_frame; + int log_once; } VTContext; char *videotoolbox_pixfmt; @@ -44,6 +45,13 @@ static int videotoolbox_retrieve_data(AVCodecContext *s, AVFrame *frame) int linesize[4] = { 0 }; int planes, ret, i; + if (frame->format == ist->hwaccel_output_format) { + av_log_once(s, AV_LOG_INFO, AV_LOG_TRACE, &vt->log_once, + "There is no video filter for videotoolbox pix_fmt now, remove the " + "-hwaccel_output_format option if video filter doesn't work\n"); + return 0; + } + av_frame_unref(vt->tmp_frame); switch (pixel_format) { |