diff options
| author | 2025-03-05 08:35:07 -0800 | |
|---|---|---|
| committer | 2025-03-05 08:35:07 -0800 | |
| commit | ccfdcd82b086e489d8273cb5cd6d3a2bf79be2ee (patch) | |
| tree | 35ad609684ba6e7a10aa34f7bf6e882990fc829d /services/surfaceflinger/Scheduler | |
| parent | 5155aa6251094b903999b1dfb45a702318f1d8c1 (diff) | |
| parent | 4418a39cb42cf20ea0e3b0f6f93279ab3b5fe07d (diff) | |
Merge changes I6f68b884,I870d8f13 into main
* changes:
SF: Clean up helpers for thread priority
SF: Remove connected_display flag
Diffstat (limited to 'services/surfaceflinger/Scheduler')
| -rw-r--r-- | services/surfaceflinger/Scheduler/Scheduler.cpp | 4 | ||||
| -rw-r--r-- | services/surfaceflinger/Scheduler/Scheduler.h | 2 |
2 files changed, 3 insertions, 3 deletions
diff --git a/services/surfaceflinger/Scheduler/Scheduler.cpp b/services/surfaceflinger/Scheduler/Scheduler.cpp index 4da76f6ecc..e587178ec2 100644 --- a/services/surfaceflinger/Scheduler/Scheduler.cpp +++ b/services/surfaceflinger/Scheduler/Scheduler.cpp @@ -986,7 +986,7 @@ std::shared_ptr<VsyncSchedule> Scheduler::promotePacesetterDisplayLocked( if (const auto pacesetterOpt = pacesetterDisplayLocked()) { const Display& pacesetter = *pacesetterOpt; - if (!FlagManager::getInstance().connected_display() || params.toggleIdleTimer) { + if (params.toggleIdleTimer) { pacesetter.selectorPtr->setIdleTimerCallbacks( {.platform = {.onReset = [this] { idleTimerCallback(TimerState::Reset); }, .onExpired = [this] { idleTimerCallback(TimerState::Expired); }}, @@ -1018,7 +1018,7 @@ void Scheduler::applyNewVsyncSchedule(std::shared_ptr<VsyncSchedule> vsyncSchedu } void Scheduler::demotePacesetterDisplay(PromotionParams params) { - if (!FlagManager::getInstance().connected_display() || params.toggleIdleTimer) { + if (params.toggleIdleTimer) { // No need to lock for reads on kMainThreadContext. if (const auto pacesetterPtr = FTL_FAKE_GUARD(mDisplayLock, pacesetterSelectorPtrLocked())) { diff --git a/services/surfaceflinger/Scheduler/Scheduler.h b/services/surfaceflinger/Scheduler/Scheduler.h index 3fdddac52a..81389e7362 100644 --- a/services/surfaceflinger/Scheduler/Scheduler.h +++ b/services/surfaceflinger/Scheduler/Scheduler.h @@ -386,7 +386,7 @@ private: // a deadlock where the main thread joins with the timer thread as the timer thread waits to // lock a mutex held by the main thread. struct PromotionParams { - // Whether to stop and start the idle timer. Ignored unless connected_display flag is set. + // Whether to stop and start the idle timer. bool toggleIdleTimer; }; |