netdev - Re: [PATCH 1/1] smsc: Enabling SMSC LAN911 driver for ARM64 as well.

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]
Date:	Tue, 12 Jan 2016 06:32:13 +0800
From:	kbuild test robot <lkp@...el.com>
To:	Manjeet Pawar <manjeet.p@...sung.com>
Cc:	kbuild-all@...org, davem@...emloft.net,
	paul.gortmaker@...driver.com, schmitzmic@...il.com,
	geert@...ux-m68k.org, nico@...xnic.net, netdev@...r.kernel.org,
	linux-kernel@...r.kernel.org, manjeet.p@...sung.com,
	akhilesh.k@...sung.com, pankaj.m@...sung.com
Subject: Re: [PATCH 1/1] smsc: Enabling SMSC LAN911 driver for ARM64 as well.

Hi Manjeet,

[auto build test WARNING on net-next/master]
[also build test WARNING on v4.4 next-20160111]
[if your patch is applied to the wrong git tree, please drop us a note to help improving the system]

url:    https://github.com/0day-ci/linux/commits/Manjeet-Pawar/smsc-Enabling-SMSC-LAN911-driver-for-ARM64-as-well/20160111-170643
config: arm64-allmodconfig (attached as .config)
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=arm64 

All warnings (new ones prefixed by >>):

   drivers/net/ethernet/smsc/smc911x.c: In function 'smc911x_hardware_send_pkt':
>> drivers/net/ethernet/smsc/smc911x.c:475:16: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
     buf = (char*)((u32)skb->data & ~0x3);
                   ^
>> drivers/net/ethernet/smsc/smc911x.c:475:8: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
     buf = (char*)((u32)skb->data & ~0x3);
           ^
   drivers/net/ethernet/smsc/smc911x.c:476:25: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
     len = (skb->len + 3 + ((u32)skb->data & 3)) & ~0x3;
                            ^
   drivers/net/ethernet/smsc/smc911x.c:477:11: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
     cmdA = (((u32)skb->data & 0x3) << 16) |
              ^

vim +475 drivers/net/ethernet/smsc/smc911x.c

dcdf8710 drivers/net/ethernet/smsc/smc911x.c Ben Boeckel     2013-11-01  459  	DBG(SMC_DEBUG_FUNC | SMC_DEBUG_TX, dev, "--> %s\n", __func__);
0a0c72c9 drivers/net/smc911x.c               Dustin McIntire 2006-04-19  460  	BUG_ON(lp->pending_tx_skb == NULL);
0a0c72c9 drivers/net/smc911x.c               Dustin McIntire 2006-04-19  461  
0a0c72c9 drivers/net/smc911x.c               Dustin McIntire 2006-04-19  462  	skb = lp->pending_tx_skb;
0a0c72c9 drivers/net/smc911x.c               Dustin McIntire 2006-04-19  463  	lp->pending_tx_skb = NULL;
0a0c72c9 drivers/net/smc911x.c               Dustin McIntire 2006-04-19  464  
0a0c72c9 drivers/net/smc911x.c               Dustin McIntire 2006-04-19  465  	/* cmdA {25:24] data alignment [20:16] start offset [10:0] buffer length */
0a0c72c9 drivers/net/smc911x.c               Dustin McIntire 2006-04-19  466  	/* cmdB {31:16] pkt tag [10:0] length */
0a0c72c9 drivers/net/smc911x.c               Dustin McIntire 2006-04-19  467  #ifdef SMC_USE_DMA
0a0c72c9 drivers/net/smc911x.c               Dustin McIntire 2006-04-19  468  	/* 16 byte buffer alignment mode */
0a0c72c9 drivers/net/smc911x.c               Dustin McIntire 2006-04-19  469  	buf = (char*)((u32)(skb->data) & ~0xF);
0a0c72c9 drivers/net/smc911x.c               Dustin McIntire 2006-04-19  470  	len = (skb->len + 0xF + ((u32)skb->data & 0xF)) & ~0xF;
0a0c72c9 drivers/net/smc911x.c               Dustin McIntire 2006-04-19  471  	cmdA = (1<<24) | (((u32)skb->data & 0xF)<<16) |
0a0c72c9 drivers/net/smc911x.c               Dustin McIntire 2006-04-19  472  			TX_CMD_A_INT_FIRST_SEG_ | TX_CMD_A_INT_LAST_SEG_ |
0a0c72c9 drivers/net/smc911x.c               Dustin McIntire 2006-04-19  473  			skb->len;
0a0c72c9 drivers/net/smc911x.c               Dustin McIntire 2006-04-19  474  #else
0a0c72c9 drivers/net/smc911x.c               Dustin McIntire 2006-04-19 @475  	buf = (char*)((u32)skb->data & ~0x3);
0a0c72c9 drivers/net/smc911x.c               Dustin McIntire 2006-04-19  476  	len = (skb->len + 3 + ((u32)skb->data & 3)) & ~0x3;
0a0c72c9 drivers/net/smc911x.c               Dustin McIntire 2006-04-19  477  	cmdA = (((u32)skb->data & 0x3) << 16) |
0a0c72c9 drivers/net/smc911x.c               Dustin McIntire 2006-04-19  478  			TX_CMD_A_INT_FIRST_SEG_ | TX_CMD_A_INT_LAST_SEG_ |
0a0c72c9 drivers/net/smc911x.c               Dustin McIntire 2006-04-19  479  			skb->len;
0a0c72c9 drivers/net/smc911x.c               Dustin McIntire 2006-04-19  480  #endif
0a0c72c9 drivers/net/smc911x.c               Dustin McIntire 2006-04-19  481  	/* tag is packet length so we can use this in stats update later */
0a0c72c9 drivers/net/smc911x.c               Dustin McIntire 2006-04-19  482  	cmdB = (skb->len  << 16) | (skb->len & 0x7FF);
0a0c72c9 drivers/net/smc911x.c               Dustin McIntire 2006-04-19  483  

:::::: The code at line 475 was first introduced by commit
:::::: 0a0c72c9118c4e63080eb409f0cfdf15808d23a4 [PATCH] RE: [PATCH 1/1] net driver: Add support for SMSC LAN911x line of ethernet chips

:::::: TO: Dustin McIntire <dustin@...soria.com>
:::::: CC: Jeff Garzik <jeff@...zik.org>

---
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" (46847 bytes)