[GIT PULL][2.6.31] tracing: have recordmcount.pl handle a sectionwith only weak functions

From: Steven Rostedt
Date: Thu Aug 06 2009 - 20:01:53 EST



Ingo,

Please pull the latest tip/tracing/urgent tree, which can be found at:

git://git.kernel.org/pub/scm/linux/kernel/git/rostedt/linux-2.6-trace.git
tip/tracing/urgent


Steven Rostedt (1):
tracing: have recordmcount.pl handle a section with only weak functions

----
scripts/recordmcount.pl | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
---------------------------
commit 0484219980fd985528f66bf620362859e617bb37
Author: Steven Rostedt <srostedt@xxxxxxxxxx>
Date: Thu Aug 6 19:53:18 2009 -0400

tracing: have recordmcount.pl handle a section with only weak functions

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, than that section will not be recorded.

Reported-by: Roland Dreier <rdreier@xxxxxxxxx>
Signed-off-by: Steven Rostedt <rostedt@xxxxxxxxxxx>

diff --git a/scripts/recordmcount.pl b/scripts/recordmcount.pl
index 4889c44..911ba7f 100755
--- a/scripts/recordmcount.pl
+++ b/scripts/recordmcount.pl
@@ -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) {


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/