From 28a209466f43c22db42dd02baa136ac7ac25069b Mon Sep 17 00:00:00 2001 From: Bartosz Golaszewski Date: Wed, 16 Oct 2013 19:18:05 +0200 Subject: bloat-o-meter: add usage info Signed-off-by: Bartosz Golaszewski Signed-off-by: Denys Vlasenko --- docs/keep_data_small.txt | 15 ++++++++------- 1 file changed, 8 insertions(+), 7 deletions(-) (limited to 'docs') diff --git a/docs/keep_data_small.txt b/docs/keep_data_small.txt index 21d7326..9fc7996 100644 --- a/docs/keep_data_small.txt +++ b/docs/keep_data_small.txt @@ -138,13 +138,6 @@ less readable, use #defines: #define sector (G.sector) - Word of caution - -If applet doesn't use much of global data, converting it to use -one of above methods is not worth the resulting code obfuscation. -If you have less than ~300 bytes of global data - don't bother. - - Finding non-shared duplicated strings strings busybox | sort | uniq -c | sort -nr @@ -224,6 +217,14 @@ Result (non-static busybox built against glibc): Keeping code small +Use scripts/bloat-o-meter to check whether introduced changes +didn't generate unnecessary bloat. This script needs unstripped binaries +to generate a detailed report. To automate this, just use +"make bloatcheck". It requires busybox_old binary to be present, +use "make baseline" to generate it from unmodified source, or +copy busybox_unstripped to busybox_old before modifying sources +and rebuilding. + Set CONFIG_EXTRA_CFLAGS="-fno-inline-functions-called-once", produce "make bloatcheck", see the biggest auto-inlined functions. Now, set CONFIG_EXTRA_CFLAGS back to "", but add NOINLINE -- cgit v1.1