Skip to content

Commit 5fee64f

Browse files
author
Christoph Hellwig
committed
init: add an init_mknod helper
Add a simple helper to mknod with a kernel space file name and switch the early init code over to it. Remove the now unused ksys_mknod. Signed-off-by: Christoph Hellwig <hch@lst.de>
1 parent 83ff98c commit 5fee64f

File tree

8 files changed

+30
-16
lines changed

8 files changed

+30
-16
lines changed

fs/init.c

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -122,6 +122,31 @@ int __init init_eaccess(const char *filename)
122122
return error;
123123
}
124124

125+
int __init init_mknod(const char *filename, umode_t mode, unsigned int dev)
126+
{
127+
struct dentry *dentry;
128+
struct path path;
129+
int error;
130+
131+
if (S_ISFIFO(mode) || S_ISSOCK(mode))
132+
dev = 0;
133+
else if (!(S_ISBLK(mode) || S_ISCHR(mode)))
134+
return -EINVAL;
135+
136+
dentry = kern_path_create(AT_FDCWD, filename, &path, 0);
137+
if (IS_ERR(dentry))
138+
return PTR_ERR(dentry);
139+
140+
if (!IS_POSIXACL(path.dentry->d_inode))
141+
mode &= ~current_umask();
142+
error = security_path_mknod(&path, dentry, mode, dev);
143+
if (!error)
144+
error = vfs_mknod(path.dentry->d_inode, dentry, mode,
145+
new_decode_dev(dev));
146+
done_path_create(&path, dentry);
147+
return error;
148+
}
149+
125150
int __init init_link(const char *oldname, const char *newname)
126151
{
127152
struct dentry *new_dentry;

fs/internal.h

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -62,8 +62,6 @@ extern int filename_lookup(int dfd, struct filename *name, unsigned flags,
6262
struct path *path, struct path *root);
6363
extern int vfs_path_lookup(struct dentry *, struct vfsmount *,
6464
const char *, unsigned int, struct path *);
65-
long do_mknodat(int dfd, const char __user *filename, umode_t mode,
66-
unsigned int dev);
6765
long do_rmdir(int dfd, struct filename *name);
6866
long do_unlinkat(int dfd, struct filename *name);
6967
int may_linkat(struct path *link);

fs/namei.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3564,7 +3564,7 @@ static int may_mknod(umode_t mode)
35643564
}
35653565
}
35663566

3567-
long do_mknodat(int dfd, const char __user *filename, umode_t mode,
3567+
static long do_mknodat(int dfd, const char __user *filename, umode_t mode,
35683568
unsigned int dev)
35693569
{
35703570
struct dentry *dentry;

include/linux/init_syscalls.h

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ int __init init_chroot(const char *filename);
88
int __init init_chown(const char *filename, uid_t user, gid_t group, int flags);
99
int __init init_chmod(const char *filename, umode_t mode);
1010
int __init init_eaccess(const char *filename);
11+
int __init init_mknod(const char *filename, umode_t mode, unsigned int dev);
1112
int __init init_link(const char *oldname, const char *newname);
1213
int __init init_symlink(const char *oldname, const char *newname);
1314
int __init init_unlink(const char *pathname);

include/linux/syscalls.h

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1270,15 +1270,6 @@ int compat_ksys_ipc(u32 call, int first, int second,
12701270
* The following kernel syscall equivalents are just wrappers to fs-internal
12711271
* functions. Therefore, provide stubs to be inlined at the callsites.
12721272
*/
1273-
extern long do_mknodat(int dfd, const char __user *filename, umode_t mode,
1274-
unsigned int dev);
1275-
1276-
static inline long ksys_mknod(const char __user *filename, umode_t mode,
1277-
unsigned int dev)
1278-
{
1279-
return do_mknodat(AT_FDCWD, filename, mode, dev);
1280-
}
1281-
12821273
extern int do_fchownat(int dfd, const char __user *filename, uid_t user,
12831274
gid_t group, int flag);
12841275

init/do_mounts.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ extern int root_mountflags;
1717
static inline __init int create_dev(char *name, dev_t dev)
1818
{
1919
init_unlink(name);
20-
return ksys_mknod(name, S_IFBLK|0600, new_encode_dev(dev));
20+
return init_mknod(name, S_IFBLK | 0600, new_encode_dev(dev));
2121
}
2222

2323
#ifdef CONFIG_BLK_DEV_RAM

init/initramfs.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -355,7 +355,7 @@ static int __init do_name(void)
355355
} else if (S_ISBLK(mode) || S_ISCHR(mode) ||
356356
S_ISFIFO(mode) || S_ISSOCK(mode)) {
357357
if (maybe_link() == 0) {
358-
ksys_mknod(collected, mode, rdev);
358+
init_mknod(collected, mode, rdev);
359359
init_chown(collected, uid, gid, 0);
360360
init_chmod(collected, mode);
361361
do_utime(collected, mtime);

init/noinitramfs.c

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,7 @@ static int __init default_rootfs(void)
2222
if (err < 0)
2323
goto out;
2424

25-
err = ksys_mknod((const char __user __force *) "/dev/console",
26-
S_IFCHR | S_IRUSR | S_IWUSR,
25+
err = init_mknod("/dev/console", S_IFCHR | S_IRUSR | S_IWUSR,
2726
new_encode_dev(MKDEV(5, 1)));
2827
if (err < 0)
2928
goto out;

0 commit comments

Comments
 (0)