创建proc项，write函数调用copy_from_user读取换行符的疑问

superwujc

请教各位

在/proc下创建了一个测试文件，并手动指定了write回调函数，write中调用copy_from_user，将输入在proc文件中的内容写入到内核空间
现在的问题是，如果proc文件中包含换行符时，copy_from_user会对内容分段写入多次

程序：proc_rw.c

1. #include <linux/module.h>
   
2. #include <linux/kernel.h>
   
3. #include <linux/proc_fs.h>
   
4. #include<linux/sched.h>
   
5. #include <asm/uaccess.h>
   
6. #include <linux/slab.h>
   
7. #include <linux/string.h>
   
8. 
   
9. int len,temp;
   
10. char *msg;
    
11. 
    
12. int read_proc(struct file *filp,char *buf,size_t count,loff_t *offp)
    
13. {
    
14.         if(count>temp)
    
15.         {
    
16.                 count=temp;
    
17.         }
    
18.         temp=temp-count;
    
19.         printk("read\n");
    
20.         printk("count = %d\n", count);
    
21.         printk("strlen(msg) = %d\n", strlen(msg));
    
22.         copy_to_user(buf,msg, count);
    
23.         if(count==0)
    
24.                 temp=len;
    
25. 
    
26.         return count;
    
27. }
    
28. 
    
29. int write_proc(struct file *filp,const char *buf,size_t count,loff_t *offp)
    
30. {
    
31.         if (copy_from_user(msg,buf,count))
    
32.                 return -EFAULT;
    
33. 
    
34.         len=count;
    
35.         temp=len;
    
36.         printk("write\n");
    
37.         printk("count = %d\n", count);
    
38.         printk("strlen(msg) = %d\n", strlen(msg));
    
39. 
    
40.         return count;
    
41. }
    
42. 
    
43. struct file_operations proc_fops = {
    
44.         read: read_proc,
    
45.         write: write_proc
    
46. };
    
47. 
    
48. void create_new_proc_entry()
    
49. {
    
50.         proc_create("hello",0,NULL,&proc_fops);
    
51.         msg=kmalloc(GFP_KERNEL,10*sizeof(char));
    
52. }
    
53. 
    
54. int proc_init (void) {
    
55.         create_new_proc_entry();
    
56.         return 0;
    
57. }
    
58. 
    
59. void proc_cleanup(void) {
    
60.         remove_proc_entry("hello",NULL);
    
61. }
    
62. 
    
63. MODULE_LICENSE("GPL");
    
64. module_init(proc_init);
    
65. module_exit(proc_cleanup);

复制代码

Makefile

1. obj-m += proc_rw.o
   
2. KDIR := /lib/modules/$(shell uname -r)/build
   
3. PWD := $(shell pwd)
   
4.     EXTRA_CFLAGS = -g -O0 -std=gnu99 -Wno-declaration-after-statement
   
5. all:
   
6.     make -C $(KDIR) M=$(PWD) modules
   
7.     rm -rf *.ko.unsigned *.mod.c *.mod.o *.symvers *.o
   
8. .PHONY: clean
   
9. clean:
   
10.     make -C $(KDIR) M=$(PWD) clean

复制代码

make并insmod

写入后查看内容

1. echo -ne "a b c" > /proc/hello

复制代码

1. # cat /proc/hello
   
2. a b c

复制代码

写入包含换行符的字符序列

1. echo -ne "a b c\td \ne   f" > /proc/hello

复制代码

1. # cat /proc/hello
   
2. e   f

复制代码

请各位大神指点一下，怎样copy_from_user()一次读取包含换行符的字符序列

nuclearxin

这个不是 copy_from_user 的问题。你的count 会在遇到回车的时候 设置，cut 然后进入下一轮的write.

所以我建议想要达到你说的效果，请仿照 dmesg  使用的 ring buff  的数据结构 来 代替 msg。

两个指针一个负责写。一个负责读取就好了。。

superwujc

回复 2# nuclearxin

感谢楼上，请教几个问题

write回调函数的buffer是用户内存空间，但我在内核源码中并未找到分配该内存空间的代码
如果count参数以'\n'计数，那么怎样改变这种行为？

proc的read与write，buffer都是在对proc文件读取和写入时自动分配的，怎样使用手动指定的缓冲区？

superwujc

回复 2# nuclearxin

再次请教，

a个'\n'会把输入分割成a+1段，write随之被调用了a+1次，因而读取时，read也被调用a+1次
在write的起始部分，copy_from_user调用之前，打印一下user buffer的长度，会观察到这种行为

1. int write_proc(struct file *filp,const char *buf,size_t count,loff_t *offp)
   
2. {
   
3.     printk("buf size: %d\n", strlen_user(buf));
   
4.     if (copy_from_user(msg,buf,count))
   
5.         return -EFAULT;
   
6. 
   
7.     len=count;
   
8.     temp=len;
   
9.     printk("write\n");
   
10.     printk("count = %d\n", count);
    
11.     printk("strlen(msg) = %d\n", strlen(msg));
    
12. 
    
13.     return count;
    
14. }

复制代码

1. echo -ne "a b c\td \ne   f" > /proc/hello

复制代码

1. # dmesg
   
2. [76143.461257] buf size: 1613
   
3. [76143.461261] write
   
4. [76143.461262] count = 9
   
5. [76143.461262] strlen(msg) = 16
   
6. [76143.461268] buf size: 1613
   
7. [76143.461269] write
   
8. [76143.461270] count = 5
   
9. [76143.461270] strlen(msg) = 16

复制代码

1. cat /proc/hello
   
2. e   f

复制代码

1. # dmesg
   
2. [76143.461257] buf size: 1613
   
3. [76143.461261] write
   
4. [76143.461262] count = 9
   
5. [76143.461262] strlen(msg) = 16
   
6. [76143.461268] buf size: 1613
   
7. [76143.461269] write
   
8. [76143.461270] count = 5
   
9. [76143.461270] strlen(msg) = 16
   
10. [76157.239084] read
    
11. [76157.239088] count = 5
    
12. [76157.239090] strlen(msg) = 16
    
13. [76157.239102] read
    
14. [76157.239103] count = 0
    
15. [76157.239104] strlen(msg) = 16

复制代码

nuclearxin

回复 3# superwujc

内核空间的内存 要额外小心分配。和用户空间内存随意分配不一样。
count的机制 我不建议你去修改。  工作方式很合理你改这里就是坑。会影响其他所有proc下的文件类的行为。

还是申请的 circular buff 环形链表  作为固定 大小 内存来使用吧。

等我有空写下

nuclearxin

superwujc 发表于 2017-03-30 10:19
回复 2# nuclearxin

再次请教，

没看懂你请教什么 了。

你的这个code 在 write的时候会执行很多次。 只保留最后一次的 len 和buff 的内容

然后你的读取只运行1次啊。 所以只打最后一次write进来的内容。

superwujc

回复 6# nuclearxin

read和write调用的次数是一样的，只不过最后一次read出来的内容把前面的覆盖掉了

1. [76143.461257] buf size: 1613
   
2. [76143.461261] write
   
3. [76143.461262] count = 9
   
4. [76143.461262] strlen(msg) = 16
   
5. [76143.461268] buf size: 1613
   
6. [76143.461269] write
   
7. [76143.461270] count = 5
   
8. [76143.461270] strlen(msg) = 16
   
9. [76157.239084] read
   
10. [76157.239088] count = 5
    
11. [76157.239090] strlen(msg) = 16
    
12. [76157.239102] read
    
13. [76157.239103] count = 0
    
14. [76157.239104] strlen(msg) = 16

复制代码

cat文件然后dmesg

1.     printk("read\n");
   
2.     printk("count = %d\n", count);
   
3.     printk("strlen(msg) = %d\n", strlen(msg));

复制代码

这段read函数中的代码被执行了2次，因为打印了2次

write被执行多次是因为user buf中的'\n'把输入在proc文件中的内容分割成多段，所以copy_from_user复制了多次user buf到kernel buf
read被执行多次是因为write被执行了多次，因而copy_to_user向user buf复制了多次

nuclearxin

superwujc 发表于 2017-03-30 10:40
回复 6# nuclearxin

read和write调用的次数是一样的，只不过最后一次read出来的内容把前面的覆盖掉了

read 被执行量2次 是cat  读完一行会尝试在读一次。

你的write会不断的使用\n分割的内容  不断重复覆盖 buff  和len。

你自己测试下

你可以连续 echo 不带 \n的 字符 然后最后cat 下看看是不是只触发上面的2次 read。

nuclearxin

superwujc 发表于 2017-03-30 10:40
回复 6# nuclearxin

read和write调用的次数是一样的，只不过最后一次read出来的内容把前面的覆盖掉了

确实 挺像 描述的行为。  太巧了

superwujc

回复 9# nuclearxin

so问题来了

如果proc的user buf包含换行符，write怎样一次性全部把这些内容写入到内核空间？

求教