[PATCH v4] sched/fair: Remove SCHED_LOAD_SHIFT and SCHED_LOAD_SCALE

From: Ingo Molnar
Date: Thu May 05 2016 - 03:47:03 EST



* Yuyang Du <yuyang.du@xxxxxxxxx> wrote:

> After cleaning up the sched metrics, these two definitions that cause
> ambiguity are not needed any more. Use NICE_0_LOAD_SHIFT and NICE_0_LOAD
> instead (the names suggest clearly who they are).
>
> Suggested-by: Ben Segall <bsegall@xxxxxxxxxx>
> Signed-off-by: Yuyang Du <yuyang.du@xxxxxxxxx>

Yeah, so this patch was a bit of a trainwreck:

- it didn't build on 32-bit kernels

- a stale SCHED_LOAD_SHIFT definition was left around

- the title and th changelog lies actively: it's not a removal, but a complex
combination of a rename, replace and removal ...

I've fixed that all with the patch below, but _please_ be more careful in the
future when changing scheduler code, and please also read your changelogs and
patch titles before sending them out to make sure the label matches contents.

I'll push it all out in tip:sched/core if it passes testing.

Thanks,

Ingo

=======================>