commit 028508cad5ef63ef9fbd42c14e76658e4fd9ebf2
author Mathias Agopian <mathias@google.com> Wed Jul 25 21:12:12 2012 -0700
committer Mathias Agopian <mathias@google.com> Wed Jul 25 21:13:10 2012 -0700
tree ff611b4577a59d6314fb5369477d4356da0f223a
parent 8630320433bd15aca239522e54e711ef6372ab07

hopefully fixe a race condition in sf initialization

if we received a vsync event during SF init, we could crash
as not all objects were ready to go.

Change-Id: Ie11b46e3eb1b37a709dd8757843d444f93dd0189

services/surfaceflinger/SurfaceFlinger.cpp

diff --git a/services/surfaceflinger/SurfaceFlinger.cpp b/services/surfaceflinger/SurfaceFlinger.cpp
index 2ecdeb8..992c779 100644
--- a/services/surfaceflinger/SurfaceFlinger.cpp
+++ b/services/surfaceflinger/SurfaceFlinger.cpp
@@ -406,6 +406,10 @@
     EGLSurface surface = hw->getEGLSurface();
     initializeGL(display, surface);
 
+    // start the EventThread
+    mEventThread = new EventThread(this);
+    mEventQueue.setEventThread(mEventThread);
+
     // initialize the H/W composer
     mHwc = new HWComposer(this,
             *static_cast<HWComposer::EventHandler *>(this),
@@ -414,10 +418,6 @@
         mHwc->setFrameBuffer(display, surface);
     }
 
-    // start the EventThread
-    mEventThread = new EventThread(this);
-    mEventQueue.setEventThread(mEventThread);
-
     // We're now ready to accept clients...
     mReadyToRunBarrier.open();