From 90186aa1f3e6c79a210618014523016bd241c248 Mon Sep 17 00:00:00 2001 From: wenchangliu Date: Tue, 9 Jan 2024 08:18:03 +0000 Subject: [PATCH 1/3] Update media_codecs_performance_c2.xml for meet device performance Fine tune the performance table since the change in CPU policy for power saving cause some encoder performance results from the CTS are not able to meet pass criteria from the original performance measurement results. Bug: 317441297 Test: run cts -m CtsVideoTestCases -t android.video.cts.VideoEncoderDecoderTest Test: run cts -m CtsVideoTestCases -t android.video.cts.CodecDecoderPerformanceTest Change-Id: Ib75d3f35c5ea2e2dc80ba192d738d54140a09d68 --- media_codecs_performance_c2.xml | 60 ++++++++++++++++----------------- 1 file changed, 30 insertions(+), 30 deletions(-) diff --git a/media_codecs_performance_c2.xml b/media_codecs_performance_c2.xml index 4dcb0b4b..146a3171 100644 --- a/media_codecs_performance_c2.xml +++ b/media_codecs_performance_c2.xml @@ -193,40 +193,40 @@ - - - - - - + + + + + + - - - - - - + + + + + + - - - - - - - - + + + + + + + + - - - - - - - - + + + + + + + + @@ -280,8 +280,8 @@ - - + + From a90a2e4a6ec3807ccb303be26a8b10ac5f8c505f Mon Sep 17 00:00:00 2001 From: wenchangliu Date: Tue, 9 Jan 2024 08:18:03 +0000 Subject: [PATCH 2/3] Update media_codecs_performance_c2.xml for meet device performance Fine tune the performance table since the change in CPU policy for power saving cause some encoder performance results from the CTS are not able to meet pass criteria from the original performance measurement results. Bug: 317441297 Test: run cts -m CtsVideoTestCases -t android.video.cts.VideoEncoderDecoderTest Test: run cts -m CtsVideoTestCases -t android.video.cts.CodecDecoderPerformanceTest Change-Id: Ib75d3f35c5ea2e2dc80ba192d738d54140a09d68 --- media_codecs_performance_c2.xml | 60 ++++++++++++++++----------------- 1 file changed, 30 insertions(+), 30 deletions(-) diff --git a/media_codecs_performance_c2.xml b/media_codecs_performance_c2.xml index 924b1c3b..db8c8380 100644 --- a/media_codecs_performance_c2.xml +++ b/media_codecs_performance_c2.xml @@ -187,40 +187,40 @@ - - - - - - + + + + + + - - - - - - + + + + + + - - - - - - - - + + + + + + + + - - - - - - - - + + + + + + + + @@ -274,8 +274,8 @@ - - + + From 9164eae7cdee032381d8bf045fbbed33f62b00bf Mon Sep 17 00:00:00 2001 From: Roy Luo Date: Thu, 4 Jan 2024 01:15:16 +0000 Subject: [PATCH 3/3] usb: dump flag value in the constructor Dump flag value in the construtor for easier debugging. Feature flags are not expected to change in runtime, dumping the value once should be enough. Bug: 296119135 Test: manual test Change-Id: Ie5aca2dff23e59704fb306833cde0e592b9f430d --- usb/usb/Usb.cpp | 5 +++++ usb/usb/UsbDataSessionMonitor.cpp | 3 +++ 2 files changed, 8 insertions(+) diff --git a/usb/usb/Usb.cpp b/usb/usb/Usb.cpp index c0813d9e..a98eca7b 100644 --- a/usb/usb/Usb.cpp +++ b/usb/usb/Usb.cpp @@ -573,6 +573,11 @@ Usb::Usb() ALOGE("mDisplayPortActivateTimer timerfd failed: %s", strerror(errno)); abort(); } + + ALOGI("feature flag enable_usb_data_compliance_warning: %d", + usb_flags::enable_usb_data_compliance_warning()); + ALOGI("feature flag enable_input_power_limited_warning: %d", + usb_flags::enable_input_power_limited_warning()); } ScopedAStatus Usb::switchRole(const string& in_portName, const PortRole& in_role, diff --git a/usb/usb/UsbDataSessionMonitor.cpp b/usb/usb/UsbDataSessionMonitor.cpp index bae261d7..bff52f06 100644 --- a/usb/usb/UsbDataSessionMonitor.cpp +++ b/usb/usb/UsbDataSessionMonitor.cpp @@ -164,6 +164,9 @@ UsbDataSessionMonitor::UsbDataSessionMonitor( ALOGE("pthread creation failed %d", errno); abort(); } + + ALOGI("feature flag enable_report_usb_data_compliance_warning: %d", + usb_flags::enable_report_usb_data_compliance_warning()); } UsbDataSessionMonitor::~UsbDataSessionMonitor() {}