写一个块设备驱动 insmod后机器死了（完全控制不了）

5doumi

本人最近想学习linux驱动程序的编写，网上正好有一篇与此相关的精彩文章，有zhaolei前辈写的《写一个块设备驱动
》（原帖地址http://bbs.chinaunix.net/thread-2017377-1-1.html）。

按照上述第一章的内容，编写了下述程序。原文章中使用的一些结构和函数比较老，在新内核中有所变化（sectors调用blk_rq_pos得到， current_nr_sectors调用blk_rq_sectors得到， elv_next_request换成blk_peek_request， end_request换成blk_end_request_cur）。
可是我make完了之后，insmod的时候悲剧了，机器直接卡那不受控制了。我baidu了下，网上有的说是在处理request的时候加锁了，但blk_end_request_cur时没正确释放。（参考帖地址http://bbs.chinaunix.net/viewthr ... p;extra=&page=2）
我参照了网上的修改意见修改了我的程序，把blk_end_request_cur改成了__blk_end_request_cur , blk_end_request_all, __blk_end_request_all,都未能解决问题，还是一样的，insmod之后就死在那了，悲催了。。。。

有哪位高人能指出其中错误之处阿！

下面是我的代码

1. #include <linux/module.h>
   
2. #include <linux/slab.h>
   
3. #include <linux/kernel.h>
   
4. #include <linux/list.h>
   
5. #include <linux/fs.h>
   
6. #include <linux/blkdev.h>
   
7. #include <linux/blkpg.h>
   
8. #include <linux/elevator.h>
   
9. 
   
10. 
    
11. #define SIMP_BLKDEV_DEVICEMAJOR         COMPAQ_SMART2_MAJOR
    
12. #define SIMP_BLKDEV_DISKNAME                 "simp_blkdev"
    
13. #define SIMP_BLKDEV_BYTES                (16*1024*1024)
    
14. 
    
15. 
    
16. unsigned char simp_blkdev_data[SIMP_BLKDEV_BYTES];
    
17. 
    
18. static struct gendisk *simp_blkdev_disk;
    
19. 
    
20. struct block_device_operations simp_blkdev_ops = {
    
21.         .owner        = THIS_MODULE
    
22. };
    
23. 
    
24. static struct request_queue *simp_blkdev_queue;
    
25. 
    
26. static void simp_blkdev_do_request(struct request_queue *q);
    
27. 
    
28. static int __init simp_blk_init(void)
    
29. {
    
30.         int ret;
    
31. 
    
32.         simp_blkdev_queue = blk_init_queue(simp_blkdev_do_request, NULL);
    
33.         if(!simp_blkdev_queue)
    
34.         {
    
35.                 ret = -ENOMEM;
    
36.                 goto err_init_queue;
    
37.         }       
    
38. 
    
39.         simp_blkdev_disk = alloc_disk(1);
    
40.         if(!simp_blkdev_disk)
    
41.         {
    
42.                 ret = -ENOMEM;
    
43.                 goto err_alloc_disk;
    
44.         }
    
45.        
    
46.         strcpy(simp_blkdev_disk->disk_name, SIMP_BLKDEV_DISKNAME);
    
47.         simp_blkdev_disk->major = SIMP_BLKDEV_DEVICEMAJOR;
    
48.         simp_blkdev_disk->first_minor = 0;
    
49.         simp_blkdev_disk->fops = &simp_blkdev_ops;
    
50.         simp_blkdev_disk->queue = simp_blkdev_queue;
    
51.         set_capacity(simp_blkdev_disk, SIMP_BLKDEV_BYTES>>9);       
    
52. 
    
53.         add_disk(simp_blkdev_disk);
    
54.         return 0;
    
55. 
    
56.         err_alloc_disk:
    
57.                 blk_cleanup_queue(simp_blkdev_queue);
    
58.                 return ret;
    
59. 
    
60.         err_init_queue:
    
61.                 return ret;
    
62. }
    
63. 
    
64. static void __exit simp_blk_exit(void)
    
65. {
    
66.         del_gendisk(simp_blkdev_disk);
    
67.         put_disk(simp_blkdev_disk);
    
68.         blk_cleanup_queue(simp_blkdev_queue);
    
69. }
    
70. 
    
71. static void simp_blkdev_do_request(struct request_queue *q)
    
72. {
    
73.         struct request *req;
    
74.         while( (req = blk_fetch_request(q)) != NULL )
    
75.         {
    
76.         if( (blk_rq_pos(req) + blk_rq_sectors(req))<<9
    
77.             > SIMP_BLKDEV_BYTES         )
    
78.                 {
    
79.                 printk(KERN_ERR SIMP_BLKDEV_DISKNAME ": bad request: block = %llu, count = %u\n", (unsigned long long)(blk_rq_pos(req)),blk_rq_sectors(req));       
    
80.                 __blk_end_request_all(req, 0);
    
81.                 continue;
    
82.                 }       
    
83.        
    
84.         switch(rq_data_dir(req))
    
85.         {
    
86.         case READ:
    
87.                 memcpy(req->buffer, simp_blkdev_data + ( blk_rq_pos(req) << 9) , blk_rq_sectors(req)<<9);
    
88.                 __blk_end_request_all(req, 1);
    
89.                 break;
    
90. 
    
91.         case WRITE:
    
92.                 memcpy(simp_blkdev_data + (blk_rq_pos(req)<< 9), req->buffer, blk_rq_sectors(req)<<9);       
    
93.                 __blk_end_request_all(req , 1);
    
94.                 break;
    
95. 
    
96.         default:
    
97.                 /*No default, because the return value is 1 bit*/
    
98.                 break;
    
99.         }
    
100.        
     
101.         }
     
102. 
     
103. 
     
104. }
     
105. 
     
106. module_init(simp_blk_init);
     
107. module_exit(simp_blk_exit);

复制代码

sting99999

你把所有__blk_end_request_all(req , 1);改为 __blk_end_request_cur(req , 0);注意参数也要改为0  。。试试。。。有问题qq联系，1193533825

鬼鬼一哈

你的内核多新，blk_rq_pos(req) + blk_rq_sectors(req)换为req->sector + req->current_nr_sectors。
在2.6.27上测试通过。。insmod无异常

insmod死机，kdum不好么，定位一下