diff options
author | Nikolay Borisov <nborisov@suse.com> | 2020-08-06 23:17:32 -0700 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2020-08-07 11:33:21 -0700 |
commit | 1d35b6054a9b94b48511e23a7fab6cd1dbf3fde6 (patch) | |
tree | e0f9ee183644ce2657e7e20f1e8d209ef4516b70 | |
parent | 4f491bb6ea2aef2f5b184f385904a73796d98554 (diff) | |
download | lwn-1d35b6054a9b94b48511e23a7fab6cd1dbf3fde6.tar.gz lwn-1d35b6054a9b94b48511e23a7fab6cd1dbf3fde6.zip |
scripts/bloat-o-meter: Support comparing library archives
Library archives (.a) usually contain multiple object files so their
output of nm --size-sort contains lines like:
<omitted for brevity>
00000000000003a8 t run_test
extent-map-tests.o:
<omitted for brevity>
bloat-o-meter currently doesn't handle them which results in errors when
calling .split() on them. Fix this by simply ignoring them. This enables
diffing subsystems which generate built-in.a files.
Signed-off-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Link: http://lkml.kernel.org/r/20200603103513.3712-1-nborisov@suse.com
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rwxr-xr-x | scripts/bloat-o-meter | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/scripts/bloat-o-meter b/scripts/bloat-o-meter index 8c965f6a9881..d7ca46c612b3 100755 --- a/scripts/bloat-o-meter +++ b/scripts/bloat-o-meter @@ -26,6 +26,8 @@ def getsizes(file, format): sym = {} with os.popen("nm --size-sort " + file) as f: for line in f: + if line.startswith("\n") or ":" in line: + continue size, type, name = line.split() if type in format: # strip generated symbols |