On Tue, 2016-07-19 at 14:07 +0200, Dario Faggioli wrote: > --- a/xen/common/sched_credit2.c > +++ b/xen/common/sched_credit2.c > @@ -656,7 +656,8 @@ __update_runq_load(const struct scheduler *ops, > rqd->load += change; > rqd->load_last_update = now; > > - ASSERT(rqd->avgload <= STIME_MAX && rqd->b_avgload <= > STIME_MAX); > + /* Overflow, capable of making the load look negative, must not > occur. */ > + ASSERT(rqd->avgload > 0 && rqd->b_avgload > 0); > Wait! This quite obviously wants to be '>= 0' !!
Sorry for the glaring mistake. v2 coming... Dario -- <<This happens because I choose it to happen!>> (Raistlin Majere) ----------------------------------------------------------------- Dario Faggioli, Ph.D, http://about.me/dario.faggioli Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)
signature.asc
Description: This is a digitally signed message part
_______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org https://lists.xen.org/xen-devel
