Re: [PATCH 2/3] sched: introduce per-cpu var next_cpu to track search limit

From: Peter Zijlstra
Date: Tue Apr 24 2018 - 08:47:25 EST


On Mon, Apr 23, 2018 at 05:41:15PM -0700, subhra mazumdar wrote:
> @@ -17,6 +17,7 @@
> #include <trace/events/sched.h>
>
> DEFINE_PER_CPU_SHARED_ALIGNED(struct rq, runqueues);
> +DEFINE_PER_CPU_SHARED_ALIGNED(int, next_cpu);
>
> #if defined(CONFIG_SCHED_DEBUG) && defined(HAVE_JUMP_LABEL)
> /*
> @@ -6018,6 +6019,7 @@ void __init sched_init(void)
> struct rq *rq;
>
> rq = cpu_rq(i);
> + per_cpu(next_cpu, i) = -1;

If you leave it uninitialized it'll be 0, and we can avoid that extra
branch in the next patch, no?