insmod驱动模块时 内核会自动重启 提示：kernel BUG at kernel/timer.c:662!

牡丹岩

# insmod inputk2_drv.ko    Linux内核2.6.35.7 加载驱动模块时直接导致系统自动重启，请教为何？

有高手请速回！小弟急死了。。。
input: gec_input as /devices/virtual/input/input1
kernel BUG at kernel/timer.c:662!
[ 1051.330588] Unable to handle kernel NULL pointer dereference at virtual address 00000000

amarant

这是kernel通过BUG()打印出来的，你看看源代码。也可对比主线上的代码，可能问题已经修复了

牡丹岩

这是我的bug代码，不是每次insmod都会出错的，问题真心难找，麻烦各位大仙看看！谢过~

1. [size=6][size=5][code]
   
2. #include <linux/module.h>
   
3. #include <linux/init.h>
   
4. #include <linux/input.h>
   
5. #include <linux/interrupt.h>
   
6. #include <asm/gpio.h>
   
7. #include <linux/timer.h>
   
8. 
   
9. 
   
10. static struct input_dev *p_button_dev = NULL;//定义输入设备
    
11. 
    
12. struct timer_list my_timer;//定义定时器
    
13. 
    
14. 
    
15. //定义定时器超时处理函数
    
16. void timer_func(unsigned long data)
    
17. {
    
18.         int key_value = gpio_get_value(S5PV210_GPH2(0));
    
19. 
    
20.         //上报事件给input核心层
    
21.         input_report_key(p_button_dev, KEY_A, !key_value);//按下为1,释放为0
    
22. 
    
23.              //告诉input子系统上报已经完成
    
24.         input_sync(p_button_dev);
    
25. }
    
26. 
    
27. //中断处理函数
    
28. static irqreturn_t button_interrupt(int irq, void *dev_id)
    
29. {
    
30.              mod_timer(&my_timer, jiffies + 5);//启动定时器以及设置超时时间
    
31.    
    
32.         return IRQ_HANDLED;
    
33. }
    
34. 
    
35. //初始化按钮
    
36. static int __init button_init(void)
    
37. {
    
38.         int ret;
    
39.         ret = gpio_request(S5PV210_GPH2(0), "key2");
    
40.         if (ret)
    
41.              {
    
42.                 printk(KERN_ERR "gpio_request Failed to register device\r\n");
    
43. 
    
44.                           goto error1;
    
45.         }
    
46.    
    
47.              //为新输入设备分配内存并初始化
    
48.         p_button_dev = input_allocate_device();
    
49.         if (!p_button_dev)
    
50.              {
    
51.                 printk(KERN_ERR "can't allocate input mem!\r\n");
    
52. 
    
53.                          goto error2;
    
54.         }
    
55.    
    
56.         p_button_dev->name = "gec_input";
    
57.         p_button_dev->id.bustype = 0x1;
    
58.         p_button_dev->id.product = 0x2;
    
59.         p_button_dev->id.vendor  = 0x3;
    
60.         p_button_dev->id.version = 0x4;
    
61.         p_button_dev->evbit[BIT_WORD(EV_KEY)] = BIT_MASK(EV_KEY);       
    
62.              p_button_dev->keybit[BIT_WORD(KEY_A)] = BIT_MASK(KEY_A);   
    
63.              //注册一个输入设备
    
64.         ret = input_register_device(p_button_dev);
    
65.         if (ret)
    
66.              {
    
67.                 printk(KERN_ERR "Failed to register device\r\n");
    
68. 
    
69.                           goto error3;
    
70.         }
    
71.    
    
72.                //申请中断注册中断处理函数
    
73.         ret = request_irq(IRQ_EINT(16), button_interrupt,
    
74.                                  IRQF_TRIGGER_FALLING | IRQF_TRIGGER_RISING | IRQF_DISABLED,
    
75.                                       "button", NULL);
    
76.         if (ret)
    
77.         {
    
78.                 printk(KERN_ERR "Can't request irq %d\r\n", IRQ_EINT(16));
    
79. 
    
80.                           goto error4;
    
81.         }   
    
82.    
    
83.         //定时器
    
84.         init_timer(&my_timer);//初始化定时器
    
85.         my_timer.function = timer_func;//注册定时器超时处理函数
    
86. 
    
87.          return 0;
    
88.    
    
89. 
    
90. error4:
    
91.             free_irq(IRQ_EINT(16), NULL);//释放分配给已定中断的内存
    
92.          input_unregister_device(p_button_dev);
    
93.    
    
94. error3:
    
95.             input_free_device(p_button_dev);
    
96. 
    
97. error2:
    
98.            ret = -ENOMEM;
    
99. 
    
100. error1:
     
101.            gpio_free(S5PV210_GPH2(0));
     
102.    
     
103.            return ret;
     
104. }
     
105. 
     
106. static void __exit button_exit(void)
     
107. {
     
108.         gpio_free(S5PV210_GPH2(0));       
     
109.         free_irq(IRQ_EINT(16), NULL);
     
110.         input_unregister_device(p_button_dev);
     
111.         del_timer(&my_timer);//删除内核定时器
     
112. }
     
113. 
     
114. module_init(button_init);
     
115. module_exit(button_exit);
     
116. 
     
117. MODULE_LICENSE("Dual BSD/GPL");
     
118. MODULE_LICENSE("GPL");
     
119. 
     

复制代码

[/code]

zszkismet

应该先初始化定时器，然后再注册中断处理函数
如果注册完中断，中断就来了，这个时候定时器还没初始化，就会出错

牡丹岩

zszkismet 发表于 2014-08-28 13:54
应该先初始化定时器，然后再注册中断处理函数
如果注册完中断，中断就来了，这个时候定时器还没初始化，就 ...

我的中断是通过按按键产生的，不太可能出现你说的情况哦。
insmod 导致自动重启的情况 也不是每次都会出现，这是最蛋碎的地方，
等测试确定了再交流！
谢了！

madang

回复 4# zszkismet


    说的有道理，或者终端注册完成以后，先把irq disable掉，等timer初始化以后再enable irq。