* [FFmpeg-devel] [PATCH] avcodec/mediacodecenc: set quality in cq mode @ 2023-12-17 12:50 Zhao Zhili 2023-12-19 10:42 ` Matthieu Bouron 0 siblings, 1 reply; 3+ messages in thread From: Zhao Zhili @ 2023-12-17 12:50 UTC (permalink / raw) To: ffmpeg-devel; +Cc: Zhao Zhili From: Zhao Zhili <zhilizhao@tencent.com> From AOSP doc, these values are device and codec specific, but lower values generally result in more efficient (smaller-sized) encoding. For example, global_quality 50 on Pixel 6 results a 1080P 30 FPS HEVC with 3744 kb/s, while global_quality 80 results 28178 kb/s. Fix #10689 Signed-off-by: Zhao Zhili <zhilizhao@tencent.com> --- libavcodec/mediacodecenc.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/libavcodec/mediacodecenc.c b/libavcodec/mediacodecenc.c index a1ae5d5ec5..7eae542248 100644 --- a/libavcodec/mediacodecenc.c +++ b/libavcodec/mediacodecenc.c @@ -268,8 +268,11 @@ static av_cold int mediacodec_init(AVCodecContext *avctx) if (avctx->bit_rate) ff_AMediaFormat_setInt32(format, "bitrate", avctx->bit_rate); - if (s->bitrate_mode >= 0) + if (s->bitrate_mode >= 0) { ff_AMediaFormat_setInt32(format, "bitrate-mode", s->bitrate_mode); + if (s->bitrate_mode == BITRATE_MODE_CQ && avctx->global_quality > 0) + ff_AMediaFormat_setInt32(format, "quality", avctx->global_quality); + } // frame-rate and i-frame-interval are required to configure codec if (avctx->framerate.num >= avctx->framerate.den && avctx->framerate.den > 0) { s->fps = avctx->framerate.num / avctx->framerate.den; -- 2.34.1 _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe". ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [FFmpeg-devel] [PATCH] avcodec/mediacodecenc: set quality in cq mode 2023-12-17 12:50 [FFmpeg-devel] [PATCH] avcodec/mediacodecenc: set quality in cq mode Zhao Zhili @ 2023-12-19 10:42 ` Matthieu Bouron 2023-12-19 14:16 ` Zhao Zhili 0 siblings, 1 reply; 3+ messages in thread From: Matthieu Bouron @ 2023-12-19 10:42 UTC (permalink / raw) To: FFmpeg development discussions and patches; +Cc: Zhao Zhili On Sun, Dec 17, 2023 at 1:51 PM Zhao Zhili <quinkblack@foxmail.com> wrote: > > From: Zhao Zhili <zhilizhao@tencent.com> > > From AOSP doc, these values are device and codec specific, but lower > values generally result in more efficient (smaller-sized) encoding. > > For example, global_quality 50 on Pixel 6 results a 1080P 30 FPS > HEVC with 3744 kb/s, while global_quality 80 results 28178 kb/s. > > Fix #10689 > > Signed-off-by: Zhao Zhili <zhilizhao@tencent.com> > --- > libavcodec/mediacodecenc.c | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/libavcodec/mediacodecenc.c b/libavcodec/mediacodecenc.c > index a1ae5d5ec5..7eae542248 100644 > --- a/libavcodec/mediacodecenc.c > +++ b/libavcodec/mediacodecenc.c > @@ -268,8 +268,11 @@ static av_cold int mediacodec_init(AVCodecContext *avctx) > > if (avctx->bit_rate) > ff_AMediaFormat_setInt32(format, "bitrate", avctx->bit_rate); > - if (s->bitrate_mode >= 0) > + if (s->bitrate_mode >= 0) { > ff_AMediaFormat_setInt32(format, "bitrate-mode", s->bitrate_mode); > + if (s->bitrate_mode == BITRATE_MODE_CQ && avctx->global_quality > 0) > + ff_AMediaFormat_setInt32(format, "quality", avctx->global_quality); > + } > // frame-rate and i-frame-interval are required to configure codec > if (avctx->framerate.num >= avctx->framerate.den && avctx->framerate.den > 0) { > s->fps = avctx->framerate.num / avctx->framerate.den; > -- > 2.34.1 > Does the `global_quality` value need to be checked/clamped against `MediaCodecInfo.EncoderCapabilities.getQualityRange()` before use ? Otherwise, LGTM. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe". ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [FFmpeg-devel] [PATCH] avcodec/mediacodecenc: set quality in cq mode 2023-12-19 10:42 ` Matthieu Bouron @ 2023-12-19 14:16 ` Zhao Zhili 0 siblings, 0 replies; 3+ messages in thread From: Zhao Zhili @ 2023-12-19 14:16 UTC (permalink / raw) To: ffmpeg-devel On 2023/12/19 18:42, Matthieu Bouron wrote: > On Sun, Dec 17, 2023 at 1:51 PM Zhao Zhili <quinkblack@foxmail.com> wrote: >> From: Zhao Zhili <zhilizhao@tencent.com> >> >> From AOSP doc, these values are device and codec specific, but lower >> values generally result in more efficient (smaller-sized) encoding. >> >> For example, global_quality 50 on Pixel 6 results a 1080P 30 FPS >> HEVC with 3744 kb/s, while global_quality 80 results 28178 kb/s. >> >> Fix #10689 >> >> Signed-off-by: Zhao Zhili <zhilizhao@tencent.com> >> --- >> libavcodec/mediacodecenc.c | 5 ++++- >> 1 file changed, 4 insertions(+), 1 deletion(-) >> >> diff --git a/libavcodec/mediacodecenc.c b/libavcodec/mediacodecenc.c >> index a1ae5d5ec5..7eae542248 100644 >> --- a/libavcodec/mediacodecenc.c >> +++ b/libavcodec/mediacodecenc.c >> @@ -268,8 +268,11 @@ static av_cold int mediacodec_init(AVCodecContext *avctx) >> >> if (avctx->bit_rate) >> ff_AMediaFormat_setInt32(format, "bitrate", avctx->bit_rate); >> - if (s->bitrate_mode >= 0) >> + if (s->bitrate_mode >= 0) { >> ff_AMediaFormat_setInt32(format, "bitrate-mode", s->bitrate_mode); >> + if (s->bitrate_mode == BITRATE_MODE_CQ && avctx->global_quality > 0) >> + ff_AMediaFormat_setInt32(format, "quality", avctx->global_quality); >> + } >> // frame-rate and i-frame-interval are required to configure codec >> if (avctx->framerate.num >= avctx->framerate.den && avctx->framerate.den > 0) { >> s->fps = avctx->framerate.num / avctx->framerate.den; >> -- >> 2.34.1 >> > Does the `global_quality` value need to be checked/clamped against > `MediaCodecInfo.EncoderCapabilities.getQualityRange()` before use ? Thanks for the review. Firstly, since 'quality' is device and codec specific, not only valid range but also the relationship between 'quality' value and the real encoding quality (and bitrate), some prerequisite knowledge(or trail and error) is required. With prerequisite knowledge, check valid range is unnecessary. Secondly, from my test the device can do clamp automatically. For example, on pixel 6, set H.265 quality to 100 or 1000 get the same results (I think it's lossless with very high bitrate, which should never be used except in rare case). Last but not the least, there is no NDK API for MediaCodecInfo. I want to avoid complex JNI calls unless there are a lot of gains. > Otherwise, LGTM. > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe". _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe". ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-12-19 14:16 UTC | newest] Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2023-12-17 12:50 [FFmpeg-devel] [PATCH] avcodec/mediacodecenc: set quality in cq mode Zhao Zhili 2023-12-19 10:42 ` Matthieu Bouron 2023-12-19 14:16 ` Zhao Zhili
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