Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1602313

'perf test tsc' failing, bisected to "sched/clock: Provide better clock continuity"

Show all headers | View raw

Hi, this entry is failing for a while:

[root@jouet ~]# perf test -v tsc
55: Convert perf time to TSC                   :
--- start ---
test child forked, pid 3008
mmap size 528384B
1st event perf time 93133455486631 tsc 15369449468752
rdtsc          time 93133464598760 tsc 15369473104358
2nd event perf time 93133455506961 tsc 15369449521485
test child finished with -1
---- end ----
Convert perf time to TSC: FAILED!
[root@jouet ~]#

I bisected it to the following kernel change, ideas?

[acme@felicio linux]$ git bisect good
5680d8094ffa9e5cfc81afdd865027ee6417c263 is the first bad commit
commit 5680d8094ffa9e5cfc81afdd865027ee6417c263
Author: Peter Zijlstra <peterz@infradead.org>
Date:   Thu Dec 15 13:36:17 2016 +0100

    sched/clock: Provide better clock continuity
    
    When switching between the unstable and stable variants it is
    currently possible that clock discontinuities occur.
    
    And while these will mostly be 'small', attempt to do better.
    
    As observed on my IVB-EP, the sched_clock() is ~1.5s ahead of the
    ktime_get_ns() based timeline at the point of switchover
    (sched_clock_init_late()) after SMP bringup.
    
    Equally, when the TSC is later found to be unstable -- typically
    because SMM tries to hide its SMI latencies by mucking with the TSC --
    we want to avoid large jumps.
    
    Since the clocksource watchdog reports the issue after the fact we
    cannot exactly fix up time, but since SMI latencies are typically
    small (~10ns range), the discontinuity is mainly due to drift between
    sched_clock() and ktime_get_ns() (which on my desktop is ~79s over
    24days).
    
    I dislike this patch because it adds overhead to the good case in
    favour of dealing with badness. But given the widespread failure of
    TSC stability this is worth it.
    
    Note that in case the TSC makes drastic jumps after SMP bringup we're
    still hosed. There's just not much we can do in that case without
    stupid overhead.
    
    If we were to somehow expose tsc_clocksource_reliable (which is hard
    because this code is also used on ia64 and parisc) we could avoid some
    of the newly introduced overhead.
    
    Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
    Cc: Linus Torvalds <torvalds@linux-foundation.org>
    Cc: Mike Galbraith <efault@gmx.de>
    Cc: Peter Zijlstra <peterz@infradead.org>
    Cc: Thomas Gleixner <tglx@linutronix.de>
    Cc: linux-kernel@vger.kernel.org
    Signed-off-by: Ingo Molnar <mingo@kernel.org>

:040000 040000 152545abe3b879aaa3cf053cdd58ef998c285529 3afcd0a5bc643fdd0fc994ee11cbfd87cfe4c30f M	kernel
[acme@felicio linux]$ 

Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread

Thread

'perf test tsc' failing, bisected to "sched/clock: Provide better  clock continuity" Arnaldo Carvalho de Melo <acme@kernel.org> - 2017-03-16 15:10 +0100
  Re: 'perf test tsc' failing, bisected to "sched/clock: Provide  better clock continuity" Peter Zijlstra <peterz@infradead.org> - 2017-03-16 16:00 +0100
    Re: 'perf test tsc' failing, bisected to "sched/clock: Provide  better clock continuity" Peter Zijlstra <peterz@infradead.org> - 2017-03-16 19:20 +0100
      Re: 'perf test tsc' failing, bisected to "sched/clock: Provide  better clock continuity" Arnaldo Carvalho de Melo <acme@kernel.org> - 2017-03-16 19:40 +0100
        Re: 'perf test tsc' failing, bisected to "sched/clock: Provide  better clock continuity" Arnaldo Carvalho de Melo <acme@kernel.org> - 2017-03-16 20:30 +0100
          Re: 'perf test tsc' failing, bisected to "sched/clock: Provide  better clock continuity" Arnaldo Carvalho de Melo <acme@kernel.org> - 2017-03-20 14:30 +0100
        Re: 'perf test tsc' failing, bisected to "sched/clock: Provide  better clock continuity" Arnaldo Carvalho de Melo <acme@kernel.org> - 2017-03-16 20:40 +0100

csiph-web