我注意到从linux 2.6.37开始,lock_kernel()似乎已被弃用.我读了“ioctl()的新方法”,发现设备驱动程序现在应该在操作之前执行特定的锁定.

所以如果可能的话,我想要一些建议.

我在块文件夹部分找到了当前代码中我认为相关的两个部分.

Source 
      block->io.c
           ->ctrldev.c

我把每个片段都放在你的考虑范围内.

io.c包含对lock_kernel的一次调用：

NDAS_SAL_API xbool     sal_file_get_size(sal_file file,xuint64* size)
{
    definitions and declarations etc..

lock_kernel();

#ifdef HAVE_UNLOCKED_IOCTL
    if (filp->f_op->unlocked_ioctl) {   
       some small statements

       error = filp->f_op->unlocked_ioctl(filp,BLKGETSIZE64,(unsigned long)size);

       actions if error or not etc.
   }
#endif

   unlock_kernel(); 
   return ret;
}

并且ctrldev.c包含主要的io函数：

#include <linux/spinlock.h> // spinklock_t
#include <linux/semaphore.h> // struct semaphore
#include <asm/atomic.h> // atomic
#include <linux/interrupt.h>
#include <linux/fs.h>
#include <asm/uaccess.h>
#include <linux/ide.h>
#include <linux/smp_lock.h>
#include <linux/time.h>

......

int ndas_ctrldev_ioctl(struct inode *inode,struct file *filp,unsigned int cmd,unsigned long arg)
{
  lots of operations and functions. 

  return result;
}

后来ndas_ctrldev_ioctl函数被设置为前者.ioctl.

static struct file_operations ndasctrl_fops = {
    .write = ndas_ctrldev_write,.read = ndas_ctrldev_read,.open = ndas_ctrldev_open,.release = ndas_ctrldev_release,.ioctl = ndas_ctrldev_ioctl,};

现在我想转换它以避免使用lock_kernel();

根据我的理解,我将修改前面的部分如下：

NDAS_SAL_API xbool     sal_file_get_size(sal_file file,xuint64* size)
{
    definitions and declarations etc..

#ifndef HAVE_UNLOCKED_IOCTL
    lock_kernel();
#endif

#ifdef HAVE_UNLOCKED_IOCTL
    if (filp->f_op->unlocked_ioctl) {   
       some small statements

       error = filp->f_op->unlocked_ioctl(filp,(unsigned long)size);

       actions if error or not etc.
   }
#endif

#ifndef HAVE_UNLOCKED_IOCTL
   unlock_kernel(); 
#endif
   return ret;

}

#ifdef HAVE_UNLOCKED_IOCTL
long ndas_ctrldev_ioctl(struct file *filp,unsigned long arg)
#else
int ndas_ctrldev_ioctl(struct inode *inode,unsigned long arg)
#endif
{

#ifdef HAVE_UNLOCKED_IOCTL
  ! add some sort of lock here !
#endif

  lots of operations and functions. 

#ifdef HAVE_UNLOCKED_IOCTL
  ! add unlock statement here  !
#endif
  return result;
}

static struct file_operations ndasctrl_fops = {
    .write = ndas_ctrldev_write,#ifdef HAVE_UNLOCKED_IOCTL
    .unlocked_ioctl = ndas_ctrldev_ioctl,#else
    .ioctl = ndas_ctrldev_ioctl,#endif
};

所以,我会问以下建议.

>这看起来是正确的吗？
proceedure？
>我理解是否正确移动
锁定io功能？
>根据crtrldev.c中的包含,你能吗？
推荐顶部的任何锁定
你的头？ (我试着研究一下
其他司机处理filp和
lock_kernel,但我太过分了
菜鸟马上找到答案.)