Re: [PATCH v1] perf stat: avoid 10ms limit for printing event counts
From: Andi Kleen
Date: Tue Mar 27 2018 - 05:07:07 EST
> When running perf stat -I for monitoring e.g. PCIe uncore counters and
> at the same time profiling some I/O workload by perf record e.g. for
> cpu-cycles and context switches, it is then possible to build and
> observe good-enough consolidated CPU/OS/IO(Uncore) performance picture
> for that workload.
At some point I still hope we can make uncore measurements in
perf record work. Kan tried at some point to allow multiple
PMUs in a group, but was not successfull. But perhaps we
can sample them from a software event instead.
>
> The warning on possible runtime overhead is still preserved, however
> it is only visible when specifying -v option.
I would print it unconditionally. Very few people use -v.
BTW better of course would be to occasionally measure the perf stat
cpu time and only print the warning if it's above some percentage
of a CPU. But that would be much more work.
Rest looks ok.
-Andi