Re: [PATCH] net: ifb: record drop reason when redirect device is missing

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

Hi Murari,

kernel test robot noticed the following build errors:

[auto build test ERROR on net/main]
[also build test ERROR on net-next/main linus/master v6.19-rc8 next-20260202]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Murari-Prasad-Samal/net-ifb-record-drop-reason-when-redirect-device-is-missing/20260203-031526
base:   net/main
patch link:    https://lore.kernel.org/r/20260202191135.7668-1-murariprasadsamal57%40gmail.com
patch subject: [PATCH] net: ifb: record drop reason when redirect device is missing
config: i386-allnoconfig-bpf (https://download.01.org/0day-ci/archive/20260203/202602031242.kPM0xECv-lkp@intel.com/config)
compiler: clang version 20.1.8 (https://github.com/llvm/llvm-project 87f0227cb60147a26a1eeb4fb06e3b505e9c7261)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260203/202602031242.kPM0xECv-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/202602031242.kPM0xECv-lkp@intel.com/

All errors (new ones prefixed by >>):

>> drivers/net/ifb.c:118:26: error: use of undeclared identifier 'SKB_DROP_REASON_NO_DEV'; did you mean 'SKB_DROP_REASON_NOMEM'?
     118 |                         kfree_skb_reason(skb, SKB_DROP_REASON_NO_DEV);
         |                                               ^~~~~~~~~~~~~~~~~~~~~~
         |                                               SKB_DROP_REASON_NOMEM
   ./include/net/dropreason-core.h:446:2: note: 'SKB_DROP_REASON_NOMEM' declared here
     446 |         SKB_DROP_REASON_NOMEM,
         |         ^
   1 error generated.


vim +118 drivers/net/ifb.c

    88	
    89	static void ifb_ri_tasklet(struct tasklet_struct *t)
    90	{
    91		struct ifb_q_private *txp = from_tasklet(txp, t, ifb_tasklet);
    92		struct netdev_queue *txq;
    93		struct sk_buff *skb;
    94	
    95		txq = netdev_get_tx_queue(txp->dev, txp->txqnum);
    96		skb = skb_peek(&txp->tq);
    97		if (!skb) {
    98			if (!__netif_tx_trylock(txq))
    99				goto resched;
   100			skb_queue_splice_tail_init(&txp->rq, &txp->tq);
   101			__netif_tx_unlock(txq);
   102		}
   103	
   104		while ((skb = __skb_dequeue(&txp->tq)) != NULL) {
   105			/* Skip tc and netfilter to prevent redirection loop. */
   106			skb->redirected = 0;
   107	#ifdef CONFIG_NET_CLS_ACT
   108			skb->tc_skip_classify = 1;
   109	#endif
   110			nf_skip_egress(skb, true);
   111	
   112			ifb_update_q_stats(&txp->tx_stats, skb->len);
   113	
   114			rcu_read_lock();
   115			skb->dev = dev_get_by_index_rcu(dev_net(txp->dev), skb->skb_iif);
   116			if (!skb->dev) {
   117				rcu_read_unlock();
 > 118				kfree_skb_reason(skb, SKB_DROP_REASON_NO_DEV);
   119				txp->dev->stats.tx_dropped++;
   120				if (skb_queue_len(&txp->tq) != 0)
   121					goto resched;
   122				break;
   123			}
   124			rcu_read_unlock();
   125			skb->skb_iif = txp->dev->ifindex;
   126	
   127			if (!skb->from_ingress) {
   128				dev_queue_xmit(skb);
   129			} else {
   130				skb_pull_rcsum(skb, skb->mac_len);
   131				netif_receive_skb(skb);
   132			}
   133		}
   134	
   135		if (__netif_tx_trylock(txq)) {
   136			skb = skb_peek(&txp->rq);
   137			if (!skb) {
   138				txp->tasklet_pending = 0;
   139				if (netif_tx_queue_stopped(txq))
   140					netif_tx_wake_queue(txq);
   141			} else {
   142				__netif_tx_unlock(txq);
   143				goto resched;
   144			}
   145			__netif_tx_unlock(txq);
   146		} else {
   147	resched:
   148			txp->tasklet_pending = 1;
   149			tasklet_schedule(&txp->ifb_tasklet);
   150		}
   151	

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