diff options
| author | Jan Ekström <jeebjp@gmail.com> | 2023-03-09 20:27:06 +0200 |
|---|---|---|
| committer | Jan Ekström <jeebjp@gmail.com> | 2023-03-14 00:14:52 +0200 |
| commit | fba9d9609fe00c8c4d04b76f9ecdd274aad318dd (patch) | |
| tree | f2967bc22be86aadd11f4c96f23a34728971baf2 /libavcodec | |
| parent | 1231003c3c6d4839a9e838d06f8e16ee7690958f (diff) | |
| download | ffmpeg-fba9d9609fe00c8c4d04b76f9ecdd274aad318dd.tar.gz | |
avcodec/libsvtav1: use larger of bit rate and max rate for buffer size
Generally if maxrate is set, the calculation should be maxrate over
bufsize. This additionally enables CRF + maxrate & bufsize usage.
In order to keep negative values from enabling zero to be treated
as larger and causing a division by zero, check that one of the
variables is larger than zero.
Diffstat (limited to 'libavcodec')
| -rw-r--r-- | libavcodec/libsvtav1.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/libavcodec/libsvtav1.c b/libavcodec/libsvtav1.c index 56e1e22b51..9174e2753c 100644 --- a/libavcodec/libsvtav1.c +++ b/libavcodec/libsvtav1.c @@ -184,8 +184,10 @@ static int config_enc_params(EbSvtAv1EncConfiguration *param, param->min_qp_allowed = avctx->qmin; } param->max_bit_rate = avctx->rc_max_rate; - if (avctx->bit_rate && avctx->rc_buffer_size) - param->maximum_buffer_size_ms = avctx->rc_buffer_size * 1000LL / avctx->bit_rate; + if ((avctx->bit_rate > 0 || avctx->rc_max_rate > 0) && avctx->rc_buffer_size) + param->maximum_buffer_size_ms = + avctx->rc_buffer_size * 1000LL / + FFMAX(avctx->bit_rate, avctx->rc_max_rate); if (svt_enc->crf > 0) { param->qp = svt_enc->crf; @@ -302,7 +304,8 @@ static int config_enc_params(EbSvtAv1EncConfiguration *param, avctx->bit_rate = param->rate_control_mode > 0 ? param->target_bit_rate : 0; avctx->rc_max_rate = param->max_bit_rate; - avctx->rc_buffer_size = param->maximum_buffer_size_ms * avctx->bit_rate / 1000LL; + avctx->rc_buffer_size = param->maximum_buffer_size_ms * + FFMAX(avctx->bit_rate, avctx->rc_max_rate) / 1000LL; if (avctx->bit_rate || avctx->rc_max_rate || avctx->rc_buffer_size) { AVCPBProperties *cpb_props = ff_add_cpb_side_data(avctx); |