请教一个spin_lock问题

amarant

回复 10# azzurris


    也就是说，忙等待一定不能被中断，如果这么写就是程序员的错误是吗。。但是我觉得就如我前面说的，自旋锁就算被中断打断也没有什么严重后果吧。。

azzurris

回复 9# amarant


    我的意思是非SMP环境下且非中断上下文，这个时候你要用spin_lock也可以，但是之前要关中断，这么一来简单点就直接spin_lock_irqsave就ok了。但是您刚才提到的例子里他还要去操作rcu锁，而在这个rcu操作内已经关了中断了，所以后面的自旋锁就不用关本地中断了。

简单的列举一下我的总结，格式比较乱：

                            UP单处理器                                                                                         SMP多处理器

中断上下文             不用自旋锁                                                                                            spin_lock
进程上下文            local_irq_save或spin_lock_irqsave（内核抢占开启的情况）                         spin_lock_irqsave

azzurris

回复 11# amarant


    自旋锁被中断打断，就有可能出现两种情况：
1.ISR中未使用自旋锁获取操作受保护数据的权限，那么受保护数据就乱掉了
2.ISR是使用了自旋锁来获取操作受保护数据的权限，那么因为这个权限还在你被中断的自旋锁那里hold着，所以这里就会死锁

amarant

回复 13# azzurris


    我是说如果ISR中不会对保护的数据访问（也不会尝试去获得锁）就如LZ说的情况，那么在进程上下文不关中断直接spin_lock不会有什么问题吧。

（刚睡醒，小憩了下）

azzurris

我个人感觉是不会有问题。
但这玩意防不胜防啊，你现在是知道ISR不会动到受保护的数据，6个月后修修改改，谁还记得这里不能动那个数据啊。直接用spin_lock_irqsave，简单方便不会给后续代码留漏洞。

amarant

呵呵，受教了！大侠以后多来cu逛逛阿。

kouu

这个函数在kernel/rcupreempt.c里面实现为__rcu_read_lock，在该函数内会去关本地中断                local_irq_save(flags);
azzurris 发表于 2011-04-27 12:00

__rcu_read_lock没有这么暴力吧，为什么需要关本地中断呢？ 貌似是这样的吧：

1. 
   
2.         local_irq_save(flags);
   
3.         /* some update */
   
4.         local_irq_restore(flags);
   

复制代码

azzurris

回复 17# kouu


    您贴的代码在这里的含义是。。。

amarant

我这里代码有点不一样
__exit_signal:
        sighand = rcu_dereference(tsk->sighand);
        spin_lock(&sighand->siglock);

但在之前确实关了中断：
release_task:
        write_lock_irq(&tasklist_lock);
        tracehook_finish_release_task(p);
        __exit_signal(p);

azzurris

我了解了，kouu的意思是关了中断但是最后又打开了中断，所以函数结束后，仍然中断是打开着的。
是我看错了。
我的理解是进程上下文用到spin_lock之前要关本地中断，但amarant举的代码例子里没关，这我现在无法解释了。各位大神发表下意见吧。