From 001a09bd31df7b48095344ea657165a1d420c441 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bart=C5=82omiej=20Grzesik?= <bag@semihalf.com>
Date: Wed, 29 Jun 2022 09:51:27 +0000
Subject: [PATCH] V4L2EncodeComponent: allow user to disable VBR
This patch adds a Android system property persist.vendor.v4l2_codec2.disable_vbr.
The property allows user to disable option to encode with variable bitrate
in runtime.
This change is required for a temporary workaround for b/235771157.
Bug: 235771157, 238835187
Test: android.video.cts.VideoEncoderDecoderTest#testVp8Other0Perf0640x0360
Change-Id: I4a9931431354193968f8255d0d501aed02ee3508
(cherry picked from commit f9f4b5681ff4fc22cd83192b17693d90922134c7)
---
components/V4L2EncodeComponent.cpp | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/components/V4L2EncodeComponent.cpp b/components/V4L2EncodeComponent.cpp
index c2a2679..f9d735d 100644
--- a/components/V4L2EncodeComponent.cpp
+++ b/components/V4L2EncodeComponent.cpp
@@ -652,6 +652,12 @@ bool V4L2EncodeComponent::initializeEncoder() {
// Get the requested bitrate mode and bitrate. The C2 framework doesn't offer a parameter to
// configure the peak bitrate, so we use a multiple of the target bitrate.
mBitrateMode = mInterface->getBitrateMode();
+ if (property_get_bool("persist.vendor.v4l2_codec2.disable_vbr", false)) {
+ // NOTE: This is a workaround for b/235771157.
+ ALOGW("VBR is disabled on this device");
+ mBitrateMode = C2Config::BITRATE_CONST;
+ }
+
mBitrate = mInterface->getBitrate();
mEncoder = V4L2Encoder::create(
--
GitLab