From 2f579e15da634c673e764a2e9dd309dcf0ca88ee Mon Sep 17 00:00:00 2001 From: Naomi Luis Date: Thu, 26 Jan 2012 12:42:01 -0800 Subject: [PATCH] libgenlock: Remove the call to GENLOCK_IOC_RELEASE. The GENLOCK_IOC_RELEASE ioctl is now deprecated. This was done because to avoid race conditions where the handle was released before another client tried to unlock it. (cherry picked from commit d1514357422ce130d3e2776f984294a9efeea83e) Change-Id: Iaf433a79dd0dad992cbf67180765718beec4eb1c CRs-fixed: 333141 --- libgenlock/genlock.cpp | 6 ------ 1 file changed, 6 deletions(-) diff --git a/libgenlock/genlock.cpp b/libgenlock/genlock.cpp index 299161d..1309af6 100644 --- a/libgenlock/genlock.cpp +++ b/libgenlock/genlock.cpp @@ -186,12 +186,6 @@ genlock_status_t genlock_release_lock(native_handle_t *buffer_handle) return GENLOCK_FAILURE; } - if (ioctl(hnd->genlockPrivFd, GENLOCK_IOC_RELEASE, NULL)) { - LOGE("%s: GENLOCK_IOC_RELEASE failed (err=%s)", __FUNCTION__, - strerror(errno)); - ret = GENLOCK_FAILURE; - } - // Close the fd and reset the parameters. close_genlock_fd_and_handle(hnd->genlockPrivFd, hnd->genlockHandle); }