softirq&timer中还用得着bh版本spin_lock_bh吗？

allenlu1212

spin_lock_bh通常用在进程中，用来禁止抢断和禁止软中断。
而软中断本身不可嵌套，所以，在软中断中就不需要bh版本的spin_lock_bh。
Linux的timer是通过软中断TIMER_SOFTIRQ实现的，所以timer的超时处理函数应该也不需要bh版本的spin_lock_bh。

但kernel中br_fdb.c中的br_fdb_cleanup()为什么用spin_lock_bh？

void br_fdb_cleanup(unsigned long _data)
{
struct net_bridge *br = (struct net_bridge *)_data;
unsigned long delay = hold_time(br);
int i;

spin_lock_bh(&br->hash_lock);
for (i = 0; i < BR_HASH_SIZE; i++) {
struct net_bridge_fdb_entry *f;
struct hlist_node *h, *n;

hlist_for_each_entry_safe(f, h, n, &br->hash, hlist) {
if (!f->is_static &&
    time_before_eq(f->ageing_timer + delay, jiffies))
fdb_delete(f);
}
}
spin_unlock_bh(&br->hash_lock);

mod_timer(&br->gc_timer, jiffies + HZ/10);
}

恳请大家来讨论下。

scutan

相同的软中断不会嵌套, 但是, 不同的软中断可以嵌套啊, 而且相同的软中断也可以运行在不同的CPU上

allenlu1212

第一： 同一个cpu上所有的软中断是不会嵌套的，参见do_softirq()
asmlinkage void do_softirq(void)
{
        __u32 pending;
        unsigned long flags;

        if (in_interrupt())  //这里决定了，软中断在一个cpu上不会嵌套
                return;

        local_irq_save(flags);

        pending = local_softirq_pending();

        if (pending)
                __do_softirq();

        local_irq_restore(flags);
}

第二： 禁止软中断只能禁止本地的，即本cpu的软中断。

scutan

谢谢指正，我搞错了，软中断在一个CPU上是串行的。

scutan

原帖由 allenlu1212 于 2008-12-11 22:34 发表
第二： 禁止软中断只能禁止本地的，即本cpu的软中断。

这句话能给一下出处吗？
我没有找到。
LKD上面说的是：
The function spin_lock_bh() obtains the given lock and disables all bottom halves.
没有提到是本CPU的。

thanks.

allenlu1212

1.  参看内核：
    spin_lock_bh()->
        local_bh_disable()->....
              add_preempt_count(SOFTIRQ_OFFSET)->
                           preempt_count() += val;

   而preempt_count()定义为：
   #define preempt_count()        (current_thread_info()->preempt_count) //注意这里

2. spin_lock_bh中的"spin"是全局的，"bh"是本地的。

3. 禁止软中断和禁止硬中断都是指本地。

baohuaihuai

不需要

allenlu1212

但为什么br_fdb_cleanup()用spin_lock_bh，相信linux的code不是随便写的。

vn42

lz什么版本的kernel？bh2.6已经没有了吧

allenlu1212

2.6.26