commit 82b74d6f1538e01ab47907d38aaa9275fb23f1b9
author Treehugger Robot <treehugger-gerrit@google.com> Tue Feb 08 05:52:22 2022 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Tue Feb 08 05:52:22 2022 +0000
tree 545f4eeb0e368707e8e944a781d0de8e4d0209f0
parent 4b7400756ca0c9e7fe20f93d97d203b187d1eca8
parent 0455a2c39c781a8153c1090a4638193b4181379d

Merge "libutils: don't call SetTaskProfiles around priority"

libutils/Threads.cpp

diff --git a/libutils/Threads.cpp b/libutils/Threads.cpp
index 3bf5779..a85ff20 100644
--- a/libutils/Threads.cpp
+++ b/libutils/Threads.cpp
@@ -84,14 +84,6 @@
         delete t;
         setpriority(PRIO_PROCESS, 0, prio);
 
-        // A new thread will be in its parent's sched group by default,
-        // so we just need to handle the background case.
-        // currently set to system_background group which is different
-        // from background group for app.
-        if (prio >= ANDROID_PRIORITY_BACKGROUND) {
-            SetTaskProfiles(0, {"SCHED_SP_SYSTEM"}, true);
-        }
-
         if (name) {
             androidSetThreadName(name);
             free(name);
@@ -314,12 +306,6 @@
         return rc;
     }
 
-    if (pri >= ANDROID_PRIORITY_BACKGROUND) {
-        rc = SetTaskProfiles(tid, {"SCHED_SP_SYSTEM"}, true) ? 0 : -1;
-    } else if (curr_pri >= ANDROID_PRIORITY_BACKGROUND) {
-        rc = SetTaskProfiles(tid, {"SCHED_SP_FOREGROUND"}, true) ? 0 : -1;
-    }
-
     if (rc) {
         lasterr = errno;
     }