On 25 May 2015 at 03:49, Yuyang Du <yuyang...@intel.com> wrote: [snip]
> > @@ -2585,334 +2583,156 @@ static __always_inline int > __update_entity_runnable_avg(u64 now, int cpu, > periods = delta / 1024; > delta %= 1024; > > - sa->runnable_avg_sum = decay_load(sa->runnable_avg_sum, > - periods + 1); > - sa->running_avg_sum = decay_load(sa->running_avg_sum, > - periods + 1); > - sa->avg_period = decay_load(sa->avg_period, > - periods + 1); > + sa->load_sum = decay_load(sa->load_sum, periods + 1); > + sa->util_sum = decay_load(u64(sa->util_sum), periods + 1); Hi Yuyang, Brackets are missing around u64 to cast util_sum > > /* Efficiently calculate \sum (1..n_period) 1024*y^i */ > - runnable_contrib = __compute_runnable_contrib(periods); > - if (runnable) > - sa->runnable_avg_sum += runnable_contrib; > + contrib = __compute_runnable_contrib(periods); > + if (weight) > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
