为什么在ip报文重组的时候要关闭中断？

wangjun0629

2.6.28  nf_defrag_ipv4.c

/* Returns new sk_buff, or NULL */
static int nf_ct_ipv4_gather_frags(struct sk_buff *skb, u_int32_t user)
{
        int err;

        skb_orphan(skb);

        local_bh_disable();
        err = ip_defrag(skb, user);
        local_bh_enable();

        if (!err)
                ip_send_check(ip_hdr(skb));

        return err;
}

这段代码本来就运行在软中断上下文，这里调用local_bh_disable()没看明白，难道有定时器和相关资源互斥？

瀚海书香

回复 1# wangjun0629

2.6.28  nf_defrag_ipv4.c

/* Returns new sk_buff, or NULL */
static int nf_ct_ipv4_gather_frags(struct sk_buff *skb, u_int32_t user)
{
        int err;

        skb_orphan(skb);

        local_bh_disable();
        err = ip_defrag(skb, user);
        local_bh_enable();

        if (!err)
                ip_send_check(ip_hdr(skb));

        return err;
}

这段代码本来就运行在软中断上下文，这里调用local_bh_disable()没看明白，难道有定时器和相关资源互斥？

有可能运行在ksoftirqd的进程上下文中，所以需要local_bh_disable()

   

wangjun0629

哦,好像你说的是对的,但是如果是这样的话,网络协议栈里面的自旋锁,那不是都应该是 spin_lock_bh() , 但是
我看见链接跟踪的锁只是spin_lock(),没有关闭bh.

瀚海书香

回复 3# wangjun0629

哦,好像你说的是对的,但是如果是这样的话,网络协议栈里面的自旋锁,那不是都应该是 spin_lock_bh() , 但是
我看见链接跟踪的锁只是spin_lock(),没有关闭bh.

spin_lock和spin_lock_bh的区别是是否禁止软中断。

你所说的情况下，所包含的代码极可能永远运行在进程上下文中。比如发送数据包的系统调用、添加协议处理代码等等

   

smalloc

回复 2# 瀚海书香


    即使是ksoftirqd中软中断函数也应是运行在关闭local软中断下运行的。因为一个软中断函数如果要同时考虑2种情形实在太费事了。
asmlinkage void __do_softirq(void)
{
        struct softirq_action *h;
        __u32 pending;
        int max_restart = MAX_SOFTIRQ_RESTART;
        int cpu;

        pending = local_softirq_pending();

        local_bh_disable();

wangjun0629

回复 4# 瀚海书香
不好意思，后面翻了一下内核的实现，发现内核中都是用的spin_lock_bh(),没有我说的情况。

但有一事不明。

如果资源在netfilter的框架中运行，正常情况下属于软中断。但如果忙的时候，开了softirqd时，处理报文的逻辑就运行在了
进程上下文。 这时就需要考虑软中断和内核线程的互斥。 那岂不是在netfilter框架中，随意加个链表，岂不是每次访问都要加 local_disable_bh().

   

wangjun0629

回复 5# smalloc

按照你的讲法，那这段代码就不应该调用  local_bh_disable();这段代码只可能运行在软中断和内核线程。

如果在内核线程,按照你的说法，调用前已经关闭了。


2.6.28  nf_defrag_ipv4.c

/* Returns new sk_buff, or NULL */
static int nf_ct_ipv4_gather_frags(struct sk_buff *skb, u_int32_t user)
{
        int err;

        skb_orphan(skb);

        local_bh_disable();
        err = ip_defrag(skb, user);
        local_bh_enable();

        if (!err)
                ip_send_check(ip_hdr(skb));

        return err;
}



   

smalloc

回复 7# wangjun0629


    瞎猜可能是段老代码。BH定义的机制不相同。

wangjun0629

求高手回答问题啊~~~~~~