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

Commit 31e52649 authored by Pascal Mütschard's avatar Pascal Mütschard
Browse files

Add the expected and actual frame duration to the SF jank data. 

Adds the expected and actual frame duration, that is the frame durations
as shown in the frame timeline in Perfetto, to the jank data exposed by
SurfaceFlinger.

Bug: b/354763298
Test: libsurfaceflinger_unittest manual
Flag: com.android.internal.jank.use_sf_frame_duration
Change-Id: I43035a15592985c91891c033c667314ec6e15430
parent 2e679ca6

libs/gui/aidl/android/gui/JankData.aidl

+11 −1
Original line number Diff line number Diff line
@@ -29,7 +29,17 @@ parcelable JankData { 
  int jankType;



  /**

   * Expected duration in nanoseconds of this frame.

   * Time between frames in nanoseconds.

   */

  long frameIntervalNs;



  /**

   * Time allocated to the application to render this frame.

   */

  long scheduledAppFrameTimeNs;



  /**

   * Time taken by the application to render this frame.

   */

  long actualAppFrameTimeNs;

}

services/surfaceflinger/FrameTimeline/FrameTimeline.cpp

+17 −0
Original line number Diff line number Diff line
@@ -697,6 +697,23 @@ void SurfaceFrame::onPresent(nsecs_t presentTime, int32_t displayFrameJankType, 
        jd.jankType = mJankType;

        jd.frameIntervalNs =

                (mRenderRate ? *mRenderRate : mDisplayFrameRenderRate).getPeriodNsecs();



        if (mPredictionState == PredictionState::Valid) {

            jd.scheduledAppFrameTimeNs = mPredictions.endTime - mPredictions.startTime;



            // Using expected start, rather than actual, to measure the entire frame time. That is

            // if the application starts the frame later than scheduled, include that delay in the

            // frame time, as it usually means main thread being busy with non-rendering work.

            if (mPresentState == PresentState::Dropped) {

                jd.actualAppFrameTimeNs = mDropTime - mPredictions.startTime;

            } else {

                jd.actualAppFrameTimeNs = mActuals.endTime - mPredictions.startTime;

            }

        } else {

            jd.scheduledAppFrameTimeNs = 0;

            jd.actualAppFrameTimeNs = 0;

        }



        JankTracker::onJankData(mLayerId, jd);

    }

}