commit 343626c30d16bd0979860e094d0bf3a57570b303
author Parth Sane <parthsane@google.com> Thu Apr 06 16:17:04 2023 +0000
committer Parth Sane <parthsane@google.com> Wed Apr 12 09:10:22 2023 +0000
tree df6a1c5dc344fe70501272003585e3db0579bd68
parent 6f77de4c76c189694c0dff172c975c25b5dcec7c

Add Traces to SurfaceFlinger constructor and init

This will help debug boot time regressions

Test: Manually checked that boot trace has the init and constructor
slices in perfetto
Bug: 271576143
Change-Id: I91287e3810052ae08933b3631453698ab2d30a59

services/surfaceflinger/SurfaceFlinger.cpp

diff --git a/services/surfaceflinger/SurfaceFlinger.cpp b/services/surfaceflinger/SurfaceFlinger.cpp
index 2f024ce..14c757d 100644
--- a/services/surfaceflinger/SurfaceFlinger.cpp
+++ b/services/surfaceflinger/SurfaceFlinger.cpp
@@ -373,6 +373,7 @@
 }
 
 SurfaceFlinger::SurfaceFlinger(Factory& factory) : SurfaceFlinger(factory, SkipInitialization) {
+    ATRACE_CALL();
     ALOGI("SurfaceFlinger is starting");
 
     hasSyncFramework = running_without_sync_framework(true);
@@ -799,6 +800,7 @@
 // Do not call property_set on main thread which will be blocked by init
 // Use StartPropertySetThread instead.
 void SurfaceFlinger::init() FTL_FAKE_GUARD(kMainThreadContext) {
+    ATRACE_CALL();
     ALOGI(  "SurfaceFlinger's main thread ready to run. "
             "Initializing graphics H/W...");
     addTransactionReadyFilters();