RE: [PATCH V3 11/12] perf top: switch default mode to overwrite mode

2018-01-11 Thread Liang, Kan
> On Thu, Dec 21, 2017 at 10:08:53AM -0800, kan.li...@intel.com wrote: > > SNIP > > > .max_stack = sysctl_perf_event_max_stack, > > .sym_pcnt_filter = 5, > > diff --git a/tools/perf/ui/browsers/hists.c b/tools/perf/ui/browsers/hists.c > > index

RE: [PATCH V3 11/12] perf top: switch default mode to overwrite mode

2018-01-11 Thread Liang, Kan
> On Thu, Dec 21, 2017 at 10:08:53AM -0800, kan.li...@intel.com wrote: > > SNIP > > > .max_stack = sysctl_perf_event_max_stack, > > .sym_pcnt_filter = 5, > > diff --git a/tools/perf/ui/browsers/hists.c b/tools/perf/ui/browsers/hists.c > > index

RE: [PATCH V3 11/12] perf top: switch default mode to overwrite mode

2018-01-11 Thread Liang, Kan
> SNIP > > > .max_stack = sysctl_perf_event_max_stack, > > .sym_pcnt_filter = 5, > > diff --git a/tools/perf/ui/browsers/hists.c b/tools/perf/ui/browsers/hists.c > > index 68146f4..56023e4 100644 > > --- a/tools/perf/ui/browsers/hists.c > > +++

RE: [PATCH V3 11/12] perf top: switch default mode to overwrite mode

2018-01-11 Thread Liang, Kan
> SNIP > > > .max_stack = sysctl_perf_event_max_stack, > > .sym_pcnt_filter = 5, > > diff --git a/tools/perf/ui/browsers/hists.c b/tools/perf/ui/browsers/hists.c > > index 68146f4..56023e4 100644 > > --- a/tools/perf/ui/browsers/hists.c > > +++

Re: [PATCH V3 11/12] perf top: switch default mode to overwrite mode

2018-01-11 Thread Jiri Olsa
On Thu, Dec 21, 2017 at 10:08:53AM -0800, kan.li...@intel.com wrote: > From: Kan Liang > > perf_top__mmap_read has severe performance issue in > Knights Landing/Mill, when monitoring in heavy load system. It costs > several minutes to finish, which is unacceptable. > >

Re: [PATCH V3 11/12] perf top: switch default mode to overwrite mode

2018-01-11 Thread Jiri Olsa
On Thu, Dec 21, 2017 at 10:08:53AM -0800, kan.li...@intel.com wrote: > From: Kan Liang > > perf_top__mmap_read has severe performance issue in > Knights Landing/Mill, when monitoring in heavy load system. It costs > several minutes to finish, which is unacceptable. > > Currently, perf top is

Re: [PATCH V3 11/12] perf top: switch default mode to overwrite mode

2018-01-11 Thread Jiri Olsa
On Thu, Dec 21, 2017 at 10:08:53AM -0800, kan.li...@intel.com wrote: SNIP > .max_stack = sysctl_perf_event_max_stack, > .sym_pcnt_filter = 5, > diff --git a/tools/perf/ui/browsers/hists.c b/tools/perf/ui/browsers/hists.c > index 68146f4..56023e4 100644 >

Re: [PATCH V3 11/12] perf top: switch default mode to overwrite mode

2018-01-11 Thread Jiri Olsa
On Thu, Dec 21, 2017 at 10:08:53AM -0800, kan.li...@intel.com wrote: SNIP > .max_stack = sysctl_perf_event_max_stack, > .sym_pcnt_filter = 5, > diff --git a/tools/perf/ui/browsers/hists.c b/tools/perf/ui/browsers/hists.c > index 68146f4..56023e4 100644 >

[PATCH V3 11/12] perf top: switch default mode to overwrite mode

2017-12-21 Thread kan . liang
From: Kan Liang perf_top__mmap_read has severe performance issue in Knights Landing/Mill, when monitoring in heavy load system. It costs several minutes to finish, which is unacceptable. Currently, perf top is non overwrite mode. For non overwrite mode, it tries to read

[PATCH V3 11/12] perf top: switch default mode to overwrite mode

2017-12-21 Thread kan . liang
From: Kan Liang perf_top__mmap_read has severe performance issue in Knights Landing/Mill, when monitoring in heavy load system. It costs several minutes to finish, which is unacceptable. Currently, perf top is non overwrite mode. For non overwrite mode, it tries to read everything in the