diff options
author | Bartosz Golaszewski <bartekgola@gmail.com> | 2013-10-16 19:18:05 +0200 |
---|---|---|
committer | Denys Vlasenko <vda.linux@googlemail.com> | 2013-10-16 19:18:05 +0200 |
commit | 28a209466f43c22db42dd02baa136ac7ac25069b (patch) | |
tree | bd8fc6cdbe9bfc9e2a586ff01b9fadcaecd60560 /docs | |
parent | 64938011f3ba06a8f425926397172dc361bce851 (diff) | |
download | busybox-w32-28a209466f43c22db42dd02baa136ac7ac25069b.tar.gz busybox-w32-28a209466f43c22db42dd02baa136ac7ac25069b.tar.bz2 busybox-w32-28a209466f43c22db42dd02baa136ac7ac25069b.zip |
bloat-o-meter: add usage info
Signed-off-by: Bartosz Golaszewski <bartekgola@gmail.com>
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'docs')
-rw-r--r-- | docs/keep_data_small.txt | 15 |
1 files changed, 8 insertions, 7 deletions
diff --git a/docs/keep_data_small.txt b/docs/keep_data_small.txt index 21d732674..9fc799646 100644 --- a/docs/keep_data_small.txt +++ b/docs/keep_data_small.txt | |||
@@ -138,13 +138,6 @@ less readable, use #defines: | |||
138 | #define sector (G.sector) | 138 | #define sector (G.sector) |
139 | 139 | ||
140 | 140 | ||
141 | Word of caution | ||
142 | |||
143 | If applet doesn't use much of global data, converting it to use | ||
144 | one of above methods is not worth the resulting code obfuscation. | ||
145 | If you have less than ~300 bytes of global data - don't bother. | ||
146 | |||
147 | |||
148 | Finding non-shared duplicated strings | 141 | Finding non-shared duplicated strings |
149 | 142 | ||
150 | strings busybox | sort | uniq -c | sort -nr | 143 | strings busybox | sort | uniq -c | sort -nr |
@@ -224,6 +217,14 @@ Result (non-static busybox built against glibc): | |||
224 | 217 | ||
225 | Keeping code small | 218 | Keeping code small |
226 | 219 | ||
220 | Use scripts/bloat-o-meter to check whether introduced changes | ||
221 | didn't generate unnecessary bloat. This script needs unstripped binaries | ||
222 | to generate a detailed report. To automate this, just use | ||
223 | "make bloatcheck". It requires busybox_old binary to be present, | ||
224 | use "make baseline" to generate it from unmodified source, or | ||
225 | copy busybox_unstripped to busybox_old before modifying sources | ||
226 | and rebuilding. | ||
227 | |||
227 | Set CONFIG_EXTRA_CFLAGS="-fno-inline-functions-called-once", | 228 | Set CONFIG_EXTRA_CFLAGS="-fno-inline-functions-called-once", |
228 | produce "make bloatcheck", see the biggest auto-inlined functions. | 229 | produce "make bloatcheck", see the biggest auto-inlined functions. |
229 | Now, set CONFIG_EXTRA_CFLAGS back to "", but add NOINLINE | 230 | Now, set CONFIG_EXTRA_CFLAGS back to "", but add NOINLINE |