内核协议栈， ip层和tcp层在skb中地址相同

firocu

Hi, all

简单说啦就是
skb_transport_header()和 skb_network_header();
的值，竟然是相同。
内核，为什么这么做呢？
组长说，自己做了好多年，也没明白内核为什么这么做。
求解啊。

Godbach

回复 1# firocu

因为现在数据包还在网络层，所以三层四层 header 都先指向网络层头部。因为尚未解析四层呢。

这个时候，你是不能用所谓的 skb 中的四层 header 去读取四层数据的。要想读取的话，要自己通过偏移获取。

等到数据包在三层处理完毕，提交给四层时，四层header 就指向真正的四层头部了。

   

瀚海书香

回复 1# firocu

简单说啦就是
skb_transport_header()和 skb_network_header();
的值，竟然是相同。
内核，为什么这么做呢？
组长说，自己做了好多年，也没明白内核为什么这么做。
求解啊。

看2楼God的解释，你组长做好多年了，可能关注的方向不再netdev部分吧

   

firocu

原来如此。
我搜了下2.6.37的代码是在

1. static int ip_local_deliver_finish(struct sk_buff *skb)
   
2. {
   
3.         __skb_pull(skb , ip_hdrlen(skb)) ;
   
4.         skb_reset_transport_header(skb) ;
   
5. }
   

复制代码

ip层结束是更新下tcp header 的指针。
我看了下最新的代码3.11，死活也找不到skb_reset_transport_header(skb) ;这句了。。。而且其他地方也没有更新tcp header的函数。
在接下来的tcp_v4_rcv就直接用tcp_hdr(skb)调用skb_transport_header(skb)了。。。。这个慢慢找。
有一个问题。就是关于skb中存储的数据包的内容。
组长每次访问头部都会小心翼翼地先调用：

1. iph = skb_header_pointer(skb, skb_network_offset(skb), sizeof(iph_buff), &iph_buff)

复制代码

我觉得没必要。。。直接用

1. ip_hdr(skb);

复制代码

不就完了吗？ 会有什么问题？

===================SOLVED====================

因为可能skb中由分散聚合的数据， 直接指针访问有问题：
1 读取超过1 byte数据，比如short，可能是上一个page尾部一个byte，下一个page的开始的那一个byte，你是不能通过头部指针找到的。
2 或者访问的数据就在第二个page 上头部指针加成员偏移就是非法的。

smalloc

回复 4# firocu


    没看懂，这个和page有啥关系？

firocu

回复 5# smalloc



    struct skb_shared_info 这个结构的frag_list和frags成员都涉及到了page， 可能不连续。

henrystark

回复 6# firocu
好久没上论坛~
用ip_hdr(skb);就可以了吧，我以前打印过发送信息，用这个取没啥问题。


   

Godbach

回复 4# firocu

我看了下, 3.11-rc1 中还是有的，查了下 git log。如下：

git log 21d1196a35f5686c4323e42a62fdb4b23b0ab4a3
ipv4: set transport header earlier

   
后面的修改把四层 header 的设置提前了，放到 ip_rcv() 函数中了

     30 diff --git a/net/ipv4/ip_input.c b/net/ipv4/ip_input.c
     31 index 3da817b..15e3e68 100644
     32 --- a/net/ipv4/ip_input.c
     33 +++ b/net/ipv4/ip_input.c
     34 @@ -190,10 +190,7 @@ static int ip_local_deliver_finish(struct sk_buff *skb)
     35  {
     36         struct net *net = dev_net(skb->dev);
     37  
     38 -       __skb_pull(skb, ip_hdrlen(skb));
     39 -
     40 -       /* Point into the IP datagram, just past the header. */
     41 -       skb_reset_transport_header(skb);
     42 +       __skb_pull(skb, skb_network_header_len(skb));
     43  
     44         rcu_read_lock();
     45         {
     46 @@ -437,6 +434,8 @@ int ip_rcv(struct sk_buff *skb, struct net_device *dev, struct packet_type *pt,
     47                 goto drop;
     48         }
     49  
     50 +       skb->transport_header = skb->network_header + iph->ihl*4;
     51 +
     52         /* Remove any debris in the socket control block */
     53         memset(IPCB(skb), 0, sizeof(struct inet_skb_parm));

Line 50 应该就是你想找到的。

firocu

回复 7# henrystark
有潜在的内存不连续的问题。 我觉得不会出现。