linux-kernel - Re: [PATCH] usb: gadget: ether: Allow changing the MTU

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] [day] [month] [year] [list]

Date:	Mon, 30 Nov 2015 17:14:16 +0800
From:	kbuild test robot <lkp@...el.com>
To:	Mike Looijmans <mike.looijmans@...ic.nl>
Cc:	kbuild-all@...org, linux-usb@...r.kernel.org, balbi@...com,
	gregkh@...uxfoundation.org, linux-kernel@...r.kernel.org,
	Mike Looijmans <mike.looijmans@...ic.nl>
Subject: Re: [PATCH] usb: gadget: ether: Allow changing the MTU

Hi Mike,

[auto build test WARNING on: balbi-usb/next]
[also build test WARNING on: v4.4-rc3 next-20151127]

url:    https://github.com/0day-ci/linux/commits/Mike-Looijmans/usb-gadget-ether-Allow-changing-the-MTU/20151130-162847
base:   https://git.kernel.org/pub/scm/linux/kernel/git/balbi/usb.git next
config: i386-randconfig-s1-201548 (attached as .config)
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

All warnings (new ones prefixed by >>):

   drivers/usb/gadget/function/u_ether.c: In function 'ueth_change_mtu':
>> drivers/usb/gadget/function/u_ether.c:146:18: warning: unused variable 'dev' [-Wunused-variable]
     struct eth_dev *dev = netdev_priv(net);
                     ^

vim +/dev +146 drivers/usb/gadget/function/u_ether.c

2b3d942c drivers/usb/gadget/u_ether.c          David Brownell    2008-06-19  130  #else
2b3d942c drivers/usb/gadget/u_ether.c          David Brownell    2008-06-19  131  #define VDBG(dev, fmt, args...) \
2b3d942c drivers/usb/gadget/u_ether.c          David Brownell    2008-06-19  132  	do { } while (0)
2b3d942c drivers/usb/gadget/u_ether.c          David Brownell    2008-06-19  133  #endif /* DEBUG */
2b3d942c drivers/usb/gadget/u_ether.c          David Brownell    2008-06-19  134  
2b3d942c drivers/usb/gadget/u_ether.c          David Brownell    2008-06-19  135  #define ERROR(dev, fmt, args...) \
2b3d942c drivers/usb/gadget/u_ether.c          David Brownell    2008-06-19  136  	xprintk(dev , KERN_ERR , fmt , ## args)
2b3d942c drivers/usb/gadget/u_ether.c          David Brownell    2008-06-19  137  #define INFO(dev, fmt, args...) \
2b3d942c drivers/usb/gadget/u_ether.c          David Brownell    2008-06-19  138  	xprintk(dev , KERN_INFO , fmt , ## args)
2b3d942c drivers/usb/gadget/u_ether.c          David Brownell    2008-06-19  139  
2b3d942c drivers/usb/gadget/u_ether.c          David Brownell    2008-06-19  140  /*-------------------------------------------------------------------------*/
2b3d942c drivers/usb/gadget/u_ether.c          David Brownell    2008-06-19  141  
2b3d942c drivers/usb/gadget/u_ether.c          David Brownell    2008-06-19  142  /* NETWORK DRIVER HOOKUP (to the layer above this driver) */
2b3d942c drivers/usb/gadget/u_ether.c          David Brownell    2008-06-19  143  
ccad637b drivers/usb/gadget/u_ether.c          Stephen Hemminger 2008-11-19  144  static int ueth_change_mtu(struct net_device *net, int new_mtu)
2b3d942c drivers/usb/gadget/u_ether.c          David Brownell    2008-06-19  145  {
2b3d942c drivers/usb/gadget/u_ether.c          David Brownell    2008-06-19 @146  	struct eth_dev	*dev = netdev_priv(net);
1616c4ef drivers/usb/gadget/function/u_ether.c Mike Looijmans    2015-11-30  147  
1616c4ef drivers/usb/gadget/function/u_ether.c Mike Looijmans    2015-11-30  148  	if (new_mtu <= ETH_HLEN || new_mtu > GETHER_MAX_ETH_FRAME_LEN)
1616c4ef drivers/usb/gadget/function/u_ether.c Mike Looijmans    2015-11-30  149  		return -ERANGE;
2b3d942c drivers/usb/gadget/u_ether.c          David Brownell    2008-06-19  150  	net->mtu = new_mtu;
2b3d942c drivers/usb/gadget/u_ether.c          David Brownell    2008-06-19  151  
1616c4ef drivers/usb/gadget/function/u_ether.c Mike Looijmans    2015-11-30  152  	return 0;
2b3d942c drivers/usb/gadget/u_ether.c          David Brownell    2008-06-19  153  }
2b3d942c drivers/usb/gadget/u_ether.c          David Brownell    2008-06-19  154  

:::::: The code at line 146 was first introduced by commit
:::::: 2b3d942c4878084a37991a65e66512c02b8fa2ad usb ethernet gadget: split out network core

:::::: TO: David Brownell <dbrownell@...rs.sourceforge.net>
:::::: CC: Greg Kroah-Hartman <gregkh@...e.de>

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