sysprog21 / simplefs

A simple native file system for Linux kernel
Other
362 stars 91 forks source link

inode.c #25

Closed cbkadal closed 1 year ago

cbkadal commented 1 year ago

FILE: inode.c

At line 199 and 222: inode_init_owner(&nop_mnt_idmap, inode, dir, inode->mode); s/mode/i_mode/

jserv commented 1 year ago

Hi @cbkadal, Can you please send a pull request accordingly?

cbkadal commented 1 year ago

My apology for not knowing what/how to "pull request." I will find out how to do that properly. BTW, I tried with kernel version 6.1.32 and 6.3.6, and found that:

Linux 6.1.32
total 2
-rw-r--r-- 1 root root  0 Jun  8 10:34 file1.txt
-rw-r--r-- 1 root root 30 Jun  8 10:34 file2.txt
-rw-r--r-- 1 root root  0 Jun  8 10:34 file3.txt

Linux 6.3.6
total 2
?--------- 1 root root  0 Jun  8 10:30 file1.txt
?--------- 1 root root 30 Jun  8 10:30 file2.txt
?--------- 1 root root  0 Jun  8 10:30 file3.txt
jserv commented 1 year ago

I defer to @fewletter .

jserv commented 1 year ago

Close in favor of #27 .