Re: [PATCH v13 1/8] perf tools: Drop redundant evsel->overwrite indicator

2016-07-06 Thread Wangnan (F)
On 2016/7/6 18:53, Jiri Olsa wrote: On Mon, Jul 04, 2016 at 06:20:02AM +, Wang Nan wrote: SNIP diff --git a/tools/perf/util/evsel.c b/tools/perf/util/evsel.c index 0fea724..3abe519 100644 --- a/tools/perf/util/evsel.c +++ b/tools/perf/util/evsel.c @@ -1359,6 +1359,9 @@ static int __perf_

Re: [PATCH v13 1/8] perf tools: Drop redundant evsel->overwrite indicator

2016-07-06 Thread Jiri Olsa
On Mon, Jul 04, 2016 at 06:20:02AM +, Wang Nan wrote: SNIP > diff --git a/tools/perf/util/evsel.c b/tools/perf/util/evsel.c > index 0fea724..3abe519 100644 > --- a/tools/perf/util/evsel.c > +++ b/tools/perf/util/evsel.c > @@ -1359,6 +1359,9 @@ static int __perf_evsel__open(struct perf_evsel *

[PATCH v13 1/8] perf tools: Drop redundant evsel->overwrite indicator

2016-07-03 Thread Wang Nan
From: Arnaldo Carvalho de Melo evsel->overwrite indicator means an event should be put into overwritable ring buffer. In current implementation, it equals to evsel->attr.write_backward. To reduce compliexity, remove evsel->overwrite, use evsel->attr.write_backward instead. Signed-off-by: Arnaldo