diff options
author | Rasmus Villemoes <linux@rasmusvillemoes.dk> | 2015-01-22 12:01:23 -0500 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2015-01-22 15:03:01 -0500 |
commit | 605a3069161dd966d6cea795133c673fb6706e52 (patch) | |
tree | b7033e68d785b18e925370a4ef95f1b545be6f0c /tools | |
parent | f816b3cc99804a9f771315331deb36abec47f5b4 (diff) |
perf tests: Fix typo in sample-parsing.c
It was testing the same buffer for differences:
memcmp(s1->user_stack.data, s1->user_stack.data, s1->user_stack.size)
I'm pretty sure this wasn't supposed to be dead code.
Signed-off-by: Rasmus Villemoes <linux@rasmusvillemoes.dk>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/1421946083-29863-1-git-send-email-linux@rasmusvillemoes.dk
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools')
-rw-r--r-- | tools/perf/tests/sample-parsing.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/perf/tests/sample-parsing.c b/tools/perf/tests/sample-parsing.c index 4908c648a597..30c02181e78b 100644 --- a/tools/perf/tests/sample-parsing.c +++ b/tools/perf/tests/sample-parsing.c | |||
@@ -110,7 +110,7 @@ static bool samples_same(const struct perf_sample *s1, | |||
110 | 110 | ||
111 | if (type & PERF_SAMPLE_STACK_USER) { | 111 | if (type & PERF_SAMPLE_STACK_USER) { |
112 | COMP(user_stack.size); | 112 | COMP(user_stack.size); |
113 | if (memcmp(s1->user_stack.data, s1->user_stack.data, | 113 | if (memcmp(s1->user_stack.data, s2->user_stack.data, |
114 | s1->user_stack.size)) { | 114 | s1->user_stack.size)) { |
115 | pr_debug("Samples differ at 'user_stack'\n"); | 115 | pr_debug("Samples differ at 'user_stack'\n"); |
116 | return false; | 116 | return false; |