diff options
author | TreeHugger Robot <treehugger-gerrit@google.com> | 2019-09-20 22:39:47 +0000 |
---|---|---|
committer | Android (Google) Code Review <android-gerrit@google.com> | 2019-09-20 22:39:47 +0000 |
commit | fcf2c201ad8d353f6268a6c6f0363c7751f34a7c (patch) | |
tree | 5f4e6e1e97a162958415a0a931f223b3660fff92 | |
parent | 8f8c3ef0ef4112f1f5957ae9204936d8a2167f2c (diff) | |
parent | 0d772e2642ede5ae36542de4eecfe87b6fe21dad (diff) | |
download | native-fcf2c201ad8d353f6268a6c6f0363c7751f34a7c.tar.gz |
Merge "SurfaceFlinger: tune number of buffers for low activity layers" into qt-qpr1-dev
-rw-r--r-- | services/surfaceflinger/Scheduler/LayerInfo.h | 5 | ||||
-rw-r--r-- | services/surfaceflinger/Scheduler/SchedulerUtils.h | 6 |
2 files changed, 7 insertions, 4 deletions
diff --git a/services/surfaceflinger/Scheduler/LayerInfo.h b/services/surfaceflinger/Scheduler/LayerInfo.h index a7337817e3..17afddac28 100644 --- a/services/surfaceflinger/Scheduler/LayerInfo.h +++ b/services/surfaceflinger/Scheduler/LayerInfo.h @@ -109,7 +109,7 @@ class LayerInfo { bool isLowActivityLayer() const { // We want to make sure that we received more than two frames from the layer // in order to check low activity. - if (mElements.size() < 2) { + if (mElements.size() < scheduler::LOW_ACTIVITY_BUFFERS + 1) { return false; } @@ -118,7 +118,8 @@ class LayerInfo { // Check the frame before last to determine whether there is low activity. // If that frame is older than LOW_ACTIVITY_EPSILON_NS, the layer is sending // infrequent updates. - if (mElements.at(mElements.size() - 2) < obsoleteEpsilon) { + if (mElements.at(mElements.size() - (scheduler::LOW_ACTIVITY_BUFFERS + 1)) < + obsoleteEpsilon) { return true; } diff --git a/services/surfaceflinger/Scheduler/SchedulerUtils.h b/services/surfaceflinger/Scheduler/SchedulerUtils.h index ced1899109..ac10f83ad9 100644 --- a/services/surfaceflinger/Scheduler/SchedulerUtils.h +++ b/services/surfaceflinger/Scheduler/SchedulerUtils.h @@ -42,9 +42,11 @@ static constexpr uint32_t HWC2_SCREEN_OFF_CONFIG_ID = 0xffffffff; // or waiting idle in messaging app, when cursor is blinking. static constexpr std::chrono::nanoseconds OBSOLETE_TIME_EPSILON_NS = 1200ms; -// Layer is considered low activity if the buffers come more than LOW_ACTIVITY_EPSILON_NS -// apart. This is helping SF to vote for lower refresh rates when there is not activity +// Layer is considered low activity if the LOW_ACTIVITY_BUFFERS buffers come more than +// LOW_ACTIVITY_EPSILON_NS apart. +// This is helping SF to vote for lower refresh rates when there is not activity // in screen. +static constexpr int LOW_ACTIVITY_BUFFERS = 2; static constexpr std::chrono::nanoseconds LOW_ACTIVITY_EPSILON_NS = 250ms; // Calculates the statistical mean (average) in the data structure (array, vector). The |