Re: [PATCH] [SCSI] bfa: mark symbols static where possible

[kbuild test robot <lkp@intel.com>]

[-- Attachment #1: Type: text/plain, Size: 3329 bytes --]

Hi Baoyou,

[auto build test WARNING on scsi/for-next]
[also build test WARNING on v4.8-rc5 next-20160906]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
[Suggest to use git(>=2.9.0) format-patch --base=<commit> (or --base=auto for convenience) to record what (public, well-known) commit your patch series was built on]
[Check https://git-scm.com/docs/git-format-patch for more information]

url:    https://github.com/0day-ci/linux/commits/Baoyou-Xie/bfa-mark-symbols-static-where-possible/20160904-161528
base:   https://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi.git for-next
config: x86_64-randconfig-s1-09070420 (attached as .config)
compiler: gcc-4.4 (Debian 4.4.7-8) 4.4.7
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All warnings (new ones prefixed by >>):

   drivers/scsi/bfa/bfad_bsg.c: In function 'bfad_iocmd_handler':
>> drivers/scsi/bfa/bfad_bsg.c:3130: warning: the frame size of 2896 bytes is larger than 2048 bytes
   drivers/scsi/bfa/bfad_bsg.o: warning: objtool: bfad_fcxp_free_mem()+0xd5: function has unreachable instruction

vim +3130 drivers/scsi/bfa/bfad_bsg.c

e6826c96 Krishna Gudipati 2012-09-21  3114  		break;
e6826c96 Krishna Gudipati 2012-09-21  3115  	/* FRU */
e6826c96 Krishna Gudipati 2012-09-21  3116  	case IOCMD_FRUVPD_READ:
e6826c96 Krishna Gudipati 2012-09-21  3117  		rc = bfad_iocmd_fruvpd_read(bfad, iocmd);
e6826c96 Krishna Gudipati 2012-09-21  3118  		break;
e6826c96 Krishna Gudipati 2012-09-21  3119  	case IOCMD_FRUVPD_UPDATE:
e6826c96 Krishna Gudipati 2012-09-21  3120  		rc = bfad_iocmd_fruvpd_update(bfad, iocmd);
e6826c96 Krishna Gudipati 2012-09-21  3121  		break;
e6826c96 Krishna Gudipati 2012-09-21  3122  	case IOCMD_FRUVPD_GET_MAX_SIZE:
e6826c96 Krishna Gudipati 2012-09-21  3123  		rc = bfad_iocmd_fruvpd_get_max_size(bfad, iocmd);
e6826c96 Krishna Gudipati 2012-09-21  3124  		break;
b85daafe Krishna Gudipati 2011-06-13  3125  	default:
9afbcfab Krishna Gudipati 2011-07-20  3126  		rc = -EINVAL;
b85daafe Krishna Gudipati 2011-06-13  3127  		break;
b85daafe Krishna Gudipati 2011-06-13  3128  	}
9afbcfab Krishna Gudipati 2011-07-20  3129  	return rc;
b85daafe Krishna Gudipati 2011-06-13 @3130  }
b85daafe Krishna Gudipati 2011-06-13  3131  
b85daafe Krishna Gudipati 2011-06-13  3132  static int
b85daafe Krishna Gudipati 2011-06-13  3133  bfad_im_bsg_vendor_request(struct fc_bsg_job *job)
b85daafe Krishna Gudipati 2011-06-13  3134  {
b85daafe Krishna Gudipati 2011-06-13  3135  	uint32_t vendor_cmd = job->request->rqst_data.h_vendor.vendor_cmd[0];
b85daafe Krishna Gudipati 2011-06-13  3136  	struct bfad_im_port_s *im_port =
b85daafe Krishna Gudipati 2011-06-13  3137  			(struct bfad_im_port_s *) job->shost->hostdata[0];
b85daafe Krishna Gudipati 2011-06-13  3138  	struct bfad_s *bfad = im_port->bfad;

:::::: The code at line 3130 was first introduced by commit
:::::: b85daafe46eeb0a9ad32c4b2c3a4e09ffcae9599 [SCSI] bfa: Add BSG interface to support ELS, CT and vendor commands.

:::::: TO: Krishna Gudipati <kgudipat@brocade.com>
:::::: CC: James Bottomley <JBottomley@Parallels.com>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/octet-stream, Size: 30051 bytes --]