From d30b36d1fd0c7ab80553e784a155266be9703e47 Mon Sep 17 00:00:00 2001 From: Jamie Gennis Date: Sun, 30 Sep 2012 20:02:03 -0700 Subject: [PATCH] SurfaceFlinger: reset the FB_TGT acquire fence This change makes SurfaceFlinger reset the acquire fence fd to -1 after each call to HWComposer::commit. The HWComposer implementation is resonsible for closing the fd, so SurfaceFlinger should make sure not to pass it to HWC multiple times. Change-Id: I79554d9f6d6ef2b77d632d40251516c1f5b16ddb Bug: 7258954 --- services/surfaceflinger/DisplayHardware/HWComposer.cpp | 1 + 1 file changed, 1 insertion(+) diff --git a/services/surfaceflinger/DisplayHardware/HWComposer.cpp b/services/surfaceflinger/DisplayHardware/HWComposer.cpp index 960fb0e10..d4adad253 100644 --- a/services/surfaceflinger/DisplayHardware/HWComposer.cpp +++ b/services/surfaceflinger/DisplayHardware/HWComposer.cpp @@ -613,6 +613,7 @@ int HWComposer::getAndResetReleaseFenceFd(int32_t id) { const DisplayData& disp(mDisplayData[id]); if (disp.framebufferTarget) { fd = disp.framebufferTarget->releaseFenceFd; + disp.framebufferTarget->acquireFenceFd = -1; disp.framebufferTarget->releaseFenceFd = -1; } }