diff options
author | Jean Delvare <khali@linux-fr.org> | 2011-03-22 19:35:13 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2011-03-22 20:44:17 -0400 |
commit | c50e3f512a5a15a73acd94e6ec8ed63cd512e04f (patch) | |
tree | 7f8e4ef42dd0c946ba7d4d6b001133db83e15a4a /scripts/bloat-o-meter | |
parent | 565d76cb7d5fd7cb010fd690602280a69ab116ef (diff) |
bloat-o-meter: include read-only data section in report
I'm not sure why the read-only data section is excluded from the report,
it seems as relevant as the other data sections (b and d).
I've stripped the symbols starting with __mod_ as they can have their
names dynamically generated and thus comparison between binaries is not
possible.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Cc: Andi Kleen <andi@firstfloor.org>
Acked-by: Nathan Lynch <ntl@pobox.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'scripts/bloat-o-meter')
-rwxr-xr-x | scripts/bloat-o-meter | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/scripts/bloat-o-meter b/scripts/bloat-o-meter index 6501a50e17f0..6129020c41a9 100755 --- a/scripts/bloat-o-meter +++ b/scripts/bloat-o-meter | |||
@@ -17,7 +17,9 @@ def getsizes(file): | |||
17 | sym = {} | 17 | sym = {} |
18 | for l in os.popen("nm --size-sort " + file).readlines(): | 18 | for l in os.popen("nm --size-sort " + file).readlines(): |
19 | size, type, name = l[:-1].split() | 19 | size, type, name = l[:-1].split() |
20 | if type in "tTdDbB": | 20 | if type in "tTdDbBrR": |
21 | # strip generated symbols | ||
22 | if name[:6] == "__mod_": continue | ||
21 | # function names begin with '.' on 64-bit powerpc | 23 | # function names begin with '.' on 64-bit powerpc |
22 | if "." in name[1:]: name = "static." + name.split(".")[0] | 24 | if "." in name[1:]: name = "static." + name.split(".")[0] |
23 | sym[name] = sym.get(name, 0) + int(size, 16) | 25 | sym[name] = sym.get(name, 0) + int(size, 16) |