From f20c329a8a1fb578f5842f4056d3ebb39403d01f Mon Sep 17 00:00:00 2001 From: Eric Laurent Date: Sun, 10 Aug 2014 10:46:52 -0700 Subject: [PATCH] IPowerManager: make all binder call one way make all binder call to power manager service one way so that they don't block calling thread and are processed in the incoming order. Bug: 16408906. Change-Id: I94034df7f0a36d967e085b53fdc37fff7b44757b --- services/powermanager/IPowerManager.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/services/powermanager/IPowerManager.cpp b/services/powermanager/IPowerManager.cpp index ee730d6ec..926c050ae 100644 --- a/services/powermanager/IPowerManager.cpp +++ b/services/powermanager/IPowerManager.cpp @@ -56,7 +56,7 @@ public: data.writeString16(packageName); data.writeInt32(0); // no WorkSource data.writeString16(NULL, 0); // no history tag - return remote()->transact(ACQUIRE_WAKE_LOCK, data, &reply); + return remote()->transact(ACQUIRE_WAKE_LOCK, data, &reply, IBinder::FLAG_ONEWAY); } virtual status_t acquireWakeLockWithUid(int flags, const sp& lock, const String16& tag, @@ -70,7 +70,7 @@ public: data.writeString16(tag); data.writeString16(packageName); data.writeInt32(uid); // uid to blame for the work - return remote()->transact(ACQUIRE_WAKE_LOCK_UID, data, &reply); + return remote()->transact(ACQUIRE_WAKE_LOCK_UID, data, &reply, IBinder::FLAG_ONEWAY); } virtual status_t releaseWakeLock(const sp& lock, int flags) @@ -79,7 +79,7 @@ public: data.writeInterfaceToken(IPowerManager::getInterfaceDescriptor()); data.writeStrongBinder(lock); data.writeInt32(flags); - return remote()->transact(RELEASE_WAKE_LOCK, data, &reply); + return remote()->transact(RELEASE_WAKE_LOCK, data, &reply, IBinder::FLAG_ONEWAY); } virtual status_t updateWakeLockUids(const sp& lock, int len, const int *uids) {