Li Zefan wrote: >> static int __init slab_proc_init(void) >> { >> - proc_create("slabinfo",S_IWUSR|S_IRUGO,NULL,&proc_slabinfo_operations); >> + proc_create("slabinfo",S_IRUSR|S_IRUGO,NULL,&proc_slabinfo_operations); >> > > S_IRUSR|S_IRUGO == S_IRUGO > > Ah, yeah. Thanks! Update it. Signed-off-by: WANG Cong