关于驱动程序中llseek方法是否需要加互斥锁的疑问

wawxdyy

书上例程如下：

1. 
   
2. loff_t scull_llseek(struct file *filp, loff_t off, int whence)
   
3. {
   
4.         struct scull_dev *dev = filp->private_data;
   
5.         loff_t newpos;
   
6. 
   
7.         switch(whence) {
   
8.           case 0: /* SEEK_SET */
   
9.                 newpos = off;
   
10.                 break;
    
11. 
    
12.           case 1: /* SEEK_CUR */
    
13.                 newpos = filp->f_pos + off;
    
14.                 break;
    
15. 
    
16.           case 2: /* SEEK_END */
    
17.                 newpos = dev->size + off;
    
18.                 break;
    
19. 
    
20.           default: /* can't happen */
    
21.                 return -EINVAL;
    
22.         }
    
23.         if (newpos < 0) return -EINVAL;
    
24.         filp->f_pos = newpos;
    
25.         return newpos;
    
26. }
    

复制代码

可是我觉得应该加上互斥锁，各位觉得如何？

realtang

Post by wawxdyy
书上例程如下：

1. 
   
2. loff_t scull_llseek(struct file *filp, loff_t off, int whence)
   
3. {
   
4.         struct scull_dev *dev = filp->private_data;
   
5.         loff_t newpos;
   
6. 
   
7.         switch(whence) {
   
8.           case 0: /* SEEK_SET */
   
9.                 newpos = off;
   
10.                 break;
    
11. 
    
12.           case 1: /* SEEK_CUR */
    
13.                 newpos = filp->f_pos + off;
    
14.                 break;
    
15. 
    
16.           case 2: /* SEEK_END */
    
17.                 newpos = dev->size + off;
    
18.                 break;
    
19. 
    
20.           default: /* can't happen */
    
21.                 return -EINVAL;
    
22.         }
    
23.         if (newpos < 0) return -EINVAL;
    
24.         filp->f_pos = newpos;
    
25.         return newpos;
    
26. }
    

复制代码

可是我觉得应该加上互斥锁，各位觉得如何？

内核会保证该函数执行的串行化。
光看驱动没用，建议你先熟悉一下内核流程。