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

Commit a099bb7b authored by Android (Google) Code Review's avatar Android (Google) Code Review
Browse files

Merge change I2867746d into eclair 

* changes:
  When a thread is about to be put back onto the thread pool ensure that it is in the foreground cgroup.
parents 8b6b95ab b5c41353

libs/binder/IPCThreadState.cpp

+4 −22
Original line number Diff line number Diff line
@@ -422,31 +422,13 @@ void IPCThreadState::joinThreadPool(bool isMain) 
                    << getReturnString(cmd) << endl;

            }



            bool isTainted = false;



            {

                SchedPolicy policy;

                get_sched_policy(getpid(), &policy);



                if (policy == SP_BACKGROUND) {

                    isTainted = true;

                }

            }



            result = executeCommand(cmd);



            // Make sure that after executing the commands that we put the thread back into the

            // default cgroup.

            {

                int pid = getpid();

                SchedPolicy policy;

                get_sched_policy(pid, &policy);



                if (!isTainted && policy == SP_BACKGROUND) {

                    LOGW("*** THREAD %p (PID %p) was left in SP_BACKGROUND with a priority of %d\n",

                        (void*)pthread_self(), pid, getpriority(PRIO_PROCESS, pid));

                }

            }

            // Make sure that after executing the command that we put the thread back into the

            // default cgroup. This is just a failsafe incase the thread's priority or cgroup was 

            // not properly restored.

            set_sched_policy(getpid(), SP_FOREGROUND);

        }

        

        // Let this thread exit the thread pool if it is no longer