[18/69] eventpoll: simplify sysctl declaration with register_sysctl()

Commit Message

Andrew Morton Jan. 22, 2022, 6:12 a.m. UTC

From: Xiaoming Ni <[e-mail]>
Subject: eventpoll: simplify sysctl declaration with register_sysctl()

The kernel/sysctl.c is a kitchen sink where everyone leaves their dirty
dishes, this makes it very difficult to maintain.

To help with this maintenance let's start by moving sysctls to places
where they actually belong.  The proc sysctl maintainers do not want to
know what sysctl knobs you wish to add for your own piece of code, we just
care about the core logic.

So move the epoll_table sysctl to fs/eventpoll.c and use
register_sysctl().

Link: https://lkml.kernel.org/r/[e-mail]
Signed-off-by: Xiaoming Ni <[e-mail]>
Signed-off-by: Luis Chamberlain <[e-mail]>
Cc: Al Viro <[e-mail]>
Cc: Amir Goldstein <[e-mail]>
Cc: Andy Shevchenko <[e-mail]>
Cc: Antti Palosaari <[e-mail]>
Cc: Arnd Bergmann <[e-mail]>
Cc: Benjamin Herrenschmidt <[e-mail]>
Cc: Benjamin LaHaise <[e-mail]>
Cc: Clemens Ladisch <[e-mail]>
Cc: David Airlie <[e-mail]>
Cc: "Eric W. Biederman" <[e-mail]>
Cc: Greg Kroah-Hartman <[e-mail]>
Cc: Iurii Zaikin <[e-mail]>
Cc: Jani Nikula <[e-mail]>
Cc: Jan Kara <[e-mail]>
Cc: Joel Becker <[e-mail]>
Cc: Joonas Lahtinen <[e-mail]>
Cc: Joseph Qi <[e-mail]>
Cc: Julia Lawall <[e-mail]>
Cc: Kees Cook <[e-mail]>
Cc: Lukas Middendorf <[e-mail]>
Cc: Mark Fasheh <[e-mail]>
Cc: Paul Turner <[e-mail]>
Cc: Peter Zijlstra <[e-mail]>
Cc: Petr Mladek <[e-mail]>
Cc: Phillip Potter <[e-mail]>
Cc: Qing Wang <[e-mail]>
Cc: Rodrigo Vivi <[e-mail]>
Cc: Sebastian Reichel <[e-mail]>
Cc: Sergey Senozhatsky <[e-mail]>
Cc: Stephen Kitt <[e-mail]>
Cc: Tetsuo Handa <[e-mail]>
Cc: Douglas Gilbert <[e-mail]>
Cc: James E.J. Bottomley <[e-mail]>
Cc: Jani Nikula <[e-mail]>
Cc: John Ogness <[e-mail]>
Cc: Martin K. Petersen <[e-mail]>
Cc: "Rafael J. Wysocki" <[e-mail]>
Cc: Steven Rostedt (VMware) <[e-mail]>
Cc: Suren Baghdasaryan <[e-mail]>
Cc: "Theodore Ts'o" <[e-mail]>
Signed-off-by: Andrew Morton <[e-mail]>
---

 fs/eventpoll.c         |   10 +++++++++-
 include/linux/poll.h   |    2 --
 include/linux/sysctl.h |    1 -
 kernel/sysctl.c        |    7 -------
 4 files changed, 9 insertions(+), 11 deletions(-)

Patch

--- a/fs/eventpoll.c~eventpoll-simplify-sysctl-declaration-with-register_sysctl
+++ a/fs/eventpoll.c
@@ -307,7 +307,7 @@  static void unlist_file(struct epitems_h
 static long long_zero;
 static long long_max = LONG_MAX;
 
-struct ctl_table epoll_table[] = {
+static struct ctl_table epoll_table[] = {
 	{
 		.procname	= "max_user_watches",
 		.data		= &max_user_watches,
@@ -319,6 +319,13 @@  struct ctl_table epoll_table[] = {
 	},
 	{ }
 };
+
+static void __init epoll_sysctls_init(void)
+{
+	register_sysctl("fs/epoll", epoll_table);
+}
+#else
+#define epoll_sysctls_init() do { } while (0)
 #endif /* CONFIG_SYSCTL */
 
 static const struct file_operations eventpoll_fops;
@@ -2378,6 +2385,7 @@  static int __init eventpoll_init(void)
 	/* Allocates slab cache used to allocate "struct eppoll_entry" */
 	pwq_cache = kmem_cache_create("eventpoll_pwq",
 		sizeof(struct eppoll_entry), 0, SLAB_PANIC|SLAB_ACCOUNT, NULL);
+	epoll_sysctls_init();
 
 	ephead_cache = kmem_cache_create("ep_head",
 		sizeof(struct epitems_head), 0, SLAB_PANIC|SLAB_ACCOUNT, NULL);
--- a/include/linux/poll.h~eventpoll-simplify-sysctl-declaration-with-register_sysctl
+++ a/include/linux/poll.h
@@ -8,12 +8,10 @@ 
 #include <linux/wait.h>
 #include <linux/string.h>
 #include <linux/fs.h>
-#include <linux/sysctl.h>
 #include <linux/uaccess.h>
 #include <uapi/linux/poll.h>
 #include <uapi/linux/eventpoll.h>
 
-extern struct ctl_table epoll_table[]; /* for sysctl */
 /* ~832 bytes of stack space used max in sys_select/sys_poll before allocating
    additional memory. */
 #ifdef __clang__
--- a/include/linux/sysctl.h~eventpoll-simplify-sysctl-declaration-with-register_sysctl
+++ a/include/linux/sysctl.h
@@ -219,7 +219,6 @@  extern int no_unaligned_warning;
 extern struct ctl_table sysctl_mount_point[];
 extern struct ctl_table random_table[];
 extern struct ctl_table firmware_config_table[];
-extern struct ctl_table epoll_table[];
 
 #else /* CONFIG_SYSCTL */
 static inline struct ctl_table_header *register_sysctl_table(struct ctl_table * table)
--- a/kernel/sysctl.c~eventpoll-simplify-sysctl-declaration-with-register_sysctl
+++ a/kernel/sysctl.c
@@ -3093,13 +3093,6 @@  static struct ctl_table fs_table[] = {
 		.proc_handler	= proc_dointvec,
 	},
 #endif
-#ifdef CONFIG_EPOLL
-	{
-		.procname	= "epoll",
-		.mode		= 0555,
-		.child		= epoll_table,
-	},
-#endif
 #endif
 	{
 		.procname	= "protected_symlinks",