From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.ffmpeg.org (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTPS id 613914ECA1 for ; Mon, 16 Feb 2026 17:10:33 +0000 (UTC) Authentication-Results: ffbox; dkim=fail (body hash mismatch (got b'rMn2QFZ40p9ZhKOZ6yqfs0TFaoiMX8L84ckNut1hjdc=', expected b'w9+rBqB538DfRTi0BwFGkzTsxDof60Q9kIx/FFNTsqA=')) header.d=ffmpeg.org header.i=@ffmpeg.org header.a=rsa-sha256 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ffmpeg.org; i=@ffmpeg.org; q=dns/txt; s=mail; t=1771261819; h=mime-version : to : date : message-id : reply-to : subject : list-id : list-archive : list-archive : list-help : list-owner : list-post : list-subscribe : list-unsubscribe : from : cc : content-type : content-transfer-encoding : from; bh=rMn2QFZ40p9ZhKOZ6yqfs0TFaoiMX8L84ckNut1hjdc=; b=OGXgn1gK6honLUAe2TqF2gztDz7zOFmzki3c6UICI0ouvmSe4S64eXN5QtLbICL/PMifp 3b9I5650VX1FS84xeF2Nivnf9w8BvRqphCfKRNuE8OIxq6xlx7qNMN5dvy86iwlb2DDXSUI nyib09I3FCS4YSjsMV4uKJ0uOHGp1rby/AUzMfs5v6tYhzHtHerI4+HzNRtA5QhSi32U0GS 1TBlsCSnTRDwHVLVzmXRamyIja/WZ8/0CnUPpvBHUEilf9gncvFkd5vhIOO3zlc27nH7qo+ ri2RBCDz51DD4Tut+QmlZ8I8lOvyd4x24Kel7foMGhueuCs4YxkGc9EGMPEg== Received: from [172.20.0.3] (unknown [172.20.0.3]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTP id 460FE69153D; Mon, 16 Feb 2026 19:10:19 +0200 (EET) ARC-Seal: i=1; cv=none; a=rsa-sha256; d=ffmpeg.org; s=arc; t=1771261811; b=Ngz8B1dboig1PgYSG9vCS1nmGg8oJFChIJNE7Vxd/Wi0UPz10z/jO5s8rl8/hlW/JCf91 Rg7bcqkhC1JNMboOcgApAZXsz4ZMSsSSfHkFAuIyl0MoNz3D3OfpiRoTJ1GyjOkl7qQOyCm btomPWoh+lUQIshKA6W6DdwYn+lmWSp48tva6L36Gad8UsGA8Ul6zf0+IINhLn6mF9NmLyi iQJw6Y410pXVIDPpeIpmanQEmhLjrFwKTaQfhTni7Hm9XZ7TYqiGTHK4Xkq7OmkRnRnCghY I/dYfGHr5ydDnLuOFWR8rY/AWavi25FzS4x2acTMgK5D1vjP+IBGFGrmNYlQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=ffmpeg.org; s=arc; t=1771261811; h=from : sender : reply-to : subject : date : message-id : to : cc : mime-version : content-type : content-transfer-encoding : content-id : content-description : resent-date : resent-from : resent-sender : resent-to : resent-cc : resent-message-id : in-reply-to : references : list-id : list-help : list-unsubscribe : list-subscribe : list-post : list-owner : list-archive; bh=M1KQXzSAQu1bIhIBumzEu5put4MNStSYEht2stiPWnU=; b=avS3T2eCoA2hKCHL5TIB3bSPzDNGyMwRyuD6LE4iVPgEgX0pTSjqnL5kR61XfB9VcxjVf gTW/bDaDpK873TdG5i5SeiUyqro5xlUFf21mOqOI7YQ4Qaz4ehvWzjs+uP4q83ILY+IbHyo bHShmrZAGFvDgWZdSTgT39ZeEwz1GJUCH+zRoUmwpqITkCFRrOTteP4eSKeRsRoShjHHiM9 7S2QBiA2koMHlaUoHQj5pJI5Vi5mumvdw/vdiHqJ9IU3cHUg5h53eDuvNwwgvusNPdrEA4y ci8mud2P9K0kiRY5NX1IAu60+lw3y6Hdq5bF/fNCXwAEeqtbk2xMNzj8WpeQ== ARC-Authentication-Results: i=1; ffmpeg.org; dkim=pass header.d=ffmpeg.org header.i=@ffmpeg.org; arc=none; dmarc=pass header.from=ffmpeg.org policy.dmarc=quarantine Authentication-Results: ffmpeg.org; dkim=pass header.d=ffmpeg.org header.i=@ffmpeg.org; arc=none (Message is not ARC signed); dmarc=pass (Used From Domain Record) header.from=ffmpeg.org policy.dmarc=quarantine DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ffmpeg.org; i=@ffmpeg.org; q=dns/txt; s=mail; t=1771261805; h=content-type : mime-version : content-transfer-encoding : from : to : reply-to : subject : date : from; bh=w9+rBqB538DfRTi0BwFGkzTsxDof60Q9kIx/FFNTsqA=; b=vM9KREAq6750rwLRsX6jyRo4TZp7yEGGVwVyotc03nS6MlpXQIzx7Nkk7x1BmYd49eciA ATnv2tJ0kYwBRk6uW70T7kAG8+utiBs4TCpVIhCHmBgxsiiOBuYcxjb8WVtcz9DEg/lvYgi ooJ1D6LzRrUf4M6U+x9vqJ9aXM+hiAadoKcESgHIied7ts1EA14lsyBDafSSSFNhxLJekWr aycNhozStKhhJy//8/fSDbRFvNAM5tvYij+b/y2wWpOPIPV6C0ED9pqK7eXbSCUiErE8LmA EY3JnUiSxfmxxBdp1x67O+dhXLvqJ788ii2vZgF0mjSiGhLMXxzFpCN9I1qg== Received: from c8d966988b92 (code.ffmpeg.org [188.245.149.3]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTPS id 2BB6769151E for ; Mon, 16 Feb 2026 19:10:05 +0200 (EET) MIME-Version: 1.0 To: ffmpeg-devel@ffmpeg.org Date: Mon, 16 Feb 2026 17:10:04 -0000 Message-ID: <177126180526.25.18031864243202100929@009cbcb3d8cd> Message-ID-Hash: C7YDJQOTSNVHANI2CQMO7RZB3G33MQVC X-Message-ID-Hash: C7YDJQOTSNVHANI2CQMO7RZB3G33MQVC X-MailFrom: code@ffmpeg.org X-Mailman-Rule-Hits: nonmember-moderation X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; loop; banned-address; header-match-ffmpeg-devel.ffmpeg.org-0; header-match-ffmpeg-devel.ffmpeg.org-1; header-match-ffmpeg-devel.ffmpeg.org-2; header-match-ffmpeg-devel.ffmpeg.org-3; emergency; member-moderation X-Mailman-Version: 3.3.10 Precedence: list Reply-To: FFmpeg development discussions and patches Subject: [FFmpeg-devel] [PR] fftools/ffmpeg_sched: make trailing_dts() report max_dts instead of min_dts (PR #21770) List-Id: FFmpeg development discussions and patches Archived-At: Archived-At: List-Archive: List-Archive: List-Help: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: From: ngaullier via ffmpeg-devel Cc: ngaullier Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Archived-At: List-Archive: List-Post: PR #21770 opened by ngaullier URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/21770 Patch URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/21770.patch This is to reapply 18217bb0f5fb4ad9d93ea02edab078111cd83910. Its commit msg is still meaningful: "Using the max instead of the min avoids the progress stopping with gaps in sparse streams (subtitles)." Unfortunately, the comment "compute min output value" was left unchanged in this commit and this was pretty misleading... So, years later, the 'max_dts code' code was restored in 9b8cc36ce0b2417469d78c68780c8796886c202e and caused the regression when enabled by the following commit: d119ae2fd82a494d9430ff4d4fc262961a68c598. Also ignore streams with no data (AV_NOPTS_VALUE): when processing files having a subtitle track but no data at all, this fixes ffmpeg reporting N/A for both time and speed (also a regression). Signed-off-by: Nicolas Gaullier >>From fe6d98d8d62a2d1ab50fd26f782ac8979e695c17 Mon Sep 17 00:00:00 2001 From: Nicolas Gaullier Date: Mon, 16 Feb 2026 16:43:51 +0100 Subject: [PATCH] fftools/ffmpeg_sched: make trailing_dts() report max_dts instead of min_dts This is to reapply 18217bb0f5fb4ad9d93ea02edab078111cd83910. Its commit msg is still meaningful: "Using the max instead of the min avoids the progress stopping with gaps in sparse streams (subtitles)." Unfortunately, the comment "compute min output value" was left unchanged in this commit and this was pretty misleading... So, years later, the 'max_dts code' code was restored in 9b8cc36ce0b2417469d78c68780c8796886c202e and caused the regression when enabled by the following commit: d119ae2fd82a494d9430ff4d4fc262961a68c598. Also ignore streams with no data (AV_NOPTS_VALUE): when processing files having a subtitle track but no data at all, this fixes ffmpeg reporting N/A for both time and speed (also a regression). Signed-off-by: Nicolas Gaullier --- fftools/ffmpeg_sched.c | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/fftools/ffmpeg_sched.c b/fftools/ffmpeg_sched.c index 517ff1ea85..12e6133667 100644 --- a/fftools/ffmpeg_sched.c +++ b/fftools/ffmpeg_sched.c @@ -437,7 +437,7 @@ static void task_init(Scheduler *sch, SchTask *task, enum SchedulerNodeType type static int64_t trailing_dts(const Scheduler *sch, int count_finished) { - int64_t min_dts = INT64_MAX; + int64_t max_dts = INT64_MIN; for (unsigned i = 0; i < sch->nb_mux; i++) { const SchMux *mux = &sch->mux[i]; @@ -447,14 +447,12 @@ static int64_t trailing_dts(const Scheduler *sch, int count_finished) if (ms->source_finished && !count_finished) continue; - if (ms->last_dts == AV_NOPTS_VALUE) - return AV_NOPTS_VALUE; - - min_dts = FFMIN(min_dts, ms->last_dts); + if (ms->last_dts != AV_NOPTS_VALUE) + max_dts = FFMAX(max_dts, ms->last_dts); } } - return min_dts == INT64_MAX ? AV_NOPTS_VALUE : min_dts; + return max_dts == INT64_MIN ? AV_NOPTS_VALUE : max_dts; } void sch_remove_filtergraph(Scheduler *sch, int idx) -- 2.52.0 _______________________________________________ ffmpeg-devel mailing list -- ffmpeg-devel@ffmpeg.org To unsubscribe send an email to ffmpeg-devel-leave@ffmpeg.org