commit 3fd33ece353da336aaab92298929192b78e108b8
author Daeho Jeong <daehojeong@google.com> Thu Jun 23 16:19:58 2022 -0700
committer Daeho Jeong <daehojeong@google.com> Thu Jun 23 16:20:45 2022 -0700
tree 5b272b26829a126135ff58781590dba300234b53
parent d99898496f4652a1d6a3dc3761b85016fda3bfff

Make minimum gc sleep time tunnable

Test: check smart idle maint log of StorageManagerService
Signed-off-by: Daeho Jeong <daehojeong@google.com>
Change-Id: I5a70e4ec2ca895551b6446a9dfd4bb5003a3fbd0

IdleMaint.cpp

diff --git a/IdleMaint.cpp b/IdleMaint.cpp
index 426be58..b1565aa 100644
--- a/IdleMaint.cpp
+++ b/IdleMaint.cpp
@@ -86,7 +86,6 @@
 static const int GC_TIMEOUT_SEC = 420;
 static const int DEVGC_TIMEOUT_SEC = 120;
 static const int KBYTES_IN_SEGMENT = 2048;
-static const int MIN_GC_URGENT_SLEEP_TIME = 500;
 static const int ONE_MINUTE_IN_MS = 60000;
 static const int GC_NORMAL_MODE = 0;
 static const int GC_URGENT_MID_MODE = 3;
@@ -531,7 +530,7 @@
 }
 
 void SetGCUrgentPace(int32_t neededSegments, int32_t minSegmentThreshold, float dirtyReclaimRate,
-                     float reclaimWeight, int32_t gcPeriod) {
+                     float reclaimWeight, int32_t gcPeriod, int32_t minGCSleepTime) {
     std::list<std::string> paths;
     bool needGC = true;
     int32_t sleepTime;
@@ -593,8 +592,8 @@
             needGC = false;
         } else {
             sleepTime = gcPeriod * ONE_MINUTE_IN_MS / neededSegments;
-            if (sleepTime < MIN_GC_URGENT_SLEEP_TIME) {
-                sleepTime = MIN_GC_URGENT_SLEEP_TIME;
+            if (sleepTime < minGCSleepTime) {
+                sleepTime = minGCSleepTime;
             }
         }
     }