Skip to contentSkip to conversation

fix bloat-o-meter for ppc64

PATCH New Nathan Lynch 1 participant 1 message Dec 14, 2007

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 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 '.'.

---
 scripts/bloat-o-meter |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)
scripts/bloat-o-meter +2 -1
index ce59fc2..6501a50 100755
@@ -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
 
Signature · 1 line
1.5.3.7.1112.g9758e