请教timer_list中data域初始化为地址的用法

RUNONMIPS

使用环境大体是这样的：

static long do_ioctrl(struct file *file, unsigned int cmd, void *arg) {
struct A *handle = (struct uvc_fh *)file->private_data;
struct B *stream = handle->stream;
switch(cmd) {
  ...
  STREAMON:
   struct timer_list cap_timer = TIMER_INITIALIZER(trigger,0,(unsigned long)stream);
   ....
   mod_timer(&cap_timer, jiffies +300);
   break;
}
}
在另外一个文件中定义了trigger
void trigger(unsigned long val) {
printk(KERN_INFO "fuck!");
}
我原本只是想利用定时器做一个触发，简单写了个模板，上层程序也会调到这个ioctrl。可是当程序跑起来以后，定时器触发时就死掉了。但如果将(unsigned long)stream换成一个具体的数字（如(unsigned long)stream的地址）就可以打印出来，一切都正常。我看了定时器的源码，也只是把data域传入到（fn）的回调函数中，所以对这个现象很不解。有没有用过这种将指针作为回调参数的方法的兄弟啊，请大家指教。。

jetking

你能给出打印正常时的代码吗？

RUNONMIPS

回复 2# jetking


    只要把(unsigned long)stream换成一个常数就可以正常打印了。

jetking

回复 3# RUNONMIPS

stream的值就是一个常数阿，表示一个地址值。除非trigger里引用了stream，而stream所指向的内存区被销毁了。
其他的看不出来，帮你顶一下！   

RUNONMIPS

回复 4# jetking


    我做了个实验，在trigger函数中只把stream参数传进去，根本没有使用，但还是死掉了。所以我很补了解，所以我觉得timer_list中的data域是不是不能用变量的方法来初始化。

ant艾草

这个timer_list分配在栈里面，退出这个switch之后，timer_list就不安全了。
不知道和这个有没有关系？

fly6

应该如楼上所说

RUNONMIPS

ant艾草 发表于 2012-01-17 15:47
这个timer_list分配在栈里面，退出这个switch之后，timer_list就不安全了。
不知道和这个有没有关系？

应该不是吧，mod_timer之后定时器就可以工作了，即使推出switch也没关系吧。倒是这个stream指针有这个嫌疑，再试试