Linux SCSI subsystem development
 help / color / mirror / Atom feed
* [PATCH 0/5] bfa: fix W=1 build warnings
@ 2016-08-02 15:22 Arnd Bergmann
  2016-08-02 15:22 ` [PATCH 1/5] bfa: mark symbols static where possible Arnd Bergmann
                   ` (5 more replies)
  0 siblings, 6 replies; 10+ messages in thread
From: Arnd Bergmann @ 2016-08-02 15:22 UTC (permalink / raw)
  To: Anil Gurumurthy, Sudarsana Kalluru
  Cc: James E . J . Bottomley, Martin K . Petersen, linux-scsi,
	linux-kernel, Baoyou Xie, Arnd Bergmann

I tried to build an allmodconfig kernel with W=1, and the bfa
driver stuck out for having the most warnings in one file, so
I decided to send patches for all of these and make the driver
build cleanly with the extra warnings enabled.

As most warnings were for functions that lacked a 'static',
I marked them that way, and also used a script to look for
further functions that have a declaration but can also be static.

Overall, this saves around 10% of the size of the driver
module:

   text	   data	    bss	    dec	    hex	filename
 232027	   3012	    976	 236015	  399ef	drivers/scsi/bfa/bfa-before.o
 221851	   2996	    908	 225755	  371db	drivers/scsi/bfa/bfa-after.o

	Arnd

Arnd Bergmann (5):
  bfa: mark symbols static where possible
  bfa: remove unused variables
  bfa: rename some global variables
  bfa: remove unused functions from fbbuild.c
  bfa: remove more unused functions

 drivers/scsi/bfa/bfa.h           |  17 --
 drivers/scsi/bfa/bfa_core.c      |  54 +---
 drivers/scsi/bfa/bfa_fcbuild.c   | 609 +--------------------------------------
 drivers/scsi/bfa/bfa_fcbuild.h   |  90 ------
 drivers/scsi/bfa/bfa_fcpim.c     |  81 +++---
 drivers/scsi/bfa/bfa_fcpim.h     |  26 --
 drivers/scsi/bfa/bfa_fcs.c       |  31 +-
 drivers/scsi/bfa/bfa_fcs.h       |  54 ----
 drivers/scsi/bfa/bfa_fcs_lport.c | 211 +++-----------
 drivers/scsi/bfa/bfa_fcs_rport.c |  48 +--
 drivers/scsi/bfa/bfa_ioc.c       |  83 ++----
 drivers/scsi/bfa/bfa_ioc.h       |  25 --
 drivers/scsi/bfa/bfa_ioc_cb.c    |   3 +-
 drivers/scsi/bfa/bfa_ioc_ct.c    |  12 +-
 drivers/scsi/bfa/bfa_plog.h      |   8 -
 drivers/scsi/bfa/bfa_port.c      |   6 +-
 drivers/scsi/bfa/bfa_port.h      |   1 -
 drivers/scsi/bfa/bfa_svc.c       |  95 ++----
 drivers/scsi/bfa/bfa_svc.h       |  17 --
 drivers/scsi/bfa/bfad.c          | 128 ++++----
 drivers/scsi/bfa/bfad_attr.c     |   9 +-
 drivers/scsi/bfa/bfad_bsg.c      | 224 +++++++-------
 drivers/scsi/bfa/bfad_drv.h      |  49 +---
 drivers/scsi/bfa/bfad_im.c       |  29 +-
 drivers/scsi/bfa/bfad_im.h       |  10 -
 25 files changed, 395 insertions(+), 1525 deletions(-)

-- 
2.9.0


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

end of thread, other threads:[~2016-08-10  5:28 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-08-02 15:22 [PATCH 0/5] bfa: fix W=1 build warnings Arnd Bergmann
2016-08-02 15:22 ` [PATCH 1/5] bfa: mark symbols static where possible Arnd Bergmann
2016-08-03  4:45   ` kbuild test robot
2016-08-03  7:46     ` Arnd Bergmann
2016-08-10  5:20   ` Sudarsana Kalluru
2016-08-02 15:22 ` [PATCH 2/5] bfa: remove unused variables Arnd Bergmann
2016-08-02 15:22 ` [PATCH 3/5] bfa: rename some global variables Arnd Bergmann
2016-08-02 15:22 ` [PATCH 4/5] bfa: remove unused functions from fbbuild.c Arnd Bergmann
2016-08-02 15:22 ` [PATCH 5/5] bfa: remove more unused functions Arnd Bergmann
2016-08-10  5:28 ` [PATCH 0/5] bfa: fix W=1 build warnings Sudarsana Kalluru

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox