Re: [PATCH 06/25] qla2xxx: Reduce excessive debug print during 27xx fwdump.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Joe,

[auto build test WARNING on scsi/for-next]
[also build test WARNING on v4.12-rc1 next-20170519]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Himanshu-Madhani/qla2xxx-Bug-fixes-and-cleanups/20170521-131406
base:   https://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi.git for-next
config: i386-allmodconfig (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

All warnings (new ones prefixed by >>):

   drivers/scsi/qla2xxx/qla_init.c: In function 'qla24xx_load_risc_flash':
>> drivers/scsi/qla2xxx/qla_init.c:6359:50: warning: format '%lx' expects argument of type 'long unsigned int', but argument 5 has type 'uint32_t {aka unsigned int}' [-Wformat=]
          "Failed fwdump template exceeds array by %lx bytes\n",
                                                     ^
   drivers/scsi/qla2xxx/qla_init.c: In function 'qla24xx_load_risc_blob':
   drivers/scsi/qla2xxx/qla_init.c:6661:50: warning: format '%lx' expects argument of type 'long unsigned int', but argument 5 has type 'uint32_t {aka unsigned int}' [-Wformat=]
          "Failed fwdump template exceeds array by %lx bytes\n",
                                                     ^

vim +6359 drivers/scsi/qla2xxx/qla_init.c

  6343		dcode = ha->fw_dump_template;
  6344		qla24xx_read_flash_data(vha, dcode, faddr, risc_size);
  6345		for (i = 0; i < risc_size; i++)
  6346			dcode[i] = le32_to_cpu(dcode[i]);
  6347	
  6348		if (!qla27xx_fwdt_template_valid(dcode)) {
  6349			ql_log(ql_log_warn, vha, 0x0165,
  6350			    "Failed fwdump template validate\n");
  6351			goto default_template;
  6352		}
  6353	
  6354		dlen = qla27xx_fwdt_template_size(dcode);
  6355		ql_dbg(ql_dbg_init, vha, 0x0166,
  6356		    "-> template size %x bytes\n", dlen);
  6357		if (dlen > risc_size * sizeof(*dcode)) {
  6358			ql_log(ql_log_warn, vha, 0x0167,
> 6359			    "Failed fwdump template exceeds array by %lx bytes\n",
  6360			    dlen - risc_size * sizeof(*dcode));
  6361			goto default_template;
  6362		}
  6363		ha->fw_dump_template_len = dlen;
  6364		return rval;
  6365	
  6366	default_template:
  6367		ql_log(ql_log_warn, vha, 0x0168, "Using default fwdump template\n");

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

Attachment: .config.gz
Description: application/gzip


[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]

  Powered by Linux