Re: [PATCH 2/2] sched: Trace point sched_stat_sleep should cover iowait case

[Steven Rostedt <rostedt@goodmis.org>]

[ Adding Peter Zijlstra to this ]

-- Steve


On Mon, 27 Jul 2015 09:11:52 -0400
yangoliver <yang_oliver@hotmail.com> wrote:

> Per sched_stat_sleep definition in sched.h, it should include
> iowait case. This can also relect the design of sum_sleep_runtime
> statistic, as this counter also includes the io_wait.
> 
> Signed-off-by: Yong Yang <yangoliver@gmail.com>
> ---
>  kernel/sched/fair.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c
> index d113c3b..85677bf 100644
> --- a/kernel/sched/fair.c
> +++ b/kernel/sched/fair.c
> @@ -3018,6 +3018,8 @@ static void enqueue_sleeper(struct cfs_rq *cfs_rq, struct sched_entity *se)
>  		se->statistics.sum_sleep_runtime += delta;
>  
>  		if (tsk) {
> +			trace_sched_stat_sleep(tsk, delta);
> +
>  			if (tsk->in_iowait) {
>  				se->statistics.iowait_sum += delta;
>  				se->statistics.iowait_count++;