commit 9cf9e51cf20a1d907b1fecd6dd59b6071b011f47
author Marin Shalamanov <shalamanov@google.com> Wed Dec 02 13:28:06 2020 +0100
committer Marin Shalamanov <shalamanov@google.com> Tue Dec 08 19:25:49 2020 +0100
tree 27bbccd98592932f1fd99a9e120942c48ea09ec4
parent 1876e2eac0b9fb79c1633b0386d72938fbc3051d

SF: Log hotplug events

When debugging hotplug related issues it is very useful to have
hotplug events in the logs. These events occur rarely so the
overhead of this additional log is low.

Bug: 174644816
Test: adb logcat | grep onHotplug
Change-Id: If1383a4dea2e95060fa3c4332406b80fc42af40c

services/surfaceflinger/SurfaceFlinger.cpp

diff --git a/services/surfaceflinger/SurfaceFlinger.cpp b/services/surfaceflinger/SurfaceFlinger.cpp
index fe80f96..6c8a1c1 100644
--- a/services/surfaceflinger/SurfaceFlinger.cpp
+++ b/services/surfaceflinger/SurfaceFlinger.cpp
@@ -1618,7 +1618,7 @@
 
 void SurfaceFlinger::onHotplugReceived(int32_t sequenceId, hal::HWDisplayId hwcDisplayId,
                                        hal::Connection connection) {
-    ALOGV("%s(%d, %" PRIu64 ", %s)", __FUNCTION__, sequenceId, hwcDisplayId,
+    ALOGI("%s(%d, %" PRIu64 ", %s)", __FUNCTION__, sequenceId, hwcDisplayId,
           connection == hal::Connection::CONNECTED ? "connected" : "disconnected");
 
     // Ignore events that do not have the right sequenceId.
@@ -2385,6 +2385,8 @@
 }
 
 void SurfaceFlinger::dispatchDisplayHotplugEvent(PhysicalDisplayId displayId, bool connected) {
+    ALOGI("Dispatching display hotplug event displayId=%s, connected=%d",
+          to_string(displayId).c_str(), connected);
     mScheduler->onHotplugReceived(mAppConnectionHandle, displayId, connected);
     mScheduler->onHotplugReceived(mSfConnectionHandle, displayId, connected);
 }