linux-scsi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v2 0/3] scsi: reduce CONFIG_SCSI_CONSTANTS=y impact by 8k
@ 2015-11-24  9:42 Rasmus Villemoes
  2015-11-24  9:42 ` [PATCH v2 1/3] scsi: make some Additional Sense strings more grep'able Rasmus Villemoes
                   ` (5 more replies)
  0 siblings, 6 replies; 17+ messages in thread
From: Rasmus Villemoes @ 2015-11-24  9:42 UTC (permalink / raw)
  To: Christoph Hellwig, Hannes Reinecke, Bart Van Assche,
	James E.J. Bottomley, Julian Calaby
  Cc: Rasmus Villemoes, linux-scsi, linux-kernel

This reduces the impact of choosing CONFIG_SCSI_CONSTANTS by about 8KB.

2dd951ecd511 ("scsi: Conditionally compile in constants.c") updated
the Kconfig help text from 12KB to 75KB. The 12K predated git so was
certainly outdated. But I'm not sure where the 75K comes from; using
size(1) on a defconfig (with/without this config option) vmlinux shows
a difference of about 47K, and 39K after these patches are applied. In
any case, I've left the Kconfig text alone, since I'm not sure I'm
counting the same way the 75K was computed (I'm fairly certain of the
8K delta, however).

Tested with a trivial module calling scsi_extd_sense_format with a few
random known codes and comparing the result to the expected value.

v2: prepend patch to unsplit a few string literals for greppability,
leave the NULL sentinel in the .c file in 2/3 (it's removed in 3/3
either way).

Rasmus Villemoes (3):
  scsi: make some Additional Sense strings more grep'able
  scsi: move Additional Sense Codes to separate file
  scsi: reduce CONFIG_SCSI_CONSTANTS=y impact by 8k

 drivers/scsi/constants.c   | 859 ++-------------------------------------------
 drivers/scsi/sense_codes.h | 826 +++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 847 insertions(+), 838 deletions(-)
 create mode 100644 drivers/scsi/sense_codes.h

-- 
2.6.1

^ permalink raw reply	[flat|nested] 17+ messages in thread

end of thread, other threads:[~2016-03-23 20:50 UTC | newest]

Thread overview: 17+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-11-24  9:42 [PATCH v2 0/3] scsi: reduce CONFIG_SCSI_CONSTANTS=y impact by 8k Rasmus Villemoes
2015-11-24  9:42 ` [PATCH v2 1/3] scsi: make some Additional Sense strings more grep'able Rasmus Villemoes
2015-12-22 20:25   ` Douglas Gilbert
2015-11-24  9:42 ` [PATCH v2 2/3] scsi: move Additional Sense Codes to separate file Rasmus Villemoes
2015-12-22 20:25   ` Douglas Gilbert
2015-11-24  9:42 ` [PATCH v2 3/3] scsi: reduce CONFIG_SCSI_CONSTANTS=y impact by 8k Rasmus Villemoes
2015-12-22 20:25   ` Douglas Gilbert
2015-11-24 15:09 ` [PATCH v2 0/3] " Hannes Reinecke
2015-12-22 20:25 ` Douglas Gilbert
2016-03-08 19:52 ` Rasmus Villemoes
2016-03-22  7:54   ` Christoph Hellwig
2016-03-22 19:32     ` [PATCH v3 " Rasmus Villemoes
2016-03-22 19:32       ` [PATCH v3 1/3] scsi: make some Additional Sense strings more grep'able Rasmus Villemoes
2016-03-22 19:32       ` [PATCH v3 2/3] scsi: move Additional Sense Codes to separate file Rasmus Villemoes
2016-03-22 19:32       ` [PATCH v3 3/3] scsi: reduce CONFIG_SCSI_CONSTANTS=y impact by 8k Rasmus Villemoes
2016-03-23  7:39       ` [PATCH v3 0/3] " Christoph Hellwig
2016-03-23 20:50       ` Martin K. Petersen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).