Hi Eric,

I love your patch! Perhaps something to improve:

[auto build test WARNING on net-next/master]

url:    
https://github.com/0day-ci/linux/commits/Eric-Dumazet/net-add-CONFIG_PCPU_DEV_REFCNT/20210319-230417
base:   https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git 
38cb57602369cf194556460a52bd18e53c76e13d
config: arm-randconfig-r014-20210318 (attached as .config)
compiler: clang version 13.0.0 (https://github.com/llvm/llvm-project 
fcc1ce00931751ac02498986feb37744e9ace8de)
reproduce (this is a W=1 build):
        wget 
https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O 
~/bin/make.cross
        chmod +x ~/bin/make.cross
        # install arm cross compiling tool for clang build
        # apt-get install binutils-arm-linux-gnueabi
        # 
https://github.com/0day-ci/linux/commit/684c34243e0c84e496aa426734df321b7ebc088b
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review 
Eric-Dumazet/net-add-CONFIG_PCPU_DEV_REFCNT/20210319-230417
        git checkout 684c34243e0c84e496aa426734df321b7ebc088b
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <[email protected]>

All warnings (new ones prefixed by >>):

>> net/core/dev.c:10752:1: warning: unused label 'free_dev' [-Wunused-label]
   free_dev:
   ^~~~~~~~~
   net/core/dev.c:4993:1: warning: unused function 'sch_handle_ingress' 
[-Wunused-function]
   sch_handle_ingress(struct sk_buff *skb, struct packet_type **pt_prev, int 
*ret,
   ^
   net/core/dev.c:5143:19: warning: unused function 'nf_ingress' 
[-Wunused-function]
   static inline int nf_ingress(struct sk_buff *skb, struct packet_type 
**pt_prev,
                     ^
   3 warnings generated.


vim +/free_dev +10752 net/core/dev.c

8d3bdbd55a7e2a David S. Miller        2011-02-08  10703  
8d3bdbd55a7e2a David S. Miller        2011-02-08  10704         
INIT_LIST_HEAD(&dev->napi_list);
8d3bdbd55a7e2a David S. Miller        2011-02-08  10705         
INIT_LIST_HEAD(&dev->unreg_list);
5cde282938915f Eric W. Biederman      2013-10-05  10706         
INIT_LIST_HEAD(&dev->close_list);
8d3bdbd55a7e2a David S. Miller        2011-02-08  10707         
INIT_LIST_HEAD(&dev->link_watch_list);
2f268f129c2d1a Veaceslav Falico       2013-09-25  10708         
INIT_LIST_HEAD(&dev->adj_list.upper);
2f268f129c2d1a Veaceslav Falico       2013-09-25  10709         
INIT_LIST_HEAD(&dev->adj_list.lower);
7866a621043fba Salam Noureddine       2015-01-27  10710         
INIT_LIST_HEAD(&dev->ptype_all);
7866a621043fba Salam Noureddine       2015-01-27  10711         
INIT_LIST_HEAD(&dev->ptype_specific);
93642e14bd50e5 Jiri Pirko             2020-01-25  10712         
INIT_LIST_HEAD(&dev->net_notifier_list);
59cc1f61f09c26 Jiri Kosina            2016-08-10  10713  #ifdef CONFIG_NET_SCHED
59cc1f61f09c26 Jiri Kosina            2016-08-10  10714         
hash_init(dev->qdisc_hash);
59cc1f61f09c26 Jiri Kosina            2016-08-10  10715  #endif
0287587884b150 Eric Dumazet           2014-10-05  10716         dev->priv_flags 
= IFF_XMIT_DST_RELEASE | IFF_XMIT_DST_RELEASE_PERM;
8d3bdbd55a7e2a David S. Miller        2011-02-08  10717         setup(dev);
8d3bdbd55a7e2a David S. Miller        2011-02-08  10718  
a813104d923339 Phil Sutter            2016-02-17  10719         if 
(!dev->tx_queue_len) {
f84bb1eac02752 Phil Sutter            2015-08-27  10720                 
dev->priv_flags |= IFF_NO_QUEUE;
1159708432f706 Jesper Dangaard Brouer 2016-11-03  10721                 
dev->tx_queue_len = DEFAULT_TX_QUEUE_LEN;
a813104d923339 Phil Sutter            2016-02-17  10722         }
906470c19da771 Phil Sutter            2015-08-18  10723  
36909ea43814cb Tom Herbert            2011-01-09  10724         
dev->num_tx_queues = txqs;
36909ea43814cb Tom Herbert            2011-01-09  10725         
dev->real_num_tx_queues = txqs;
ed9af2e839c06c Tom Herbert            2010-11-09  10726         if 
(netif_alloc_netdev_queues(dev))
8d3bdbd55a7e2a David S. Miller        2011-02-08  10727                 goto 
free_all;
e8a0464cc95097 David S. Miller        2008-07-17  10728  
36909ea43814cb Tom Herbert            2011-01-09  10729         
dev->num_rx_queues = rxqs;
36909ea43814cb Tom Herbert            2011-01-09  10730         
dev->real_num_rx_queues = rxqs;
fe8222406c8277 Tom Herbert            2010-11-09  10731         if 
(netif_alloc_rx_queues(dev))
8d3bdbd55a7e2a David S. Miller        2011-02-08  10732                 goto 
free_all;
0a9627f2649a02 Tom Herbert            2010-03-16  10733  
^1da177e4c3f41 Linus Torvalds         2005-04-16  10734         
strcpy(dev->name, name);
c835a677331495 Tom Gundersen          2014-07-14  10735         
dev->name_assign_type = name_assign_type;
cbda10fa97d72c Vlad Dogaru            2011-01-13  10736         dev->group = 
INIT_NETDEV_GROUP;
2c60db037034d2 Eric Dumazet           2012-09-16  10737         if 
(!dev->ethtool_ops)
2c60db037034d2 Eric Dumazet           2012-09-16  10738                 
dev->ethtool_ops = &default_ethtool_ops;
e687ad60af0901 Pablo Neira            2015-05-13  10739  
357b6cc5834eab Daniel Borkmann        2020-03-18  10740         
nf_hook_ingress_init(dev);
e687ad60af0901 Pablo Neira            2015-05-13  10741  
^1da177e4c3f41 Linus Torvalds         2005-04-16  10742         return dev;
ab9c73ccb52f40 Jiri Pirko             2009-05-08  10743  
8d3bdbd55a7e2a David S. Miller        2011-02-08  10744  free_all:
8d3bdbd55a7e2a David S. Miller        2011-02-08  10745         
free_netdev(dev);
8d3bdbd55a7e2a David S. Miller        2011-02-08  10746         return NULL;
8d3bdbd55a7e2a David S. Miller        2011-02-08  10747  
29b4433d991c88 Eric Dumazet           2010-10-11  10748  free_pcpu:
684c34243e0c84 Eric Dumazet           2021-03-19  10749  #ifdef 
CONFIG_PCPU_DEV_REFCNT
29b4433d991c88 Eric Dumazet           2010-10-11  10750         
free_percpu(dev->pcpu_refcnt);
684c34243e0c84 Eric Dumazet           2021-03-19  10751  #endif
74d332c13b2148 Eric Dumazet           2013-10-30 @10752  free_dev:
74d332c13b2148 Eric Dumazet           2013-10-30  10753         
netdev_freemem(dev);
ab9c73ccb52f40 Jiri Pirko             2009-05-08  10754         return NULL;
^1da177e4c3f41 Linus Torvalds         2005-04-16  10755  }
36909ea43814cb Tom Herbert            2011-01-09  10756  
EXPORT_SYMBOL(alloc_netdev_mqs);
^1da177e4c3f41 Linus Torvalds         2005-04-16  10757  

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/[email protected]

Attachment: .config.gz
Description: application/gzip

Reply via email to