• Paul E. McKenney's avatar
    clocksource: Improve "skew is too large" messages · dd029269
    Paul E. McKenney authored
    When clocksource_watchdog() detects excessive clocksource skew compared
    to the watchdog clocksource, it marks the clocksource under test as
    unstable and prints several lines worth of message.  But that message
    is unclear to anyone unfamiliar with the code:
    
    clocksource: timekeeping watchdog on CPU2: Marking clocksource 'wdtest-ktime' as unstable because the skew is too large:
    clocksource:                       'kvm-clock' wd_nsec: 400744390 wd_now: 612625c2c wd_last: 5fa7f7c66 mask: ffffffffffffffff
    clocksource:                       'wdtest-ktime' cs_nsec: 600744034 cs_now: 173081397a292d4f cs_last: 17308139565a8ced mask: ffffffffffffffff
    clocksource:                       'kvm-clock' (not 'wdtest-ktime') is current clocksource.
    
    Therefore, add the following line near the end of that message:
    
    Clocksource 'wdtest-ktime' skewed 199999644 ns (199 ms) over watchdog 'kvm-clock' interval of 400744390 ns (400 ms)
    
    This new line clearly indicates the amount of skew between the two
    clocksources, along with the duration of the time interval over which
    the skew occurred, both in nanoseconds and milliseconds.
    Signed-off-by: default avatarPaul E. McKenney <paulmck@kernel.org>
    Cc: John Stultz <jstultz@google.com>
    Cc: Thomas Gleixner <tglx@linutronix.de>
    Cc: Stephen Boyd <sboyd@kernel.org>
    Cc: Feng Tang <feng.tang@intel.com>
    dd029269
clocksource.c 41.9 KB