From: James Almer via ffmpeg-devel <ffmpeg-devel@ffmpeg.org>
To: ffmpeg-devel@ffmpeg.org
Cc: James Almer <code@ffmpeg.org>
Subject: [FFmpeg-devel] [PATCH] tests/fate-run: pass threads argument to tests using certain pseudo muxers (PR #20960)
Date: Tue, 18 Nov 2025 17:01:42 -0000
Message-ID: <176348530314.25.1371914612123517358@2cb04c0e5124> (raw)
PR #20960 opened by James Almer (jamrial)
URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20960
Patch URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20960.patch
>From 8030577241e9439a1ee94c7b9bf1d3d205878d1f Mon Sep 17 00:00:00 2001
From: James Almer <jamrial@gmail.com>
Date: Tue, 18 Nov 2025 00:41:12 -0300
Subject: [PATCH 1/2] tests/fate/ffmpeg: remove max_muxing_queue_size option
from fate-ffmpeg-attached_pics
It's a no-op given that the CLI will only look at that value after the size
threshold is reached, which is by default about 50mb, way bigger than the
output of this test.
Instead disable lavf's max_interleave_delta, which has a default of about 1
second worth of packets. This lets us remove the hardcoded threads values as
the output frame order will no longer change based on it (lavf's generic muxer
code will no longer drain buffered frames after 1 second worth of audio is
reached before it ever sees a video frame, which used to happen when enabling
multi threaded encoding).
Signed-off-by: James Almer <jamrial@gmail.com>
---
tests/fate/ffmpeg.mak | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tests/fate/ffmpeg.mak b/tests/fate/ffmpeg.mak
index 57028a7936..270c99d857 100644
--- a/tests/fate/ffmpeg.mak
+++ b/tests/fate/ffmpeg.mak
@@ -7,7 +7,7 @@ fate-ffmpeg-filter_complex_audio: CMD = framecrc -auto_conversion_filters -filte
# Ticket 6375, use case of NoX
FATE_SAMPLES_FFMPEG-$(call FRAMECRC, MOV, PNG ALAC, ARESAMPLE_FILTER) += fate-ffmpeg-attached_pics
-fate-ffmpeg-attached_pics: CMD = threads=2 framecrc -i $(TARGET_SAMPLES)/lossless-audio/inside.m4a -threads 1 -max_muxing_queue_size 16 -af aresample
+fate-ffmpeg-attached_pics: CMD = framecrc -i $(TARGET_SAMPLES)/lossless-audio/inside.m4a -max_interleave_delta 0 -af aresample
FATE_SAMPLES_FFMPEG-$(call FILTERDEMDEC, COLORKEY OVERLAY SCALE, MPEGPS IMAGE_PPM_PIPE, CAVS PPM, CAVSVIDEO_PARSER) += fate-ffmpeg-filter_colorkey
fate-ffmpeg-filter_colorkey: tests/data/filtergraphs/colorkey
--
2.49.1
>From b80d92f29441c91edfe65c79f2fd54d93ab11fb4 Mon Sep 17 00:00:00 2001
From: James Almer <jamrial@gmail.com>
Date: Tue, 18 Nov 2025 00:43:23 -0300
Subject: [PATCH 2/2] tests/fate-run: pass threads argument to tests using
certain pseudo muxers
---
tests/fate-run.sh | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/tests/fate-run.sh b/tests/fate-run.sh
index 6d1fe1185c..210fcc2eb5 100755
--- a/tests/fate-run.sh
+++ b/tests/fate-run.sh
@@ -162,15 +162,15 @@ ffprobe_demux(){
}
framecrc(){
- ffmpeg "$@" -bitexact -f framecrc -
+ ffmpeg "$@" -bitexact -threads $threads -f framecrc -
}
ffmetadata(){
- ffmpeg "$@" -bitexact -f ffmetadata -
+ ffmpeg "$@" -bitexact -threads $threads -f ffmetadata -
}
framemd5(){
- ffmpeg "$@" -bitexact -f framemd5 -
+ ffmpeg "$@" -bitexact -threads $threads -f framemd5 -
}
crc(){
--
2.49.1
_______________________________________________
ffmpeg-devel mailing list -- ffmpeg-devel@ffmpeg.org
To unsubscribe send an email to ffmpeg-devel-leave@ffmpeg.org
reply other threads:[~2025-11-18 17:02 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=176348530314.25.1371914612123517358@2cb04c0e5124 \
--to=ffmpeg-devel@ffmpeg.org \
--cc=code@ffmpeg.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
This inbox may be cloned and mirrored by anyone:
git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
ffmpegdev@gitmailbox.com
public-inbox-index ffmpegdev
Example config snippet for mirrors.
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git