From: Dario Faggioli Date: Tue, 3 May 2016 21:46:50 +0000 (+0200) Subject: xen: credit2: fix 2 (minor) issues in load tracking logic X-Git-Tag: archive/raspbian/4.8.0-1+rpi1~1^2~1160 X-Git-Url: https://dgit.raspbian.org/?a=commitdiff_plain;h=adf7d04317b4c4e60db42314ad874a45d80e65b9;p=xen.git xen: credit2: fix 2 (minor) issues in load tracking logic All calculations that involve load_last_update uses quantities shifted by LOADAVG_GRANULARITY_SHIFT, so make sure that this is true even when the field is assigned a value for the first time, during vcpu allocation. Also, during migration, while the loads of both the source and destination runqueues certainly need changing, the vcpu being moved does not change its running/non-running status, and its calculated load should hence not be affected. Signed-off-by: Dario Faggioli Reviewed-by: George Dunlap Release-acked-by: Wei Liu --- diff --git a/xen/common/sched_credit2.c b/xen/common/sched_credit2.c index dee961b9e5..f3b62acfca 100644 --- a/xen/common/sched_credit2.c +++ b/xen/common/sched_credit2.c @@ -867,7 +867,7 @@ csched2_alloc_vdata(const struct scheduler *ops, struct vcpu *vc, void *dd) svc->weight = svc->sdom->weight; /* Starting load of 50% */ svc->avgload = 1ULL << (CSCHED2_PRIV(ops)->load_window_shift - 1); - svc->load_last_update = NOW(); + svc->load_last_update = NOW() >> LOADAVG_GRANULARITY_SHIFT; } else { @@ -1301,7 +1301,7 @@ static void migrate(const struct scheduler *ops, if ( __vcpu_on_runq(svc) ) { __runq_remove(svc); - update_load(ops, svc->rqd, svc, -1, now); + update_load(ops, svc->rqd, NULL, -1, now); on_runq=1; } __runq_deassign(svc); @@ -1314,7 +1314,7 @@ static void migrate(const struct scheduler *ops, __runq_assign(svc, trqd); if ( on_runq ) { - update_load(ops, svc->rqd, svc, 1, now); + update_load(ops, svc->rqd, NULL, 1, now); runq_insert(ops, svc->vcpu->processor, svc); runq_tickle(ops, svc->vcpu->processor, svc, now); SCHED_STAT_CRANK(migrate_on_runq);