diff options
| author | Alex Naidis <alex.naidis@linux.com> | 2017-06-10 22:34:23 +0200 |
|---|---|---|
| committer | Alex Naidis <alex.naidis@linux.com> | 2017-06-16 21:30:49 +0000 |
| commit | 2d998ea99bd650aafc270f5d0599fc0b8d2c0ce7 (patch) | |
| tree | 618242a85305532287c23404af669294fbeeece6 /libcutils/sched_policy.cpp | |
| parent | 6794bc7d37957b3192dbb9868038d5ee4349fde5 (diff) | |
| download | system_core-2d998ea99bd650aafc270f5d0599fc0b8d2c0ce7.tar.gz system_core-2d998ea99bd650aafc270f5d0599fc0b8d2c0ce7.tar.bz2 system_core-2d998ea99bd650aafc270f5d0599fc0b8d2c0ce7.zip | |
libcutils: Clarify schedboost_enabled()
Commit 795267d4c748 ("Removed cpusets/schedboost build time dependency.")
turned the cpusets and schedtune options into runtime
decisions.
However the kernel option which is mentioned in the
code comment (CONFIG_SCHEDTUNE) is very misleading
as it doesn't exist (CONFIG_SCHED_TUNE does exist)
and it doesn't describe the real functionality of
the method. schedboost_enabled() will still return
false if CONFIG_SCHED_TUNE is set in the kernel but
CONFIG_CGROUP_SCHEDTUNE is not.
So to clarify this, we need to change the comment
to reflect that CONFIG_CGROUP_SCHEDTUNE, which depends
on CONFIG_SCHED_TUNE, is required.
Signed-off-by: Alex Naidis <alex.naidis@linux.com>
Diffstat (limited to 'libcutils/sched_policy.cpp')
| -rw-r--r-- | libcutils/sched_policy.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libcutils/sched_policy.cpp b/libcutils/sched_policy.cpp index 7e385355f..4c303e6b1 100644 --- a/libcutils/sched_policy.cpp +++ b/libcutils/sched_policy.cpp @@ -136,7 +136,7 @@ bool cpusets_enabled() { /* Similar to CONFIG_CPUSETS above, but with a different configuration - CONFIG_SCHEDTUNE that's in Android common Linux kernel and Linaro + CONFIG_CGROUP_SCHEDTUNE that's in Android common Linux kernel and Linaro Stable Kernel (LSK), but not in mainline Linux as of v4.9. With runtime check using the following function, build time |