Re: [PATCH 16/24] aacraid: Add task management functionality

kbuild test robot
2017-01-24T03:40:21Z

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

Hi Raghava,

[auto build test WARNING on scsi/for-next]
[also build test WARNING on v4.10-rc5 next-20170123]
[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/Raghava-Aditya-Renukunta/aacraid-Patchset-for-Smart-Family-card-support/20170124-093617
base: https://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi.git for-next
config: parisc-allyesconfig (attached as .config)
compiler: hppa-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705
reproduce:
wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
make.cross ARCH=parisc

All warnings (new ones prefixed by >>):

In file included from include/linux/byteorder/big_endian.h:4:0,
from arch/parisc/include/uapi/asm/byteorder.h:4,
from arch/parisc/include/asm/bitops.h:10,
from include/linux/bitops.h:36,
from include/linux/kernel.h:10,
from drivers/scsi/aacraid/aachba.c:27:
drivers/scsi/aacraid/aachba.c: In function 'aac_build_sghba':
include/uapi/linux/byteorder/big_endian.h:32:26: warning: large integer implicitly truncated to unsigned type [-Woverflow]
#define __cpu_to_le32(x) ((__force __le32)__swab32((x)))
^
include/linux/byteorder/generic.h:87:21: note: in expansion of macro '__cpu_to_le32'
#define cpu_to_le32 __cpu_to_le32
^~~~~~~~~~~~~
>> drivers/scsi/aacraid/aachba.c:4000:34: note: in expansion of macro 'cpu_to_le32'
hbacmd->emb_data_desc_count = cpu_to_le32(1);
^~~~~~~~~~~
drivers/scsi/aacraid/aachba.c: In function 'aac_report_phys_luns':
drivers/scsi/aacraid/aachba.c:1801:2: warning: 'addr' may be used uninitialized in this function [-Wmaybe-uninitialized]
pci_free_consistent(dev->pdev, datasize, (void *) phys_luns, addr);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/scsi/aacraid/aachba.c:1802:9: warning: 'rcode' may be used uninitialized in this function [-Wmaybe-uninitialized]
return rcode;
^~~~~

vim +/cpu_to_le32 +4000 drivers/scsi/aacraid/aachba.c

3984
3985 sge--;
3986 /* hba wants the size to be exact */
3987 if (byte_count > scsi_bufflen(scsicmd)) {
3988 u32 temp = le32_to_cpu(sge->len) -
3989 (byte_count - scsi_bufflen(scsicmd));
3990 sge->len = cpu_to_le32(temp);
3991 byte_count = scsi_bufflen(scsicmd);
3992 }
3993
3994 if (nseg <= HBA_MAX_SG_EMBEDDED) {
3995 hbacmd->emb_data_desc_count = cpu_to_le32(nseg);
3996 sge->flags = cpu_to_le32(0x40000000);
3997 } else {
3998 /* not embedded */
3999 hbacmd->sge[0].flags = cpu_to_le32(0x80000000);
> 4000 hbacmd->emb_data_desc_count = cpu_to_le32(1);
4001 hbacmd->sge[0].addr_hi =
4002 cpu_to_le32((u32)(sg_address >> 32));
4003 hbacmd->sge[0].addr_lo =
4004 cpu_to_le32((u32)(sg_address & 0xffffffff));
4005 }
4006
4007 /* Check for command underflow */
4008 if (scsicmd->underflow && (byte_count < scsicmd->underflow)) {

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

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 48402 bytes --]