关于构造Http Response包的问题

platinum

如果未来的数据内容大于以前的，要看 tailroom 有没有空间，没有的话要扩展 skb 空间
我以前总结过但不知道对不对，也希望有高人帮我指点一下
http://blog.chinaunix.net/u/311/showart_1131002.html

Godbach

原帖由 platinum 于 2008-9-4 17:13 发表
如果未来的数据内容大于以前的，要看 tailroom 有没有空间，没有的话要扩展 skb 空间
我以前总结过但不知道对不对，也希望有高人帮我指点一下
http://blog.chinaunix.net/u/311/showart_1131002.html

偶这里是小于报文以前的内容，调整起来应该难度不大。

白金兄的Blog，小弟去看了一遍。对于函数skb_put，如果tailroom的空间不够用的话，是会出错并且调用skb_over_panic。

如果白金兄的方法是可以的，那么skb_put函数为什么不做这样的处理呢。

本身向一个报文的当前数据区末尾追加数据的应用极少出现（引自《Linux网络体系结构》），那么想白金兄这样的用法是不是更少出现了呢。
不知小弟理解的是否正确。

platinum

skb_put(len)：用于在当前有效数据末尾添加数据
如果 tailroom 够用的话是没有问题的，不够就要扩展了
skb_put() 的具体实现是这样的

1. 
   
2. 901 /**
   
3. 902  *      skb_put - add data to a buffer
   
4. 903  *      @skb: buffer to use
   
5. 904  *      @len: amount of data to add
   
6. 905  *
   
7. 906  *      This function extends the used data area of the buffer. If this would
   
8. 907  *      exceed the total buffer size the kernel will panic. A pointer to the
   
9. 908  *      first byte of the extra data is returned.
   
10. 909  */
    
11. 910 static inline unsigned char *skb_put(struct sk_buff *skb, unsigned int len)
    
12. 911 {
    
13. 912         unsigned char *tmp = skb_tail_pointer(skb);
    
14. 913         SKB_LINEAR_ASSERT(skb);
    
15. 914         skb->tail += len;
    
16. 915         skb->len  += len;
    
17. 916         if (unlikely(skb->tail > skb->end))
    
18. 917                 skb_over_panic(skb, len, current_text_addr());
    
19. 918         return tmp;
    
20. 919 }
    

复制代码

这个 len 的长度一定不会超过 tailroom，否则会出现问题，916 行就是做了这样的判断，并调用 skb_over_panic 提示错误并 hang
skb_over_panic 的处理如下

1. 
   
2.   74 /*
   
3.   75  *      Keep out-of-line to prevent kernel bloat.
   
4.   76  *      __builtin_return_address is not used because it is not always
   
5.   77  *      reliable.
   
6.   78  */
   
7.   79
   
8.   80 /**
   
9.   81  *      skb_over_panic  -       private function
   
10.   82  *      @skb: buffer
    
11.   83  *      @sz: size
    
12.   84  *      @here: address
    
13.   85  *
    
14.   86  *      Out of line support code for skb_put(). Not user callable.
    
15.   87  */
    
16.   88 void skb_over_panic(struct sk_buff *skb, int sz, void *here)
    
17.   89 {
    
18.   90         printk(KERN_EMERG "skb_over_panic: text:%p len:%d put:%d head:%p "
    
19.   91                           "data:%p tail:%#lx end:%#lx dev:%s\n",
    
20.   92                here, skb->len, sz, skb->head, skb->data,
    
21.   93                (unsigned long)skb->tail, (unsigned long)skb->end,
    
22.   94                skb->dev ? skb->dev->name : "<NULL>");
    
23.   95         BUG();
    
24.   96 }
    

复制代码

Godbach

如果 tailroom 够用的话是没有问题的，不够就要扩展了

这里应该如何扩展啊，内核代码中有扩展tailroom的地方吗？

skb_put中判断如果超过tailroom的话，就报错了，他自己没有去做相应的扩展

platinum

原帖由 Godbach 于 2008-9-4 18:17 发表


这里应该如何扩展啊，内核代码中有扩展tailroom的地方吗？

skb_put中判断如果超过tailroom的话，就报错了，他自己没有去做相应的扩展

好像是 skb_make_writable，我在 21 楼有提到，你再仔细看一下我 blog 里的那个东西
虽然我也不是很确定，但我觉得像

Godbach

原帖由 platinum 于 2008-9-4 18:56 发表

好像是 skb_make_writable，我在 21 楼有提到，你再仔细看一下我 blog 里的那个东西
虽然我也不是很确定，但我觉得像

应该是2.6中添加的，2.4没有。

感觉都是NAT方面的文件调用了该函数

platinum

原帖由 Godbach 于 2008-9-4 19:16 发表


应该是2.6中添加的，2.4没有。

感觉都是NAT方面的文件调用了该函数

这样，你看一下 2.4 内核里有没有 ipt_TCPMSS.c，如果有的话，他一定也做了相应的处理，参考他的实现方式
我看了一下，用的是 skb_copy_expand()

1. 
   
2.         /*
   
3.          * MSS Option not found ?! add it..
   
4.          */
   
5.         if (skb_tailroom((*pskb)) < TCPOLEN_MSS) {
   
6.                 struct sk_buff *newskb;
   
7. 
   
8.                 newskb = skb_copy_expand(*pskb, skb_headroom(*pskb),
   
9.                                          TCPOLEN_MSS, GFP_ATOMIC);
   
10.                 if (!newskb) {
    
11.                         if (net_ratelimit())
    
12.                                 printk(KERN_ERR "ipt_tcpmss_target:"
    
13.                                        " unable to allocate larger skb\n");
    
14.                         return NF_DROP;
    
15.                 }
    
16. 
    
17.                 kfree_skb(*pskb);
    
18.                 *pskb = newskb;
    
19.                 iph = (*pskb)->nh.iph;
    
20.                 tcph = (void *)iph + iph->ihl*4;
    
21.         }
    
22. 
    
23.         skb_put((*pskb), TCPOLEN_MSS);
    

复制代码

[ 本帖最后由 platinum 于 2008-9-4 19:20 编辑 ]

Godbach

我的是2.4.22内核，ipt_tcpmss.c没有这个函数啊

Godbach

2.6中把TCPMSS作为Target，2.4中把这个作为Match，呵呵，变化真大啊

platinum

原帖由 Godbach 于 2008-9-4 19:26 发表
2.6中把TCPMSS作为Target，2.4中把这个作为Match，呵呵，变化真大啊

小写的是 match，大写的是 target
我这里有，给你