Re: [PATCH 1/3] net: core,vrf: Change pcpu_dstat fields to u64_stats_t

[kernel test robot <lkp@intel.com>]

Hi Jeremy,

kernel test robot noticed the following build warnings:

[auto build test WARNING on 32f88d65f01bf6f45476d7edbe675e44fb9e1d58]

url:    https://github.com/intel-lab-lkp/linux/commits/Jeremy-Kerr/net-core-vrf-Change-pcpu_dstat-fields-to-u64_stats_t/20240605-143942
base:   32f88d65f01bf6f45476d7edbe675e44fb9e1d58
patch link:    https://lore.kernel.org/r/20240605-dstats-v1-1-1024396e1670%40codeconstruct.com.au
patch subject: [PATCH 1/3] net: core,vrf: Change pcpu_dstat fields to u64_stats_t
config: i386-randconfig-062-20240607 (https://download.01.org/0day-ci/archive/20240607/202406070424.JtWImJfu-lkp@intel.com/config)
compiler: gcc-10 (Ubuntu 10.5.0-1ubuntu1) 10.5.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240607/202406070424.JtWImJfu-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202406070424.JtWImJfu-lkp@intel.com/

sparse warnings: (new ones prefixed by >>)
>> drivers/net/vrf.c:414:35: sparse: sparse: incorrect type in argument 1 (different address spaces) @@     expected struct u64_stats_t [usertype] *p @@     got struct u64_stats_t [noderef] __percpu * @@
   drivers/net/vrf.c:414:35: sparse:     expected struct u64_stats_t [usertype] *p
   drivers/net/vrf.c:414:35: sparse:     got struct u64_stats_t [noderef] __percpu *
   drivers/net/vrf.c: note: in included file (through include/linux/smp.h, include/linux/alloc_tag.h, include/linux/percpu.h, ...):
   include/linux/list.h:83:21: sparse: sparse: self-comparison always evaluates to true

vim +414 drivers/net/vrf.c

   391	
   392	/* Local traffic destined to local address. Reinsert the packet to rx
   393	 * path, similar to loopback handling.
   394	 */
   395	static int vrf_local_xmit(struct sk_buff *skb, struct net_device *dev,
   396				  struct dst_entry *dst)
   397	{
   398		int len = skb->len;
   399	
   400		skb_orphan(skb);
   401	
   402		skb_dst_set(skb, dst);
   403	
   404		/* set pkt_type to avoid skb hitting packet taps twice -
   405		 * once on Tx and again in Rx processing
   406		 */
   407		skb->pkt_type = PACKET_LOOPBACK;
   408	
   409		skb->protocol = eth_type_trans(skb, dev);
   410	
   411		if (likely(__netif_rx(skb) == NET_RX_SUCCESS))
   412			vrf_rx_stats(dev, len);
   413		else
 > 414			u64_stats_inc(&dev->dstats->rx_drops);
   415	
   416		return NETDEV_TX_OK;
   417	}
   418	

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki