commit 50858e61d95220b67a0eac75429c5b5f8a6076a5
author Vishnu Nair <vishnun@google.com> Fri Oct 13 23:28:24 2023 +0000
committer Vishnu Nair <vishnun@google.com> Fri Oct 13 23:28:24 2023 +0000
tree 333cd828132b427e1db6677ee1b4629a5387b9a3
parent 3e69a659a17262949576eb20661abb3c3163a9ca

Destroy transaction tracing from main thread

Fixes: 304996860
Test: presubmit
Change-Id: I3b11570fdb3ea5ba42124f23667be0dfd7d8af11

services/surfaceflinger/SurfaceFlinger.cpp

diff --git a/services/surfaceflinger/SurfaceFlinger.cpp b/services/surfaceflinger/SurfaceFlinger.cpp
index e08690a..358aab9 100644
--- a/services/surfaceflinger/SurfaceFlinger.cpp
+++ b/services/surfaceflinger/SurfaceFlinger.cpp
@@ -7060,7 +7060,7 @@
                 if (mTransactionTracing) {
                     int arg = data.readInt32();
                     if (arg == -1) {
-                        mTransactionTracing.reset();
+                        mScheduler->schedule([&]() { mTransactionTracing.reset(); }).get();
                     } else if (arg > 0) {
                         // Transaction tracing is always running but allow the user to temporarily
                         // increase the buffer when actively debugging.