sched: Use rq->clock_task instead of rq->clock for correctly maintaining load averages
authorPaul Turner <pjt@google.com>
Sat, 22 Jan 2011 04:45:02 +0000 (20:45 -0800)
committerIngo Molnar <mingo@elte.hu>
Wed, 26 Jan 2011 11:31:03 +0000 (12:31 +0100)
The delta in clock_task is a more fair attribution of how much time a tg has
been contributing load to the current cpu.

While not really important it also means we're more in sync (by magnitude)
with respect to periodic updates (since __update_curr deltas are clock_task
based).

Signed-off-by: Paul Turner <pjt@google.com>
Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
LKML-Reference: <20110122044852.007092349@google.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
kernel/sched_fair.c

index 1997383ba4d606b12bc5e067b70cd9607ac934cd..0c26e2df450ee534e79f1265851100245b30a9cd 100644 (file)
@@ -725,7 +725,7 @@ static void update_cfs_load(struct cfs_rq *cfs_rq, int global_update)
        if (cfs_rq->tg == &root_task_group)
                return;
 
-       now = rq_of(cfs_rq)->clock;
+       now = rq_of(cfs_rq)->clock_task;
        delta = now - cfs_rq->load_stamp;
 
        /* truncate load history at 4 idle periods */