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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <201609070649.nsmY4rum%fengguang.wu@intel.com>
Date:   Wed, 7 Sep 2016 06:15:34 +0800
From:   kbuild test robot <lkp@...el.com>
To:     Baoyou Xie <baoyou.xie@...aro.org>
Cc:     kbuild-all@...org, anil.gurumurthy@...gic.com,
        sudarsana.kalluru@...gic.com, jejb@...ux.vnet.ibm.com,
        martin.petersen@...cle.com, linux-scsi@...r.kernel.org,
        linux-kernel@...r.kernel.org, arnd@...db.de, baoyou.xie@...aro.org,
        xie.baoyou@....com.cn
Subject: Re: [PATCH] [SCSI] bfa: mark symbols static where possible

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@...cade.com>
:::::: CC: James Bottomley <JBottomley@...allels.com>

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

Download attachment ".config.gz" of type "application/octet-stream" (30051 bytes)