Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 68e261a0 authored by Rick Yiu's avatar Rick Yiu
Browse files

Revert "Add setSchedPolicyCgroup to AM binder" 

This reverts commit b12941cf.

Reason for revert: may have problem under race condition

Change-Id: I67941e7b5026484df856554dc66e1ae0c621a93c
parent ad321cd9

libs/binder/ActivityManager.cpp

+0 −9
Original line number Diff line number Diff line
@@ -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) {

libs/binder/IActivityManager.cpp

+0 −11
Original line number Diff line number Diff line
@@ -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();

    }

};



// ------------------------------------------------------------------------------------

libs/binder/include/binder/ActivityManager.h

+1 −1
Original line number Diff line number Diff line
@@ -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);

libs/binder/include/binder/IActivityManager.h

+1 −3
Original line number Diff line number Diff line
@@ -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

    };

};