Re: [PATCH 3/3] sched/debug: remove unnecessary CONFIG_SCHEDSTATS checks

[Josh Poimboeuf]

On Fri, Jun 03, 2016 at 03:44:41PM -0500, Josh Poimboeuf wrote:
> --- a/kernel/sched/fair.c
> +++ b/kernel/sched/fair.c
> @@ -3447,7 +3447,7 @@ set_next_entity(struct cfs_rq *cfs_rq, struct sched_entity *se)
>  
>  	update_stats_curr_start(cfs_rq, se);
>  	cfs_rq->curr = se;
> -#ifdef CONFIG_SCHEDSTATS
> +
>  	/*
>  	 * Track our maximum slice length, if the CPU's load is at
>  	 * least twice that of our own weight (i.e. dont track it
> @@ -3457,7 +3457,7 @@ set_next_entity(struct cfs_rq *cfs_rq, struct sched_entity *se)
>  		se->statistics.slice_max = max(se->statistics.slice_max,
>  			se->sum_exec_runtime - se->prev_sum_exec_runtime);
>  	}
> -#endif
> +
>  	se->prev_sum_exec_runtime = se->sum_exec_runtime;

Oops.  This part fails to build without CONFIG_SCHEDSTATS.  This hunk
can just be removed from the patch.

-- 
Josh