OSDN Git Service

Fix deadlock when transitioning to vr flinger
authorSteven Thomas <steventhomas@google.com>
Thu, 28 Sep 2017 22:30:23 +0000 (15:30 -0700)
committerSteven Thomas <steventhomas@google.com>
Fri, 29 Sep 2017 18:03:15 +0000 (11:03 -0700)
commitbe6cbae3b61a55ab87c131e79dba03d159961aa3
tree8f5eca867ce05ec0fb0aca7950b556aa870d2b47
parent7e70ec2c0d38aeaf7bd06519e6bd12f4528eb23d
Fix deadlock when transitioning to vr flinger

If we were unlucky with the timing when switching to vr flinger we would
sometimes deadlock. The vr dispatch thread would request the display
from surface flinger, locking mStateLock in the process. mStateLock was
being held by the surface flinger main thread, which was processing a
previous request to switch to vr flinger. The main thread was trying to
connect to the vr hardware composer service, which sent a request to the
vr dispatch thread as part of its initialization, leading to the
deadlock.

The deadlock is easily fixed by posting a message to request the vr
flinger switch to the surface flinger main thread, instead of doing it
on the vr dispatch thread.

Bug: 66916578

Test: Confirmed I can still get into/out of vr flinger on
Marlin. There's no longer code to acquire mStateLock in the vr dispatch
thread, so the deadlock is no longer possible.

Change-Id: I3de5476f698ed798f6b9afe927cc733f0f38c56e
services/surfaceflinger/SurfaceFlinger.cpp