diff options
| -rw-r--r-- | tools/perf/builtin-report.c | 2 | ||||
| -rw-r--r-- | tools/perf/util/callchain.c | 35 | ||||
| -rw-r--r-- | tools/perf/util/callchain.h | 2 |
3 files changed, 20 insertions, 19 deletions
diff --git a/tools/perf/builtin-report.c b/tools/perf/builtin-report.c index 359205782964..fd7407c7205c 100644 --- a/tools/perf/builtin-report.c +++ b/tools/perf/builtin-report.c | |||
| @@ -107,7 +107,7 @@ static int perf_session__add_hist_entry(struct perf_session *self, | |||
| 107 | goto out_free_syms; | 107 | goto out_free_syms; |
| 108 | err = 0; | 108 | err = 0; |
| 109 | if (symbol_conf.use_callchain) { | 109 | if (symbol_conf.use_callchain) { |
| 110 | err = append_chain(he->callchain, data->callchain, syms); | 110 | err = append_chain(he->callchain, data->callchain, syms, data->period); |
| 111 | if (err) | 111 | if (err) |
| 112 | goto out_free_syms; | 112 | goto out_free_syms; |
| 113 | } | 113 | } |
diff --git a/tools/perf/util/callchain.c b/tools/perf/util/callchain.c index 62b69ad4aa73..52c777e451ed 100644 --- a/tools/perf/util/callchain.c +++ b/tools/perf/util/callchain.c | |||
| @@ -230,7 +230,7 @@ fill_node(struct callchain_node *node, struct resolved_chain *chain, int start) | |||
| 230 | 230 | ||
| 231 | static void | 231 | static void |
| 232 | add_child(struct callchain_node *parent, struct resolved_chain *chain, | 232 | add_child(struct callchain_node *parent, struct resolved_chain *chain, |
| 233 | int start) | 233 | int start, u64 period) |
| 234 | { | 234 | { |
| 235 | struct callchain_node *new; | 235 | struct callchain_node *new; |
| 236 | 236 | ||
| @@ -238,7 +238,7 @@ add_child(struct callchain_node *parent, struct resolved_chain *chain, | |||
| 238 | fill_node(new, chain, start); | 238 | fill_node(new, chain, start); |
| 239 | 239 | ||
| 240 | new->children_hit = 0; | 240 | new->children_hit = 0; |
| 241 | new->hit = 1; | 241 | new->hit = period; |
| 242 | } | 242 | } |
| 243 | 243 | ||
| 244 | /* | 244 | /* |
| @@ -248,7 +248,8 @@ add_child(struct callchain_node *parent, struct resolved_chain *chain, | |||
| 248 | */ | 248 | */ |
| 249 | static void | 249 | static void |
| 250 | split_add_child(struct callchain_node *parent, struct resolved_chain *chain, | 250 | split_add_child(struct callchain_node *parent, struct resolved_chain *chain, |
| 251 | struct callchain_list *to_split, int idx_parents, int idx_local) | 251 | struct callchain_list *to_split, int idx_parents, int idx_local, |
| 252 | u64 period) | ||
| 252 | { | 253 | { |
| 253 | struct callchain_node *new; | 254 | struct callchain_node *new; |
| 254 | struct list_head *old_tail; | 255 | struct list_head *old_tail; |
| @@ -275,41 +276,41 @@ split_add_child(struct callchain_node *parent, struct resolved_chain *chain, | |||
| 275 | /* create a new child for the new branch if any */ | 276 | /* create a new child for the new branch if any */ |
| 276 | if (idx_total < chain->nr) { | 277 | if (idx_total < chain->nr) { |
| 277 | parent->hit = 0; | 278 | parent->hit = 0; |
| 278 | add_child(parent, chain, idx_total); | 279 | add_child(parent, chain, idx_total, period); |
| 279 | parent->children_hit++; | 280 | parent->children_hit += period; |
| 280 | } else { | 281 | } else { |
| 281 | parent->hit = 1; | 282 | parent->hit = period; |
| 282 | } | 283 | } |
| 283 | } | 284 | } |
| 284 | 285 | ||
| 285 | static int | 286 | static int |
| 286 | __append_chain(struct callchain_node *root, struct resolved_chain *chain, | 287 | __append_chain(struct callchain_node *root, struct resolved_chain *chain, |
| 287 | unsigned int start); | 288 | unsigned int start, u64 period); |
| 288 | 289 | ||
| 289 | static void | 290 | static void |
| 290 | __append_chain_children(struct callchain_node *root, | 291 | __append_chain_children(struct callchain_node *root, |
| 291 | struct resolved_chain *chain, | 292 | struct resolved_chain *chain, |
| 292 | unsigned int start) | 293 | unsigned int start, u64 period) |
| 293 | { | 294 | { |
| 294 | struct callchain_node *rnode; | 295 | struct callchain_node *rnode; |
| 295 | 296 | ||
| 296 | /* lookup in childrens */ | 297 | /* lookup in childrens */ |
| 297 | chain_for_each_child(rnode, root) { | 298 | chain_for_each_child(rnode, root) { |
| 298 | unsigned int ret = __append_chain(rnode, chain, start); | 299 | unsigned int ret = __append_chain(rnode, chain, start, period); |
| 299 | 300 | ||
| 300 | if (!ret) | 301 | if (!ret) |
| 301 | goto inc_children_hit; | 302 | goto inc_children_hit; |
| 302 | } | 303 | } |
| 303 | /* nothing in children, add to the current node */ | 304 | /* nothing in children, add to the current node */ |
| 304 | add_child(root, chain, start); | 305 | add_child(root, chain, start, period); |
| 305 | 306 | ||
| 306 | inc_children_hit: | 307 | inc_children_hit: |
| 307 | root->children_hit++; | 308 | root->children_hit += period; |
| 308 | } | 309 | } |
| 309 | 310 | ||
| 310 | static int | 311 | static int |
| 311 | __append_chain(struct callchain_node *root, struct resolved_chain *chain, | 312 | __append_chain(struct callchain_node *root, struct resolved_chain *chain, |
| 312 | unsigned int start) | 313 | unsigned int start, u64 period) |
| 313 | { | 314 | { |
| 314 | struct callchain_list *cnode; | 315 | struct callchain_list *cnode; |
| 315 | unsigned int i = start; | 316 | unsigned int i = start; |
| @@ -345,18 +346,18 @@ __append_chain(struct callchain_node *root, struct resolved_chain *chain, | |||
| 345 | 346 | ||
| 346 | /* we match only a part of the node. Split it and add the new chain */ | 347 | /* we match only a part of the node. Split it and add the new chain */ |
| 347 | if (i - start < root->val_nr) { | 348 | if (i - start < root->val_nr) { |
| 348 | split_add_child(root, chain, cnode, start, i - start); | 349 | split_add_child(root, chain, cnode, start, i - start, period); |
| 349 | return 0; | 350 | return 0; |
| 350 | } | 351 | } |
| 351 | 352 | ||
| 352 | /* we match 100% of the path, increment the hit */ | 353 | /* we match 100% of the path, increment the hit */ |
| 353 | if (i - start == root->val_nr && i == chain->nr) { | 354 | if (i - start == root->val_nr && i == chain->nr) { |
| 354 | root->hit++; | 355 | root->hit += period; |
| 355 | return 0; | 356 | return 0; |
| 356 | } | 357 | } |
| 357 | 358 | ||
| 358 | /* We match the node and still have a part remaining */ | 359 | /* We match the node and still have a part remaining */ |
| 359 | __append_chain_children(root, chain, i); | 360 | __append_chain_children(root, chain, i, period); |
| 360 | 361 | ||
| 361 | return 0; | 362 | return 0; |
| 362 | } | 363 | } |
| @@ -380,7 +381,7 @@ static void filter_context(struct ip_callchain *old, struct resolved_chain *new, | |||
| 380 | 381 | ||
| 381 | 382 | ||
| 382 | int append_chain(struct callchain_node *root, struct ip_callchain *chain, | 383 | int append_chain(struct callchain_node *root, struct ip_callchain *chain, |
| 383 | struct map_symbol *syms) | 384 | struct map_symbol *syms, u64 period) |
| 384 | { | 385 | { |
| 385 | struct resolved_chain *filtered; | 386 | struct resolved_chain *filtered; |
| 386 | 387 | ||
| @@ -397,7 +398,7 @@ int append_chain(struct callchain_node *root, struct ip_callchain *chain, | |||
| 397 | if (!filtered->nr) | 398 | if (!filtered->nr) |
| 398 | goto end; | 399 | goto end; |
| 399 | 400 | ||
| 400 | __append_chain_children(root, filtered, 0); | 401 | __append_chain_children(root, filtered, 0, period); |
| 401 | end: | 402 | end: |
| 402 | free(filtered); | 403 | free(filtered); |
| 403 | 404 | ||
diff --git a/tools/perf/util/callchain.h b/tools/perf/util/callchain.h index 22dbaec003d5..f2e9ee164bd8 100644 --- a/tools/perf/util/callchain.h +++ b/tools/perf/util/callchain.h | |||
| @@ -61,7 +61,7 @@ static inline u64 cumul_hits(struct callchain_node *node) | |||
| 61 | 61 | ||
| 62 | int register_callchain_param(struct callchain_param *param); | 62 | int register_callchain_param(struct callchain_param *param); |
| 63 | int append_chain(struct callchain_node *root, struct ip_callchain *chain, | 63 | int append_chain(struct callchain_node *root, struct ip_callchain *chain, |
| 64 | struct map_symbol *syms); | 64 | struct map_symbol *syms, u64 period); |
| 65 | 65 | ||
| 66 | bool ip_callchain__valid(struct ip_callchain *chain, event_t *event); | 66 | bool ip_callchain__valid(struct ip_callchain *chain, event_t *event); |
| 67 | #endif /* __PERF_CALLCHAIN_H */ | 67 | #endif /* __PERF_CALLCHAIN_H */ |
