Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 57ef007e authored by Daeho Jeong's avatar Daeho Jeong
Browse files

Make minimum gc sleep time tunnable 



Bug: 241601436
Test: check smart idle maint log of StorageManagerService
Signed-off-by: default avatarDaeho Jeong <daehojeong@google.com>
Change-Id: I004bb6752e84d26b041cfdd27848949f6e49d05e
Merged-In: I004bb6752e84d26b041cfdd27848949f6e49d05e
parent 38c28f3b

services/core/java/com/android/server/StorageManagerService.java

+9 −2
Original line number Diff line number Diff line
@@ -370,6 +370,8 @@ class StorageManagerService extends IStorageManager.Stub 
    private static final float DEFAULT_LOW_BATTERY_LEVEL = 20F;

    // Decide whether charging is required to turn on the feature

    private static final boolean DEFAULT_CHARGING_REQUIRED = true;

    // Minimum GC interval sleep time in ms

    private static final int DEFAULT_MIN_GC_SLEEPTIME = 10000;



    private volatile int mLifetimePercentThreshold;

    private volatile int mMinSegmentsThreshold;
@@ -377,6 +379,7 @@ class StorageManagerService extends IStorageManager.Stub 
    private volatile float mSegmentReclaimWeight;

    private volatile float mLowBatteryLevel;

    private volatile boolean mChargingRequired;

    private volatile int mMinGCSleepTime;

    private volatile boolean mNeedGC;



    private volatile boolean mPassedLifetimeThresh;
@@ -2712,6 +2715,8 @@ class StorageManagerService extends IStorageManager.Stub 
                "low_battery_level", DEFAULT_LOW_BATTERY_LEVEL);

            mChargingRequired = DeviceConfig.getBoolean(DeviceConfig.NAMESPACE_STORAGE_NATIVE_BOOT,

                "charging_required", DEFAULT_CHARGING_REQUIRED);

            mMinGCSleepTime = DeviceConfig.getInt(DeviceConfig.NAMESPACE_STORAGE_NATIVE_BOOT,

                "min_gc_sleeptime", DEFAULT_MIN_GC_SLEEPTIME);



            // If we use the smart idle maintenance, we need to turn off GC in the traditional idle

            // maintenance to avoid the conflict
@@ -2851,9 +2856,11 @@ class StorageManagerService extends IStorageManager.Stub 
                            ", min segment threshold: " + mMinSegmentsThreshold +

                            ", dirty reclaim rate: " + mDirtyReclaimRate +

                            ", segment reclaim weight: " + mSegmentReclaimWeight +

                            ", period: " + sSmartIdleMaintPeriod);

                            ", period(min): " + sSmartIdleMaintPeriod +

                            ", min gc sleep time(ms): " + mMinGCSleepTime);

                mVold.setGCUrgentPace(avgWriteAmount, mMinSegmentsThreshold, mDirtyReclaimRate,

                                      mSegmentReclaimWeight, sSmartIdleMaintPeriod);

                                      mSegmentReclaimWeight, sSmartIdleMaintPeriod,

                                      mMinGCSleepTime);

            } else {

                Slog.i(TAG, "Skipping smart idle maintenance - block based checkpoint in progress");

            }