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

Commit ab87979a authored by Wei Wang's avatar Wei Wang
Browse files

Add system-background cpu group 

Add system-background cgroup hierarchy for uclamp and cpu bandwidth
tuning.

Bug: 170507876
Bug: 171740453
Test: boot and check cgroup
Change-Id: I41745e7b3ac5c1d7ea7dd77f5db39c6895047446
parent 6a285726

libprocessgroup/include/processgroup/sched_policy.h

+1 −1
Original line number Diff line number Diff line
@@ -42,7 +42,7 @@ typedef enum { 
    SP_DEFAULT = -1,

    SP_BACKGROUND = 0,

    SP_FOREGROUND = 1,

    SP_SYSTEM = 2,  // can't be used with set_sched_policy()

    SP_SYSTEM = 2,

    SP_AUDIO_APP = 3,

    SP_AUDIO_SYS = 4,

    SP_TOP_APP = 5,

libprocessgroup/profiles/task_profiles.json

+17 −0
Original line number Diff line number Diff line
@@ -105,6 +105,19 @@ 
        }

      ]

    },

    {

      "Name": "ServicePerformance",

      "Actions": [

        {

          "Name": "JoinCgroup",

          "Params":

          {

            "Controller": "cpu",

            "Path": "system-background"

          }

        }

      ]

    },

    {

      "Name": "HighPerformance",

      "Actions": [
@@ -590,6 +603,10 @@ 
      "Name": "SCHED_SP_TOP_APP",

      "Profiles": [ "MaxPerformance", "MaxIoPriority", "TimerSlackNormal" ]

    },

    {

      "Name": "SCHED_SP_SYSTEM",

      "Profiles": [ "ServicePerformance", "LowIoPriority", "TimerSlackNormal" ]

    },

    {

      "Name": "SCHED_SP_RT_APP",

      "Profiles": [ "RealtimePerformance", "MaxIoPriority", "TimerSlackNormal" ]

libprocessgroup/sched_policy.cpp

+3 −1
Original line number Diff line number Diff line
@@ -124,6 +124,8 @@ int set_sched_policy(int tid, SchedPolicy policy) { 
            return SetTaskProfiles(tid, {"SCHED_SP_FOREGROUND"}, true) ? 0 : -1;

        case SP_TOP_APP:

            return SetTaskProfiles(tid, {"SCHED_SP_TOP_APP"}, true) ? 0 : -1;

        case SP_SYSTEM:

            return SetTaskProfiles(tid, {"SCHED_SP_SYSTEM"}, true) ? 0 : -1;

        case SP_RT_APP:

            return SetTaskProfiles(tid, {"SCHED_SP_RT_APP"}, true) ? 0 : -1;

        default:
@@ -258,7 +260,7 @@ const char* get_sched_policy_profile_name(SchedPolicy policy) { 
     */

    static constexpr const char* kSchedProfiles[SP_CNT + 1] = {

            "SCHED_SP_DEFAULT", "SCHED_SP_BACKGROUND", "SCHED_SP_FOREGROUND",

            "SCHED_SP_DEFAULT", "SCHED_SP_FOREGROUND", "SCHED_SP_FOREGROUND",

            "SCHED_SP_SYSTEM",  "SCHED_SP_FOREGROUND", "SCHED_SP_FOREGROUND",

            "SCHED_SP_TOP_APP", "SCHED_SP_RT_APP",     "SCHED_SP_DEFAULT"};

    if (policy < SP_DEFAULT || policy >= SP_CNT) {

        return nullptr;

rootdir/init.rc

+7 −1
Original line number Diff line number Diff line
@@ -154,24 +154,28 @@ on init 
    mkdir /dev/cpuctl/top-app

    mkdir /dev/cpuctl/rt

    mkdir /dev/cpuctl/system

    mkdir /dev/cpuctl/system-background

    chown system system /dev/cpuctl

    chown system system /dev/cpuctl/foreground

    chown system system /dev/cpuctl/background

    chown system system /dev/cpuctl/top-app

    chown system system /dev/cpuctl/rt

    chown system system /dev/cpuctl/system

    chown system system /dev/cpuctl/system-background

    chown system system /dev/cpuctl/tasks

    chown system system /dev/cpuctl/foreground/tasks

    chown system system /dev/cpuctl/background/tasks

    chown system system /dev/cpuctl/top-app/tasks

    chown system system /dev/cpuctl/rt/tasks

    chown system system /dev/cpuctl/system/tasks

    chown system system /dev/cpuctl/system-background/tasks

    chmod 0664 /dev/cpuctl/tasks

    chmod 0664 /dev/cpuctl/foreground/tasks

    chmod 0664 /dev/cpuctl/background/tasks

    chmod 0664 /dev/cpuctl/top-app/tasks

    chmod 0664 /dev/cpuctl/rt/tasks

    chmod 0664 /dev/cpuctl/system/tasks

    chmod 0664 /dev/cpuctl/system-background/tasks



    # Create a cpu group for NNAPI HAL processes

    mkdir /dev/cpuctl/nnapi-hal
@@ -196,7 +200,7 @@ on init 
    # Surfaceflinger is in FG group so giving it a bit more

    write /dev/cpuctl/foreground/cpu.rt_runtime_us 450000

    write /dev/cpuctl/foreground/cpu.rt_period_us 1000000

    write /dev/cpuctl/background/cpu.rt_runtime_us 100000

    write /dev/cpuctl/background/cpu.rt_runtime_us 50000

    write /dev/cpuctl/background/cpu.rt_period_us 1000000

    write /dev/cpuctl/top-app/cpu.rt_runtime_us 100000

    write /dev/cpuctl/top-app/cpu.rt_period_us 1000000
@@ -204,6 +208,8 @@ on init 
    write /dev/cpuctl/rt/cpu.rt_period_us 1000000

    write /dev/cpuctl/system/cpu.rt_runtime_us 100000

    write /dev/cpuctl/system/cpu.rt_period_us 1000000

    write /dev/cpuctl/system-background/cpu.rt_runtime_us 50000

    write /dev/cpuctl/system-background/cpu.rt_period_us 1000000

    write /dev/cpuctl/nnapi-hal/cpu.rt_runtime_us 50000

    write /dev/cpuctl/nnapi-hal/cpu.rt_period_us 1000000

    write /dev/cpuctl/camera-daemon/cpu.rt_runtime_us 50000