perf evsel: Always use perf_evsel__set_sample_bit()
authorAdrian Hunter <[email protected]>
Fri, 1 Nov 2013 13:51:35 +0000 (15:51 +0200)
committerArnaldo Carvalho de Melo <[email protected]>
Mon, 4 Nov 2013 15:42:28 +0000 (12:42 -0300)
Always use perf_evsel__set_sample_bit() rather than just setting the
bit.

Signed-off-by: Adrian Hunter <[email protected]>
Cc: David Ahern <[email protected]>
Cc: Frederic Weisbecker <[email protected]>
Cc: Ingo Molnar <[email protected]>
Cc: Jiri Olsa <[email protected]>
Cc: Mike Galbraith <[email protected]>
Cc: Namhyung Kim <[email protected]>
Cc: Paul Mackerras <[email protected]>
Cc: Peter Zijlstra <[email protected]>
Cc: Stephane Eranian <[email protected]>
Link: http://lkml.kernel.org/r/[email protected]
[ Cope with 3090ffb "perf: Disable PERF_RECORD_MMAP2 support" ]
Signed-off-by: Arnaldo Carvalho de Melo <[email protected]>
tools/perf/util/evsel.c

index f0e65dec66a512a79b4a7b5ebbc1443fd28344c4..47bbf03aa7efac9c8bac15b7a778e037e033d1de 100644 (file)
@@ -663,7 +663,7 @@ void perf_evsel__config(struct perf_evsel *evsel,
        }
 
        if (opts->sample_address)
-               attr->sample_type       |= PERF_SAMPLE_DATA_SRC;
+               perf_evsel__set_sample_bit(evsel, DATA_SRC);
 
        if (opts->no_delay) {
                attr->watermark = 0;
@@ -675,13 +675,13 @@ void perf_evsel__config(struct perf_evsel *evsel,
        }
 
        if (opts->sample_weight)
-               attr->sample_type       |= PERF_SAMPLE_WEIGHT;
+               perf_evsel__set_sample_bit(evsel, WEIGHT);
 
        attr->mmap  = track;
        attr->comm  = track;
 
        if (opts->sample_transaction)
-               attr->sample_type       |= PERF_SAMPLE_TRANSACTION;
+               perf_evsel__set_sample_bit(evsel, TRANSACTION);
 
        /*
         * XXX see the function comment above