Question

我的内核驱动程序中有一个ioctl，读取时需要返回一些数据。当我在用户空间中读回它时，它没有显示预期的结果。

用户空间代码段：

typedef struct abc_T {
    int size;
    int addr;
    int data[32];
} abc_T;

// Read
int abc_read(int addr, int size, int *data) {
     abc_T abc = {};
     int fd, retval;
     abc.size = size;
     abc.addr = addr;
     fd = open("/dev/"ABC_DEV, O_RDWR);

     if (fd >=0) {
     retval = ioctl(fd, READ_ABC, &abc);
     if (retval == 0)
         memcpy(data, abc.data, size);

     }
return retval;
}

内核：

static int ABC_ioctl(struct file * file, uint cmd, ulong arg)
{
ABC_T abc;
void __user * arg_ptr;
arg_ptr = (void *)arg;
int retval;

if (!access_ok(VERIFY_READ, arg_ptr, sizeof(ABC_T))) {

    return -EACCES;
}

if (copy_from_user(&abc, arg_ptr,
                        sizeof(ABC_T)) != 0) {
    return -EFAULT;
}

switch(cmd) {
case READ_ABC:
retval = read_func(&abc);
if (retval == 0) {
    if (copy_to_user((void __user *)arg, &abc,
                     sizeof(ABC_T) != 0)) {
                retval = -EFAULT;
    } else {
        retval = 0;
    }
}
return retval;
}

我已经在read_func中打印了结果（即数据数组），并且可以看到读取是期望值，但是在copy_to_user之后，当我在用户空间中打印data时，我看到了所有零。该代码段有问题吗？

Answer 1

您的亲戚不在打copy_to_user的电话上。我认为，您的意思是测试copy_to_user的响应是否为!= 0。相反，您将sizeof(ABC_T) != 0作为最后一个参数。由于sizeof(ABC_T)不为零，因此您的通话最终为copy_to_user((void __user *)arg, &abc, true)。

修正您的括号，看看您是否获得更好的结果：

if (copy_to_user((void __user *)arg, &abc, sizeof(ABC_T)) != 0) {
    // ...
}

copy_to_user不返回预期数据

1 个答案: