From fa02bb9952ee4e3e8403d6c2910a91aefc76dda7 Mon Sep 17 00:00:00 2001 From: Rick Yiu Date: Sun, 27 Sep 2020 11:21:11 +0800 Subject: Revert "Add choice for changing sched policy when setting thread priority" This reverts commit 57affbf91d016eda1395106c4d5f0350471dca85. Reason for revert: new API is no longer used Change-Id: I97128c1d367e161aa50deaefa8b73a0f25e29af4 --- libutils/Threads.cpp | 25 ++++++++++++------------- 1 file changed, 12 insertions(+), 13 deletions(-) (limited to 'libutils/Threads.cpp') diff --git a/libutils/Threads.cpp b/libutils/Threads.cpp index 147db542d..540dcf49d 100644 --- a/libutils/Threads.cpp +++ b/libutils/Threads.cpp @@ -302,7 +302,8 @@ void androidSetCreateThreadFunc(android_create_thread_fn func) } #if defined(__ANDROID__) -int androidSetThreadPriority(pid_t tid, int pri, bool change_policy) { +int androidSetThreadPriority(pid_t tid, int pri) +{ int rc = 0; int lasterr = 0; int curr_pri = getpriority(PRIO_PROCESS, tid); @@ -311,19 +312,17 @@ int androidSetThreadPriority(pid_t tid, int pri, bool change_policy) { return rc; } - if (change_policy) { - if (pri >= ANDROID_PRIORITY_BACKGROUND) { - rc = SetTaskProfiles(tid, {"SCHED_SP_BACKGROUND"}, true) ? 0 : -1; - } else if (curr_pri >= ANDROID_PRIORITY_BACKGROUND) { - SchedPolicy policy = SP_FOREGROUND; - // Change to the sched policy group of the process. - get_sched_policy(getpid(), &policy); - rc = SetTaskProfiles(tid, {get_sched_policy_profile_name(policy)}, true) ? 0 : -1; - } + if (pri >= ANDROID_PRIORITY_BACKGROUND) { + rc = SetTaskProfiles(tid, {"SCHED_SP_BACKGROUND"}, true) ? 0 : -1; + } else if (curr_pri >= ANDROID_PRIORITY_BACKGROUND) { + SchedPolicy policy = SP_FOREGROUND; + // Change to the sched policy group of the process. + get_sched_policy(getpid(), &policy); + rc = SetTaskProfiles(tid, {get_sched_policy_profile_name(policy)}, true) ? 0 : -1; + } - if (rc) { - lasterr = errno; - } + if (rc) { + lasterr = errno; } if (setpriority(PRIO_PROCESS, tid, pri) < 0) { -- cgit v1.2.3