tracing: Fix recordmcount.pl to handle sections with only weak functions
authorSteven Rostedt <srostedt@redhat.com>
Thu, 6 Aug 2009 23:53:18 +0000 (19:53 -0400)
committerIngo Molnar <mingo@elte.hu>
Fri, 7 Aug 2009 06:50:29 +0000 (08:50 +0200)
Roland Dreier found that a section that contained only a weak
function in one of the staging drivers and this caused
recordmcount.pl to spit out a warning and fail.

Although it is strange that a driver would have a weak function, and
this function only be used in one place, it should not be something
to make recordmcount.pl fail.

This patch fixes the issue in a simple manner: if only weak
functions exist in a section, then that section will not be
recorded.

Reported-by: Roland Dreier <rdreier@cisco.com>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
scripts/recordmcount.pl

index 4889c44d71b5d3e5907cb886577f81aea63dab8c..911ba7ffab842dd3017229e269851501f19c145f 100755 (executable)
@@ -393,7 +393,7 @@ while (<IN>) {
            $read_function = 0;
        }
        # print out any recorded offsets
-       update_funcs() if ($text_found);
+       update_funcs() if (defined($ref_func));
 
        # reset all markers and arrays
        $text_found = 0;
@@ -444,7 +444,7 @@ while (<IN>) {
 }
 
 # dump out anymore offsets that may have been found
-update_funcs() if ($text_found);
+update_funcs() if (defined($ref_func));
 
 # If we did not find any mcount callers, we are done (do nothing).
 if (!$opened) {