Messages in this thread | | | From | Namhyung Kim <> | Subject | Re: [PATCH 3/3] perf timechart: add more options to IO mode | Date | Thu, 12 Jun 2014 10:03:39 +0900 |
| |
On Tue, 10 Jun 2014 19:04:54 +0400, Stanislav Fomichev wrote: > --io-skip-eagain - don't show EAGAIN errors > --io-min-time - make small io bursts visible > --io-merge-dist - merge adjacent events > > Signed-off-by: Stanislav Fomichev <stfomichev@yandex-team.ru> > --- > tools/perf/Documentation/perf-timechart.txt | 9 ++++++ > tools/perf/builtin-timechart.c | 49 +++++++++++++++++++++++++++-- > 2 files changed, 56 insertions(+), 2 deletions(-) > > diff --git a/tools/perf/Documentation/perf-timechart.txt b/tools/perf/Documentation/perf-timechart.txt > index ec6b46c7bca0..62c29656ad95 100644 > --- a/tools/perf/Documentation/perf-timechart.txt > +++ b/tools/perf/Documentation/perf-timechart.txt > @@ -64,6 +64,15 @@ TIMECHART OPTIONS > duration or tasks with given name. If number is given it's interpreted > as number of nanoseconds. If non-numeric string is given it's > interpreted as task name. > +--io-skip-eagain:: > + Don't draw EAGAIN IO events. > +--io-min-time:: > + Draw small events as if they lasted min-time. Useful when you need > + to see very small and fast IO. Default value is 1ms.
It's in nano-second unit, right? If so, it's very unconvenient for user to specify. Maybe we could support to parse unit (s, ms, us, ...) also.
> +--io-merge-dist:: > + Merge events that are merge-dist nanoseconds apart. > + Reduces number of figures on the SVG and makes it more render-friendly. > + Default value is 1us.
Ditto.
Thanks, Namhyung
| |