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

Commit 853e2f90 authored by Android Build Coastguard Worker's avatar Android Build Coastguard Worker
Browse files

Snap for 7566880 from 5ede32f2 to sc-release 

Change-Id: I6d6305154bd23f7a3b2ea0af4b7bb3a7629369b8
parents 15ec18d0 5ede32f2

services/surfaceflinger/Scheduler/VSyncDispatchTimerQueue.cpp

+5 −2
Original line number Diff line number Diff line
@@ -84,10 +84,13 @@ ScheduleResult VSyncDispatchTimerQueueEntry::schedule(VSyncDispatch::ScheduleTim 
                                                      VSyncTracker& tracker, nsecs_t now) {

    auto nextVsyncTime = tracker.nextAnticipatedVSyncTimeFrom(

            std::max(timing.earliestVsync, now + timing.workDuration + timing.readyDuration));

    auto nextWakeupTime = nextVsyncTime - timing.workDuration - timing.readyDuration;



    bool const wouldSkipAVsyncTarget =

            mArmedInfo && (nextVsyncTime > (mArmedInfo->mActualVsyncTime + mMinVsyncDistance));

    if (wouldSkipAVsyncTarget) {

    bool const wouldSkipAWakeup =

            mArmedInfo && ((nextWakeupTime > (mArmedInfo->mActualWakeupTime + mMinVsyncDistance)));

    if (wouldSkipAVsyncTarget && wouldSkipAWakeup) {

        return getExpectedCallbackTime(nextVsyncTime, timing);

    }
@@ -97,9 +100,9 @@ ScheduleResult VSyncDispatchTimerQueueEntry::schedule(VSyncDispatch::ScheduleTim 
    if (alreadyDispatchedForVsync) {

        nextVsyncTime =

                tracker.nextAnticipatedVSyncTimeFrom(*mLastDispatchTime + mMinVsyncDistance);

        nextWakeupTime = nextVsyncTime - timing.workDuration - timing.readyDuration;

    }



    auto const nextWakeupTime = nextVsyncTime - timing.workDuration - timing.readyDuration;

    auto const nextReadyTime = nextVsyncTime - timing.readyDuration;

    mScheduleTiming = timing;

    mArmedInfo = {nextWakeupTime, nextVsyncTime, nextReadyTime};

services/surfaceflinger/tests/unittests/VSyncDispatchTimerQueueTest.cpp

+21 −0
Original line number Diff line number Diff line
@@ -942,6 +942,27 @@ TEST_F(VSyncDispatchTimerQueueTest, basicAlarmSettingFutureWithReadyDuration) { 
    EXPECT_THAT(cb.mReadyTime[0], 970);

}



TEST_F(VSyncDispatchTimerQueueTest, updatesVsyncTimeForCloseWakeupTime) {

    Sequence seq;

    EXPECT_CALL(mMockClock, alarmAt(_, 600)).InSequence(seq);



    CountingCallback cb(mDispatch);



    mDispatch.schedule(cb, {.workDuration = 400, .readyDuration = 0, .earliestVsync = 1000});

    mDispatch.schedule(cb, {.workDuration = 1400, .readyDuration = 0, .earliestVsync = 1000});



    advanceToNextCallback();



    advanceToNextCallback();



    ASSERT_THAT(cb.mCalls.size(), Eq(1));

    EXPECT_THAT(cb.mCalls[0], Eq(2000));

    ASSERT_THAT(cb.mWakeupTime.size(), Eq(1));

    EXPECT_THAT(cb.mWakeupTime[0], Eq(600));

    ASSERT_THAT(cb.mReadyTime.size(), Eq(1));

    EXPECT_THAT(cb.mReadyTime[0], Eq(2000));

}



class VSyncDispatchTimerQueueEntryTest : public testing::Test {

protected:

    nsecs_t const mPeriod = 1000;