Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 587f7830 authored by Dave Sparks's avatar Dave Sparks
Browse files

Retry overlay create if it fails. Bug 2153980. 

Occasionally we see references to the overlay hanging around long
enough to cause problems in applications when they tried to destroy
the overlay and re-create it. This patch causes the camera HAL to
retry the overlay creation call if it fails every 20ms up to 50
times before it gives up.
parent b4055a63

camera/libcameraservice/CameraService.cpp

+13 −1
Original line number Diff line number Diff line
@@ -563,7 +563,19 @@ status_t CameraService::Client::setOverlay() 
    status_t ret = NO_ERROR;

    if (mSurface != 0) {

        if (mOverlayRef.get() == NULL) {



            // FIXME:

            // Surfaceflinger may hold onto the previous overlay reference for some

            // time after we try to destroy it. retry a few times. In the future, we

            // should make the destroy call block, or possibly specify that we can

            // wait in the createOverlay call if the previous overlay is in the 

            // process of being destroyed.

            for (int retry = 0; retry < 50; ++retry) {

                mOverlayRef = mSurface->createOverlay(w, h, OVERLAY_FORMAT_DEFAULT);

                if (mOverlayRef != NULL) break;

                LOGD("Overlay create failed - retrying");

                usleep(20000);

            }

            if ( mOverlayRef.get() == NULL )

            {

                LOGE("Overlay Creation Failed!");