The patch titled cpuidle-add-a-repeating-pattern-detector-to-the-menu-governor-fix has been added to the -mm tree. Its filename is cpuidle-add-a-repeating-pattern-detector-to-the-menu-governor-fix.patch Before you just go and hit "reply", please: a) Consider who else should be cc'ed b) Prefer to cc a suitable mailing list as well c) Ideally: find the original patch on the mailing list and do a reply-to-all to that, adding suitable additional cc's *** Remember to use Documentation/SubmitChecklist when testing your code *** See http://userweb.kernel.org/~akpm/stuff/added-to-mm.txt to find out what to do about this The current -mm tree may be found at http://userweb.kernel.org/~akpm/mmotm/ ------------------------------------------------------ Subject: cpuidle-add-a-repeating-pattern-detector-to-the-menu-governor-fix From: Arjan van de Ven <arjan@xxxxxxxxxxxxx> Frank Rowand found a bug in the new pattern detector code where the average value was looked at before the sum of values got divided by the number of samples. Signed-off-by: Arjan van de Ven <arjan@xxxxxxxxxxxxxxx> Cc: Corrado Zoccolo <czoccolo@xxxxxxxxx> Cc: Frank Rowand <frank.rowand@xxxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- drivers/cpuidle/governors/menu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff -puN drivers/cpuidle/governors/menu.c~cpuidle-add-a-repeating-pattern-detector-to-the-menu-governor-fix drivers/cpuidle/governors/menu.c --- a/drivers/cpuidle/governors/menu.c~cpuidle-add-a-repeating-pattern-detector-to-the-menu-governor-fix +++ a/drivers/cpuidle/governors/menu.c @@ -205,12 +205,12 @@ static void detect_repeating_patterns(st /* first calculate average and standard deviation of the past */ for (i = 0; i < INTERVALS; i++) avg += data->intervals[i]; + avg = avg / INTERVALS; /* if the avg is beyond the known next tick, it's worthless */ if (avg > data->expected_us) return; - avg = avg / INTERVALS; for (i = 0; i < INTERVALS; i++) stddev += (data->intervals[i] - avg) * (data->intervals[i] - avg); _ Patches currently in -mm which might be from arjan@xxxxxxxxxxxxx are origin.patch linux-next.patch timer-add-on-stack-deferrable-timer-interfaces.patch x86-platform-driver-intelligent-power-sharing-driver.patch cpuidle-add-a-repeating-pattern-detector-to-the-menu-governor-fix.patch -- To unsubscribe from this list: send the line "unsubscribe mm-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html