Skip to content

Commit bdd2039

Browse files
Treehugger RobotAndroid (Google) Code Review
authored andcommitted
Merge "Revert "Resync on the transaction."" into main
2 parents 07b0a37 + 529a0a7 commit bdd2039

2 files changed

Lines changed: 1 addition & 2 deletions

File tree

services/surfaceflinger/Scheduler/Scheduler.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -209,7 +209,6 @@ class Scheduler : public IEventThreadCallback, android::impl::MessageQueue {
209209
ftl::FakeGuard guard(kMainThreadContext);
210210
resyncToHardwareVsyncLocked(id, allowToEnable, modePtr);
211211
}
212-
void resync() override EXCLUDES(mDisplayLock);
213212
void forceNextResync() { mLastResyncTime = 0; }
214213

215214
// Passes a vsync sample to VsyncController. Returns true if
@@ -471,6 +470,7 @@ class Scheduler : public IEventThreadCallback, android::impl::MessageQueue {
471470
bool throttleVsync(TimePoint, uid_t) override;
472471
// Get frame interval
473472
Period getVsyncPeriod(uid_t) override EXCLUDES(mDisplayLock);
473+
void resync() override EXCLUDES(mDisplayLock);
474474
void onExpectedPresentTimePosted(TimePoint expectedPresentTime) override EXCLUDES(mDisplayLock);
475475

476476
std::unique_ptr<EventThread> mRenderEventThread;

services/surfaceflinger/SurfaceFlinger.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4605,7 +4605,6 @@ void SurfaceFlinger::setTransactionFlags(uint32_t mask, TransactionSchedule sche
46054605
SFTRACE_INT("mTransactionFlags", transactionFlags);
46064606

46074607
if (const bool scheduled = transactionFlags & mask; !scheduled) {
4608-
mScheduler->resync();
46094608
scheduleCommit(frameHint);
46104609
} else if (frameHint == FrameHint::kActive) {
46114610
// Even if the next frame is already scheduled, we should reset the idle timer

0 commit comments

Comments
 (0)