Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1742786 > unrolled thread

Re: [PATCH v3 03/13] perf report: create real callchain entries for inlined frames

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH v3 03/13] perf report: create real callchain entries for inlined frames Milian Wolff <milian.wolff@kdab.com> - 2017-10-01 14:40 +0200
    Re: [PATCH v3 03/13] perf report: create real callchain entries for  inlined frames Joe Perches <joe@perches.com> - 2017-10-01 19:50 +0200

#1742786 — Re: [PATCH v3 03/13] perf report: create real callchain entries for inlined frames

On Dienstag, 19. September 2017 14:27:11 CEST Jiri Olsa wrote:
> On Wed, Sep 06, 2017 at 03:54:51PM +0200, Milian Wolff wrote:
> 
> SNIP
> 
> > -struct inline_node *dso__parse_addr_inlines(struct dso *dso, u64 addr);
> > +// parse inlined frames for the given address
> > +struct inline_node *dso__parse_addr_inlines(struct dso *dso, u64 addr,
> > +					    struct symbol *sym);
> > +// free resources associated to the inline node list
> > 
> >  void inline_node__delete(struct inline_node *node);
> > 
> > +// insert the inline node list into the DSO, which will take ownership
> > +void inlines__tree_insert(struct rb_root *tree, struct inline_node
> > *inlines); +// find previously inserted inline node list
> > +struct inline_node *inlines__tree_find(struct rb_root *tree, u64 addr);
> > +// delete all nodes within the tree of inline_node s
> > +void inlines__tree_delete(struct rb_root *tree);
> 
> nit, think we prefer the /* */ comment style

I'll fix this, but note that there are more places outside this patch series 
where this style has crept in. Some of them also from past patches of mine - 
sorry for that.

I looks to me as if the checkpatch.pl should catch this, but it doesn't for me 
which is odd. Does anyone know what I need to do to enable the C99_COMMENTS 
check?

~~~~~
┌milian@agathebauer:~/projects/src/linux/tools/perf|wip/distinguish-inliners<>
└$ grep '// ' 0009-perf-report-properly-handle-branch-count-in-match_ch.patch 
-               // if no srcline is available, fallback to symbol name
-               // otherwise fall-back to IP-based comparison below
-               // compare inlined frames based on their symbol name because
-               // different inlined frames will have the same symbol start
+               // else fall-through
+                       // compare inlined frames based on their symbol name
+                       // because different inlined frames will have the same
+                       // symbol start. otherwise do a faster comparison 
based
+                       // on the symbol start address
+               // else fall-through
┌milian@agathebauer:~/projects/src/linux/tools/perf|wip/distinguish-inliners<>
└$ ../../scripts/checkpatch.pl --strict 0009-perf-report-properly-handle-
branch-count-in-match_ch.patch 
total: 0 errors, 0 warnings, 0 checks, 136 lines checked

0009-perf-report-properly-handle-branch-count-in-match_ch.patch has no obvious 
style problems and is ready for submission.
~~~~~

Cheers

-- 
Milian Wolff | milian.wolff@kdab.com | Senior Software Engineer
KDAB (Deutschland) GmbH&Co KG, a KDAB Group company
Tel: +49-30-521325470
KDAB - The Qt Experts

[toc] | [next] | [standalone]

#1742854 — Re: [PATCH v3 03/13] perf report: create real callchain entries for inlined frames

On Sun, 2017-10-01 at 14:37 +0200, Milian Wolff wrote:
> On Dienstag, 19. September 2017 14:27:11 CEST Jiri Olsa wrote:
[]
> > nit, think we prefer the /* */ comment style
> 
> I'll fix this, but note that there are more places outside this patch series 
> where this style has crept in. Some of them also from past patches of mine - 
> sorry for that.

I think c99 comments are fine.
So did Linus Torvalds.

http://lkml.iu.edu/hypermail/linux/kernel/1607.1/00627.html

> I looks to me as if the checkpatch.pl should catch this, but it doesn't for me 
> which is odd. Does anyone know what I need to do to enable the C99_COMMENTS 
> check?

Modify checkpatch

my $allow_c99_comments = 1;

change the 1 to 0

Add a runtime option if you want.

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web