On Wed, Nov 9, 2016 at 8:55 PM, Chen Yu <yu.c.c...@intel.com> wrote: > Previously we encountered some memory overflow issues due to > the bogus sleep time brought by inconsistent rtc, which is > triggered when pm_trace is enabled, and we have fixed it > in recent kernel. However it's improper in the first place > to call __timekeeping_inject_sleeptime() in case that pm_trace > is enabled simply because that "hash" time value will wreckage > the timekeeping subsystem. > > This patch is originally written by Thomas, which would bypass > the bogus rtc interval when pm_trace is enabled. > Meanwhile, if system succeed to resume back with pm_trace set, the > users are warned to adjust the bogus rtc either by ntp-date or rdate, > by resetting pm_trace_rtc_abused to false, otherwise above tools might > not work as expected. > > Originally-from: Thomas Gleixner <t...@linutronix.de> > Cc: "Rafael J. Wysocki" <r...@rjwysocki.net> > Cc: John Stultz <john.stu...@linaro.org> > Cc: Xunlei Pang <xlp...@redhat.com> > Cc: Ingo Molnar <mi...@redhat.com> > Cc: Len Brown <l...@kernel.org> > Cc: "H. Peter Anvin" <h...@zytor.com> > Cc: Pavel Machek <pa...@ucw.cz> > Cc: Thomas Gleixner <t...@linutronix.de> > Signed-off-by: Chen Yu <yu.c.c...@intel.com>
Looks ok to me. I'm queueing it up for testing. If no one objects (and I don't see any issues) I'll merge it to my tree and push it to -tip through Thomas. thanks -john
