Merge "Add the expected and actual frame duration to the SF jank data." into main (8f07bb31) · Commits · e / os / android_frameworks_native

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;
		}

+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);
		}
		}