summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorEric Laurent2014-08-10 12:46:52 -0500
committerEric Laurent2014-09-05 11:14:19 -0500
commitf20c329a8a1fb578f5842f4056d3ebb39403d01f (patch)
treeaa307eafc1cdd1c23ebfe4e6f143764ab8ee42d0 /services/powermanager/IPowerManager.cpp
parentbcbf97ef8fb6eaf527014ed9e7d979ad4fe9fdd2 (diff)
downloadframeworks-native-f20c329a8a1fb578f5842f4056d3ebb39403d01f.tar.gz
frameworks-native-f20c329a8a1fb578f5842f4056d3ebb39403d01f.tar.xz
frameworks-native-f20c329a8a1fb578f5842f4056d3ebb39403d01f.zip
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
Diffstat (limited to 'services/powermanager/IPowerManager.cpp')
-rw-r--r--services/powermanager/IPowerManager.cpp6
1 files 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:
56 data.writeString16(packageName); 56 data.writeString16(packageName);
57 data.writeInt32(0); // no WorkSource 57 data.writeInt32(0); // no WorkSource
58 data.writeString16(NULL, 0); // no history tag 58 data.writeString16(NULL, 0); // no history tag
59 return remote()->transact(ACQUIRE_WAKE_LOCK, data, &reply); 59 return remote()->transact(ACQUIRE_WAKE_LOCK, data, &reply, IBinder::FLAG_ONEWAY);
60 } 60 }
61 61
62 virtual status_t acquireWakeLockWithUid(int flags, const sp<IBinder>& lock, const String16& tag, 62 virtual status_t acquireWakeLockWithUid(int flags, const sp<IBinder>& lock, const String16& tag,
@@ -70,7 +70,7 @@ public:
70 data.writeString16(tag); 70 data.writeString16(tag);
71 data.writeString16(packageName); 71 data.writeString16(packageName);
72 data.writeInt32(uid); // uid to blame for the work 72 data.writeInt32(uid); // uid to blame for the work
73 return remote()->transact(ACQUIRE_WAKE_LOCK_UID, data, &reply); 73 return remote()->transact(ACQUIRE_WAKE_LOCK_UID, data, &reply, IBinder::FLAG_ONEWAY);
74 } 74 }
75 75
76 virtual status_t releaseWakeLock(const sp<IBinder>& lock, int flags) 76 virtual status_t releaseWakeLock(const sp<IBinder>& lock, int flags)
@@ -79,7 +79,7 @@ public:
79 data.writeInterfaceToken(IPowerManager::getInterfaceDescriptor()); 79 data.writeInterfaceToken(IPowerManager::getInterfaceDescriptor());
80 data.writeStrongBinder(lock); 80 data.writeStrongBinder(lock);
81 data.writeInt32(flags); 81 data.writeInt32(flags);
82 return remote()->transact(RELEASE_WAKE_LOCK, data, &reply); 82 return remote()->transact(RELEASE_WAKE_LOCK, data, &reply, IBinder::FLAG_ONEWAY);
83 } 83 }
84 84
85 virtual status_t updateWakeLockUids(const sp<IBinder>& lock, int len, const int *uids) { 85 virtual status_t updateWakeLockUids(const sp<IBinder>& lock, int len, const int *uids) {