Gitweb:     
http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=8998979cc1f90da5a48b2e8a13833217c63f7c4a
Commit:     8998979cc1f90da5a48b2e8a13833217c63f7c4a
Parent:     bb8e8bcce7eceacb52eb0a3ebb64202ad6bcc438
Author:     Nathan Lynch <[EMAIL PROTECTED]>
AuthorDate: Mon Dec 17 16:20:09 2007 -0800
Committer:  Linus Torvalds <[EMAIL PROTECTED]>
CommitDate: Mon Dec 17 19:28:17 2007 -0800

    fix bloat-o-meter for ppc64
    
    bloat-o-meter assumes that a '.' anywhere in a symbol's name means that it
    is static and prepends 'static.' to the first part of the symbol name,
    discarding the portion of the name that follows the '.'.  However, the
    names of function entry points begin with '.' in the ppc64 ABI.  This
    causes all function text size changes to be accounted to a single 'static.'
    entry in the output when comparing ppc64 kernels.
    
    Change getsizes() to ignore the first character of the symbol name when
    searching for '.'.
    
    Signed-off-by: Nathan Lynch <[EMAIL PROTECTED]>
    Cc: Matt Mackall <[EMAIL PROTECTED]>
    Signed-off-by: Andrew Morton <[EMAIL PROTECTED]>
    Signed-off-by: Linus Torvalds <[EMAIL PROTECTED]>
---
 scripts/bloat-o-meter |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/scripts/bloat-o-meter b/scripts/bloat-o-meter
index ce59fc2..6501a50 100755
--- a/scripts/bloat-o-meter
+++ b/scripts/bloat-o-meter
@@ -18,7 +18,8 @@ def getsizes(file):
     for l in os.popen("nm --size-sort " + file).readlines():
         size, type, name = l[:-1].split()
         if type in "tTdDbB":
-            if "." in name: name = "static." + name.split(".")[0]
+            # function names begin with '.' on 64-bit powerpc
+            if "." in name[1:]: name = "static." + name.split(".")[0]
             sym[name] = sym.get(name, 0) + int(size, 16)
     return sym
 
-
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to