diff options
| author | Rick Yiu <rickyiu@google.com> | 2020-09-27 11:21:11 +0800 |
|---|---|---|
| committer | Rick Yiu <rickyiu@google.com> | 2020-09-27 11:21:11 +0800 |
| commit | fa02bb9952ee4e3e8403d6c2910a91aefc76dda7 (patch) | |
| tree | a999643560df7ca11be213fd1e51a5885ceb22d0 /libutils/Threads.cpp | |
| parent | 290efe38b6e1695915ed77a0b8593dd1f75ecae9 (diff) | |
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
Diffstat (limited to 'libutils/Threads.cpp')
| -rw-r--r-- | libutils/Threads.cpp | 25 |
1 files changed, 12 insertions, 13 deletions
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) { |