diff options
author | 2020-08-28 05:10:55 +0000 | |
---|---|---|
committer | 2020-08-28 05:23:13 +0000 | |
commit | 68e261a0aa6c423c4e79bd2d6070e5180282102e (patch) | |
tree | 9a2ed82d11ca809c7e8374a0d26aa2fc42a204ec | |
parent | ad321cd93840d11440f80f49452d4631697033e5 (diff) |
Revert "Add setSchedPolicyCgroup to AM binder"
This reverts commit b12941cf9b1e3c3456eb44a19927570d99ba108d.
Reason for revert: may have problem under race condition
Change-Id: I67941e7b5026484df856554dc66e1ae0c621a93c
-rw-r--r-- | libs/binder/ActivityManager.cpp | 9 | ||||
-rw-r--r-- | libs/binder/IActivityManager.cpp | 11 | ||||
-rw-r--r-- | libs/binder/include/binder/ActivityManager.h | 2 | ||||
-rw-r--r-- | libs/binder/include/binder/IActivityManager.h | 4 |
4 files changed, 2 insertions, 24 deletions
diff --git a/libs/binder/ActivityManager.cpp b/libs/binder/ActivityManager.cpp index d005058abe..5e4c98fc7a 100644 --- a/libs/binder/ActivityManager.cpp +++ b/libs/binder/ActivityManager.cpp @@ -98,15 +98,6 @@ int32_t ActivityManager::getUidProcessState(const uid_t uid, const String16& cal return PROCESS_STATE_UNKNOWN; } -bool ActivityManager::setSchedPolicyCgroup(const int32_t tid, const int32_t group) -{ - sp<IActivityManager> service = getService(); - if (service != nullptr) { - return service->setSchedPolicyCgroup(tid, group); - } - return false; -} - status_t ActivityManager::linkToDeath(const sp<IBinder::DeathRecipient>& recipient) { sp<IActivityManager> service = getService(); if (service != nullptr) { diff --git a/libs/binder/IActivityManager.cpp b/libs/binder/IActivityManager.cpp index a30211225d..1eb5363ae2 100644 --- a/libs/binder/IActivityManager.cpp +++ b/libs/binder/IActivityManager.cpp @@ -104,17 +104,6 @@ public: } return reply.readInt32(); } - - virtual bool setSchedPolicyCgroup(const int32_t tid, const int32_t group) - { - Parcel data, reply; - data.writeInterfaceToken(IActivityManager::getInterfaceDescriptor()); - data.writeInt32(tid); - data.writeInt32(group); - remote()->transact(SET_SCHED_POLICY_CGROUP_TRANSACTION, data, &reply); - if (reply.readExceptionCode() != 0) return false; - return reply.readBool(); - } }; // ------------------------------------------------------------------------------------ diff --git a/libs/binder/include/binder/ActivityManager.h b/libs/binder/include/binder/ActivityManager.h index 7043b17b53..9108e31758 100644 --- a/libs/binder/include/binder/ActivityManager.h +++ b/libs/binder/include/binder/ActivityManager.h @@ -77,7 +77,7 @@ public: void unregisterUidObserver(const sp<IUidObserver>& observer); bool isUidActive(const uid_t uid, const String16& callingPackage); int getUidProcessState(const uid_t uid, const String16& callingPackage); - bool setSchedPolicyCgroup(const int32_t tid, const int32_t group); + status_t linkToDeath(const sp<IBinder::DeathRecipient>& recipient); status_t unlinkToDeath(const sp<IBinder::DeathRecipient>& recipient); diff --git a/libs/binder/include/binder/IActivityManager.h b/libs/binder/include/binder/IActivityManager.h index fe58a41546..e0248f6624 100644 --- a/libs/binder/include/binder/IActivityManager.h +++ b/libs/binder/include/binder/IActivityManager.h @@ -39,15 +39,13 @@ public: virtual void unregisterUidObserver(const sp<IUidObserver>& observer) = 0; virtual bool isUidActive(const uid_t uid, const String16& callingPackage) = 0; virtual int32_t getUidProcessState(const uid_t uid, const String16& callingPackage) = 0; - virtual bool setSchedPolicyCgroup(const int32_t tid, const int32_t group) = 0; enum { OPEN_CONTENT_URI_TRANSACTION = IBinder::FIRST_CALL_TRANSACTION, REGISTER_UID_OBSERVER_TRANSACTION, UNREGISTER_UID_OBSERVER_TRANSACTION, IS_UID_ACTIVE_TRANSACTION, - GET_UID_PROCESS_STATE_TRANSACTION, - SET_SCHED_POLICY_CGROUP_TRANSACTION + GET_UID_PROCESS_STATE_TRANSACTION }; }; |