diff options
author | Jiri Olsa <jolsa@kernel.org> | 2019-05-08 09:20:07 -0400 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2019-05-28 17:37:44 -0400 |
commit | 4f600bcf657d4d0476d0d96cb38077a72b8fb2af (patch) | |
tree | 5035dd3c90434c7c7ce977a5ced41a35d0b47ba4 | |
parent | 1c4924220c96392d17e0222c113509fd7b9a0854 (diff) |
perf tests: Add map_groups__merge_in test
Add map_groups__merge_in test to test the map_groups__merge_in function
usage - merging kcore maps into existing eBPF maps.
Committer testing:
# perf test merge
59: map_groups__merge_in : Ok
# perf test -v merge
59: map_groups__merge_in :
--- start ---
test child forked, pid 8349
test child finished with 0
---- end ----
map_groups__merge_in: Ok
#
Signed-off-by: Jiri Olsa <jolsa@kernel.org>
Acked-by: Song Liu <songliubraving@fb.com>
Tested-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Andi Kleen <ak@linux.intel.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Stanislav Fomichev <sdf@google.com>
Link: http://lkml.kernel.org/r/20190508132010.14512-10-jolsa@kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
-rw-r--r-- | tools/perf/tests/Build | 1 | ||||
-rw-r--r-- | tools/perf/tests/builtin-test.c | 4 | ||||
-rw-r--r-- | tools/perf/tests/map_groups.c | 120 | ||||
-rw-r--r-- | tools/perf/tests/tests.h | 1 | ||||
-rw-r--r-- | tools/perf/util/map_groups.h | 2 | ||||
-rw-r--r-- | tools/perf/util/symbol.c | 2 |
6 files changed, 129 insertions, 1 deletions
diff --git a/tools/perf/tests/Build b/tools/perf/tests/Build index 0b2b8305c965..4afb6319ed51 100644 --- a/tools/perf/tests/Build +++ b/tools/perf/tests/Build | |||
@@ -50,6 +50,7 @@ perf-y += perf-hooks.o | |||
50 | perf-y += clang.o | 50 | perf-y += clang.o |
51 | perf-y += unit_number__scnprintf.o | 51 | perf-y += unit_number__scnprintf.o |
52 | perf-y += mem2node.o | 52 | perf-y += mem2node.o |
53 | perf-y += map_groups.o | ||
53 | 54 | ||
54 | $(OUTPUT)tests/llvm-src-base.c: tests/bpf-script-example.c tests/Build | 55 | $(OUTPUT)tests/llvm-src-base.c: tests/bpf-script-example.c tests/Build |
55 | $(call rule_mkdir) | 56 | $(call rule_mkdir) |
diff --git a/tools/perf/tests/builtin-test.c b/tools/perf/tests/builtin-test.c index 9852b5d624a5..941c5456d625 100644 --- a/tools/perf/tests/builtin-test.c +++ b/tools/perf/tests/builtin-test.c | |||
@@ -290,6 +290,10 @@ static struct test generic_tests[] = { | |||
290 | .func = test__mem2node, | 290 | .func = test__mem2node, |
291 | }, | 291 | }, |
292 | { | 292 | { |
293 | .desc = "map_groups__merge_in", | ||
294 | .func = test__map_groups__merge_in, | ||
295 | }, | ||
296 | { | ||
293 | .func = NULL, | 297 | .func = NULL, |
294 | }, | 298 | }, |
295 | }; | 299 | }; |
diff --git a/tools/perf/tests/map_groups.c b/tools/perf/tests/map_groups.c new file mode 100644 index 000000000000..70d96acc6dcf --- /dev/null +++ b/tools/perf/tests/map_groups.c | |||
@@ -0,0 +1,120 @@ | |||
1 | #include <linux/compiler.h> | ||
2 | #include <linux/kernel.h> | ||
3 | #include "tests.h" | ||
4 | #include "map.h" | ||
5 | #include "map_groups.h" | ||
6 | #include "dso.h" | ||
7 | #include "debug.h" | ||
8 | |||
9 | struct map_def { | ||
10 | const char *name; | ||
11 | u64 start; | ||
12 | u64 end; | ||
13 | }; | ||
14 | |||
15 | static int check_maps(struct map_def *merged, unsigned int size, struct map_groups *mg) | ||
16 | { | ||
17 | struct map *map; | ||
18 | unsigned int i = 0; | ||
19 | |||
20 | map = map_groups__first(mg); | ||
21 | while (map) { | ||
22 | TEST_ASSERT_VAL("wrong map start", map->start == merged[i].start); | ||
23 | TEST_ASSERT_VAL("wrong map end", map->end == merged[i].end); | ||
24 | TEST_ASSERT_VAL("wrong map name", !strcmp(map->dso->name, merged[i].name)); | ||
25 | TEST_ASSERT_VAL("wrong map refcnt", refcount_read(&map->refcnt) == 2); | ||
26 | |||
27 | i++; | ||
28 | map = map_groups__next(map); | ||
29 | |||
30 | TEST_ASSERT_VAL("less maps expected", (map && i < size) || (!map && i == size)); | ||
31 | } | ||
32 | |||
33 | return TEST_OK; | ||
34 | } | ||
35 | |||
36 | int test__map_groups__merge_in(struct test *t __maybe_unused, int subtest __maybe_unused) | ||
37 | { | ||
38 | struct map_groups mg; | ||
39 | unsigned int i; | ||
40 | struct map_def bpf_progs[] = { | ||
41 | { "bpf_prog_1", 200, 300 }, | ||
42 | { "bpf_prog_2", 500, 600 }, | ||
43 | { "bpf_prog_3", 800, 900 }, | ||
44 | }; | ||
45 | struct map_def merged12[] = { | ||
46 | { "kcore1", 100, 200 }, | ||
47 | { "bpf_prog_1", 200, 300 }, | ||
48 | { "kcore1", 300, 500 }, | ||
49 | { "bpf_prog_2", 500, 600 }, | ||
50 | { "kcore1", 600, 800 }, | ||
51 | { "bpf_prog_3", 800, 900 }, | ||
52 | { "kcore1", 900, 1000 }, | ||
53 | }; | ||
54 | struct map_def merged3[] = { | ||
55 | { "kcore1", 100, 200 }, | ||
56 | { "bpf_prog_1", 200, 300 }, | ||
57 | { "kcore1", 300, 500 }, | ||
58 | { "bpf_prog_2", 500, 600 }, | ||
59 | { "kcore1", 600, 800 }, | ||
60 | { "bpf_prog_3", 800, 900 }, | ||
61 | { "kcore1", 900, 1000 }, | ||
62 | { "kcore3", 1000, 1100 }, | ||
63 | }; | ||
64 | struct map *map_kcore1, *map_kcore2, *map_kcore3; | ||
65 | int ret; | ||
66 | |||
67 | map_groups__init(&mg, NULL); | ||
68 | |||
69 | for (i = 0; i < ARRAY_SIZE(bpf_progs); i++) { | ||
70 | struct map *map; | ||
71 | |||
72 | map = dso__new_map(bpf_progs[i].name); | ||
73 | TEST_ASSERT_VAL("failed to create map", map); | ||
74 | |||
75 | map->start = bpf_progs[i].start; | ||
76 | map->end = bpf_progs[i].end; | ||
77 | map_groups__insert(&mg, map); | ||
78 | map__put(map); | ||
79 | } | ||
80 | |||
81 | map_kcore1 = dso__new_map("kcore1"); | ||
82 | TEST_ASSERT_VAL("failed to create map", map_kcore1); | ||
83 | |||
84 | map_kcore2 = dso__new_map("kcore2"); | ||
85 | TEST_ASSERT_VAL("failed to create map", map_kcore2); | ||
86 | |||
87 | map_kcore3 = dso__new_map("kcore3"); | ||
88 | TEST_ASSERT_VAL("failed to create map", map_kcore3); | ||
89 | |||
90 | /* kcore1 map overlaps over all bpf maps */ | ||
91 | map_kcore1->start = 100; | ||
92 | map_kcore1->end = 1000; | ||
93 | |||
94 | /* kcore2 map hides behind bpf_prog_2 */ | ||
95 | map_kcore2->start = 550; | ||
96 | map_kcore2->end = 570; | ||
97 | |||
98 | /* kcore3 map hides behind bpf_prog_3, kcore1 and adds new map */ | ||
99 | map_kcore3->start = 880; | ||
100 | map_kcore3->end = 1100; | ||
101 | |||
102 | ret = map_groups__merge_in(&mg, map_kcore1); | ||
103 | TEST_ASSERT_VAL("failed to merge map", !ret); | ||
104 | |||
105 | ret = check_maps(merged12, ARRAY_SIZE(merged12), &mg); | ||
106 | TEST_ASSERT_VAL("merge check failed", !ret); | ||
107 | |||
108 | ret = map_groups__merge_in(&mg, map_kcore2); | ||
109 | TEST_ASSERT_VAL("failed to merge map", !ret); | ||
110 | |||
111 | ret = check_maps(merged12, ARRAY_SIZE(merged12), &mg); | ||
112 | TEST_ASSERT_VAL("merge check failed", !ret); | ||
113 | |||
114 | ret = map_groups__merge_in(&mg, map_kcore3); | ||
115 | TEST_ASSERT_VAL("failed to merge map", !ret); | ||
116 | |||
117 | ret = check_maps(merged3, ARRAY_SIZE(merged3), &mg); | ||
118 | TEST_ASSERT_VAL("merge check failed", !ret); | ||
119 | return TEST_OK; | ||
120 | } | ||
diff --git a/tools/perf/tests/tests.h b/tools/perf/tests/tests.h index 399f18ca71a3..e5e3a57cd373 100644 --- a/tools/perf/tests/tests.h +++ b/tools/perf/tests/tests.h | |||
@@ -107,6 +107,7 @@ const char *test__clang_subtest_get_desc(int subtest); | |||
107 | int test__clang_subtest_get_nr(void); | 107 | int test__clang_subtest_get_nr(void); |
108 | int test__unit_number__scnprint(struct test *test, int subtest); | 108 | int test__unit_number__scnprint(struct test *test, int subtest); |
109 | int test__mem2node(struct test *t, int subtest); | 109 | int test__mem2node(struct test *t, int subtest); |
110 | int test__map_groups__merge_in(struct test *t, int subtest); | ||
110 | 111 | ||
111 | bool test__bp_signal_is_supported(void); | 112 | bool test__bp_signal_is_supported(void); |
112 | bool test__wp_is_supported(void); | 113 | bool test__wp_is_supported(void); |
diff --git a/tools/perf/util/map_groups.h b/tools/perf/util/map_groups.h index 4dcda33e0fdf..5f25efa6d6bc 100644 --- a/tools/perf/util/map_groups.h +++ b/tools/perf/util/map_groups.h | |||
@@ -88,4 +88,6 @@ int map_groups__fixup_overlappings(struct map_groups *mg, struct map *map, FILE | |||
88 | 88 | ||
89 | struct map *map_groups__find_by_name(struct map_groups *mg, const char *name); | 89 | struct map *map_groups__find_by_name(struct map_groups *mg, const char *name); |
90 | 90 | ||
91 | int map_groups__merge_in(struct map_groups *kmaps, struct map *new_map); | ||
92 | |||
91 | #endif // __PERF_MAP_GROUPS_H | 93 | #endif // __PERF_MAP_GROUPS_H |
diff --git a/tools/perf/util/symbol.c b/tools/perf/util/symbol.c index 29780fcd049c..f4540f8bbed1 100644 --- a/tools/perf/util/symbol.c +++ b/tools/perf/util/symbol.c | |||
@@ -1170,7 +1170,7 @@ static int kcore_mapfn(u64 start, u64 len, u64 pgoff, void *data) | |||
1170 | * Merges map into map_groups by splitting the new map | 1170 | * Merges map into map_groups by splitting the new map |
1171 | * within the existing map regions. | 1171 | * within the existing map regions. |
1172 | */ | 1172 | */ |
1173 | static int map_groups__merge_in(struct map_groups *kmaps, struct map *new_map) | 1173 | int map_groups__merge_in(struct map_groups *kmaps, struct map *new_map) |
1174 | { | 1174 | { |
1175 | struct map *old_map; | 1175 | struct map *old_map; |
1176 | LIST_HEAD(merged); | 1176 | LIST_HEAD(merged); |