• Namhyung Kim's avatar
    perf lock contention: Fix debug stat if no contention · 35bf007e
    Namhyung Kim authored
    It should not divide if the total number is 0.  Otherwise it'd show
    NaN in the bad rate output.  Also add a whitespace in the "output
    for debug" message.
    
      $ sudo perf lock contention -abv true
      Looking at the vmlinux_path (8 entries long)
      symsrc__init: cannot get elf header.
      Using /proc/kcore for kernel data
      Using /proc/kallsyms for symbols
       contended   total wait     max wait     avg wait         type   caller
    
      === output for debug===
    
      bad: 0, total: 0
      bad rate: -nan %     <-------------------------  (here)
      histogram of events caused bad sequence
          acquire: 0
         acquired: 0
        contended: 0
          release: 0
    Signed-off-by: default avatarNamhyung Kim <namhyung@kernel.org>
    Cc: Adrian Hunter <adrian.hunter@intel.com>
    Cc: Hao Luo <haoluo@google.com>
    Cc: Ian Rogers <irogers@google.com>
    Cc: Ingo Molnar <mingo@kernel.org>
    Cc: Jiri Olsa <jolsa@kernel.org>
    Cc: Peter Zijlstra <peterz@infradead.org>
    Cc: bpf@vger.kernel.org
    Link: https://lore.kernel.org/r/20230327225711.245738-1-namhyung@kernel.orgSigned-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
    35bf007e
builtin-lock.c 54.8 KB