看这段代码是不是有问题

C、C++和Java语言
回复
头像
nobrain
帖子: 808
注册时间: 2005-08-25 13:58
来自: ustc
送出感谢: 0
接收感谢: 0
联系:

看这段代码是不是有问题

#1

帖子 nobrain » 2006-08-12 21:56

这是Linux Device Drivers (3rd Edition) 里面 ioctl 函数里面的一段代码:

代码: 全选

if (_IOC_DIR(cmd) & _IOC_READ)
		err = !access_ok(VERIFY_WRITE, (void __user *)arg, _IOC_SIZE(cmd));
else if (_IOC_DIR(cmd) & _IOC_WRITE)
		err =  !access_ok(VERIFY_READ, (void __user *)arg, _IOC_SIZE(cmd));
if (err) return -EFAULT;
pdf文档和源代码里面都是这样写的,
考虑到 _IOC_DIR(cmd) 可能等于 _IOC_READ | _IOC_WRITE,应该是if条件里面的_IOC_READ和_IOC_WRITE写反了吧?
爱喝真猪奶茶的夜鸣猪
头像
eexpress
帖子: 58428
注册时间: 2005-08-14 21:55
来自: 长沙
送出感谢: 4 次
接收感谢: 256 次

#2

帖子 eexpress » 2006-08-12 22:01

_IOC_DIR()的返回不一定能参与&操作。
● 鸣学
头像
nobrain
帖子: 808
注册时间: 2005-08-25 13:58
来自: ustc
送出感谢: 0
接收感谢: 0
联系:

#3

帖子 nobrain » 2006-08-12 22:15

_IOC_DIR是一个宏,把一个32bits的cmd的最高两位,移到最低两位,同时高30位都是零,为什么说
eexpress
_IOC_DIR()的返回不一定能参与&操作
没明白你的意思
爱喝真猪奶茶的夜鸣猪
头像
eexpress
帖子: 58428
注册时间: 2005-08-14 21:55
来自: 长沙
送出感谢: 4 次
接收感谢: 256 次

#4

帖子 eexpress » 2006-08-12 22:22

&是位操作。如果函数返回是结构/类指针这样的。是不能参与位操作的吧。
因为我不确定返回的是dword还是什么哦。
● 鸣学
头像
nobrain
帖子: 808
注册时间: 2005-08-25 13:58
来自: ustc
送出感谢: 0
接收感谢: 0
联系:

#5

帖子 nobrain » 2006-08-12 22:29

刚才在瀚海星云上遇达人,解决了这个问题:
因为_IOC_WRITE 和 _IOC_READ是针对用户空间来说的,而access_ok是针对内核空间来说的,这样read和write操作就是反过来的。
对用户来说的read,是要把数据从设备读入然后写入用户空间的buffer里,用copy_to_user等函数完成,所以对内核来说是write用户空间的buffer.
对用户来说的write,是内核从用户空间的buffer里将数据读出写入设备,用copy_from_user等函数完成,所以对内核来说是read用户空间的buffer.
爱喝真猪奶茶的夜鸣猪
头像
jiangpeng
帖子: 223
注册时间: 2006-07-25 9:33
送出感谢: 0
接收感谢: 0
联系:

#6

帖子 jiangpeng » 2006-08-12 23:51

呵呵,书上(翻译的)这一段是有注释地
Take what man makes and use it, But do not worship it, For it shall pass. -- Anonymous

Twitter @jiangpeng
回复

回到 “C/C++/Java”