scsi: fnic: make array dev_cmd_err static const, makes object smaller
authorColin Ian King <colin.king@canonical.com>
Fri, 6 Sep 2019 16:39:45 +0000 (17:39 +0100)
committerMartin K. Petersen <martin.petersen@oracle.com>
Tue, 1 Oct 2019 02:45:36 +0000 (22:45 -0400)
Don't populate the array dev_cmd_err on the stack but instead make it
static const. Makes the object code smaller by 80 bytes.

Before:
   text    data     bss     dec     hex filename
  21461    1564       0   23025    59f1 drivers/scsi/fnic/vnic_dev.o

After:
   text    data     bss     dec     hex filename
  21318    1628       0   22946    59a2 drivers/scsi/fnic/vnic_dev.o

(gcc version 9.2.1, amd64)

Link: https://lore.kernel.org/r/20190906163945.3889-1-colin.king@canonical.com
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/fnic/vnic_dev.c

index 78af9cc..1f55b9e 100644 (file)
@@ -259,7 +259,7 @@ int vnic_dev_cmd1(struct vnic_dev *vdev, enum vnic_devcmd_cmd cmd, int wait)
        struct vnic_devcmd __iomem *devcmd = vdev->devcmd;
        int delay;
        u32 status;
-       int dev_cmd_err[] = {
+       static const int dev_cmd_err[] = {
                /* convert from fw's version of error.h to host's version */
                0,      /* ERR_SUCCESS */
                EINVAL, /* ERR_EINVAL */