[PATCH] sysctl: Add a group of macro functions to initcall the sysctl table of each feature
From: Xiaoming Ni
Date: Mon Dec 06 2021 - 20:13:43 EST
To avoid duplicated code, add a set of macro functions to initialize the
sysctl table for each feature.
The system initialization process is as follows:
start_kernel () {
...
/* init proc and sysctl base,
* proc_root_init()-->proc_sys_init()-->sysctl_init_bases()
*/
proc_root_init(); /* init proc and sysctl base */
...
arch_call_rest_init();
}
arch_call_rest_init()-->rest_init()-->kernel_init()
kernel_init() {
...
kernel_init_freeable(); /* do all initcalls */
...
do_sysctl_args(); /* Process the sysctl parameter: sysctl.*= */
}
kernel_init_freeable()--->do_basic_setup()-->do_initcalls()
do_initcalls() {
for (level = 0; level < ARRAY_SIZE(initcall_levels) - 1; level++) {
do_initcall_level
}
The sysctl interface of each subfeature should be registered after
sysctl_init_bases() and before do_sysctl_args(). It seems that the sysctl
interface does not depend on initcall_levels. To prevent the sysctl
interface from being initialized before the feature itself. The
lowest-level late_initcall() is used as the common sysctl interface
registration level.
Signed-off-by: Xiaoming Ni <nixiaoming@xxxxxxxxxx>
---
fs/coredump.c | 7 +------
fs/dcache.c | 7 +------
fs/exec.c | 8 +-------
fs/file_table.c | 7 +------
fs/inode.c | 7 +------
fs/locks.c | 7 +------
fs/namei.c | 8 +-------
fs/namespace.c | 7 +------
include/linux/sysctl.h | 15 +++++++++++++++
kernel/stackleak.c | 7 +------
10 files changed, 24 insertions(+), 56 deletions(-)
diff --git a/fs/coredump.c b/fs/coredump.c
index 570d98398668..8f6c6322651d 100644
--- a/fs/coredump.c
+++ b/fs/coredump.c
@@ -943,12 +943,7 @@ static struct ctl_table coredump_sysctls[] = {
{ }
};
-static int __init init_fs_coredump_sysctls(void)
-{
- register_sysctl_init("kernel", coredump_sysctls);
- return 0;
-}
-fs_initcall(init_fs_coredump_sysctls);
+kernel_sysctl_initcall(coredump_sysctls);
#endif /* CONFIG_SYSCTL */
/*
diff --git a/fs/dcache.c b/fs/dcache.c
index 0eef1102f460..c1570243aaee 100644
--- a/fs/dcache.c
+++ b/fs/dcache.c
@@ -195,12 +195,7 @@ static struct ctl_table fs_dcache_sysctls[] = {
{ }
};
-static int __init init_fs_dcache_sysctls(void)
-{
- register_sysctl_init("fs", fs_dcache_sysctls);
- return 0;
-}
-fs_initcall(init_fs_dcache_sysctls);
+fs_sysctl_initcall(fs_dcache_sysctls);
#endif
/*
diff --git a/fs/exec.c b/fs/exec.c
index cc5ec43df028..3c30e686af79 100644
--- a/fs/exec.c
+++ b/fs/exec.c
@@ -2124,11 +2124,5 @@ static struct ctl_table fs_exec_sysctls[] = {
{ }
};
-static int __init init_fs_exec_sysctls(void)
-{
- register_sysctl_init("fs", fs_exec_sysctls);
- return 0;
-}
-
-fs_initcall(init_fs_exec_sysctls);
+fs_sysctl_initcall(fs_exec_sysctls);
#endif /* CONFIG_SYSCTL */
diff --git a/fs/file_table.c b/fs/file_table.c
index 57edef16dce4..52b60e9378a7 100644
--- a/fs/file_table.c
+++ b/fs/file_table.c
@@ -116,12 +116,7 @@ static struct ctl_table fs_stat_sysctls[] = {
{ }
};
-static int __init init_fs_stat_sysctls(void)
-{
- register_sysctl_init("fs", fs_stat_sysctls);
- return 0;
-}
-fs_initcall(init_fs_stat_sysctls);
+fs_sysctl_initcall(fs_stat_sysctls);
#endif
static struct file *__alloc_file(int flags, const struct cred *cred)
diff --git a/fs/inode.c b/fs/inode.c
index bef6ba9b8eb4..cd20d6dd8ffa 100644
--- a/fs/inode.c
+++ b/fs/inode.c
@@ -132,12 +132,7 @@ static struct ctl_table inodes_sysctls[] = {
{ }
};
-static int __init init_fs_inode_sysctls(void)
-{
- register_sysctl_init("fs", inodes_sysctls);
- return 0;
-}
-early_initcall(init_fs_inode_sysctls);
+fs_sysctl_initcall(inodes_sysctls);
#endif
static int no_open(struct inode *inode, struct file *file)
diff --git a/fs/locks.c b/fs/locks.c
index 8c6df10cd9ed..d86a74d19ed3 100644
--- a/fs/locks.c
+++ b/fs/locks.c
@@ -113,12 +113,7 @@ static struct ctl_table locks_sysctls[] = {
{}
};
-static int __init init_fs_locks_sysctls(void)
-{
- register_sysctl_init("fs", locks_sysctls);
- return 0;
-}
-early_initcall(init_fs_locks_sysctls);
+fs_sysctl_initcall(locks_sysctls);
#endif /* CONFIG_SYSCTL */
/*
diff --git a/fs/namei.c b/fs/namei.c
index 8d4f832f94aa..7eb636796fd4 100644
--- a/fs/namei.c
+++ b/fs/namei.c
@@ -1066,13 +1066,7 @@ static struct ctl_table namei_sysctls[] = {
{ }
};
-static int __init init_fs_namei_sysctls(void)
-{
- register_sysctl_init("fs", namei_sysctls);
- return 0;
-}
-fs_initcall(init_fs_namei_sysctls);
-
+fs_sysctl_initcall(namei_sysctls);
#endif /* CONFIG_SYSCTL */
/**
diff --git a/fs/namespace.c b/fs/namespace.c
index 647af66f313d..7117214b7a85 100644
--- a/fs/namespace.c
+++ b/fs/namespace.c
@@ -4626,11 +4626,6 @@ static struct ctl_table fs_namespace_sysctls[] = {
{ }
};
-static int __init init_fs_namespace_sysctls(void)
-{
- register_sysctl_init("fs", fs_namespace_sysctls);
- return 0;
-}
-fs_initcall(init_fs_namespace_sysctls);
+fs_sysctl_initcall(fs_namespace_sysctls);
#endif /* CONFIG_SYSCTL */
diff --git a/include/linux/sysctl.h b/include/linux/sysctl.h
index acf0805cf3a0..228be2197348 100644
--- a/include/linux/sysctl.h
+++ b/include/linux/sysctl.h
@@ -231,6 +231,21 @@ extern int sysctl_init_bases(void);
extern void __register_sysctl_init(const char *path, struct ctl_table *table,
const char *table_name);
#define register_sysctl_init(path, table) __register_sysctl_init(path, table, #table)
+
+#define sysctl_initcall(path, table) \
+ static int __init init_##table(void) \
+ { \
+ register_sysctl_init(path, table); \
+ return 0;\
+ } \
+ late_initcall(init_##table)
+
+#define kernel_sysctl_initcall(table) sysctl_initcall("kernel", table)
+#define fs_sysctl_initcall(table) sysctl_initcall("fs", table)
+#define vm_sysctl_initcall(table) sysctl_initcall("vm", table)
+#define debug_sysctl_initcall(table) sysctl_initcall("debug", table)
+#define dev_sysctl_initcall(table) sysctl_initcall("dev", table)
+
extern struct ctl_table_header *register_sysctl_mount_point(const char *path);
void do_sysctl_args(void);
diff --git a/kernel/stackleak.c b/kernel/stackleak.c
index 66b8af394e58..f084c1c787ba 100644
--- a/kernel/stackleak.c
+++ b/kernel/stackleak.c
@@ -57,12 +57,7 @@ static struct ctl_table stackleak_sysctls[] = {
{}
};
-static int __init stackleak_sysctls_init(void)
-{
- register_sysctl_init("kernel", stackleak_sysctls);
- return 0;
-}
-late_initcall(stackleak_sysctls_init);
+kernel_sysctl_initcall(stackleak_sysctls);
#endif /* CONFIG_SYSCTL */
#define skip_erasing() static_branch_unlikely(&stack_erasing_bypass)
--
2.12.3