有关netpoll的2个疑问。。。

chishanmingshen

from 3.7.0

1.

1. static void netpoll_poll_dev(struct net_device *dev)
   
2. {
   
3.         const struct net_device_ops *ops;
   
4.         struct netpoll_info *ni = rcu_dereference_bh(dev->npinfo);
   
5. 
   
6.         if (!dev || !netif_running(dev))
   
7.                 return;
   
8. 
   
9.         ops = dev->netdev_ops;
   
10.         if (!ops->ndo_poll_controller)<-------------提前跳出了？！
    
11.                 return;
    
12. 
    
13.         /* Process pending work on NIC */
    
14.         ops->ndo_poll_controller(dev);
    
15. 
    
16.         poll_napi(dev);<-----------------------此处为何在netpoll实例不需要收包时跳过？
    
17. 
    
18.         （以下省略部分代码）
    
19. }
    
20. 
    

复制代码

既然调用poll_napi()可以释放部分内存，为何在netpoll实例不需要收包时跳过？？



2.

1. 
   
2. /* call with IRQ disabled */
   
3. void netpoll_send_skb_on_dev(struct netpoll *np, struct sk_buff *skb,
   
4.                              struct net_device *dev)
   
5. {
   
6.         int status = NETDEV_TX_BUSY;
   
7.         unsigned long tries;
   
8.         const struct net_device_ops *ops = dev->netdev_ops;
   
9.         /* It is up to the caller to keep npinfo alive. */
   
10.         struct netpoll_info *npinfo;
    
11. 
    
12.         WARN_ON_ONCE(!irqs_disabled());
    
13. 
    
14.         npinfo = rcu_dereference_bh(np->dev->npinfo);
    
15.         if (!npinfo || !netif_running(dev) || !netif_device_present(dev)) {
    
16.                 __kfree_skb(skb);
    
17.                 return;
    
18.         }
    
19. 
    
20.         /* don't get messages out of order, and no recursion */
    
21.         if (skb_queue_len(&npinfo->txq) == 0 && !netpoll_owner_active(dev)) {<--------此处为何要判断队列为空这个条件？
    
22.                 struct netdev_queue *txq;
    
23. 
    
24.                 txq = netdev_pick_tx(dev, skb);
    
25. 
    
26.                 /* try until next clock tick */
    
27.                 for (tries = jiffies_to_usecs(1)/USEC_PER_POLL;
    
28.                      tries > 0; --tries) {
    
29.                         if (__netif_tx_trylock(txq)) {
    
30.                                 if (!netif_xmit_stopped(txq)) {
    
31.                                         if (vlan_tx_tag_present(skb) &&
    
32.                                             !(netif_skb_features(skb) & NETIF_F_HW_VLAN_TX)) {
    
33.                                                 skb = __vlan_put_tag(skb, vlan_tx_tag_get(skb));
    
34.                                                 if (unlikely(!skb))
    
35.                                                         break;
    
36.                                                 skb->vlan_tci = 0;
    
37.                                         }
    
38. 
    
39.                                         status = ops->ndo_start_xmit(skb, dev);
    
40.                                         if (status == NETDEV_TX_OK)
    
41.                                                 txq_trans_update(txq);
    
42.                                 }
    
43.                                 __netif_tx_unlock(txq);
    
44. 
    
45.                                 if (status == NETDEV_TX_OK)
    
46.                                         break;
    
47. 
    
48.                         }
    
49. 
    
50.                         /* tickle device maybe there is some cleanup */
    
51.                         netpoll_poll_dev(np->dev);
    
52. 
    
53.                         udelay(USEC_PER_POLL);
    
54.                 }
    
55. 
    
56.                 WARN_ONCE(!irqs_disabled(),
    
57.                         "netpoll_send_skb_on_dev(): %s enabled interrupts in poll (%pF)\n",
    
58.                         dev->name, ops->ndo_start_xmit);
    
59. 
    
60.         }
    
61. 
    
62.         if (status != NETDEV_TX_OK) {
    
63.                 skb_queue_tail(&npinfo->txq, skb);
    
64.                 schedule_delayed_work(&npinfo->tx_work,0);
    
65.         }
    
66. }
    

复制代码

为何要加上队列为空这个条件？



疑惑的2个问题，请高手指点，谢谢先！！！

灌水菜鸟

我的理解：
正常工作模式：网卡硬件触发中断，软件介入之后，到驱动这一层的时候，会首先应答一下网卡，比如读，清中断控制，状态寄存器，然后处理正事，NAPI正式介入干活。

poll模式呢，不依赖中断的模式：管你有没有中断，先应答一下网卡，然后处理正事，就像中断模式一样。

这里面“应答”这个动作就被单拎出来，包装出一个ndo_poll_controller的接口，其他的都一样。所以，这个接口是否实现就比较重要了，实现了，说明驱动支持poll模式，没实现就不支持。就不要再继续干活了，于是就直接退出，而不是强行调用napi接口。

个人理解，欢迎斧正。

hmsghnh

前面注释写了啊，
skb_queue_len(&npinfo->txq) == 0  避免 消息乱序
&& !netpoll_owner_active(dev)   避免递归嵌套


如果队列不为空，说明前面还有数据包还没发送出去，要先发完前面的包，才能发送现在的包，不然顺序就乱了。
那个for循环里面就是马上把包发出去。

前面还有一个 queue_process  函数，那个会处理这恶搞队列里面的包的。

chishanmingshen

哦。明白了。谢谢！

就是说这个分支：(skb_queue_len(&npinfo->txq) == 0 && !netpoll_owner_active(dev))
其实没有也行，有它仅仅是提高了一点发包效率。

我理解没错吧，那么1楼的第1个问题如何理解呢？

回复 2# hmsghnh


   

灌水菜鸟

关于第一个问题：

ops->ndo_poll_controller 是用来“主动”处理“中断”事件的。poll模式么，不依赖你中断了，无论如何，先poll一下controller。

poll_napi(dev) 这个呢，就是用来实际干活的，poll模式下主动干活的程序段。这个呢，逻辑上是要依赖于上面这个controller的操作的。当然，实际上很多驱动实现中，这个函数功能上不依赖controller，但是逻辑上还是依赖的。你都没有实现对controller的poll，剩下的咱们也就算了吧，说明人家驱动就没打算让咱们这个干，万一强行poll_napi出了事算谁的？poll还是driver？

chishanmingshen

回复 4# 灌水菜鸟

你都没有实现对controller的poll，剩下的咱们也就算了吧，说明人家驱动就没打算让咱们这个干，万一强行poll_napi出了事算谁的？poll还是driver？
----->没明白啊，没有实现controller的poll就不能执行poll_napi()?恕我愚钝，请再指点下吧！





   

chishanmingshen

尝试理解你说的！谢谢！

看了代码，你说的“应答”：包括读，清中断控制，状态寄存器，还包括将此设备的napi结构体放到sd中吧？！
就是这个：
list_add_tail(&napi->poll_list, &sd->poll_list);


回复 6# 灌水菜鸟


   

灌水菜鸟

是，这个也是必要的。呵呵，这个还真是没有注意到。只想到控制器的操作了，数据结构的准备确实没想到，多谢了。

chishanmingshen

回复 8# 灌水菜鸟


初步了解了，谢谢！

只有支持“应答”才能支持poll（这里是netpoll），尽管很多设备都支持poll。

其实我感觉搞个flag就好理解了。