diff --git a/healthd/Android.mk b/healthd/Android.mk index 80bf84ae8..2127b9603 100644 --- a/healthd/Android.mk +++ b/healthd/Android.mk @@ -109,6 +109,7 @@ CHARGER_STATIC_LIBRARIES := \ libbase \ libutils \ libcutils \ + libprocessgroup \ liblog \ libm \ libc \ diff --git a/libcutils/Android.bp b/libcutils/Android.bp index 4291212a1..0dbbc3f2f 100644 --- a/libcutils/Android.bp +++ b/libcutils/Android.bp @@ -66,7 +66,6 @@ cc_library { "load_file.cpp", "native_handle.cpp", "record_stream.cpp", - "sched_policy.cpp", "sockets.cpp", "strdup16to8.cpp", "strdup8to16.cpp", @@ -178,8 +177,12 @@ cc_library { "libbase_headers", "libcutils_headers", "libutils_headers", + "libprocessgroup_headers", + ], + export_header_lib_headers: [ + "libcutils_headers", + "libprocessgroup_headers", ], - export_header_lib_headers: ["libcutils_headers"], local_include_dirs: ["include"], cflags: [ diff --git a/libcutils/include/cutils/sched_policy.h b/libcutils/include/cutils/sched_policy.h index cf91b76f4..538ff6b9b 100644 --- a/libcutils/include/cutils/sched_policy.h +++ b/libcutils/include/cutils/sched_policy.h @@ -17,67 +17,10 @@ #ifndef __CUTILS_SCHED_POLICY_H #define __CUTILS_SCHED_POLICY_H -#include - -#ifdef __cplusplus -extern "C" { -#endif - /* - * Check if Linux kernel enables CPUSETS feature. - * - * Return value: 1 if Linux kernel CONFIG_CPUSETS=y; 0 otherwise. + * For backwards compatibility only + * New users should include processgroup/sched_policy.h directly */ -extern bool cpusets_enabled(); - -/* - * Check if Linux kernel enables SCHEDTUNE feature (only available in Android - * common kernel or Linaro LSK, not in mainline Linux as of v4.9) - * - * Return value: 1 if Linux kernel CONFIG_CGROUP_SCHEDTUNE=y; 0 otherwise. - */ -extern bool schedboost_enabled(); - -/* Keep in sync with THREAD_GROUP_* in frameworks/base/core/java/android/os/Process.java */ -typedef enum { - SP_DEFAULT = -1, - SP_BACKGROUND = 0, - SP_FOREGROUND = 1, - SP_SYSTEM = 2, // can't be used with set_sched_policy() - SP_AUDIO_APP = 3, - SP_AUDIO_SYS = 4, - SP_TOP_APP = 5, - SP_RT_APP = 6, - SP_RESTRICTED = 7, - SP_CNT, - SP_MAX = SP_CNT - 1, - SP_SYSTEM_DEFAULT = SP_FOREGROUND, -} SchedPolicy; - -extern int set_cpuset_policy(int tid, SchedPolicy policy); - -/* Assign thread tid to the cgroup associated with the specified policy. - * If the thread is a thread group leader, that is it's gettid() == getpid(), - * then the other threads in the same thread group are _not_ affected. - * On platforms which support gettid(), zero tid means current thread. - * Return value: 0 for success, or -errno for error. - */ -extern int set_sched_policy(int tid, SchedPolicy policy); - -/* Return the policy associated with the cgroup of thread tid via policy pointer. - * On platforms which support gettid(), zero tid means current thread. - * Return value: 0 for success, or -1 for error and set errno. - */ -extern int get_sched_policy(int tid, SchedPolicy *policy); - -/* Return a displayable string corresponding to policy. - * Return value: non-NULL NUL-terminated name of unspecified length; - * the caller is responsible for displaying the useful part of the string. - */ -extern const char *get_sched_policy_name(SchedPolicy policy); - -#ifdef __cplusplus -} -#endif +#include #endif /* __CUTILS_SCHED_POLICY_H */ diff --git a/libcutils/tests/Android.bp b/libcutils/tests/Android.bp index 788419038..72ae55921 100644 --- a/libcutils/tests/Android.bp +++ b/libcutils/tests/Android.bp @@ -59,6 +59,7 @@ test_libraries = [ "libcutils", "liblog", "libbase", + "libprocessgroup", ] cc_test { diff --git a/libprocessgroup/Android.bp b/libprocessgroup/Android.bp index c38279df4..d04a79a66 100644 --- a/libprocessgroup/Android.bp +++ b/libprocessgroup/Android.bp @@ -1,10 +1,45 @@ +cc_library_headers { + name: "libprocessgroup_headers", + vendor_available: true, + recovery_available: true, + host_supported: true, + export_include_dirs: ["include"], + target: { + linux_bionic: { + enabled: true, + }, + windows: { + enabled: true, + }, + }, +} + cc_library { - srcs: ["processgroup.cpp"], + srcs: [ + "processgroup.cpp", + "sched_policy.cpp", + ], name: "libprocessgroup", host_supported: true, recovery_available: true, - shared_libs: ["libbase"], + vendor_available: true, + vndk: { + enabled: true, + support_system_process: true, + }, + shared_libs: [ + "libbase", + "liblog", + ], + // for cutils/android_filesystem_config.h + header_libs: [ + "libcutils_headers", + "libprocessgroup_headers", + ], export_include_dirs: ["include"], + export_header_lib_headers: [ + "libprocessgroup_headers", + ], cflags: [ "-Wall", "-Werror", diff --git a/libprocessgroup/include/processgroup/sched_policy.h b/libprocessgroup/include/processgroup/sched_policy.h new file mode 100644 index 000000000..79a32fdeb --- /dev/null +++ b/libprocessgroup/include/processgroup/sched_policy.h @@ -0,0 +1,80 @@ +/* + * Copyright (C) 2018 The Android Open Source Project + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +#pragma once + +#include + +#ifdef __cplusplus +extern "C" { +#endif + +/* + * Check if Linux kernel enables CPUSETS feature. + * + * Return value: 1 if Linux kernel CONFIG_CPUSETS=y; 0 otherwise. + */ +extern bool cpusets_enabled(); + +/* + * Check if Linux kernel enables SCHEDTUNE feature (only available in Android + * common kernel or Linaro LSK, not in mainline Linux as of v4.9) + * + * Return value: 1 if Linux kernel CONFIG_CGROUP_SCHEDTUNE=y; 0 otherwise. + */ +extern bool schedboost_enabled(); + +/* Keep in sync with THREAD_GROUP_* in frameworks/base/core/java/android/os/Process.java */ +typedef enum { + SP_DEFAULT = -1, + SP_BACKGROUND = 0, + SP_FOREGROUND = 1, + SP_SYSTEM = 2, // can't be used with set_sched_policy() + SP_AUDIO_APP = 3, + SP_AUDIO_SYS = 4, + SP_TOP_APP = 5, + SP_RT_APP = 6, + SP_RESTRICTED = 7, + SP_CNT, + SP_MAX = SP_CNT - 1, + SP_SYSTEM_DEFAULT = SP_FOREGROUND, +} SchedPolicy; + +extern int set_cpuset_policy(int tid, SchedPolicy policy); + +/* Assign thread tid to the cgroup associated with the specified policy. + * If the thread is a thread group leader, that is it's gettid() == getpid(), + * then the other threads in the same thread group are _not_ affected. + * On platforms which support gettid(), zero tid means current thread. + * Return value: 0 for success, or -errno for error. + */ +extern int set_sched_policy(int tid, SchedPolicy policy); + +/* Return the policy associated with the cgroup of thread tid via policy pointer. + * On platforms which support gettid(), zero tid means current thread. + * Return value: 0 for success, or -1 for error and set errno. + */ +extern int get_sched_policy(int tid, SchedPolicy *policy); + +/* Return a displayable string corresponding to policy. + * Return value: non-NULL NUL-terminated name of unspecified length; + * the caller is responsible for displaying the useful part of the string. + */ +extern const char *get_sched_policy_name(SchedPolicy policy); + +#ifdef __cplusplus +} +#endif diff --git a/libprocessgroup/processgroup.cpp b/libprocessgroup/processgroup.cpp index 9df8dd96f..8d2ac3d57 100644 --- a/libprocessgroup/processgroup.cpp +++ b/libprocessgroup/processgroup.cpp @@ -42,7 +42,7 @@ #include #include #include -#include +#include #include diff --git a/libcutils/sched_policy.cpp b/libprocessgroup/sched_policy.cpp similarity index 99% rename from libcutils/sched_policy.cpp rename to libprocessgroup/sched_policy.cpp index 3fa548f78..f95d7e48f 100644 --- a/libcutils/sched_policy.cpp +++ b/libprocessgroup/sched_policy.cpp @@ -14,7 +14,7 @@ ** limitations under the License. */ -#include +#include #define LOG_TAG "SchedPolicy" diff --git a/libutils/Android.bp b/libutils/Android.bp index 3e8417eb9..fb7ca3254 100644 --- a/libutils/Android.bp +++ b/libutils/Android.bp @@ -22,11 +22,13 @@ cc_library_headers { "liblog_headers", "libsystem_headers", "libcutils_headers", + "libprocessgroup_headers", ], export_header_lib_headers: [ "liblog_headers", "libsystem_headers", "libcutils_headers", + "libprocessgroup_headers", ], export_include_dirs: ["include"], @@ -82,6 +84,7 @@ cc_defaults { shared_libs: [ "libcutils", + "libprocessgroup", "libdl", "libvndksupport", ], diff --git a/libutils/Threads.cpp b/libutils/Threads.cpp index 64bc4025d..31ca1383e 100644 --- a/libutils/Threads.cpp +++ b/libutils/Threads.cpp @@ -36,7 +36,7 @@ #include -#include +#include #if defined(__ANDROID__) # define __android_unused diff --git a/lmkd/Android.bp b/lmkd/Android.bp index 903d0e243..f9ed57cce 100644 --- a/lmkd/Android.bp +++ b/lmkd/Android.bp @@ -5,6 +5,7 @@ cc_binary { shared_libs: [ "libcutils", "liblog", + "libprocessgroup", ], static_libs: [ "libstatslogc", diff --git a/logcat/Android.bp b/logcat/Android.bp index 0543aba73..5030b1563 100644 --- a/logcat/Android.bp +++ b/logcat/Android.bp @@ -24,8 +24,8 @@ cc_defaults { ], shared_libs: [ "libbase", - "libcutils", "libpcrecpp", + "libprocessgroup", ], static_libs: ["liblog"], logtags: ["event.logtags"], diff --git a/logcat/logcat.cpp b/logcat/logcat.cpp index 87bc6ae84..15e07feef 100644 --- a/logcat/logcat.cpp +++ b/logcat/logcat.cpp @@ -49,11 +49,11 @@ #include #include #include -#include #include #include #include #include +#include #include #include diff --git a/logd/Android.bp b/logd/Android.bp index 3abfc2171..bdbdf12e4 100644 --- a/logd/Android.bp +++ b/logd/Android.bp @@ -73,6 +73,7 @@ cc_binary { "libcutils", "libbase", "libpackagelistparser", + "libprocessgroup", "libcap", ], diff --git a/logd/main.cpp b/logd/main.cpp index 8c38d9aa3..fd3cdf877 100644 --- a/logd/main.cpp +++ b/logd/main.cpp @@ -38,12 +38,12 @@ #include #include #include -#include #include #include #include #include #include +#include #include #include "CommandListener.h"