如何使用避免内核定时器函数调用可休眠函数引发的系统崩溃问题？

song0071000

1. stm_enable()
   
2. {
   
3.         init_timer(timer);
   
4.         timer->data = (unsigned long) data;
   
5.         timer->function = timer_function;
   
6.         timer->expires = jiffies + 1*HZ;
   
7.         add_timer(timer);
   
8. }
   
9. 
   
10. /*省略其他相关细节*/
    
11. timer_function()
    
12. {
    
13.         stm_one_step();//一步状态机操作，该操作函数会调用到休眠函数。。。
    
14. 
    
15.         timer.expires = jiffies + 1*HZ;//每隔一秒执行一步状态机
    
16.         add_timer(&timer);
    
17. }

复制代码

在google查到说可以使用workqueue来避免这种情况.....
但是具体该如何操作呢？

把stm_one_step放在哪里执行？
哪位大侠给出一个大致框架也好啊！！！
我已试过将timer_function()或stm_one_step改为工作队列的执行函数，都会出错。。。。。。

asuka2001

回复 1# song0071000

1. static DECLARE_WORK(my_work, stm_one_step);
   
2. 
   
3. timer_function()
   
4. {
   
5.         schedule_work(&my_work);
   
6.         ......
   
7. }

复制代码

另外最好使用 mod_timer()而不是 add_timer()


   

song0071000

  感谢回复。。该方法亲测还真可行！！！
不过想不明白，为什么我之前使用自己创建的workqueue来queue_work添加任务就不行呢，还引发系统崩溃？
而使用系统共享队列就没这个问题。求赐教！

asuka2001 发表于 2013-11-08 17:48
回复 1# song0071000 另外最好使用 mod_timer()而不是 add_timer()

asuka2001

回复 3# song0071000

。。。不明白你怎么做的，请把源码贴下！
   

studying_linux

song0071000 发表于 2013-11-08 17:29
在google查到说可以使用workqueue来避免这种情况.....
但是具体该如何操作呢？

看了一下timer的实现，是在bottom havles上实现的，所以不能再timer的callback里运行睡眠相关的函数吧。

tick_periodic --》 update_process_times --》 run_local_timers ：
raise_softirq(TIMER_SOFTIRQ);

tick的handler里raise了TIMER_SOFTIRQ,所以在top havels退出的时候会去运行对应的下半部：run_timer_softirq
所以你的callback函数是在下半部的context上运行的，即在中断上下文，所以不能睡眠。

studying_linux

asuka2001 发表于 2013-11-08 17:48
回复 1# song0071000 另外最好使用 mod_timer()而不是 add_timer()

大侠能解释一下为什么这时应该使用 mod_timer()而不是 add_timer() 吗？

asuka2001

回复 6# studying_linux

直接摘录自 LKD3

Timer Race Conditions
Because timers run asynchronously with respect to the currently executing code, several
potential race conditions exist. First, never do the following as a substitute for a mere
mod_timer(), because this is unsafe on multiprocessing machines:
del_timer(my_timer)
my_timer->expires = jiffies + new_delay;
add_timer(my_timer);

asuka2001

回复 7# asuka2001

现在 add_timer已经是使用 mod_timer作为实现了，不过如果 timer已经激活那会报告 BUG。

void add_timer(struct timer_list *timer)
{
         BUG_ON(timer_pending(timer));
         mod_timer(timer, timer->expires);
}
EXPORT_SYMBOL(add_timer);

studying_linux

回复 7# asuka2001


Thanks. 我再细读一下这章