diff options
author | Rasmus Villemoes <[email protected]> | 2016-03-22 20:32:05 +0100 |
---|---|---|
committer | Martin K. Petersen <[email protected]> | 2016-04-11 16:57:09 -0400 |
commit | e1f0bce3a0db95007fec756225801e50477f32fd (patch) | |
tree | 4bcc48e7ff93364ac1ca0c88404a459b9c9c7774 /lib/test-string_helpers.c | |
parent | 9d99a2e33a02c3290cb53013150ee06a304588ae (diff) |
scsi: reduce CONFIG_SCSI_CONSTANTS=y impact by 8k
On 64 bit, struct error_info has 6 bytes of padding, which amounts to
over 4k of wasted space in the additional[] array. We could easily get
rid of that by instead using separate arrays for the codes and the
pointers. However, we can do even better than that and save an
additional 6 bytes per entry: In the table, just store the sizeof()
the corresponding string literal. The cumulative sum of these is then
the appropriate offset into additional_text, which is built from the
concatenation (with '\0's inbetween) of the strings.
$ scripts/bloat-o-meter /tmp/vmlinux vmlinux
add/remove: 0/0 grow/shrink: 1/1 up/down: 24/-8488 (-8464)
function old new delta
scsi_extd_sense_format 136 160 +24
additional 11312 2824 -8488
The Kconfig help text used to say that CONFIG_SCSI_CONSTANTS=y costs
around 75 KB, but that was a little exaggerated. The actual number was
closer to 44K, and 36K with this patch.
Signed-off-by: Rasmus Villemoes <[email protected]>
Reviewed-by: Hannes Reinecke <[email protected]>
Tested-by: Douglas Gilbert <[email protected]>
Reviewed-by: Christoph Hellwig <[email protected]>
Signed-off-by: Martin K. Petersen <[email protected]>
Diffstat (limited to 'lib/test-string_helpers.c')
0 files changed, 0 insertions, 0 deletions