[06/10] scheduler: Remove the now superfluous sentinel elements from ctl_table array

Commit Message

Joel Granados via B4 Relay Nov. 7, 2023, 1:45 p.m. UTC

From: Joel Granados <[e-mail]>

This commit comes at the tail end of a greater effort to remove the
empty elements at the end of the ctl_table arrays (sentinels) which
will reduce the overall build time size of the kernel and run time
memory bloat by ~64 bytes per sentinel (further information Link :
https://lore.kernel.org/all/[e-mail]/)

rm sentinel element from ctl_table arrays

Signed-off-by: Joel Granados <[e-mail]>
---
 kernel/sched/autogroup.c | 1 -
 kernel/sched/core.c      | 1 -
 kernel/sched/deadline.c  | 1 -
 kernel/sched/fair.c      | 1 -
 kernel/sched/rt.c        | 1 -
 kernel/sched/topology.c  | 1 -
 6 files changed, 6 deletions(-)

Comments

Peter Zijlstra Nov. 7, 2023, 2:49 p.m. UTC | #1

On Tue, Nov 07, 2023 at 02:45:06PM +0100, Joel Granados via B4 Relay wrote:
> From: Joel Granados <[e-mail]>
> 
> This commit comes at the tail end of a greater effort to remove the
> empty elements at the end of the ctl_table arrays (sentinels) which
> will reduce the overall build time size of the kernel and run time
> memory bloat by ~64 bytes per sentinel (further information Link :
> https://lore.kernel.org/all/[e-mail]/)
> 
> rm sentinel element from ctl_table arrays
> 
> Signed-off-by: Joel Granados <[e-mail]>
> ---
>  kernel/sched/autogroup.c | 1 -
>  kernel/sched/core.c      | 1 -
>  kernel/sched/deadline.c  | 1 -
>  kernel/sched/fair.c      | 1 -
>  kernel/sched/rt.c        | 1 -
>  kernel/sched/topology.c  | 1 -
>  6 files changed, 6 deletions(-)

Acked-by: Peter Zijlstra (Intel) <[e-mail]>

Patch

diff --git a/kernel/sched/autogroup.c b/kernel/sched/autogroup.c
index 991fc9002535..db68a964e34e 100644
--- a/kernel/sched/autogroup.c
+++ b/kernel/sched/autogroup.c
@@ -19,7 +19,6 @@  static struct ctl_table sched_autogroup_sysctls[] = {
 		.extra1         = SYSCTL_ZERO,
 		.extra2         = SYSCTL_ONE,
 	},
-	{}
 };
 
 static void __init sched_autogroup_sysctl_init(void)
diff --git a/kernel/sched/core.c b/kernel/sched/core.c
index 2299a5cfbfb9..d45cd41f08b5 100644
--- a/kernel/sched/core.c
+++ b/kernel/sched/core.c
@@ -4712,7 +4712,6 @@  static struct ctl_table sched_core_sysctls[] = {
 		.extra2		= SYSCTL_FOUR,
 	},
 #endif /* CONFIG_NUMA_BALANCING */
-	{}
 };
 static int __init sched_core_sysctl_init(void)
 {
diff --git a/kernel/sched/deadline.c b/kernel/sched/deadline.c
index 58b542bf2893..11f6e9a279d0 100644
--- a/kernel/sched/deadline.c
+++ b/kernel/sched/deadline.c
@@ -43,7 +43,6 @@  static struct ctl_table sched_dl_sysctls[] = {
 		.proc_handler   = proc_douintvec_minmax,
 		.extra2         = (void *)&sysctl_sched_dl_period_max,
 	},
-	{}
 };
 
 static int __init sched_dl_sysctl_init(void)
diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c
index cb225921bbca..8fd5b8435a9e 100644
--- a/kernel/sched/fair.c
+++ b/kernel/sched/fair.c
@@ -172,7 +172,6 @@  static struct ctl_table sched_fair_sysctls[] = {
 		.extra1		= SYSCTL_ZERO,
 	},
 #endif /* CONFIG_NUMA_BALANCING */
-	{}
 };
 
 static int __init sched_fair_sysctl_init(void)
diff --git a/kernel/sched/rt.c b/kernel/sched/rt.c
index 0597ba0f85ff..a6bcbef33e68 100644
--- a/kernel/sched/rt.c
+++ b/kernel/sched/rt.c
@@ -52,7 +52,6 @@  static struct ctl_table sched_rt_sysctls[] = {
 		.mode           = 0644,
 		.proc_handler   = sched_rr_handler,
 	},
-	{}
 };
 
 static int __init sched_rt_sysctl_init(void)
diff --git a/kernel/sched/topology.c b/kernel/sched/topology.c
index 05a5bc678c08..af998dbe097f 100644
--- a/kernel/sched/topology.c
+++ b/kernel/sched/topology.c
@@ -250,7 +250,6 @@  static struct ctl_table sched_energy_aware_sysctls[] = {
 		.extra1         = SYSCTL_ZERO,
 		.extra2         = SYSCTL_ONE,
 	},
-	{}
 };
 
 static int __init sched_energy_aware_sysctl_init(void)