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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <1648232.PM1aBqIB3I@agathebauer>
Date:   Sun, 01 Oct 2017 14:37:56 +0200
From:   Milian Wolff <milian.wolff@...b.com>
To:     Jiri Olsa <jolsa@...hat.com>
Cc:     acme@...nel.org, Linux-kernel@...r.kernel.org,
        linux-perf-users@...r.kernel.org,
        Arnaldo Carvalho de Melo <acme@...hat.com>,
        Namhyung Kim <namhyung@...nel.org>
Subject: 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@...thebauer:~/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@...thebauer:~/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@...b.com | Senior Software Engineer
KDAB (Deutschland) GmbH&Co KG, a KDAB Group company
Tel: +49-30-521325470
KDAB - The Qt Experts