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

Commit 6be8244a authored by Pablo Gamito's avatar Pablo Gamito
Browse files

Dump last committed vSyncId on layers trace start 

Keep track of the last committed vSyncId to dump it when we start the layers trace

Bug: 230462538

Test: Check a vSyncId is dumped in the first entry of the layers trace
Change-Id: Ie4c48acf99057cdb8bfebbf5cea3729ab5014bd4
parent b877579d

services/surfaceflinger/SurfaceFlinger.cpp

+2 −1
Original line number Diff line number Diff line
@@ -2170,6 +2170,7 @@ bool SurfaceFlinger::commit(nsecs_t frameTime, int64_t vsyncId, nsecs_t expected 
        // This will block and tracing should only be enabled for debugging.

        mLayerTracing.notify(mVisibleRegionsDirty, frameTime, vsyncId);

    }

    mLastCommittedVsyncId = vsyncId;



    persistDisplayBrightness(mustComposite);
@@ -5847,7 +5848,7 @@ status_t SurfaceFlinger::onTransact(uint32_t code, const Parcel& data, Parcel* r 
                        mScheduler

                                ->schedule([&]() FTL_FAKE_GUARD(mStateLock) {

                                    mLayerTracing.notify(true /* visibleRegionDirty */,

                                                         startingTime, -1 /* vsyncId */);

                                                         startingTime, mLastCommittedVsyncId);

                                })

                                .wait();

                    }

services/surfaceflinger/SurfaceFlinger.h

+2 −0
Original line number Diff line number Diff line
@@ -1266,6 +1266,8 @@ private: 
    const std::unique_ptr<FrameTracer> mFrameTracer;

    const std::unique_ptr<frametimeline::FrameTimeline> mFrameTimeline;



    int64_t mLastCommittedVsyncId = -1;



    // If blurs should be enabled on this device.

    bool mSupportsBlur = false;

    // If blurs are considered expensive and should require high GPU frequency.