[FFmpeg-devel] [PATCH] avcodec/rkmppdec: Don't fill the decoder in low-delay mode (PR #20518)

[FFmpeg-devel] [PATCH] avcodec/rkmppdec: Don't fill the decoder in low-delay mode (PR #20518)

[cgutman via ffmpeg-devel]

PR #20518 opened by cgutman
URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20518
Patch URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20518.patch

For low delay usecases, we should not trade latency for throughput. This addresses my comment in https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20511#issuecomment-6821


>From 1c4dbb424307a0f411e24e6fa980010340e0b614 Mon Sep 17 00:00:00 2001
From: Cameron Gutman <aicommander@gmail.com>
Date: Wed, 3 Sep 2025 20:35:21 -0500
Subject: [PATCH] avcodec/rkmppdec: Don't fill the decoder in low-delay mode

For low delay usecases, we should not trade latency for throughput.

Signed-off-by: Cameron Gutman <aicommander@gmail.com>
---
 libavcodec/rkmppdec.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/libavcodec/rkmppdec.c b/libavcodec/rkmppdec.c
index a6bc565776..a5c1397be5 100644
--- a/libavcodec/rkmppdec.c
+++ b/libavcodec/rkmppdec.c
@@ -518,8 +518,8 @@ static int rkmpp_receive_frame(AVCodecContext *avctx, AVFrame *frame)
             }
         }
 
-        // make sure we keep decoder full
-        if (freeslots > 1 && !decoder->eos_reached)
+        // make sure we keep decoder full unless in low-delay mode
+        if (freeslots > 1 && !decoder->eos_reached && !(avctx->flags & AV_CODEC_FLAG_LOW_DELAY))
             return AVERROR(EAGAIN);
     }
 
-- 
2.49.1

_______________________________________________
ffmpeg-devel mailing list -- ffmpeg-devel@ffmpeg.org
To unsubscribe send an email to ffmpeg-devel-leave@ffmpeg.org