sk_buff的修改问题

duoniK

想在struct sk_buff中添加一个字段，如下
struct sk_buff{
        ....
        __u16                mac_len,
                                hdr_len,
                                mytcp_len; //mytcp首部
        ....
        sk_buff_data_t                network_header;
        sk_buff_data_t                mac_header;
        sk_buff_data_t      mytcp_header; //指向mytcp头部
        ....
};

结构体中添加了2个字段 mytcp_len，mytcp_header。 不知道为什么编译能通过，lilo加载内核时就出错


[<c188569f>] skb_release_data+0x8f/0xa0
....

想问下是内存空间分配不足的原因，还是 未初始化呢？
觉得在 alloc_skb中这些内容是不用关心的
还请大神帮忙

瀚海书香

回复 1# duoniK
最好不要修改sk_buff结构体，使用skb->cb那48个字节吧。

   

duoniK

cb[]是字符型的组数。 主要是想添加一个像mac_header这样指向一个头的指针字段。回复 2# 瀚海书香


   

瀚海书香

回复 3# duoniK

cb[]是字符型的组数。 主要是想添加一个像mac_header这样指向一个头的指针字段。

你可以用cb[]中的前四个字节（x86_32)表示一个指针啊。

另外，如果你非要想修改sk_buff的话也可以。你可以参考当年添加secmark的那个patch。

git log 984bc16c -p
commit 984bc16cc92ea3c247bf34ad667cfb95331b9d3c
Author: James Morris <jmorris@namei.org>
Date:   Fri Jun 9 00:29:17 2006 -0700

    [SECMARK]: Add secmark support to core networking.
   
    Add a secmark field to the skbuff structure, to allow security subsystems to
    place security markings on network packets.  This is similar to the nfmark
    field, except is intended for implementing security policy, rather than than
    networking policy.
   
    This patch was already acked in principle by Dave Miller.
   
    Signed-off-by: James Morris <jmorris@namei.org>
    Signed-off-by: Andrew Morton <akpm@osdl.org>
    Signed-off-by: David S. Miller <davem@davemloft.net>

diff --git a/include/linux/skbuff.h b/include/linux/skbuff.h
index 23bad3b..fe2c58e 100644
--- a/include/linux/skbuff.h
+++ b/include/linux/skbuff.h
@@ -210,6 +210,7 @@ enum {
  *     @nf_bridge: Saved data about a bridged frame - see br_netfilter.c
  *     @tc_index: Traffic control index
  *     @tc_verd: traffic control verdict
+ *     @secmark: security marking
  */

struct sk_buff {
@@ -289,6 +290,9 @@ struct sk_buff {
#ifdef CONFIG_NET_DMA
        dma_cookie_t            dma_cookie;
#endif
+#ifdef CONFIG_NETWORK_SECMARK
+       __u32                   secmark;
+#endif

   

duoniK

不是必须改。只是想在数据段前，tcp头部后添加一个额外的头--mytcp头，所以想向操作tcp头，ip头那样来操作这些字段
回复 4# 瀚海书香


   

humjb_1983

瀚海书香 发表于 2013-12-20 11:04
回复 3# duoniK

你可以用cb[]中的前四个字节（x86_32)表示一个指针啊。

这个补丁的修改方法好像跟直接改的效果差不多呢~~，除了增加宏控制，好像没有特别的地方呢？

瀚海书香

回复 6# humjb_1983
那就照着改呗。如果不能启动的话，应该是代码写的问题，而不是实现方式问题

   

humjb_1983

瀚海书香 发表于 2013-12-20 13:12
回复 6# humjb_1983
那就照着改呗。如果不能启动的话，应该是代码写的问题，而不是实现方式问题

呵呵，原来是这个意思，感觉也应该是其他原因导致的。