From patchwork Mon Oct 5 09:26:40 2015 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Rasmus Villemoes X-Patchwork-Id: 7325801 Return-Path: X-Original-To: patchwork-linux-scsi@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork1.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.136]) by patchwork1.web.kernel.org (Postfix) with ESMTP id 963B89F302 for ; Mon, 5 Oct 2015 09:28:03 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 9391F20684 for ; Mon, 5 Oct 2015 09:28:02 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 8FA8620680 for ; Mon, 5 Oct 2015 09:28:01 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752619AbbJEJ1J (ORCPT ); Mon, 5 Oct 2015 05:27:09 -0400 Received: from mail-la0-f46.google.com ([209.85.215.46]:36033 "EHLO mail-la0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752344AbbJEJ1A (ORCPT ); Mon, 5 Oct 2015 05:27:00 -0400 Received: by laddr2 with SMTP id dr2so4143226lad.3 for ; Mon, 05 Oct 2015 02:26:58 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=rasmusvillemoes.dk; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=R+Cw1JAV9urkC4V5JHamc7MNpyTTcO0ZCQ0tekLI+n4=; b=Mqa0x1r5Cj4OWX6ASSDmSTXoF6Ell/XapUR6kCOp2zvop5FsJHkARRVu1qK1TxWe26 uZ8VkM4xgJ+q+bgmyN+0FS4CqdmxCY+VV+WTgIUTTNGgpNZjDtRgFmJtecqrNG6bSoSj zNJ5sSqGvVK62d7pEyeHbWh3CQ1JULR5+EyP8= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=R+Cw1JAV9urkC4V5JHamc7MNpyTTcO0ZCQ0tekLI+n4=; b=kUDZgfj5Cu6L16BCv0pzoQihaa4x/uT0brTtJgJ5lKwmY109D56RjetaGDY1ZZl1oI O6KT4rn+aP2niuTtz7STz+h84/FA/t2uFbbdWzpn/xcqHdU5F0RMeo8Tv2vtV5WyKKzb jDZHBVtMlYGd37r2KWtV7rKw/n1CB7osKOBlhgcJ1+IO3XjAge4otn9BiB8rmgNVifNu /gEyb9xFQiCzQ3KEcfytDybn1cEa8O4NLnxGLAvrZDyq6U5HgHwQ8eHHqexWdeBFHxCV xC5riPmvgBate510wo8ZKFn7f6ftR5YSQVu04qbNrualQb/CNp4byt1aJIUiau72U8te cAqQ== X-Gm-Message-State: ALoCoQl59RFI1VhsPH/GxSYm3bX5TxfN3apWCYxLCYBnhTs4WFr7wA//og4GpPxb1iSYHAR/Ccpd X-Received: by 10.112.199.70 with SMTP id ji6mr11386512lbc.73.1444037218778; Mon, 05 Oct 2015 02:26:58 -0700 (PDT) Received: from spencer.imf.au.dk ([130.225.20.51]) by smtp.gmail.com with ESMTPSA id a138sm4097810lfe.19.2015.10.05.02.26.58 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 05 Oct 2015 02:26:58 -0700 (PDT) From: Rasmus Villemoes To: Christoph Hellwig , Hannes Reinecke , "James E.J. Bottomley" Cc: linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org, Rasmus Villemoes Subject: [PATCH 2/2] scsi: reduce CONFIG_SCSI_CONSTANTS=y impact by 8k Date: Mon, 5 Oct 2015 11:26:40 +0200 Message-Id: <1444037200-14684-3-git-send-email-linux@rasmusvillemoes.dk> X-Mailer: git-send-email 2.1.3 In-Reply-To: <1444037200-14684-1-git-send-email-linux@rasmusvillemoes.dk> References: <20151004080217.GA22816@infradead.org> <1444037200-14684-1-git-send-email-linux@rasmusvillemoes.dk> Sender: linux-scsi-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-scsi@vger.kernel.org X-Spam-Status: No, score=-6.8 required=5.0 tests=BAYES_00,DKIM_SIGNED, RCVD_IN_DNSWL_HI,T_DKIM_INVALID,T_RP_MATCHES_RCVD,UNPARSEABLE_RELAY autolearn=unavailable version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP 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 Signed-off-by: Rasmus Villemoes --- drivers/scsi/constants.c | 25 +++++++++++++++++++++---- drivers/scsi/sense_codes.h | 2 -- 2 files changed, 21 insertions(+), 6 deletions(-) diff --git a/drivers/scsi/constants.c b/drivers/scsi/constants.c index 47aaccd5e68e..ccd34b0481cd 100644 --- a/drivers/scsi/constants.c +++ b/drivers/scsi/constants.c @@ -292,17 +292,31 @@ bool scsi_opcode_sa_name(int opcode, int service_action, struct error_info { unsigned short code12; /* 0x0302 looks better than 0x03,0x02 */ - const char * text; + unsigned short size; }; +/* + * There are 700+ entries in this table. To save space, we don't store + * (code, pointer) pairs, which would make sizeof(struct + * error_info)==16 on 64 bits. Rather, the second element just stores + * the size (including \0) of the corresponding string, and we use the + * sum of these to get the appropriate offset into additional_text + * defined below. This approach saves 12 bytes per entry. + */ static const struct error_info additional[] = { -#define SENSE_CODE(c, s) {c, s}, +#define SENSE_CODE(c, s) {c, sizeof(s)}, #include "sense_codes.h" #undef SENSE_CODE }; +static const char *additional_text = +#define SENSE_CODE(c, s) s "\0" +#include "sense_codes.h" +#undef SENSE_CODE + ; + struct error_info2 { unsigned char code1, code2_min, code2_max; const char * str; @@ -364,11 +378,14 @@ scsi_extd_sense_format(unsigned char asc, unsigned char ascq, const char **fmt) { int i; unsigned short code = ((asc << 8) | ascq); + unsigned offset = 0; *fmt = NULL; - for (i = 0; additional[i].text; i++) + for (i = 0; i < ARRAY_SIZE(additional); i++) { if (additional[i].code12 == code) - return additional[i].text; + return additional_text + offset; + offset += additional[i].size; + } for (i = 0; additional2[i].fmt; i++) { if (additional2[i].code1 == asc && ascq >= additional2[i].code2_min && diff --git a/drivers/scsi/sense_codes.h b/drivers/scsi/sense_codes.h index 54b3939d6309..da84d53b3379 100644 --- a/drivers/scsi/sense_codes.h +++ b/drivers/scsi/sense_codes.h @@ -833,5 +833,3 @@ SENSE_CODE(0x746E, "External data encryption control timeout") SENSE_CODE(0x746F, "External data encryption control error") SENSE_CODE(0x7471, "Logical unit access not authorized") SENSE_CODE(0x7479, "Security conflict in translated device") - -SENSE_CODE(0, NULL)