Skip to content

Commit 39c0a53

Browse files
Al Grantacmel
authored andcommitted
perf tools: Correct SNOOPX field offset
perf_event.h has macros that define the field offsets in the data_src bitmask in perf records. The SNOOPX and REMOTE offsets were both 37. These are distinct fields, and the bitfield layout in perf_mem_data_src confirms that SNOOPX should be at offset 38. Committer notes: This was extracted from a larger patch that also contained kernel changes. Fixes: 52839e6 ("perf tools: Add support for printing new mem_info encodings") Signed-off-by: Al Grant <al.grant@arm.com> Reviewed-by: Andi Kleen <ak@linux.intel.com> Cc: Adrian Hunter <adrian.hunter@intel.com> Cc: Ian Rogers <irogers@google.com> Cc: Jiri Olsa <jolsa@kernel.org> Cc: Namhyung Kim <namhyung@kernel.org> Cc: Peter Zijlstra <peterz@infradead.org> Link: http://lore.kernel.org/lkml/9974f2d0-bf7f-518e-d9f7-4520e5ff1bb0@foss.arm.com Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
1 parent a347306 commit 39c0a53

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

tools/include/uapi/linux/perf_event.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1196,7 +1196,7 @@ union perf_mem_data_src {
11961196

11971197
#define PERF_MEM_SNOOPX_FWD 0x01 /* forward */
11981198
/* 1 free */
1199-
#define PERF_MEM_SNOOPX_SHIFT 37
1199+
#define PERF_MEM_SNOOPX_SHIFT 38
12001200

12011201
/* locked instruction */
12021202
#define PERF_MEM_LOCK_NA 0x01 /* not available */

0 commit comments

Comments
 (0)