On Mon, Mar 30, 2015 at 3:24 PM, David Ahern <dsah...@gmail.com> wrote: > On 3/30/15 4:19 PM, Stephane Eranian wrote: >> >> @@ -761,6 +762,12 @@ void perf_evsel__config(struct perf_evsel *evsel, >> struct record_opts *opts) >> attr->disabled = 0; >> attr->enable_on_exec = 0; >> } >> + >> + if (opts->clockid >= 0) { >> + attr->use_clockid = 1; >> + attr->clockid = opts->clockid; >> + } else >> + attr->clockid = -1; >> } > > > The else part needs to be removed (can't set clockid to -1). And then ... > >> diff --git a/tools/perf/util/header.c b/tools/perf/util/header.c >> index fb43215..164441a 100644 >> --- a/tools/perf/util/header.c >> +++ b/tools/perf/util/header.c >> @@ -1098,6 +1098,8 @@ static void print_event_desc(struct perf_header *ph, >> int fd, FILE *fp) >> } >> fprintf(fp, " }"); >> } >> + fprintf(fp, ", clockid = %d", evsel->attr.clockid); >> + > > > this should be prefaced with: > > if (evsel->attr.use_clockid) > > (use_clockid bit says whether the clockid value was set)
Ok, will fix in V7 or Arnaldo grabs the patch and merges it. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/