The patch titled clocksource: save mult_orig in clocksource_disable() has been added to the -mm tree. Its filename is clocksource-save-mult_orig-in-clocksource_disable.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: clocksource: save mult_orig in clocksource_disable() From: Magnus Damm <damm@xxxxxxxxxx> Save clocksource mult_orig in clocksource_disable(). To fix the common case where ->enable() does not setup mult, make sure mult_orig is saved in mult on disable. Also add comments to explain why we do this. Signed-off-by: Magnus Damm <damm@xxxxxxxxxx> Acked-by: John Stultz <johnstul@xxxxxxxxxx> Cc: Thomas Gleixner <tglx@xxxxxxxxxxxxx> Cc: Ingo Molnar <mingo@xxxxxxx> Cc: Paul Mundt <lethal@xxxxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- include/linux/clocksource.h | 12 +++++++++++- 1 file changed, 11 insertions(+), 1 deletion(-) diff -puN include/linux/clocksource.h~clocksource-save-mult_orig-in-clocksource_disable include/linux/clocksource.h --- a/include/linux/clocksource.h~clocksource-save-mult_orig-in-clocksource_disable +++ a/include/linux/clocksource.h @@ -293,7 +293,11 @@ static inline int clocksource_enable(str if (cs->enable) ret = cs->enable(cs); - /* save mult_orig on enable */ + /* The frequency may have changed while the clocksource + * was disabled. If so the code in ->enable() must update + * the mult value to reflect the new frequency. Make sure + * mult_orig follows this change. + */ cs->mult_orig = cs->mult; return ret; @@ -309,6 +313,12 @@ static inline int clocksource_enable(str */ static inline void clocksource_disable(struct clocksource *cs) { + /* Save mult_orig in mult so clocksource_enable() can + * restore the value regardless if ->enable() updates + * the value of mult or not. + */ + cs->mult = cs->mult_orig; + if (cs->disable) cs->disable(cs); } _ Patches currently in -mm which might be from damm@xxxxxxxxxx are linux-next.patch clocksource-save-mult_orig-in-clocksource_disable.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