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

firocu

Hi, all

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

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 上头部指针加成员偏移就是非法的。

firocu

回复 5# smalloc



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

firocu

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