应用程序通过iotcl往驱动传参数，居然收到是错的。。。。。

zheguzai

今天遇到这个问题，简直让我很吐血，搞了好久还是不行，因为感觉这个根本就不会出问题的，可是居然出了这种问题
以前的驱动要添加接口，所以我就在原来驱动的ioctl中添加了一项，并且通过应用程序调用该ioctl，可是奇怪的事情发生了，我传递给
驱动的数据，居然不是我想要的，搞了半天都不行，期间试了两种方法，第一种：驱动中不用copy_from_user()函数来接收从应用传来的指针，可是这样也不行，第二种，就是用copy_from_user()函数来接受从应用传来的指针，可是还是不行。不说了上代码，大家帮我看看，哪里出问题了。。。。

应用层代码，调用的代码很简单就两句。。。。我从应用传过去的数据就是1和2，其他的数据都会报错的。

1. 
   
2.         int channel= 2；
   
3.         if (ioctl(inputFd, VPFE_CMD_SELECT_TVP5158_CHANNEL, &channel) == -1)
   
4.                         {
   
5.                                 printf("VPFE_CMD_SELECT_TVP5158_CHANNEL failed\n");
   
6.                         }

复制代码

驱动代码：

1. case VPFE_CMD_SELECT_TVP5158_CHANNEL:
   
2.         {   
   
3.    
   
4.             int  channel_id;
   
5. 
   
6.             copy_from_user(&channel_id, (int *)arg, sizeof(int));
   
7.            
   
8.             printk("channel_id = %d\n",channel_id);//本来这句打印出来应该是2的，可是它不是，是个垃圾数值，
   
9. 
   
10.             if (channel_id < 1 || channel_id > 2)
    
11.             {   
    
12.                 ret = -EINVAL;
    
13.                 break;
    
14.             }   
    
15. 
    
16.             tvp5158_ctrl(TVP5158_SELECT_CHANNEL, &channel_id);
    
17.             break;
    
18.         }   
    
19. 
    

复制代码

驱动中刚开始用这种方式实现的。一样是不行的

1. case VPFE_CMD_SELECT_TVP5158_CHANNEL:
   
2.         {   
   
3.    
   
4.             int  *channel_id = (int *)arg;
   
5.            
   
6.             printk("channel_id = %d\n",*channel_id);//本来这句打印出来应该是2的，可是它不是，是个垃圾数值，
   
7. 
   
8.             if (*channel_id < 1 || *channel_id > 2)
   
9.             {   
   
10.                 ret = -EINVAL;
    
11.                 break;
    
12.             }   
    
13. 
    
14.             tvp5158_ctrl(TVP5158_SELECT_CHANNEL, channel_id);
    
15.             break;
    
16.         }   

复制代码

大家帮我看看，看是哪里出错了,谢谢了 。。。。。。。。。。

xxw19840406

void __user *arg = (void __user *)arg;
驱动中arg有没有这个

然后copy_from_user(&channel_id, (int *)arg, sizeof(int));
改成 copy_from_user(&channel_id, arg, sizeof(int));

代码贴全点

zheguzai

xxw19840406 发表于 2012-05-04 18:03
void __user *arg = (void __user *)arg;
驱动中arg有没有这个

驱动里面没有你说的  void __user *arg = (void __user *)arg; 这句的。

yyplc

不应该用copy_from_user的:wink:

xxw19840406

zheguzai 发表于 2012-05-04 19:35
驱动里面没有你说的  void __user *arg = (void __user *)arg; 这句的。

这个不一定是真正的原因
一般copy_from_user都是要检查返回的，LZ可以看看是否出错

zheguzai

xxw19840406 发表于 2012-05-07 09:40
这个不一定是真正的原因
一般copy_from_user都是要检查返回的，LZ可以看看是否出错

查过了，返回值就是错误的。。。。

yxchugo

ldd3 中的做法是：因为ioctl 通常涉及到小的数据，所以先用access_ok()验证用户空间地址有效性，
然后使用不做类型检查的快速的函数：  put_user(), __put_user(), get_user,__get_user().

你把代码改成这样试试。

1. case VPFE_CMD_SELECT_TVP5158_CHANNEL:
   
2.         {   
   
3.    
   
4.             int  channel_id;
   
5. 
   
6.             int err = access_ok(VERIFY_READ, (void __user*)arg, _IOC_SIZE(cmd));
   
7.             if(!err)    return -EFAULT；
   
8.             __get_user(channel_id, (int __user *)arg);
   
9.            
   
10.             printk("channel_id = %d\n",channel_id);//本来这句打印出来应该是2的，可是它不是，是个垃圾数值，
    
11. 
    
12.             if (channel_id < 1 || channel_id > 2)
    
13.             {   
    
14.                 ret = -EINVAL;
    
15.                 break;
    
16.             }   
    
17. 
    
18.             tvp5158_ctrl(TVP5158_SELECT_CHANNEL, &channel_id);
    
19.             break;
    
20.         }   

复制代码

azzurris

友情提醒：你检查一下你的ioctl的原型和你的ioctl函数的实现，看看两者的参数个数是不是一致。

shaneqi

做个记号，下次好找！

zheguzai

非常感谢大家的回帖，那个davinci VPFE驱动里对IOCTL又进行了一次封装，本来我们只写一个ioctl函数的，它又写了一个调用真正的ioctl,并且对参数进行了处理。。。所以我的那个参数传进去之后就不对了。。。