shinys000114/linux
1
0
Fork 0
mirror of https://github.com/hardkernel/linux.git synced 2026-03-31 18:23:00 +09:00
Code Issues Packages Projects Releases Wiki Activity

ANDROID: sched/tune: Add support for negative boost values

Browse Source 
Change-Id: I164ee04ba98c3a776605f18cb65ee61b3e917939

Contains also:

eas/stune: schedtune cpu boost_max must be non-negative.

This is to avoid under-accounting cpu capacity which may
cause task stacking and frequency spikes.

Change-Id: Ie1c1cbd52a6edb77b4c15a830030aa748dff6f29
Signed-off-by: Andres Oportus <andresoportus@google.com>
This commit is contained in:
Srinath Sridharan
2016-07-28 17:28:55 +01:00
committed by Dmitry Shmidt
parent c5b2042fbc
commit e71c425516
3 changed files with 47 additions and 34 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
include/trace/events/sched.h
View File

@@ -706,14 +706,14 @@ TRACE_EVENT(sched_tune_config,
*/
TRACE_EVENT(sched_boost_cpu,
TP_PROTO(int cpu, unsigned long util, unsigned long margin),
TP_PROTO(int cpu, unsigned long util, long margin),
TP_ARGS(cpu, util, margin),
TP_STRUCT__entry(
__field( int, cpu )
__field( unsigned long, util )
__field( unsigned long, margin )
__field(long, margin )
),
TP_fast_assign(
@@ -722,7 +722,7 @@ TRACE_EVENT(sched_boost_cpu,
__entry->margin = margin;
),
TP_printk("cpu=%d util=%lu margin=%lu",
TP_printk("cpu=%d util=%lu margin=%ld",
__entry->cpu,
__entry->util,
__entry->margin)
@@ -734,7 +734,7 @@ TRACE_EVENT(sched_boost_cpu,
TRACE_EVENT(sched_tune_tasks_update,
TP_PROTO(struct task_struct *tsk, int cpu, int tasks, int idx,
unsigned int boost, unsigned int max_boost),
int boost, int max_boost),
TP_ARGS(tsk, cpu, tasks, idx, boost, max_boost),
@@ -744,8 +744,8 @@ TRACE_EVENT(sched_tune_tasks_update,
__field( int, cpu )
__field( int, tasks )
__field( int, idx )
__field( unsigned int, boost )
__field( unsigned int, max_boost )
__field( int, boost )
__field( int, max_boost )
),
TP_fast_assign(
@@ -759,7 +759,7 @@ TRACE_EVENT(sched_tune_tasks_update,
),
TP_printk("pid=%d comm=%s "
"cpu=%d tasks=%d idx=%d boost=%u max_boost=%u",
"cpu=%d tasks=%d idx=%d boost=%d max_boost=%d",
__entry->pid, __entry->comm,
__entry->cpu, __entry->tasks, __entry->idx,
__entry->boost, __entry->max_boost)
@@ -795,7 +795,7 @@ TRACE_EVENT(sched_tune_boostgroup_update,
*/
TRACE_EVENT(sched_boost_task,
TP_PROTO(struct task_struct *tsk, unsigned long util, unsigned long margin),
TP_PROTO(struct task_struct *tsk, unsigned long util, long margin),
TP_ARGS(tsk, util, margin),
@@ -803,7 +803,7 @@ TRACE_EVENT(sched_boost_task,
__array( char, comm, TASK_COMM_LEN )
__field( pid_t, pid )
__field( unsigned long, util )
__field( unsigned long, margin )
__field( long, margin )
),
@@ -814,7 +814,7 @@ TRACE_EVENT(sched_boost_task,
__entry->margin = margin;
),
TP_printk("comm=%s pid=%d util=%lu margin=%lu",
TP_printk("comm=%s pid=%d util=%lu margin=%ld",
__entry->comm, __entry->pid,
__entry->util,
__entry->margin)

36
kernel/sched/fair.c
View File

@@ -5709,22 +5709,26 @@ static bool cpu_overutilized(int cpu)
#ifdef CONFIG_SCHED_TUNE
static unsigned long
schedtune_margin(unsigned long signal, unsigned long boost)
static long
schedtune_margin(unsigned long signal, long boost)
{
unsigned long long margin = 0;
long long margin = 0;
/*
* Signal proportional compensation (SPC)
*
* The Boost (B) value is used to compute a Margin (M) which is
* proportional to the complement of the original Signal (S):
* M = B * (SCHED_LOAD_SCALE - S)
* M = B * (SCHED_LOAD_SCALE - S), if B is positive
* M = B * S, if B is negative
* The obtained M could be used by the caller to "boost" S.
*/
margin = SCHED_CAPACITY_SCALE - signal;
margin *= boost;
if (boost >= 0) {
margin = SCHED_CAPACITY_SCALE - signal;
margin *= boost;
} else
margin = -signal * boost;
/*
* Fast integer division by constant:
* Constant : (C) = 100
@@ -5740,13 +5744,15 @@ schedtune_margin(unsigned long signal, unsigned long boost)
margin *= 1311;
margin >>= 17;
if (boost < 0)
margin *= -1;
return margin;
}
static inline unsigned int
static inline int
schedtune_cpu_margin(unsigned long util, int cpu)
{
unsigned int boost;
int boost;
#ifdef CONFIG_CGROUP_SCHEDTUNE
boost = schedtune_cpu_boost(cpu);
@@ -5759,12 +5765,12 @@ schedtune_cpu_margin(unsigned long util, int cpu)
return schedtune_margin(util, boost);
}
static inline unsigned long
static inline long
schedtune_task_margin(struct task_struct *task)
{
unsigned int boost;
int boost;
unsigned long util;
unsigned long margin;
long margin;
#ifdef CONFIG_CGROUP_SCHEDTUNE
boost = schedtune_task_boost(task);
@@ -5782,13 +5788,13 @@ schedtune_task_margin(struct task_struct *task)
#else /* CONFIG_SCHED_TUNE */
static inline unsigned int
static inline int
schedtune_cpu_margin(unsigned long util, int cpu)
{
return 0;
}
static inline unsigned int
static inline int
schedtune_task_margin(struct task_struct *task)
{
return 0;
@@ -5800,7 +5806,7 @@ static inline unsigned long
boosted_cpu_util(int cpu)
{
unsigned long util = cpu_util(cpu);
unsigned long margin = schedtune_cpu_margin(util, cpu);
long margin = schedtune_cpu_margin(util, cpu);
trace_sched_boost_cpu(cpu, util, margin);
@@ -5811,7 +5817,7 @@ static inline unsigned long
boosted_task_util(struct task_struct *task)
{
unsigned long util = task_util(task);
unsigned long margin = schedtune_task_margin(task);
long margin = schedtune_task_margin(task);
trace_sched_boost_task(task, util, margin);

25
kernel/sched/tune.c
View File

@@ -213,10 +213,11 @@ static struct schedtune *allocated_group[BOOSTGROUPS_COUNT] = {
*/
struct boost_groups {
/* Maximum boost value for all RUNNABLE tasks on a CPU */
unsigned boost_max;
bool idle;
int boost_max;
struct {
/* The boost for tasks on that boost group */
unsigned boost;
int boost;
/* Count of RUNNABLE tasks on that boost group */
unsigned tasks;
} group[BOOSTGROUPS_COUNT];
@@ -229,7 +230,7 @@ static void
schedtune_cpu_update(int cpu)
{
struct boost_groups *bg;
unsigned boost_max;
int boost_max;
int idx;
bg = &per_cpu(cpu_boost_groups, cpu);
@@ -243,9 +244,13 @@ schedtune_cpu_update(int cpu)
*/
if (bg->group[idx].tasks == 0)
continue;
boost_max = max(boost_max, bg->group[idx].boost);
}
/* Ensures boost_max is non-negative when all cgroup boost values
* are neagtive. Avoids under-accounting of cpu capacity which may cause
* task stacking and frequency spikes.*/
boost_max = max(boost_max, 0);
bg->boost_max = boost_max;
}
@@ -391,7 +396,7 @@ int schedtune_task_boost(struct task_struct *p)
return task_boost;
}
static u64
static s64
boost_read(struct cgroup_subsys_state *css, struct cftype *cft)
{
struct schedtune *st = css_st(css);
@@ -401,11 +406,13 @@ boost_read(struct cgroup_subsys_state *css, struct cftype *cft)
static int
boost_write(struct cgroup_subsys_state *css, struct cftype *cft,
u64 boost)
s64 boost)
{
struct schedtune *st = css_st(css);
unsigned threshold_idx;
int boost_pct;
if (boost < 0 || boost > 100)
if (boost < -100 || boost > 100)
return -EINVAL;
st->boost = boost;
@@ -423,8 +430,8 @@ boost_write(struct cgroup_subsys_state *css, struct cftype *cft,
static struct cftype files[] = {
{
.name = "boost",
.read_u64 = boost_read,
.write_u64 = boost_write,
.read_s64 = boost_read,
.write_s64 = boost_write,
},
{ } /* terminate */
};