* [PATCH 00/10] sysctl: Remove sentinel elements from kernel dir
@ 2023-11-07 13:45 ` Joel Granados
0 siblings, 0 replies; 51+ messages in thread
From: Joel Granados via B4 Relay @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
From: Joel Granados <j.granados@samsung.com>
What?
These commits remove the sentinel element (last empty element) from the
sysctl arrays of all the files under the "kernel/" directory that use a
sysctl array for registration. The merging of the preparation patches
(in https://lore.kernel.org/all/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
to mainline allows us to just remove sentinel elements without changing
behavior (more info here [1]).
These commits are part of a bigger set (here
https://github.com/Joelgranados/linux/tree/tag/sysctl_remove_empty_elem_V5)
that remove the ctl_table sentinel. Make the review process easier by
chunking the commits into manageable pieces. Each chunk can be reviewed
separately without noise from parallel sets.
Sending the "kernel/*" chunk now that the "drivers/" has been mostly
reviewed [6]. After this and the "fs/*" are reviewed we only miss 2 more
chunks ("net/*" and miscellaneous) to complete the sentinel removal.
Hurray!!!
Why?
By removing the sysctl sentinel elements we avoid kernel bloat as
ctl_table arrays get moved out of kernel/sysctl.c into their own
respective subsystems. This move was started long ago to avoid merge
conflicts; the sentinel removal bit came after Mathew Wilcox suggested
it to avoid bloating the kernel by one element as arrays moved out. This
patchset will reduce the overall build time size of the kernel and run
time memory bloat by about ~64 bytes per declared ctl_table array. I
have consolidated some links that shed light on the history of this
effort [2].
Testing:
* Ran sysctl selftests (./tools/testing/selftests/sysctl/sysctl.sh)
* Ran this through 0-day with no errors or warnings
Size saving after this patchset:
* bloat-o-meter
- The "yesall" config saves 1984 bytes [4]
- The "tiny" config saves 771 bytes [5]
* If you want to know how many bytes are saved after all the chunks
are merged see [3]
Base commit:
tag: sysctl-6.7-rc1 (8b793bcda61f)
Comments/feedback greatly appreciated
Best
Joel
[1]
We are able to remove a sentinel table without behavioral change by
introducing a table_size argument in the same place where procname is
checked for NULL. The idea is for it to keep stopping when it hits
->procname == NULL, while the sentinel is still present. And when the
sentinel is removed, it will stop on the table_size. You can go to
(https://lore.kernel.org/all/20230809105006.1198165-1-j.granados@samsung.com/)
for more information.
[2]
Links Related to the ctl_table sentinel removal:
* E-mail threads that summarize the sentinel effort
https://lore.kernel.org/all/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/
https://lore.kernel.org/all/ZMFizKFkVxUFtSqa@bombadil.infradead.org/
* Replacing the register functions:
https://lore.kernel.org/all/20230302204612.782387-1-mcgrof@kernel.org/
https://lore.kernel.org/all/20230302202826.776286-1-mcgrof@kernel.org/
* E-mail threads discussing prposal
https://lore.kernel.org/all/20230321130908.6972-1-frank.li@vivo.com
https://lore.kernel.org/all/20220220060626.15885-1-tangmeng@uniontech.com
[3]
Size saving after removing all sentinels:
These are the bytes that we save after removing all the sentinels
(this plus all the other chunks). I included them to get an idea of
how much memory we are talking about.
* bloat-o-meter:
- The "yesall" configuration results save 9158 bytes
https://lore.kernel.org/all/20230621091000.424843-1-j.granados@samsung.com/
- The "tiny" config + CONFIG_SYSCTL save 1215 bytes
https://lore.kernel.org/all/20230809105006.1198165-1-j.granados@samsung.com/
* memory usage:
In memory savings are measured to be 7296 bytes. (here is how to
measure [7])
[4]
add/remove: 0/0 grow/shrink: 0/31 up/down: 0/-1984 (-1984)
Function old new delta
watchdog_sysctls 576 512 -64
watchdog_hardlockup_sysctl 128 64 -64
vm_table 1344 1280 -64
uts_kern_table 448 384 -64
usermodehelper_table 192 128 -64
user_table 832 768 -64
user_event_sysctls 128 64 -64
timer_sysctl 128 64 -64
signal_debug_table 128 64 -64
seccomp_sysctl_table 192 128 -64
sched_rt_sysctls 256 192 -64
sched_fair_sysctls 256 192 -64
sched_energy_aware_sysctls 128 64 -64
sched_dl_sysctls 192 128 -64
sched_core_sysctls 384 320 -64
sched_autogroup_sysctls 128 64 -64
printk_sysctls 512 448 -64
pid_ns_ctl_table_vm 128 64 -64
pid_ns_ctl_table 128 64 -64
latencytop_sysctl 128 64 -64
kprobe_sysctls 128 64 -64
kexec_core_sysctls 256 192 -64
kern_table 2560 2496 -64
kern_reboot_table 192 128 -64
kern_panic_table 192 128 -64
kern_exit_table 128 64 -64
kern_delayacct_table 128 64 -64
kern_acct_table 128 64 -64
hung_task_sysctls 448 384 -64
ftrace_sysctls 128 64 -64
bpf_syscall_table 192 128 -64
Total: Before=429912331, After=429910347, chg -0.00%
[5]
add/remove: 0/1 grow/shrink: 0/12 up/down: 0/-771 (-771)
Function old new delta
sched_core_sysctl_init 43 40 -3
vm_table 1024 960 -64
uts_kern_table 448 384 -64
usermodehelper_table 192 128 -64
user_table 576 512 -64
signal_debug_table 128 64 -64
sched_rt_sysctls 256 192 -64
sched_fair_sysctls 128 64 -64
sched_dl_sysctls 192 128 -64
sched_core_sysctls 64 - -64
kern_table 1792 1728 -64
kern_panic_table 128 64 -64
kern_exit_table 128 64 -64
Total: Before=1886645, After=1885874, chg -0.04%
[6]
https://lore.kernel.org/all/20231002-jag-sysctl_remove_empty_elem_drivers-v2-0-02dd0d46f71e@samsung.com
[7]
To measure the in memory savings apply this on top of this patchset.
"
diff --git a/fs/proc/proc_sysctl.c b/fs/proc/proc_sysctl.c
index c88854df0b62..e0073a627bac 100644
--- a/fs/proc/proc_sysctl.c
+++ b/fs/proc/proc_sysctl.c
@@ -976,6 +976,8 @@ static struct ctl_dir *new_dir(struct ctl_table_set *set,
table[0].procname = new_name;
table[0].mode = S_IFDIR|S_IRUGO|S_IXUGO;
init_header(&new->header, set->dir.header.root, set, node, table, 1);
+ // Counts additional sentinel used for each new dir.
+ printk("%ld sysctl saved mem kzalloc \n", sizeof(struct ctl_table));
return new;
}
@@ -1199,6 +1201,9 @@ static struct ctl_table_header *new_links(struct ctl_dir *dir, struct ctl_table_
link_name += len;
link++;
}
+ // Counts additional sentinel used for each new registration
+ //
+ printk("%ld sysctl saved mem kzalloc \n", sizeof(struct ctl_table));
init_header(links, dir->header.root, dir->header.set, node, link_table,
head->ctl_table_size);
links->nreg = nr_entries;
"
and then run the following bash script in the kernel:
accum=0
for n in $(dmesg | grep kzalloc | awk '{print $3}') ; do
echo $n
accum=$(calc "$accum + $n")
done
echo $accum
---
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
Joel Granados (10):
kernel misc: Remove the now superfluous sentinel elements from ctl_table array
umh: Remove the now superfluous sentinel elements from ctl_table array
ftrace: Remove the now superfluous sentinel elements from ctl_table array
timekeeping: Remove the now superfluous sentinel elements from ctl_table array
seccomp: Remove the now superfluous sentinel elements from ctl_table array
scheduler: Remove the now superfluous sentinel elements from ctl_table array
printk: Remove the now superfluous sentinel elements from ctl_table array
kprobes: Remove the now superfluous sentinel elements from ctl_table array
delayacct: Remove the now superfluous sentinel elements from ctl_table array
bpf: Remove the now superfluous sentinel elements from ctl_table array
kernel/acct.c | 1 -
kernel/bpf/syscall.c | 1 -
kernel/delayacct.c | 1 -
kernel/exit.c | 1 -
kernel/hung_task.c | 1 -
kernel/kexec_core.c | 1 -
kernel/kprobes.c | 1 -
kernel/latencytop.c | 1 -
kernel/panic.c | 1 -
kernel/pid_namespace.c | 1 -
kernel/pid_sysctl.h | 1 -
kernel/printk/sysctl.c | 1 -
kernel/reboot.c | 1 -
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 -
kernel/seccomp.c | 1 -
kernel/signal.c | 1 -
kernel/stackleak.c | 1 -
kernel/sysctl.c | 2 --
kernel/time/timer.c | 1 -
kernel/trace/ftrace.c | 1 -
kernel/trace/trace_events_user.c | 1 -
kernel/ucount.c | 3 +--
kernel/umh.c | 1 -
kernel/utsname_sysctl.c | 1 -
kernel/watchdog.c | 2 --
30 files changed, 1 insertion(+), 33 deletions(-)
---
base-commit: 8b793bcda61f6c3ed4f5b2ded7530ef6749580cb
change-id: 20231107-jag-sysctl_remove_empty_elem_kernel-7de90cfd0c0a
Best regards,
--
Joel Granados <j.granados@samsung.com>
^ permalink raw reply related [flat|nested] 51+ messages in thread
* [PATCH 00/10] sysctl: Remove sentinel elements from kernel dir
@ 2023-11-07 13:45 ` Joel Granados
0 siblings, 0 replies; 51+ messages in thread
From: Joel Granados @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
What?
These commits remove the sentinel element (last empty element) from the
sysctl arrays of all the files under the "kernel/" directory that use a
sysctl array for registration. The merging of the preparation patches
(in https://lore.kernel.org/all/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
to mainline allows us to just remove sentinel elements without changing
behavior (more info here [1]).
These commits are part of a bigger set (here
https://github.com/Joelgranados/linux/tree/tag/sysctl_remove_empty_elem_V5)
that remove the ctl_table sentinel. Make the review process easier by
chunking the commits into manageable pieces. Each chunk can be reviewed
separately without noise from parallel sets.
Sending the "kernel/*" chunk now that the "drivers/" has been mostly
reviewed [6]. After this and the "fs/*" are reviewed we only miss 2 more
chunks ("net/*" and miscellaneous) to complete the sentinel removal.
Hurray!!!
Why?
By removing the sysctl sentinel elements we avoid kernel bloat as
ctl_table arrays get moved out of kernel/sysctl.c into their own
respective subsystems. This move was started long ago to avoid merge
conflicts; the sentinel removal bit came after Mathew Wilcox suggested
it to avoid bloating the kernel by one element as arrays moved out. This
patchset will reduce the overall build time size of the kernel and run
time memory bloat by about ~64 bytes per declared ctl_table array. I
have consolidated some links that shed light on the history of this
effort [2].
Testing:
* Ran sysctl selftests (./tools/testing/selftests/sysctl/sysctl.sh)
* Ran this through 0-day with no errors or warnings
Size saving after this patchset:
* bloat-o-meter
- The "yesall" config saves 1984 bytes [4]
- The "tiny" config saves 771 bytes [5]
* If you want to know how many bytes are saved after all the chunks
are merged see [3]
Base commit:
tag: sysctl-6.7-rc1 (8b793bcda61f)
Comments/feedback greatly appreciated
Best
Joel
[1]
We are able to remove a sentinel table without behavioral change by
introducing a table_size argument in the same place where procname is
checked for NULL. The idea is for it to keep stopping when it hits
->procname == NULL, while the sentinel is still present. And when the
sentinel is removed, it will stop on the table_size. You can go to
(https://lore.kernel.org/all/20230809105006.1198165-1-j.granados@samsung.com/)
for more information.
[2]
Links Related to the ctl_table sentinel removal:
* E-mail threads that summarize the sentinel effort
https://lore.kernel.org/all/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/
https://lore.kernel.org/all/ZMFizKFkVxUFtSqa@bombadil.infradead.org/
* Replacing the register functions:
https://lore.kernel.org/all/20230302204612.782387-1-mcgrof@kernel.org/
https://lore.kernel.org/all/20230302202826.776286-1-mcgrof@kernel.org/
* E-mail threads discussing prposal
https://lore.kernel.org/all/20230321130908.6972-1-frank.li@vivo.com
https://lore.kernel.org/all/20220220060626.15885-1-tangmeng@uniontech.com
[3]
Size saving after removing all sentinels:
These are the bytes that we save after removing all the sentinels
(this plus all the other chunks). I included them to get an idea of
how much memory we are talking about.
* bloat-o-meter:
- The "yesall" configuration results save 9158 bytes
https://lore.kernel.org/all/20230621091000.424843-1-j.granados@samsung.com/
- The "tiny" config + CONFIG_SYSCTL save 1215 bytes
https://lore.kernel.org/all/20230809105006.1198165-1-j.granados@samsung.com/
* memory usage:
In memory savings are measured to be 7296 bytes. (here is how to
measure [7])
[4]
add/remove: 0/0 grow/shrink: 0/31 up/down: 0/-1984 (-1984)
Function old new delta
watchdog_sysctls 576 512 -64
watchdog_hardlockup_sysctl 128 64 -64
vm_table 1344 1280 -64
uts_kern_table 448 384 -64
usermodehelper_table 192 128 -64
user_table 832 768 -64
user_event_sysctls 128 64 -64
timer_sysctl 128 64 -64
signal_debug_table 128 64 -64
seccomp_sysctl_table 192 128 -64
sched_rt_sysctls 256 192 -64
sched_fair_sysctls 256 192 -64
sched_energy_aware_sysctls 128 64 -64
sched_dl_sysctls 192 128 -64
sched_core_sysctls 384 320 -64
sched_autogroup_sysctls 128 64 -64
printk_sysctls 512 448 -64
pid_ns_ctl_table_vm 128 64 -64
pid_ns_ctl_table 128 64 -64
latencytop_sysctl 128 64 -64
kprobe_sysctls 128 64 -64
kexec_core_sysctls 256 192 -64
kern_table 2560 2496 -64
kern_reboot_table 192 128 -64
kern_panic_table 192 128 -64
kern_exit_table 128 64 -64
kern_delayacct_table 128 64 -64
kern_acct_table 128 64 -64
hung_task_sysctls 448 384 -64
ftrace_sysctls 128 64 -64
bpf_syscall_table 192 128 -64
Total: Before=429912331, After=429910347, chg -0.00%
[5]
add/remove: 0/1 grow/shrink: 0/12 up/down: 0/-771 (-771)
Function old new delta
sched_core_sysctl_init 43 40 -3
vm_table 1024 960 -64
uts_kern_table 448 384 -64
usermodehelper_table 192 128 -64
user_table 576 512 -64
signal_debug_table 128 64 -64
sched_rt_sysctls 256 192 -64
sched_fair_sysctls 128 64 -64
sched_dl_sysctls 192 128 -64
sched_core_sysctls 64 - -64
kern_table 1792 1728 -64
kern_panic_table 128 64 -64
kern_exit_table 128 64 -64
Total: Before=1886645, After=1885874, chg -0.04%
[6]
https://lore.kernel.org/all/20231002-jag-sysctl_remove_empty_elem_drivers-v2-0-02dd0d46f71e@samsung.com
[7]
To measure the in memory savings apply this on top of this patchset.
"
diff --git a/fs/proc/proc_sysctl.c b/fs/proc/proc_sysctl.c
index c88854df0b62..e0073a627bac 100644
--- a/fs/proc/proc_sysctl.c
+++ b/fs/proc/proc_sysctl.c
@@ -976,6 +976,8 @@ static struct ctl_dir *new_dir(struct ctl_table_set *set,
table[0].procname = new_name;
table[0].mode = S_IFDIR|S_IRUGO|S_IXUGO;
init_header(&new->header, set->dir.header.root, set, node, table, 1);
+ // Counts additional sentinel used for each new dir.
+ printk("%ld sysctl saved mem kzalloc \n", sizeof(struct ctl_table));
return new;
}
@@ -1199,6 +1201,9 @@ static struct ctl_table_header *new_links(struct ctl_dir *dir, struct ctl_table_
link_name += len;
link++;
}
+ // Counts additional sentinel used for each new registration
+ //
+ printk("%ld sysctl saved mem kzalloc \n", sizeof(struct ctl_table));
init_header(links, dir->header.root, dir->header.set, node, link_table,
head->ctl_table_size);
links->nreg = nr_entries;
"
and then run the following bash script in the kernel:
accum=0
for n in $(dmesg | grep kzalloc | awk '{print $3}') ; do
echo $n
accum=$(calc "$accum + $n")
done
echo $accum
---
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
Joel Granados (10):
kernel misc: Remove the now superfluous sentinel elements from ctl_table array
umh: Remove the now superfluous sentinel elements from ctl_table array
ftrace: Remove the now superfluous sentinel elements from ctl_table array
timekeeping: Remove the now superfluous sentinel elements from ctl_table array
seccomp: Remove the now superfluous sentinel elements from ctl_table array
scheduler: Remove the now superfluous sentinel elements from ctl_table array
printk: Remove the now superfluous sentinel elements from ctl_table array
kprobes: Remove the now superfluous sentinel elements from ctl_table array
delayacct: Remove the now superfluous sentinel elements from ctl_table array
bpf: Remove the now superfluous sentinel elements from ctl_table array
kernel/acct.c | 1 -
kernel/bpf/syscall.c | 1 -
kernel/delayacct.c | 1 -
kernel/exit.c | 1 -
kernel/hung_task.c | 1 -
kernel/kexec_core.c | 1 -
kernel/kprobes.c | 1 -
kernel/latencytop.c | 1 -
kernel/panic.c | 1 -
kernel/pid_namespace.c | 1 -
kernel/pid_sysctl.h | 1 -
kernel/printk/sysctl.c | 1 -
kernel/reboot.c | 1 -
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 -
kernel/seccomp.c | 1 -
kernel/signal.c | 1 -
kernel/stackleak.c | 1 -
kernel/sysctl.c | 2 --
kernel/time/timer.c | 1 -
kernel/trace/ftrace.c | 1 -
kernel/trace/trace_events_user.c | 1 -
kernel/ucount.c | 3 +--
kernel/umh.c | 1 -
kernel/utsname_sysctl.c | 1 -
kernel/watchdog.c | 2 --
30 files changed, 1 insertion(+), 33 deletions(-)
---
base-commit: 8b793bcda61f6c3ed4f5b2ded7530ef6749580cb
change-id: 20231107-jag-sysctl_remove_empty_elem_kernel-7de90cfd0c0a
Best regards,
--
Joel Granados <j.granados@samsung.com>
^ permalink raw reply related [flat|nested] 51+ messages in thread
* [PATCH 00/10] sysctl: Remove sentinel elements from kernel dir
@ 2023-11-07 13:45 ` Joel Granados
0 siblings, 0 replies; 51+ messages in thread
From: Joel Granados via B4 Relay @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
From: Joel Granados <j.granados@samsung.com>
What?
These commits remove the sentinel element (last empty element) from the
sysctl arrays of all the files under the "kernel/" directory that use a
sysctl array for registration. The merging of the preparation patches
(in https://lore.kernel.org/all/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
to mainline allows us to just remove sentinel elements without changing
behavior (more info here [1]).
These commits are part of a bigger set (here
https://github.com/Joelgranados/linux/tree/tag/sysctl_remove_empty_elem_V5)
that remove the ctl_table sentinel. Make the review process easier by
chunking the commits into manageable pieces. Each chunk can be reviewed
separately without noise from parallel sets.
Sending the "kernel/*" chunk now that the "drivers/" has been mostly
reviewed [6]. After this and the "fs/*" are reviewed we only miss 2 more
chunks ("net/*" and miscellaneous) to complete the sentinel removal.
Hurray!!!
Why?
By removing the sysctl sentinel elements we avoid kernel bloat as
ctl_table arrays get moved out of kernel/sysctl.c into their own
respective subsystems. This move was started long ago to avoid merge
conflicts; the sentinel removal bit came after Mathew Wilcox suggested
it to avoid bloating the kernel by one element as arrays moved out. This
patchset will reduce the overall build time size of the kernel and run
time memory bloat by about ~64 bytes per declared ctl_table array. I
have consolidated some links that shed light on the history of this
effort [2].
Testing:
* Ran sysctl selftests (./tools/testing/selftests/sysctl/sysctl.sh)
* Ran this through 0-day with no errors or warnings
Size saving after this patchset:
* bloat-o-meter
- The "yesall" config saves 1984 bytes [4]
- The "tiny" config saves 771 bytes [5]
* If you want to know how many bytes are saved after all the chunks
are merged see [3]
Base commit:
tag: sysctl-6.7-rc1 (8b793bcda61f)
Comments/feedback greatly appreciated
Best
Joel
[1]
We are able to remove a sentinel table without behavioral change by
introducing a table_size argument in the same place where procname is
checked for NULL. The idea is for it to keep stopping when it hits
->procname == NULL, while the sentinel is still present. And when the
sentinel is removed, it will stop on the table_size. You can go to
(https://lore.kernel.org/all/20230809105006.1198165-1-j.granados@samsung.com/)
for more information.
[2]
Links Related to the ctl_table sentinel removal:
* E-mail threads that summarize the sentinel effort
https://lore.kernel.org/all/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/
https://lore.kernel.org/all/ZMFizKFkVxUFtSqa@bombadil.infradead.org/
* Replacing the register functions:
https://lore.kernel.org/all/20230302204612.782387-1-mcgrof@kernel.org/
https://lore.kernel.org/all/20230302202826.776286-1-mcgrof@kernel.org/
* E-mail threads discussing prposal
https://lore.kernel.org/all/20230321130908.6972-1-frank.li@vivo.com
https://lore.kernel.org/all/20220220060626.15885-1-tangmeng@uniontech.com
[3]
Size saving after removing all sentinels:
These are the bytes that we save after removing all the sentinels
(this plus all the other chunks). I included them to get an idea of
how much memory we are talking about.
* bloat-o-meter:
- The "yesall" configuration results save 9158 bytes
https://lore.kernel.org/all/20230621091000.424843-1-j.granados@samsung.com/
- The "tiny" config + CONFIG_SYSCTL save 1215 bytes
https://lore.kernel.org/all/20230809105006.1198165-1-j.granados@samsung.com/
* memory usage:
In memory savings are measured to be 7296 bytes. (here is how to
measure [7])
[4]
add/remove: 0/0 grow/shrink: 0/31 up/down: 0/-1984 (-1984)
Function old new delta
watchdog_sysctls 576 512 -64
watchdog_hardlockup_sysctl 128 64 -64
vm_table 1344 1280 -64
uts_kern_table 448 384 -64
usermodehelper_table 192 128 -64
user_table 832 768 -64
user_event_sysctls 128 64 -64
timer_sysctl 128 64 -64
signal_debug_table 128 64 -64
seccomp_sysctl_table 192 128 -64
sched_rt_sysctls 256 192 -64
sched_fair_sysctls 256 192 -64
sched_energy_aware_sysctls 128 64 -64
sched_dl_sysctls 192 128 -64
sched_core_sysctls 384 320 -64
sched_autogroup_sysctls 128 64 -64
printk_sysctls 512 448 -64
pid_ns_ctl_table_vm 128 64 -64
pid_ns_ctl_table 128 64 -64
latencytop_sysctl 128 64 -64
kprobe_sysctls 128 64 -64
kexec_core_sysctls 256 192 -64
kern_table 2560 2496 -64
kern_reboot_table 192 128 -64
kern_panic_table 192 128 -64
kern_exit_table 128 64 -64
kern_delayacct_table 128 64 -64
kern_acct_table 128 64 -64
hung_task_sysctls 448 384 -64
ftrace_sysctls 128 64 -64
bpf_syscall_table 192 128 -64
Total: Before=429912331, After=429910347, chg -0.00%
[5]
add/remove: 0/1 grow/shrink: 0/12 up/down: 0/-771 (-771)
Function old new delta
sched_core_sysctl_init 43 40 -3
vm_table 1024 960 -64
uts_kern_table 448 384 -64
usermodehelper_table 192 128 -64
user_table 576 512 -64
signal_debug_table 128 64 -64
sched_rt_sysctls 256 192 -64
sched_fair_sysctls 128 64 -64
sched_dl_sysctls 192 128 -64
sched_core_sysctls 64 - -64
kern_table 1792 1728 -64
kern_panic_table 128 64 -64
kern_exit_table 128 64 -64
Total: Before=1886645, After=1885874, chg -0.04%
[6]
https://lore.kernel.org/all/20231002-jag-sysctl_remove_empty_elem_drivers-v2-0-02dd0d46f71e@samsung.com
[7]
To measure the in memory savings apply this on top of this patchset.
"
diff --git a/fs/proc/proc_sysctl.c b/fs/proc/proc_sysctl.c
index c88854df0b62..e0073a627bac 100644
--- a/fs/proc/proc_sysctl.c
+++ b/fs/proc/proc_sysctl.c
@@ -976,6 +976,8 @@ static struct ctl_dir *new_dir(struct ctl_table_set *set,
table[0].procname = new_name;
table[0].mode = S_IFDIR|S_IRUGO|S_IXUGO;
init_header(&new->header, set->dir.header.root, set, node, table, 1);
+ // Counts additional sentinel used for each new dir.
+ printk("%ld sysctl saved mem kzalloc \n", sizeof(struct ctl_table));
return new;
}
@@ -1199,6 +1201,9 @@ static struct ctl_table_header *new_links(struct ctl_dir *dir, struct ctl_table_
link_name += len;
link++;
}
+ // Counts additional sentinel used for each new registration
+ //
+ printk("%ld sysctl saved mem kzalloc \n", sizeof(struct ctl_table));
init_header(links, dir->header.root, dir->header.set, node, link_table,
head->ctl_table_size);
links->nreg = nr_entries;
"
and then run the following bash script in the kernel:
accum=0
for n in $(dmesg | grep kzalloc | awk '{print $3}') ; do
echo $n
accum=$(calc "$accum + $n")
done
echo $accum
---
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
Joel Granados (10):
kernel misc: Remove the now superfluous sentinel elements from ctl_table array
umh: Remove the now superfluous sentinel elements from ctl_table array
ftrace: Remove the now superfluous sentinel elements from ctl_table array
timekeeping: Remove the now superfluous sentinel elements from ctl_table array
seccomp: Remove the now superfluous sentinel elements from ctl_table array
scheduler: Remove the now superfluous sentinel elements from ctl_table array
printk: Remove the now superfluous sentinel elements from ctl_table array
kprobes: Remove the now superfluous sentinel elements from ctl_table array
delayacct: Remove the now superfluous sentinel elements from ctl_table array
bpf: Remove the now superfluous sentinel elements from ctl_table array
kernel/acct.c | 1 -
kernel/bpf/syscall.c | 1 -
kernel/delayacct.c | 1 -
kernel/exit.c | 1 -
kernel/hung_task.c | 1 -
kernel/kexec_core.c | 1 -
kernel/kprobes.c | 1 -
kernel/latencytop.c | 1 -
kernel/panic.c | 1 -
kernel/pid_namespace.c | 1 -
kernel/pid_sysctl.h | 1 -
kernel/printk/sysctl.c | 1 -
kernel/reboot.c | 1 -
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 -
kernel/seccomp.c | 1 -
kernel/signal.c | 1 -
kernel/stackleak.c | 1 -
kernel/sysctl.c | 2 --
kernel/time/timer.c | 1 -
kernel/trace/ftrace.c | 1 -
kernel/trace/trace_events_user.c | 1 -
kernel/ucount.c | 3 +--
kernel/umh.c | 1 -
kernel/utsname_sysctl.c | 1 -
kernel/watchdog.c | 2 --
30 files changed, 1 insertion(+), 33 deletions(-)
---
base-commit: 8b793bcda61f6c3ed4f5b2ded7530ef6749580cb
change-id: 20231107-jag-sysctl_remove_empty_elem_kernel-7de90cfd0c0a
Best regards,
--
Joel Granados <j.granados@samsung.com>
_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec
^ permalink raw reply related [flat|nested] 51+ messages in thread
* [PATCH 01/10] kernel misc: Remove the now superfluous sentinel elements from ctl_table array
2023-11-07 13:45 ` Joel Granados
(?)
@ 2023-11-07 13:45 ` Joel Granados
-1 siblings, 0 replies; 51+ messages in thread
From: Joel Granados via B4 Relay @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
From: Joel Granados <j.granados@samsung.com>
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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
Remove the sentinel from ctl_table arrays. Reduce by one the values used
to compare the size of the adjusted arrays.
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
kernel/acct.c | 1 -
kernel/exit.c | 1 -
kernel/hung_task.c | 1 -
kernel/kexec_core.c | 1 -
kernel/latencytop.c | 1 -
kernel/panic.c | 1 -
kernel/pid_namespace.c | 1 -
kernel/pid_sysctl.h | 1 -
kernel/reboot.c | 1 -
kernel/signal.c | 1 -
kernel/stackleak.c | 1 -
kernel/sysctl.c | 2 --
kernel/ucount.c | 3 +--
kernel/utsname_sysctl.c | 1 -
kernel/watchdog.c | 2 --
15 files changed, 1 insertion(+), 18 deletions(-)
diff --git a/kernel/acct.c b/kernel/acct.c
index 1a9f929fe629..20d09f437262 100644
--- a/kernel/acct.c
+++ b/kernel/acct.c
@@ -84,7 +84,6 @@ static struct ctl_table kern_acct_table[] = {
.mode = 0644,
.proc_handler = proc_dointvec,
},
- { }
};
static __init int kernel_acct_sysctls_init(void)
diff --git a/kernel/exit.c b/kernel/exit.c
index edb50b4c9972..aebe885e5b2f 100644
--- a/kernel/exit.c
+++ b/kernel/exit.c
@@ -90,7 +90,6 @@ static struct ctl_table kern_exit_table[] = {
.mode = 0644,
.proc_handler = proc_douintvec,
},
- { }
};
static __init int kernel_exit_sysctls_init(void)
diff --git a/kernel/hung_task.c b/kernel/hung_task.c
index 9a24574988d2..a81cb511d954 100644
--- a/kernel/hung_task.c
+++ b/kernel/hung_task.c
@@ -313,7 +313,6 @@ static struct ctl_table hung_task_sysctls[] = {
.proc_handler = proc_dointvec_minmax,
.extra1 = SYSCTL_NEG_ONE,
},
- {}
};
static void __init hung_task_sysctl_init(void)
diff --git a/kernel/kexec_core.c b/kernel/kexec_core.c
index 9dc728982d79..d7e883864f82 100644
--- a/kernel/kexec_core.c
+++ b/kernel/kexec_core.c
@@ -1005,7 +1005,6 @@ static struct ctl_table kexec_core_sysctls[] = {
.mode = 0644,
.proc_handler = kexec_limit_handler,
},
- { }
};
static int __init kexec_core_sysctl_init(void)
diff --git a/kernel/latencytop.c b/kernel/latencytop.c
index 781249098cb6..84c53285f499 100644
--- a/kernel/latencytop.c
+++ b/kernel/latencytop.c
@@ -85,7 +85,6 @@ static struct ctl_table latencytop_sysctl[] = {
.mode = 0644,
.proc_handler = sysctl_latencytop,
},
- {}
};
#endif
diff --git a/kernel/panic.c b/kernel/panic.c
index ffa037fa777d..5ae3e6ba88c9 100644
--- a/kernel/panic.c
+++ b/kernel/panic.c
@@ -99,7 +99,6 @@ static struct ctl_table kern_panic_table[] = {
.mode = 0644,
.proc_handler = proc_douintvec,
},
- { }
};
static __init int kernel_panic_sysctls_init(void)
diff --git a/kernel/pid_namespace.c b/kernel/pid_namespace.c
index 619972c78774..3978b98f6e6b 100644
--- a/kernel/pid_namespace.c
+++ b/kernel/pid_namespace.c
@@ -312,7 +312,6 @@ static struct ctl_table pid_ns_ctl_table[] = {
.extra1 = SYSCTL_ZERO,
.extra2 = &pid_max,
},
- { }
};
#endif /* CONFIG_CHECKPOINT_RESTORE */
diff --git a/kernel/pid_sysctl.h b/kernel/pid_sysctl.h
index 2ee41a3a1dfd..fe9fb991dc42 100644
--- a/kernel/pid_sysctl.h
+++ b/kernel/pid_sysctl.h
@@ -41,7 +41,6 @@ static struct ctl_table pid_ns_ctl_table_vm[] = {
.extra1 = SYSCTL_ZERO,
.extra2 = SYSCTL_TWO,
},
- { }
};
static inline void register_pid_ns_sysctl_table_vm(void)
{
diff --git a/kernel/reboot.c b/kernel/reboot.c
index 3bba88c7ffc6..a7828365d2aa 100644
--- a/kernel/reboot.c
+++ b/kernel/reboot.c
@@ -1272,7 +1272,6 @@ static struct ctl_table kern_reboot_table[] = {
.mode = 0644,
.proc_handler = proc_dointvec,
},
- { }
};
static void __init kernel_reboot_sysctls_init(void)
diff --git a/kernel/signal.c b/kernel/signal.c
index 09019017d669..89476890e3a1 100644
--- a/kernel/signal.c
+++ b/kernel/signal.c
@@ -4800,7 +4800,6 @@ static struct ctl_table signal_debug_table[] = {
.proc_handler = proc_dointvec
},
#endif
- { }
};
static int __init init_signal_sysctls(void)
diff --git a/kernel/stackleak.c b/kernel/stackleak.c
index 34c9d81eea94..d099f3affcf1 100644
--- a/kernel/stackleak.c
+++ b/kernel/stackleak.c
@@ -54,7 +54,6 @@ static struct ctl_table stackleak_sysctls[] = {
.extra1 = SYSCTL_ZERO,
.extra2 = SYSCTL_ONE,
},
- {}
};
static int __init stackleak_sysctls_init(void)
diff --git a/kernel/sysctl.c b/kernel/sysctl.c
index 354a2d294f52..64f3613c224a 100644
--- a/kernel/sysctl.c
+++ b/kernel/sysctl.c
@@ -2043,7 +2043,6 @@ static struct ctl_table kern_table[] = {
.extra2 = SYSCTL_INT_MAX,
},
#endif
- { }
};
static struct ctl_table vm_table[] = {
@@ -2249,7 +2248,6 @@ static struct ctl_table vm_table[] = {
.extra2 = (void *)&mmap_rnd_compat_bits_max,
},
#endif
- { }
};
int __init sysctl_init_bases(void)
diff --git a/kernel/ucount.c b/kernel/ucount.c
index 4aa6166cb856..e196da0204dc 100644
--- a/kernel/ucount.c
+++ b/kernel/ucount.c
@@ -87,7 +87,6 @@ static struct ctl_table user_table[] = {
UCOUNT_ENTRY("max_fanotify_groups"),
UCOUNT_ENTRY("max_fanotify_marks"),
#endif
- { }
};
#endif /* CONFIG_SYSCTL */
@@ -96,7 +95,7 @@ bool setup_userns_sysctls(struct user_namespace *ns)
#ifdef CONFIG_SYSCTL
struct ctl_table *tbl;
- BUILD_BUG_ON(ARRAY_SIZE(user_table) != UCOUNT_COUNTS + 1);
+ BUILD_BUG_ON(ARRAY_SIZE(user_table) != UCOUNT_COUNTS);
setup_sysctl_set(&ns->set, &set_root, set_is_seen);
tbl = kmemdup(user_table, sizeof(user_table), GFP_KERNEL);
if (tbl) {
diff --git a/kernel/utsname_sysctl.c b/kernel/utsname_sysctl.c
index 019e3a1566cf..76a772072557 100644
--- a/kernel/utsname_sysctl.c
+++ b/kernel/utsname_sysctl.c
@@ -120,7 +120,6 @@ static struct ctl_table uts_kern_table[] = {
.proc_handler = proc_do_uts_string,
.poll = &domainname_poll,
},
- {}
};
#ifdef CONFIG_PROC_SYSCTL
diff --git a/kernel/watchdog.c b/kernel/watchdog.c
index 5cd6d4e26915..0f546d17c544 100644
--- a/kernel/watchdog.c
+++ b/kernel/watchdog.c
@@ -924,7 +924,6 @@ static struct ctl_table watchdog_sysctls[] = {
},
#endif /* CONFIG_SMP */
#endif
- {}
};
static struct ctl_table watchdog_hardlockup_sysctl[] = {
@@ -937,7 +936,6 @@ static struct ctl_table watchdog_hardlockup_sysctl[] = {
.extra1 = SYSCTL_ZERO,
.extra2 = SYSCTL_ONE,
},
- {}
};
static void __init watchdog_sysctl_init(void)
--
2.30.2
^ permalink raw reply related [flat|nested] 51+ messages in thread
* [PATCH 01/10] kernel misc: Remove the now superfluous sentinel elements from ctl_table array
@ 2023-11-07 13:45 ` Joel Granados
0 siblings, 0 replies; 51+ messages in thread
From: Joel Granados @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
Remove the sentinel from ctl_table arrays. Reduce by one the values used
to compare the size of the adjusted arrays.
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
kernel/acct.c | 1 -
kernel/exit.c | 1 -
kernel/hung_task.c | 1 -
kernel/kexec_core.c | 1 -
kernel/latencytop.c | 1 -
kernel/panic.c | 1 -
kernel/pid_namespace.c | 1 -
kernel/pid_sysctl.h | 1 -
kernel/reboot.c | 1 -
kernel/signal.c | 1 -
kernel/stackleak.c | 1 -
kernel/sysctl.c | 2 --
kernel/ucount.c | 3 +--
kernel/utsname_sysctl.c | 1 -
kernel/watchdog.c | 2 --
15 files changed, 1 insertion(+), 18 deletions(-)
diff --git a/kernel/acct.c b/kernel/acct.c
index 1a9f929fe629..20d09f437262 100644
--- a/kernel/acct.c
+++ b/kernel/acct.c
@@ -84,7 +84,6 @@ static struct ctl_table kern_acct_table[] = {
.mode = 0644,
.proc_handler = proc_dointvec,
},
- { }
};
static __init int kernel_acct_sysctls_init(void)
diff --git a/kernel/exit.c b/kernel/exit.c
index edb50b4c9972..aebe885e5b2f 100644
--- a/kernel/exit.c
+++ b/kernel/exit.c
@@ -90,7 +90,6 @@ static struct ctl_table kern_exit_table[] = {
.mode = 0644,
.proc_handler = proc_douintvec,
},
- { }
};
static __init int kernel_exit_sysctls_init(void)
diff --git a/kernel/hung_task.c b/kernel/hung_task.c
index 9a24574988d2..a81cb511d954 100644
--- a/kernel/hung_task.c
+++ b/kernel/hung_task.c
@@ -313,7 +313,6 @@ static struct ctl_table hung_task_sysctls[] = {
.proc_handler = proc_dointvec_minmax,
.extra1 = SYSCTL_NEG_ONE,
},
- {}
};
static void __init hung_task_sysctl_init(void)
diff --git a/kernel/kexec_core.c b/kernel/kexec_core.c
index 9dc728982d79..d7e883864f82 100644
--- a/kernel/kexec_core.c
+++ b/kernel/kexec_core.c
@@ -1005,7 +1005,6 @@ static struct ctl_table kexec_core_sysctls[] = {
.mode = 0644,
.proc_handler = kexec_limit_handler,
},
- { }
};
static int __init kexec_core_sysctl_init(void)
diff --git a/kernel/latencytop.c b/kernel/latencytop.c
index 781249098cb6..84c53285f499 100644
--- a/kernel/latencytop.c
+++ b/kernel/latencytop.c
@@ -85,7 +85,6 @@ static struct ctl_table latencytop_sysctl[] = {
.mode = 0644,
.proc_handler = sysctl_latencytop,
},
- {}
};
#endif
diff --git a/kernel/panic.c b/kernel/panic.c
index ffa037fa777d..5ae3e6ba88c9 100644
--- a/kernel/panic.c
+++ b/kernel/panic.c
@@ -99,7 +99,6 @@ static struct ctl_table kern_panic_table[] = {
.mode = 0644,
.proc_handler = proc_douintvec,
},
- { }
};
static __init int kernel_panic_sysctls_init(void)
diff --git a/kernel/pid_namespace.c b/kernel/pid_namespace.c
index 619972c78774..3978b98f6e6b 100644
--- a/kernel/pid_namespace.c
+++ b/kernel/pid_namespace.c
@@ -312,7 +312,6 @@ static struct ctl_table pid_ns_ctl_table[] = {
.extra1 = SYSCTL_ZERO,
.extra2 = &pid_max,
},
- { }
};
#endif /* CONFIG_CHECKPOINT_RESTORE */
diff --git a/kernel/pid_sysctl.h b/kernel/pid_sysctl.h
index 2ee41a3a1dfd..fe9fb991dc42 100644
--- a/kernel/pid_sysctl.h
+++ b/kernel/pid_sysctl.h
@@ -41,7 +41,6 @@ static struct ctl_table pid_ns_ctl_table_vm[] = {
.extra1 = SYSCTL_ZERO,
.extra2 = SYSCTL_TWO,
},
- { }
};
static inline void register_pid_ns_sysctl_table_vm(void)
{
diff --git a/kernel/reboot.c b/kernel/reboot.c
index 3bba88c7ffc6..a7828365d2aa 100644
--- a/kernel/reboot.c
+++ b/kernel/reboot.c
@@ -1272,7 +1272,6 @@ static struct ctl_table kern_reboot_table[] = {
.mode = 0644,
.proc_handler = proc_dointvec,
},
- { }
};
static void __init kernel_reboot_sysctls_init(void)
diff --git a/kernel/signal.c b/kernel/signal.c
index 09019017d669..89476890e3a1 100644
--- a/kernel/signal.c
+++ b/kernel/signal.c
@@ -4800,7 +4800,6 @@ static struct ctl_table signal_debug_table[] = {
.proc_handler = proc_dointvec
},
#endif
- { }
};
static int __init init_signal_sysctls(void)
diff --git a/kernel/stackleak.c b/kernel/stackleak.c
index 34c9d81eea94..d099f3affcf1 100644
--- a/kernel/stackleak.c
+++ b/kernel/stackleak.c
@@ -54,7 +54,6 @@ static struct ctl_table stackleak_sysctls[] = {
.extra1 = SYSCTL_ZERO,
.extra2 = SYSCTL_ONE,
},
- {}
};
static int __init stackleak_sysctls_init(void)
diff --git a/kernel/sysctl.c b/kernel/sysctl.c
index 354a2d294f52..64f3613c224a 100644
--- a/kernel/sysctl.c
+++ b/kernel/sysctl.c
@@ -2043,7 +2043,6 @@ static struct ctl_table kern_table[] = {
.extra2 = SYSCTL_INT_MAX,
},
#endif
- { }
};
static struct ctl_table vm_table[] = {
@@ -2249,7 +2248,6 @@ static struct ctl_table vm_table[] = {
.extra2 = (void *)&mmap_rnd_compat_bits_max,
},
#endif
- { }
};
int __init sysctl_init_bases(void)
diff --git a/kernel/ucount.c b/kernel/ucount.c
index 4aa6166cb856..e196da0204dc 100644
--- a/kernel/ucount.c
+++ b/kernel/ucount.c
@@ -87,7 +87,6 @@ static struct ctl_table user_table[] = {
UCOUNT_ENTRY("max_fanotify_groups"),
UCOUNT_ENTRY("max_fanotify_marks"),
#endif
- { }
};
#endif /* CONFIG_SYSCTL */
@@ -96,7 +95,7 @@ bool setup_userns_sysctls(struct user_namespace *ns)
#ifdef CONFIG_SYSCTL
struct ctl_table *tbl;
- BUILD_BUG_ON(ARRAY_SIZE(user_table) != UCOUNT_COUNTS + 1);
+ BUILD_BUG_ON(ARRAY_SIZE(user_table) != UCOUNT_COUNTS);
setup_sysctl_set(&ns->set, &set_root, set_is_seen);
tbl = kmemdup(user_table, sizeof(user_table), GFP_KERNEL);
if (tbl) {
diff --git a/kernel/utsname_sysctl.c b/kernel/utsname_sysctl.c
index 019e3a1566cf..76a772072557 100644
--- a/kernel/utsname_sysctl.c
+++ b/kernel/utsname_sysctl.c
@@ -120,7 +120,6 @@ static struct ctl_table uts_kern_table[] = {
.proc_handler = proc_do_uts_string,
.poll = &domainname_poll,
},
- {}
};
#ifdef CONFIG_PROC_SYSCTL
diff --git a/kernel/watchdog.c b/kernel/watchdog.c
index 5cd6d4e26915..0f546d17c544 100644
--- a/kernel/watchdog.c
+++ b/kernel/watchdog.c
@@ -924,7 +924,6 @@ static struct ctl_table watchdog_sysctls[] = {
},
#endif /* CONFIG_SMP */
#endif
- {}
};
static struct ctl_table watchdog_hardlockup_sysctl[] = {
@@ -937,7 +936,6 @@ static struct ctl_table watchdog_hardlockup_sysctl[] = {
.extra1 = SYSCTL_ZERO,
.extra2 = SYSCTL_ONE,
},
- {}
};
static void __init watchdog_sysctl_init(void)
--
2.30.2
^ permalink raw reply related [flat|nested] 51+ messages in thread
* [PATCH 01/10] kernel misc: Remove the now superfluous sentinel elements from ctl_table array
@ 2023-11-07 13:45 ` Joel Granados
0 siblings, 0 replies; 51+ messages in thread
From: Joel Granados via B4 Relay @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
From: Joel Granados <j.granados@samsung.com>
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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
Remove the sentinel from ctl_table arrays. Reduce by one the values used
to compare the size of the adjusted arrays.
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
kernel/acct.c | 1 -
kernel/exit.c | 1 -
kernel/hung_task.c | 1 -
kernel/kexec_core.c | 1 -
kernel/latencytop.c | 1 -
kernel/panic.c | 1 -
kernel/pid_namespace.c | 1 -
kernel/pid_sysctl.h | 1 -
kernel/reboot.c | 1 -
kernel/signal.c | 1 -
kernel/stackleak.c | 1 -
kernel/sysctl.c | 2 --
kernel/ucount.c | 3 +--
kernel/utsname_sysctl.c | 1 -
kernel/watchdog.c | 2 --
15 files changed, 1 insertion(+), 18 deletions(-)
diff --git a/kernel/acct.c b/kernel/acct.c
index 1a9f929fe629..20d09f437262 100644
--- a/kernel/acct.c
+++ b/kernel/acct.c
@@ -84,7 +84,6 @@ static struct ctl_table kern_acct_table[] = {
.mode = 0644,
.proc_handler = proc_dointvec,
},
- { }
};
static __init int kernel_acct_sysctls_init(void)
diff --git a/kernel/exit.c b/kernel/exit.c
index edb50b4c9972..aebe885e5b2f 100644
--- a/kernel/exit.c
+++ b/kernel/exit.c
@@ -90,7 +90,6 @@ static struct ctl_table kern_exit_table[] = {
.mode = 0644,
.proc_handler = proc_douintvec,
},
- { }
};
static __init int kernel_exit_sysctls_init(void)
diff --git a/kernel/hung_task.c b/kernel/hung_task.c
index 9a24574988d2..a81cb511d954 100644
--- a/kernel/hung_task.c
+++ b/kernel/hung_task.c
@@ -313,7 +313,6 @@ static struct ctl_table hung_task_sysctls[] = {
.proc_handler = proc_dointvec_minmax,
.extra1 = SYSCTL_NEG_ONE,
},
- {}
};
static void __init hung_task_sysctl_init(void)
diff --git a/kernel/kexec_core.c b/kernel/kexec_core.c
index 9dc728982d79..d7e883864f82 100644
--- a/kernel/kexec_core.c
+++ b/kernel/kexec_core.c
@@ -1005,7 +1005,6 @@ static struct ctl_table kexec_core_sysctls[] = {
.mode = 0644,
.proc_handler = kexec_limit_handler,
},
- { }
};
static int __init kexec_core_sysctl_init(void)
diff --git a/kernel/latencytop.c b/kernel/latencytop.c
index 781249098cb6..84c53285f499 100644
--- a/kernel/latencytop.c
+++ b/kernel/latencytop.c
@@ -85,7 +85,6 @@ static struct ctl_table latencytop_sysctl[] = {
.mode = 0644,
.proc_handler = sysctl_latencytop,
},
- {}
};
#endif
diff --git a/kernel/panic.c b/kernel/panic.c
index ffa037fa777d..5ae3e6ba88c9 100644
--- a/kernel/panic.c
+++ b/kernel/panic.c
@@ -99,7 +99,6 @@ static struct ctl_table kern_panic_table[] = {
.mode = 0644,
.proc_handler = proc_douintvec,
},
- { }
};
static __init int kernel_panic_sysctls_init(void)
diff --git a/kernel/pid_namespace.c b/kernel/pid_namespace.c
index 619972c78774..3978b98f6e6b 100644
--- a/kernel/pid_namespace.c
+++ b/kernel/pid_namespace.c
@@ -312,7 +312,6 @@ static struct ctl_table pid_ns_ctl_table[] = {
.extra1 = SYSCTL_ZERO,
.extra2 = &pid_max,
},
- { }
};
#endif /* CONFIG_CHECKPOINT_RESTORE */
diff --git a/kernel/pid_sysctl.h b/kernel/pid_sysctl.h
index 2ee41a3a1dfd..fe9fb991dc42 100644
--- a/kernel/pid_sysctl.h
+++ b/kernel/pid_sysctl.h
@@ -41,7 +41,6 @@ static struct ctl_table pid_ns_ctl_table_vm[] = {
.extra1 = SYSCTL_ZERO,
.extra2 = SYSCTL_TWO,
},
- { }
};
static inline void register_pid_ns_sysctl_table_vm(void)
{
diff --git a/kernel/reboot.c b/kernel/reboot.c
index 3bba88c7ffc6..a7828365d2aa 100644
--- a/kernel/reboot.c
+++ b/kernel/reboot.c
@@ -1272,7 +1272,6 @@ static struct ctl_table kern_reboot_table[] = {
.mode = 0644,
.proc_handler = proc_dointvec,
},
- { }
};
static void __init kernel_reboot_sysctls_init(void)
diff --git a/kernel/signal.c b/kernel/signal.c
index 09019017d669..89476890e3a1 100644
--- a/kernel/signal.c
+++ b/kernel/signal.c
@@ -4800,7 +4800,6 @@ static struct ctl_table signal_debug_table[] = {
.proc_handler = proc_dointvec
},
#endif
- { }
};
static int __init init_signal_sysctls(void)
diff --git a/kernel/stackleak.c b/kernel/stackleak.c
index 34c9d81eea94..d099f3affcf1 100644
--- a/kernel/stackleak.c
+++ b/kernel/stackleak.c
@@ -54,7 +54,6 @@ static struct ctl_table stackleak_sysctls[] = {
.extra1 = SYSCTL_ZERO,
.extra2 = SYSCTL_ONE,
},
- {}
};
static int __init stackleak_sysctls_init(void)
diff --git a/kernel/sysctl.c b/kernel/sysctl.c
index 354a2d294f52..64f3613c224a 100644
--- a/kernel/sysctl.c
+++ b/kernel/sysctl.c
@@ -2043,7 +2043,6 @@ static struct ctl_table kern_table[] = {
.extra2 = SYSCTL_INT_MAX,
},
#endif
- { }
};
static struct ctl_table vm_table[] = {
@@ -2249,7 +2248,6 @@ static struct ctl_table vm_table[] = {
.extra2 = (void *)&mmap_rnd_compat_bits_max,
},
#endif
- { }
};
int __init sysctl_init_bases(void)
diff --git a/kernel/ucount.c b/kernel/ucount.c
index 4aa6166cb856..e196da0204dc 100644
--- a/kernel/ucount.c
+++ b/kernel/ucount.c
@@ -87,7 +87,6 @@ static struct ctl_table user_table[] = {
UCOUNT_ENTRY("max_fanotify_groups"),
UCOUNT_ENTRY("max_fanotify_marks"),
#endif
- { }
};
#endif /* CONFIG_SYSCTL */
@@ -96,7 +95,7 @@ bool setup_userns_sysctls(struct user_namespace *ns)
#ifdef CONFIG_SYSCTL
struct ctl_table *tbl;
- BUILD_BUG_ON(ARRAY_SIZE(user_table) != UCOUNT_COUNTS + 1);
+ BUILD_BUG_ON(ARRAY_SIZE(user_table) != UCOUNT_COUNTS);
setup_sysctl_set(&ns->set, &set_root, set_is_seen);
tbl = kmemdup(user_table, sizeof(user_table), GFP_KERNEL);
if (tbl) {
diff --git a/kernel/utsname_sysctl.c b/kernel/utsname_sysctl.c
index 019e3a1566cf..76a772072557 100644
--- a/kernel/utsname_sysctl.c
+++ b/kernel/utsname_sysctl.c
@@ -120,7 +120,6 @@ static struct ctl_table uts_kern_table[] = {
.proc_handler = proc_do_uts_string,
.poll = &domainname_poll,
},
- {}
};
#ifdef CONFIG_PROC_SYSCTL
diff --git a/kernel/watchdog.c b/kernel/watchdog.c
index 5cd6d4e26915..0f546d17c544 100644
--- a/kernel/watchdog.c
+++ b/kernel/watchdog.c
@@ -924,7 +924,6 @@ static struct ctl_table watchdog_sysctls[] = {
},
#endif /* CONFIG_SMP */
#endif
- {}
};
static struct ctl_table watchdog_hardlockup_sysctl[] = {
@@ -937,7 +936,6 @@ static struct ctl_table watchdog_hardlockup_sysctl[] = {
.extra1 = SYSCTL_ZERO,
.extra2 = SYSCTL_ONE,
},
- {}
};
static void __init watchdog_sysctl_init(void)
--
2.30.2
_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec
^ permalink raw reply related [flat|nested] 51+ messages in thread
* [PATCH 02/10] umh: Remove the now superfluous sentinel elements from ctl_table array
2023-11-07 13:45 ` Joel Granados
(?)
@ 2023-11-07 13:45 ` Joel Granados
-1 siblings, 0 replies; 51+ messages in thread
From: Joel Granados via B4 Relay @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
From: Joel Granados <j.granados@samsung.com>
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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
Remove sentinel element from usermodehelper_table
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
kernel/umh.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/kernel/umh.c b/kernel/umh.c
index 1b13c5d34624..598b3ffe1522 100644
--- a/kernel/umh.c
+++ b/kernel/umh.c
@@ -560,7 +560,6 @@ static struct ctl_table usermodehelper_table[] = {
.mode = 0600,
.proc_handler = proc_cap_handler,
},
- { }
};
static int __init init_umh_sysctls(void)
--
2.30.2
^ permalink raw reply related [flat|nested] 51+ messages in thread
* [PATCH 02/10] umh: Remove the now superfluous sentinel elements from ctl_table array
@ 2023-11-07 13:45 ` Joel Granados
0 siblings, 0 replies; 51+ messages in thread
From: Joel Granados @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
Remove sentinel element from usermodehelper_table
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
kernel/umh.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/kernel/umh.c b/kernel/umh.c
index 1b13c5d34624..598b3ffe1522 100644
--- a/kernel/umh.c
+++ b/kernel/umh.c
@@ -560,7 +560,6 @@ static struct ctl_table usermodehelper_table[] = {
.mode = 0600,
.proc_handler = proc_cap_handler,
},
- { }
};
static int __init init_umh_sysctls(void)
--
2.30.2
^ permalink raw reply related [flat|nested] 51+ messages in thread
* [PATCH 02/10] umh: Remove the now superfluous sentinel elements from ctl_table array
@ 2023-11-07 13:45 ` Joel Granados
0 siblings, 0 replies; 51+ messages in thread
From: Joel Granados via B4 Relay @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
From: Joel Granados <j.granados@samsung.com>
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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
Remove sentinel element from usermodehelper_table
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
kernel/umh.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/kernel/umh.c b/kernel/umh.c
index 1b13c5d34624..598b3ffe1522 100644
--- a/kernel/umh.c
+++ b/kernel/umh.c
@@ -560,7 +560,6 @@ static struct ctl_table usermodehelper_table[] = {
.mode = 0600,
.proc_handler = proc_cap_handler,
},
- { }
};
static int __init init_umh_sysctls(void)
--
2.30.2
_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec
^ permalink raw reply related [flat|nested] 51+ messages in thread
* [PATCH 03/10] ftrace: Remove the now superfluous sentinel elements from ctl_table array
2023-11-07 13:45 ` Joel Granados
(?)
@ 2023-11-07 13:45 ` Joel Granados
-1 siblings, 0 replies; 51+ messages in thread
From: Joel Granados via B4 Relay @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
From: Joel Granados <j.granados@samsung.com>
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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
Remove sentinel elements from ftrace_sysctls and user_event_sysctls
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
kernel/trace/ftrace.c | 1 -
kernel/trace/trace_events_user.c | 1 -
2 files changed, 2 deletions(-)
diff --git a/kernel/trace/ftrace.c b/kernel/trace/ftrace.c
index 8de8bec5f366..fd40d02a23c7 100644
--- a/kernel/trace/ftrace.c
+++ b/kernel/trace/ftrace.c
@@ -8264,7 +8264,6 @@ static struct ctl_table ftrace_sysctls[] = {
.mode = 0644,
.proc_handler = ftrace_enable_sysctl,
},
- {}
};
static int __init ftrace_sysctl_init(void)
diff --git a/kernel/trace/trace_events_user.c b/kernel/trace/trace_events_user.c
index 6f046650e527..bef1bdc62acf 100644
--- a/kernel/trace/trace_events_user.c
+++ b/kernel/trace/trace_events_user.c
@@ -2694,7 +2694,6 @@ static struct ctl_table user_event_sysctls[] = {
.mode = 0644,
.proc_handler = set_max_user_events_sysctl,
},
- {}
};
static int __init trace_events_user_init(void)
--
2.30.2
^ permalink raw reply related [flat|nested] 51+ messages in thread
* [PATCH 03/10] ftrace: Remove the now superfluous sentinel elements from ctl_table array
@ 2023-11-07 13:45 ` Joel Granados
0 siblings, 0 replies; 51+ messages in thread
From: Joel Granados @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
Remove sentinel elements from ftrace_sysctls and user_event_sysctls
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
kernel/trace/ftrace.c | 1 -
kernel/trace/trace_events_user.c | 1 -
2 files changed, 2 deletions(-)
diff --git a/kernel/trace/ftrace.c b/kernel/trace/ftrace.c
index 8de8bec5f366..fd40d02a23c7 100644
--- a/kernel/trace/ftrace.c
+++ b/kernel/trace/ftrace.c
@@ -8264,7 +8264,6 @@ static struct ctl_table ftrace_sysctls[] = {
.mode = 0644,
.proc_handler = ftrace_enable_sysctl,
},
- {}
};
static int __init ftrace_sysctl_init(void)
diff --git a/kernel/trace/trace_events_user.c b/kernel/trace/trace_events_user.c
index 6f046650e527..bef1bdc62acf 100644
--- a/kernel/trace/trace_events_user.c
+++ b/kernel/trace/trace_events_user.c
@@ -2694,7 +2694,6 @@ static struct ctl_table user_event_sysctls[] = {
.mode = 0644,
.proc_handler = set_max_user_events_sysctl,
},
- {}
};
static int __init trace_events_user_init(void)
--
2.30.2
^ permalink raw reply related [flat|nested] 51+ messages in thread
* [PATCH 03/10] ftrace: Remove the now superfluous sentinel elements from ctl_table array
@ 2023-11-07 13:45 ` Joel Granados
0 siblings, 0 replies; 51+ messages in thread
From: Joel Granados via B4 Relay @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
From: Joel Granados <j.granados@samsung.com>
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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
Remove sentinel elements from ftrace_sysctls and user_event_sysctls
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
kernel/trace/ftrace.c | 1 -
kernel/trace/trace_events_user.c | 1 -
2 files changed, 2 deletions(-)
diff --git a/kernel/trace/ftrace.c b/kernel/trace/ftrace.c
index 8de8bec5f366..fd40d02a23c7 100644
--- a/kernel/trace/ftrace.c
+++ b/kernel/trace/ftrace.c
@@ -8264,7 +8264,6 @@ static struct ctl_table ftrace_sysctls[] = {
.mode = 0644,
.proc_handler = ftrace_enable_sysctl,
},
- {}
};
static int __init ftrace_sysctl_init(void)
diff --git a/kernel/trace/trace_events_user.c b/kernel/trace/trace_events_user.c
index 6f046650e527..bef1bdc62acf 100644
--- a/kernel/trace/trace_events_user.c
+++ b/kernel/trace/trace_events_user.c
@@ -2694,7 +2694,6 @@ static struct ctl_table user_event_sysctls[] = {
.mode = 0644,
.proc_handler = set_max_user_events_sysctl,
},
- {}
};
static int __init trace_events_user_init(void)
--
2.30.2
_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec
^ permalink raw reply related [flat|nested] 51+ messages in thread
* [PATCH 04/10] timekeeping: Remove the now superfluous sentinel elements from ctl_table array
2023-11-07 13:45 ` Joel Granados
(?)
@ 2023-11-07 13:45 ` Joel Granados
-1 siblings, 0 replies; 51+ messages in thread
From: Joel Granados via B4 Relay @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
From: Joel Granados <j.granados@samsung.com>
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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
Remove sentinel element from time_sysctl
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
kernel/time/timer.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/kernel/time/timer.c b/kernel/time/timer.c
index 63a8ce7177dd..475826ad78df 100644
--- a/kernel/time/timer.c
+++ b/kernel/time/timer.c
@@ -260,7 +260,6 @@ static struct ctl_table timer_sysctl[] = {
.extra1 = SYSCTL_ZERO,
.extra2 = SYSCTL_ONE,
},
- {}
};
static int __init timer_sysctl_init(void)
--
2.30.2
^ permalink raw reply related [flat|nested] 51+ messages in thread
* [PATCH 04/10] timekeeping: Remove the now superfluous sentinel elements from ctl_table array
@ 2023-11-07 13:45 ` Joel Granados
0 siblings, 0 replies; 51+ messages in thread
From: Joel Granados @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
Remove sentinel element from time_sysctl
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
kernel/time/timer.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/kernel/time/timer.c b/kernel/time/timer.c
index 63a8ce7177dd..475826ad78df 100644
--- a/kernel/time/timer.c
+++ b/kernel/time/timer.c
@@ -260,7 +260,6 @@ static struct ctl_table timer_sysctl[] = {
.extra1 = SYSCTL_ZERO,
.extra2 = SYSCTL_ONE,
},
- {}
};
static int __init timer_sysctl_init(void)
--
2.30.2
^ permalink raw reply related [flat|nested] 51+ messages in thread
* [PATCH 04/10] timekeeping: Remove the now superfluous sentinel elements from ctl_table array
@ 2023-11-07 13:45 ` Joel Granados
0 siblings, 0 replies; 51+ messages in thread
From: Joel Granados via B4 Relay @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
From: Joel Granados <j.granados@samsung.com>
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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
Remove sentinel element from time_sysctl
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
kernel/time/timer.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/kernel/time/timer.c b/kernel/time/timer.c
index 63a8ce7177dd..475826ad78df 100644
--- a/kernel/time/timer.c
+++ b/kernel/time/timer.c
@@ -260,7 +260,6 @@ static struct ctl_table timer_sysctl[] = {
.extra1 = SYSCTL_ZERO,
.extra2 = SYSCTL_ONE,
},
- {}
};
static int __init timer_sysctl_init(void)
--
2.30.2
_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec
^ permalink raw reply related [flat|nested] 51+ messages in thread
* [PATCH 05/10] seccomp: Remove the now superfluous sentinel elements from ctl_table array
2023-11-07 13:45 ` Joel Granados
(?)
@ 2023-11-07 13:45 ` Joel Granados
-1 siblings, 0 replies; 51+ messages in thread
From: Joel Granados via B4 Relay @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
From: Joel Granados <j.granados@samsung.com>
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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
Remove sentinel element from seccomp_sysctl_table.
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
kernel/seccomp.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/kernel/seccomp.c b/kernel/seccomp.c
index 255999ba9190..b727b4351c1b 100644
--- a/kernel/seccomp.c
+++ b/kernel/seccomp.c
@@ -2445,7 +2445,6 @@ static struct ctl_table seccomp_sysctl_table[] = {
.mode = 0644,
.proc_handler = seccomp_actions_logged_handler,
},
- { }
};
static int __init seccomp_sysctl_init(void)
--
2.30.2
^ permalink raw reply related [flat|nested] 51+ messages in thread
* [PATCH 05/10] seccomp: Remove the now superfluous sentinel elements from ctl_table array
@ 2023-11-07 13:45 ` Joel Granados
0 siblings, 0 replies; 51+ messages in thread
From: Joel Granados @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
Remove sentinel element from seccomp_sysctl_table.
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
kernel/seccomp.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/kernel/seccomp.c b/kernel/seccomp.c
index 255999ba9190..b727b4351c1b 100644
--- a/kernel/seccomp.c
+++ b/kernel/seccomp.c
@@ -2445,7 +2445,6 @@ static struct ctl_table seccomp_sysctl_table[] = {
.mode = 0644,
.proc_handler = seccomp_actions_logged_handler,
},
- { }
};
static int __init seccomp_sysctl_init(void)
--
2.30.2
^ permalink raw reply related [flat|nested] 51+ messages in thread
* [PATCH 05/10] seccomp: Remove the now superfluous sentinel elements from ctl_table array
@ 2023-11-07 13:45 ` Joel Granados
0 siblings, 0 replies; 51+ messages in thread
From: Joel Granados via B4 Relay @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
From: Joel Granados <j.granados@samsung.com>
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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
Remove sentinel element from seccomp_sysctl_table.
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
kernel/seccomp.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/kernel/seccomp.c b/kernel/seccomp.c
index 255999ba9190..b727b4351c1b 100644
--- a/kernel/seccomp.c
+++ b/kernel/seccomp.c
@@ -2445,7 +2445,6 @@ static struct ctl_table seccomp_sysctl_table[] = {
.mode = 0644,
.proc_handler = seccomp_actions_logged_handler,
},
- { }
};
static int __init seccomp_sysctl_init(void)
--
2.30.2
_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec
^ permalink raw reply related [flat|nested] 51+ messages in thread
* [PATCH 06/10] scheduler: Remove the now superfluous sentinel elements from ctl_table array
2023-11-07 13:45 ` Joel Granados
(?)
@ 2023-11-07 13:45 ` Joel Granados
-1 siblings, 0 replies; 51+ messages in thread
From: Joel Granados via B4 Relay @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
From: Joel Granados <j.granados@samsung.com>
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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
rm sentinel element from ctl_table arrays
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
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(-)
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)
--
2.30.2
^ permalink raw reply related [flat|nested] 51+ messages in thread
* [PATCH 06/10] scheduler: Remove the now superfluous sentinel elements from ctl_table array
@ 2023-11-07 13:45 ` Joel Granados
0 siblings, 0 replies; 51+ messages in thread
From: Joel Granados @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
rm sentinel element from ctl_table arrays
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
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(-)
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)
--
2.30.2
^ permalink raw reply related [flat|nested] 51+ messages in thread
* [PATCH 06/10] scheduler: Remove the now superfluous sentinel elements from ctl_table array
@ 2023-11-07 13:45 ` Joel Granados
0 siblings, 0 replies; 51+ messages in thread
From: Joel Granados via B4 Relay @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
From: Joel Granados <j.granados@samsung.com>
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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
rm sentinel element from ctl_table arrays
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
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(-)
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)
--
2.30.2
_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec
^ permalink raw reply related [flat|nested] 51+ messages in thread
* [PATCH 07/10] printk: Remove the now superfluous sentinel elements from ctl_table array
2023-11-07 13:45 ` Joel Granados
(?)
@ 2023-11-07 13:45 ` Joel Granados
-1 siblings, 0 replies; 51+ messages in thread
From: Joel Granados via B4 Relay @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
From: Joel Granados <j.granados@samsung.com>
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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
rm sentinel element from printk_sysctls
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
kernel/printk/sysctl.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/kernel/printk/sysctl.c b/kernel/printk/sysctl.c
index c228343eeb97..3e47dedce9e5 100644
--- a/kernel/printk/sysctl.c
+++ b/kernel/printk/sysctl.c
@@ -76,7 +76,6 @@ static struct ctl_table printk_sysctls[] = {
.extra1 = SYSCTL_ZERO,
.extra2 = SYSCTL_TWO,
},
- {}
};
void __init printk_sysctl_init(void)
--
2.30.2
^ permalink raw reply related [flat|nested] 51+ messages in thread
* [PATCH 07/10] printk: Remove the now superfluous sentinel elements from ctl_table array
@ 2023-11-07 13:45 ` Joel Granados
0 siblings, 0 replies; 51+ messages in thread
From: Joel Granados @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
rm sentinel element from printk_sysctls
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
kernel/printk/sysctl.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/kernel/printk/sysctl.c b/kernel/printk/sysctl.c
index c228343eeb97..3e47dedce9e5 100644
--- a/kernel/printk/sysctl.c
+++ b/kernel/printk/sysctl.c
@@ -76,7 +76,6 @@ static struct ctl_table printk_sysctls[] = {
.extra1 = SYSCTL_ZERO,
.extra2 = SYSCTL_TWO,
},
- {}
};
void __init printk_sysctl_init(void)
--
2.30.2
^ permalink raw reply related [flat|nested] 51+ messages in thread
* [PATCH 07/10] printk: Remove the now superfluous sentinel elements from ctl_table array
@ 2023-11-07 13:45 ` Joel Granados
0 siblings, 0 replies; 51+ messages in thread
From: Joel Granados via B4 Relay @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
From: Joel Granados <j.granados@samsung.com>
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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
rm sentinel element from printk_sysctls
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
kernel/printk/sysctl.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/kernel/printk/sysctl.c b/kernel/printk/sysctl.c
index c228343eeb97..3e47dedce9e5 100644
--- a/kernel/printk/sysctl.c
+++ b/kernel/printk/sysctl.c
@@ -76,7 +76,6 @@ static struct ctl_table printk_sysctls[] = {
.extra1 = SYSCTL_ZERO,
.extra2 = SYSCTL_TWO,
},
- {}
};
void __init printk_sysctl_init(void)
--
2.30.2
_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec
^ permalink raw reply related [flat|nested] 51+ messages in thread
* [PATCH 08/10] kprobes: Remove the now superfluous sentinel elements from ctl_table array
2023-11-07 13:45 ` Joel Granados
(?)
@ 2023-11-07 13:45 ` Joel Granados
-1 siblings, 0 replies; 51+ messages in thread
From: Joel Granados via B4 Relay @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
From: Joel Granados <j.granados@samsung.com>
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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
Remove sentinel element from kprobe_sysclts
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
kernel/kprobes.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/kernel/kprobes.c b/kernel/kprobes.c
index 0c6185aefaef..d049b602dd41 100644
--- a/kernel/kprobes.c
+++ b/kernel/kprobes.c
@@ -968,7 +968,6 @@ static struct ctl_table kprobe_sysctls[] = {
.extra1 = SYSCTL_ZERO,
.extra2 = SYSCTL_ONE,
},
- {}
};
static void __init kprobe_sysctls_init(void)
--
2.30.2
^ permalink raw reply related [flat|nested] 51+ messages in thread
* [PATCH 08/10] kprobes: Remove the now superfluous sentinel elements from ctl_table array
@ 2023-11-07 13:45 ` Joel Granados
0 siblings, 0 replies; 51+ messages in thread
From: Joel Granados @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
Remove sentinel element from kprobe_sysclts
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
kernel/kprobes.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/kernel/kprobes.c b/kernel/kprobes.c
index 0c6185aefaef..d049b602dd41 100644
--- a/kernel/kprobes.c
+++ b/kernel/kprobes.c
@@ -968,7 +968,6 @@ static struct ctl_table kprobe_sysctls[] = {
.extra1 = SYSCTL_ZERO,
.extra2 = SYSCTL_ONE,
},
- {}
};
static void __init kprobe_sysctls_init(void)
--
2.30.2
^ permalink raw reply related [flat|nested] 51+ messages in thread
* [PATCH 08/10] kprobes: Remove the now superfluous sentinel elements from ctl_table array
@ 2023-11-07 13:45 ` Joel Granados
0 siblings, 0 replies; 51+ messages in thread
From: Joel Granados via B4 Relay @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
From: Joel Granados <j.granados@samsung.com>
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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
Remove sentinel element from kprobe_sysclts
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
kernel/kprobes.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/kernel/kprobes.c b/kernel/kprobes.c
index 0c6185aefaef..d049b602dd41 100644
--- a/kernel/kprobes.c
+++ b/kernel/kprobes.c
@@ -968,7 +968,6 @@ static struct ctl_table kprobe_sysctls[] = {
.extra1 = SYSCTL_ZERO,
.extra2 = SYSCTL_ONE,
},
- {}
};
static void __init kprobe_sysctls_init(void)
--
2.30.2
_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec
^ permalink raw reply related [flat|nested] 51+ messages in thread
* [PATCH 09/10] delayacct: Remove the now superfluous sentinel elements from ctl_table array
2023-11-07 13:45 ` Joel Granados
(?)
@ 2023-11-07 13:45 ` Joel Granados
-1 siblings, 0 replies; 51+ messages in thread
From: Joel Granados via B4 Relay @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
From: Joel Granados <j.granados@samsung.com>
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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
Remove sentinel element from kern_delayacct_table
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
kernel/delayacct.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/kernel/delayacct.c b/kernel/delayacct.c
index 6f0c358e73d8..e039b0f99a0b 100644
--- a/kernel/delayacct.c
+++ b/kernel/delayacct.c
@@ -74,7 +74,6 @@ static struct ctl_table kern_delayacct_table[] = {
.extra1 = SYSCTL_ZERO,
.extra2 = SYSCTL_ONE,
},
- { }
};
static __init int kernel_delayacct_sysctls_init(void)
--
2.30.2
^ permalink raw reply related [flat|nested] 51+ messages in thread
* [PATCH 09/10] delayacct: Remove the now superfluous sentinel elements from ctl_table array
@ 2023-11-07 13:45 ` Joel Granados
0 siblings, 0 replies; 51+ messages in thread
From: Joel Granados @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
Remove sentinel element from kern_delayacct_table
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
kernel/delayacct.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/kernel/delayacct.c b/kernel/delayacct.c
index 6f0c358e73d8..e039b0f99a0b 100644
--- a/kernel/delayacct.c
+++ b/kernel/delayacct.c
@@ -74,7 +74,6 @@ static struct ctl_table kern_delayacct_table[] = {
.extra1 = SYSCTL_ZERO,
.extra2 = SYSCTL_ONE,
},
- { }
};
static __init int kernel_delayacct_sysctls_init(void)
--
2.30.2
^ permalink raw reply related [flat|nested] 51+ messages in thread
* [PATCH 09/10] delayacct: Remove the now superfluous sentinel elements from ctl_table array
@ 2023-11-07 13:45 ` Joel Granados
0 siblings, 0 replies; 51+ messages in thread
From: Joel Granados via B4 Relay @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
From: Joel Granados <j.granados@samsung.com>
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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
Remove sentinel element from kern_delayacct_table
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
kernel/delayacct.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/kernel/delayacct.c b/kernel/delayacct.c
index 6f0c358e73d8..e039b0f99a0b 100644
--- a/kernel/delayacct.c
+++ b/kernel/delayacct.c
@@ -74,7 +74,6 @@ static struct ctl_table kern_delayacct_table[] = {
.extra1 = SYSCTL_ZERO,
.extra2 = SYSCTL_ONE,
},
- { }
};
static __init int kernel_delayacct_sysctls_init(void)
--
2.30.2
_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec
^ permalink raw reply related [flat|nested] 51+ messages in thread
* [PATCH 10/10] bpf: Remove the now superfluous sentinel elements from ctl_table array
2023-11-07 13:45 ` Joel Granados
(?)
@ 2023-11-07 13:45 ` Joel Granados
-1 siblings, 0 replies; 51+ messages in thread
From: Joel Granados via B4 Relay @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
From: Joel Granados <j.granados@samsung.com>
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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
Remove sentinel element from bpf_syscall_table.
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
kernel/bpf/syscall.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/kernel/bpf/syscall.c b/kernel/bpf/syscall.c
index eb01c31ed591..1cb5b852b4e7 100644
--- a/kernel/bpf/syscall.c
+++ b/kernel/bpf/syscall.c
@@ -5679,7 +5679,6 @@ static struct ctl_table bpf_syscall_table[] = {
.mode = 0644,
.proc_handler = bpf_stats_handler,
},
- { }
};
static int __init bpf_syscall_sysctl_init(void)
--
2.30.2
^ permalink raw reply related [flat|nested] 51+ messages in thread
* [PATCH 10/10] bpf: Remove the now superfluous sentinel elements from ctl_table array
@ 2023-11-07 13:45 ` Joel Granados
0 siblings, 0 replies; 51+ messages in thread
From: Joel Granados @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
Remove sentinel element from bpf_syscall_table.
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
kernel/bpf/syscall.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/kernel/bpf/syscall.c b/kernel/bpf/syscall.c
index eb01c31ed591..1cb5b852b4e7 100644
--- a/kernel/bpf/syscall.c
+++ b/kernel/bpf/syscall.c
@@ -5679,7 +5679,6 @@ static struct ctl_table bpf_syscall_table[] = {
.mode = 0644,
.proc_handler = bpf_stats_handler,
},
- { }
};
static int __init bpf_syscall_sysctl_init(void)
--
2.30.2
^ permalink raw reply related [flat|nested] 51+ messages in thread
* [PATCH 10/10] bpf: Remove the now superfluous sentinel elements from ctl_table array
@ 2023-11-07 13:45 ` Joel Granados
0 siblings, 0 replies; 51+ messages in thread
From: Joel Granados via B4 Relay @ 2023-11-07 13:45 UTC (permalink / raw)
To: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa
Cc: linux-kernel, kexec, linux-fsdevel, linux-trace-kernel, bpf,
Joel Granados
From: Joel Granados <j.granados@samsung.com>
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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
Remove sentinel element from bpf_syscall_table.
Signed-off-by: Joel Granados <j.granados@samsung.com>
---
kernel/bpf/syscall.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/kernel/bpf/syscall.c b/kernel/bpf/syscall.c
index eb01c31ed591..1cb5b852b4e7 100644
--- a/kernel/bpf/syscall.c
+++ b/kernel/bpf/syscall.c
@@ -5679,7 +5679,6 @@ static struct ctl_table bpf_syscall_table[] = {
.mode = 0644,
.proc_handler = bpf_stats_handler,
},
- { }
};
static int __init bpf_syscall_sysctl_init(void)
--
2.30.2
_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec
^ permalink raw reply related [flat|nested] 51+ messages in thread
* Re: [PATCH 06/10] scheduler: Remove the now superfluous sentinel elements from ctl_table array
2023-11-07 13:45 ` Joel Granados
@ 2023-11-07 14:49 ` Peter Zijlstra
-1 siblings, 0 replies; 51+ messages in thread
From: Peter Zijlstra @ 2023-11-07 14:49 UTC (permalink / raw)
To: j.granados
Cc: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Juri Lelli, Vincent Guittot,
Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa, linux-kernel,
kexec, linux-fsdevel, linux-trace-kernel, bpf
On Tue, Nov 07, 2023 at 02:45:06PM +0100, Joel Granados via B4 Relay wrote:
> From: Joel Granados <j.granados@samsung.com>
>
> 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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
>
> rm sentinel element from ctl_table arrays
>
> Signed-off-by: Joel Granados <j.granados@samsung.com>
> ---
> 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) <peterz@infradead.org>
^ permalink raw reply [flat|nested] 51+ messages in thread
* Re: [PATCH 06/10] scheduler: Remove the now superfluous sentinel elements from ctl_table array
@ 2023-11-07 14:49 ` Peter Zijlstra
0 siblings, 0 replies; 51+ messages in thread
From: Peter Zijlstra @ 2023-11-07 14:49 UTC (permalink / raw)
To: j.granados
Cc: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Juri Lelli, Vincent Guittot,
Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa, linux-kernel,
kexec, linux-fsdevel, linux-trace-kernel, bpf
On Tue, Nov 07, 2023 at 02:45:06PM +0100, Joel Granados via B4 Relay wrote:
> From: Joel Granados <j.granados@samsung.com>
>
> 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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
>
> rm sentinel element from ctl_table arrays
>
> Signed-off-by: Joel Granados <j.granados@samsung.com>
> ---
> 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) <peterz@infradead.org>
_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec
^ permalink raw reply [flat|nested] 51+ messages in thread
* Re: [PATCH 10/10] bpf: Remove the now superfluous sentinel elements from ctl_table array
2023-11-07 13:45 ` Joel Granados
@ 2023-11-07 18:15 ` Andrii Nakryiko
-1 siblings, 0 replies; 51+ messages in thread
From: Andrii Nakryiko @ 2023-11-07 18:15 UTC (permalink / raw)
To: j.granados
Cc: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa, linux-kernel,
kexec, linux-fsdevel, linux-trace-kernel, bpf
On Tue, Nov 7, 2023 at 5:45 AM Joel Granados via B4 Relay
<devnull+j.granados.samsung.com@kernel.org> wrote:
>
> From: Joel Granados <j.granados@samsung.com>
>
> 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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
>
> Remove sentinel element from bpf_syscall_table.
>
> Signed-off-by: Joel Granados <j.granados@samsung.com>
> ---
> kernel/bpf/syscall.c | 1 -
> 1 file changed, 1 deletion(-)
>
LGTM.
Acked-by: Andrii Nakryiko <andrii@kernel.org>
> diff --git a/kernel/bpf/syscall.c b/kernel/bpf/syscall.c
> index eb01c31ed591..1cb5b852b4e7 100644
> --- a/kernel/bpf/syscall.c
> +++ b/kernel/bpf/syscall.c
> @@ -5679,7 +5679,6 @@ static struct ctl_table bpf_syscall_table[] = {
> .mode = 0644,
> .proc_handler = bpf_stats_handler,
> },
> - { }
> };
>
> static int __init bpf_syscall_sysctl_init(void)
>
> --
> 2.30.2
>
^ permalink raw reply [flat|nested] 51+ messages in thread
* Re: [PATCH 10/10] bpf: Remove the now superfluous sentinel elements from ctl_table array
@ 2023-11-07 18:15 ` Andrii Nakryiko
0 siblings, 0 replies; 51+ messages in thread
From: Andrii Nakryiko @ 2023-11-07 18:15 UTC (permalink / raw)
To: j.granados
Cc: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa, linux-kernel,
kexec, linux-fsdevel, linux-trace-kernel, bpf
On Tue, Nov 7, 2023 at 5:45 AM Joel Granados via B4 Relay
<devnull+j.granados.samsung.com@kernel.org> wrote:
>
> From: Joel Granados <j.granados@samsung.com>
>
> 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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
>
> Remove sentinel element from bpf_syscall_table.
>
> Signed-off-by: Joel Granados <j.granados@samsung.com>
> ---
> kernel/bpf/syscall.c | 1 -
> 1 file changed, 1 deletion(-)
>
LGTM.
Acked-by: Andrii Nakryiko <andrii@kernel.org>
> diff --git a/kernel/bpf/syscall.c b/kernel/bpf/syscall.c
> index eb01c31ed591..1cb5b852b4e7 100644
> --- a/kernel/bpf/syscall.c
> +++ b/kernel/bpf/syscall.c
> @@ -5679,7 +5679,6 @@ static struct ctl_table bpf_syscall_table[] = {
> .mode = 0644,
> .proc_handler = bpf_stats_handler,
> },
> - { }
> };
>
> static int __init bpf_syscall_sysctl_init(void)
>
> --
> 2.30.2
>
_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec
^ permalink raw reply [flat|nested] 51+ messages in thread
* Re: [PATCH 05/10] seccomp: Remove the now superfluous sentinel elements from ctl_table array
2023-11-07 13:45 ` Joel Granados
@ 2023-11-07 19:55 ` Kees Cook
-1 siblings, 0 replies; 51+ messages in thread
From: Kees Cook @ 2023-11-07 19:55 UTC (permalink / raw)
To: j.granados
Cc: Luis Chamberlain, willy, josh, Eric Biederman, Iurii Zaikin,
Steven Rostedt, Masami Hiramatsu, Mark Rutland, Thomas Gleixner,
John Stultz, Stephen Boyd, Andy Lutomirski, Will Drewry,
Ingo Molnar, Peter Zijlstra, Juri Lelli, Vincent Guittot,
Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa, linux-kernel,
kexec, linux-fsdevel, linux-trace-kernel, bpf
On Tue, Nov 07, 2023 at 02:45:05PM +0100, Joel Granados via B4 Relay wrote:
> From: Joel Granados <j.granados@samsung.com>
>
> 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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
>
> Remove sentinel element from seccomp_sysctl_table.
>
> Signed-off-by: Joel Granados <j.granados@samsung.com>
Acked-by: Kees Cook <keescook@chromium.org>
--
Kees Cook
^ permalink raw reply [flat|nested] 51+ messages in thread
* Re: [PATCH 05/10] seccomp: Remove the now superfluous sentinel elements from ctl_table array
@ 2023-11-07 19:55 ` Kees Cook
0 siblings, 0 replies; 51+ messages in thread
From: Kees Cook @ 2023-11-07 19:55 UTC (permalink / raw)
To: j.granados
Cc: Luis Chamberlain, willy, josh, Eric Biederman, Iurii Zaikin,
Steven Rostedt, Masami Hiramatsu, Mark Rutland, Thomas Gleixner,
John Stultz, Stephen Boyd, Andy Lutomirski, Will Drewry,
Ingo Molnar, Peter Zijlstra, Juri Lelli, Vincent Guittot,
Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa, linux-kernel,
kexec, linux-fsdevel, linux-trace-kernel, bpf
On Tue, Nov 07, 2023 at 02:45:05PM +0100, Joel Granados via B4 Relay wrote:
> From: Joel Granados <j.granados@samsung.com>
>
> 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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
>
> Remove sentinel element from seccomp_sysctl_table.
>
> Signed-off-by: Joel Granados <j.granados@samsung.com>
Acked-by: Kees Cook <keescook@chromium.org>
--
Kees Cook
_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec
^ permalink raw reply [flat|nested] 51+ messages in thread
* Re: [PATCH 08/10] kprobes: Remove the now superfluous sentinel elements from ctl_table array
2023-11-07 13:45 ` Joel Granados
@ 2023-11-08 10:27 ` Masami Hiramatsu
-1 siblings, 0 replies; 51+ messages in thread
From: Masami Hiramatsu @ 2023-11-08 10:27 UTC (permalink / raw)
To: j.granados
Cc: Joel Granados via B4 Relay, Luis Chamberlain, willy, josh,
Kees Cook, Eric Biederman, Iurii Zaikin, Steven Rostedt,
Mark Rutland, Thomas Gleixner, John Stultz, Stephen Boyd,
Andy Lutomirski, Will Drewry, Ingo Molnar, Peter Zijlstra,
Juri Lelli, Vincent Guittot, Dietmar Eggemann, Ben Segall,
Mel Gorman, Daniel Bristot de Oliveira, Valentin Schneider,
Petr Mladek, John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa, linux-kernel,
kexec, linux-fsdevel, linux-trace-kernel, bpf
On Tue, 07 Nov 2023 14:45:08 +0100
Joel Granados via B4 Relay <devnull+j.granados.samsung.com@kernel.org> wrote:
> From: Joel Granados <j.granados@samsung.com>
>
> 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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
>
> Remove sentinel element from kprobe_sysclts
Looks good to me.
Acked-by: Masami Hiramatsu (Google) <mhiramat@kernel.org>
Thanks,
>
> Signed-off-by: Joel Granados <j.granados@samsung.com>
> ---
> kernel/kprobes.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/kernel/kprobes.c b/kernel/kprobes.c
> index 0c6185aefaef..d049b602dd41 100644
> --- a/kernel/kprobes.c
> +++ b/kernel/kprobes.c
> @@ -968,7 +968,6 @@ static struct ctl_table kprobe_sysctls[] = {
> .extra1 = SYSCTL_ZERO,
> .extra2 = SYSCTL_ONE,
> },
> - {}
> };
>
> static void __init kprobe_sysctls_init(void)
>
> --
> 2.30.2
>
--
Masami Hiramatsu (Google) <mhiramat@kernel.org>
^ permalink raw reply [flat|nested] 51+ messages in thread
* Re: [PATCH 08/10] kprobes: Remove the now superfluous sentinel elements from ctl_table array
@ 2023-11-08 10:27 ` Masami Hiramatsu
0 siblings, 0 replies; 51+ messages in thread
From: Masami Hiramatsu @ 2023-11-08 10:27 UTC (permalink / raw)
To: j.granados
Cc: Joel Granados via B4 Relay, Luis Chamberlain, willy, josh,
Kees Cook, Eric Biederman, Iurii Zaikin, Steven Rostedt,
Mark Rutland, Thomas Gleixner, John Stultz, Stephen Boyd,
Andy Lutomirski, Will Drewry, Ingo Molnar, Peter Zijlstra,
Juri Lelli, Vincent Guittot, Dietmar Eggemann, Ben Segall,
Mel Gorman, Daniel Bristot de Oliveira, Valentin Schneider,
Petr Mladek, John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa, linux-kernel,
kexec, linux-fsdevel, linux-trace-kernel, bpf
On Tue, 07 Nov 2023 14:45:08 +0100
Joel Granados via B4 Relay <devnull+j.granados.samsung.com@kernel.org> wrote:
> From: Joel Granados <j.granados@samsung.com>
>
> 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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
>
> Remove sentinel element from kprobe_sysclts
Looks good to me.
Acked-by: Masami Hiramatsu (Google) <mhiramat@kernel.org>
Thanks,
>
> Signed-off-by: Joel Granados <j.granados@samsung.com>
> ---
> kernel/kprobes.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/kernel/kprobes.c b/kernel/kprobes.c
> index 0c6185aefaef..d049b602dd41 100644
> --- a/kernel/kprobes.c
> +++ b/kernel/kprobes.c
> @@ -968,7 +968,6 @@ static struct ctl_table kprobe_sysctls[] = {
> .extra1 = SYSCTL_ZERO,
> .extra2 = SYSCTL_ONE,
> },
> - {}
> };
>
> static void __init kprobe_sysctls_init(void)
>
> --
> 2.30.2
>
--
Masami Hiramatsu (Google) <mhiramat@kernel.org>
_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec
^ permalink raw reply [flat|nested] 51+ messages in thread
* Re: [PATCH 03/10] ftrace: Remove the now superfluous sentinel elements from ctl_table array
2023-11-07 13:45 ` Joel Granados
@ 2023-11-08 10:29 ` Masami Hiramatsu
-1 siblings, 0 replies; 51+ messages in thread
From: Masami Hiramatsu @ 2023-11-08 10:29 UTC (permalink / raw)
To: j.granados
Cc: Joel Granados via B4 Relay, Luis Chamberlain, willy, josh,
Kees Cook, Eric Biederman, Iurii Zaikin, Steven Rostedt,
Mark Rutland, Thomas Gleixner, John Stultz, Stephen Boyd,
Andy Lutomirski, Will Drewry, Ingo Molnar, Peter Zijlstra,
Juri Lelli, Vincent Guittot, Dietmar Eggemann, Ben Segall,
Mel Gorman, Daniel Bristot de Oliveira, Valentin Schneider,
Petr Mladek, John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa, linux-kernel,
kexec, linux-fsdevel, linux-trace-kernel, bpf
On Tue, 07 Nov 2023 14:45:03 +0100
Joel Granados via B4 Relay <devnull+j.granados.samsung.com@kernel.org> wrote:
> From: Joel Granados <j.granados@samsung.com>
>
> 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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
>
> Remove sentinel elements from ftrace_sysctls and user_event_sysctls
>
Both looks good to me. (since register_sysctl_init() uses ARRAY_SIZE()
macro to get the array size.)
Acked-by: Masami Hiramatsu (Google) <mhiramat@kernel.org>
> Signed-off-by: Joel Granados <j.granados@samsung.com>
> ---
> kernel/trace/ftrace.c | 1 -
> kernel/trace/trace_events_user.c | 1 -
> 2 files changed, 2 deletions(-)
>
> diff --git a/kernel/trace/ftrace.c b/kernel/trace/ftrace.c
> index 8de8bec5f366..fd40d02a23c7 100644
> --- a/kernel/trace/ftrace.c
> +++ b/kernel/trace/ftrace.c
> @@ -8264,7 +8264,6 @@ static struct ctl_table ftrace_sysctls[] = {
> .mode = 0644,
> .proc_handler = ftrace_enable_sysctl,
> },
> - {}
> };
>
> static int __init ftrace_sysctl_init(void)
> diff --git a/kernel/trace/trace_events_user.c b/kernel/trace/trace_events_user.c
> index 6f046650e527..bef1bdc62acf 100644
> --- a/kernel/trace/trace_events_user.c
> +++ b/kernel/trace/trace_events_user.c
> @@ -2694,7 +2694,6 @@ static struct ctl_table user_event_sysctls[] = {
> .mode = 0644,
> .proc_handler = set_max_user_events_sysctl,
> },
> - {}
> };
>
> static int __init trace_events_user_init(void)
>
> --
> 2.30.2
>
--
Masami Hiramatsu (Google) <mhiramat@kernel.org>
^ permalink raw reply [flat|nested] 51+ messages in thread
* Re: [PATCH 03/10] ftrace: Remove the now superfluous sentinel elements from ctl_table array
@ 2023-11-08 10:29 ` Masami Hiramatsu
0 siblings, 0 replies; 51+ messages in thread
From: Masami Hiramatsu @ 2023-11-08 10:29 UTC (permalink / raw)
To: j.granados
Cc: Joel Granados via B4 Relay, Luis Chamberlain, willy, josh,
Kees Cook, Eric Biederman, Iurii Zaikin, Steven Rostedt,
Mark Rutland, Thomas Gleixner, John Stultz, Stephen Boyd,
Andy Lutomirski, Will Drewry, Ingo Molnar, Peter Zijlstra,
Juri Lelli, Vincent Guittot, Dietmar Eggemann, Ben Segall,
Mel Gorman, Daniel Bristot de Oliveira, Valentin Schneider,
Petr Mladek, John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa, linux-kernel,
kexec, linux-fsdevel, linux-trace-kernel, bpf
On Tue, 07 Nov 2023 14:45:03 +0100
Joel Granados via B4 Relay <devnull+j.granados.samsung.com@kernel.org> wrote:
> From: Joel Granados <j.granados@samsung.com>
>
> 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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
>
> Remove sentinel elements from ftrace_sysctls and user_event_sysctls
>
Both looks good to me. (since register_sysctl_init() uses ARRAY_SIZE()
macro to get the array size.)
Acked-by: Masami Hiramatsu (Google) <mhiramat@kernel.org>
> Signed-off-by: Joel Granados <j.granados@samsung.com>
> ---
> kernel/trace/ftrace.c | 1 -
> kernel/trace/trace_events_user.c | 1 -
> 2 files changed, 2 deletions(-)
>
> diff --git a/kernel/trace/ftrace.c b/kernel/trace/ftrace.c
> index 8de8bec5f366..fd40d02a23c7 100644
> --- a/kernel/trace/ftrace.c
> +++ b/kernel/trace/ftrace.c
> @@ -8264,7 +8264,6 @@ static struct ctl_table ftrace_sysctls[] = {
> .mode = 0644,
> .proc_handler = ftrace_enable_sysctl,
> },
> - {}
> };
>
> static int __init ftrace_sysctl_init(void)
> diff --git a/kernel/trace/trace_events_user.c b/kernel/trace/trace_events_user.c
> index 6f046650e527..bef1bdc62acf 100644
> --- a/kernel/trace/trace_events_user.c
> +++ b/kernel/trace/trace_events_user.c
> @@ -2694,7 +2694,6 @@ static struct ctl_table user_event_sysctls[] = {
> .mode = 0644,
> .proc_handler = set_max_user_events_sysctl,
> },
> - {}
> };
>
> static int __init trace_events_user_init(void)
>
> --
> 2.30.2
>
--
Masami Hiramatsu (Google) <mhiramat@kernel.org>
_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec
^ permalink raw reply [flat|nested] 51+ messages in thread
* Re: [PATCH 03/10] ftrace: Remove the now superfluous sentinel elements from ctl_table array
2023-11-08 10:29 ` Masami Hiramatsu
@ 2023-11-08 16:27 ` Steven Rostedt
-1 siblings, 0 replies; 51+ messages in thread
From: Steven Rostedt @ 2023-11-08 16:27 UTC (permalink / raw)
To: Masami Hiramatsu (Google)
Cc: j.granados, Joel Granados via B4 Relay, Luis Chamberlain, willy,
josh, Kees Cook, Eric Biederman, Iurii Zaikin, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa, linux-kernel,
kexec, linux-fsdevel, linux-trace-kernel, bpf
On Wed, 8 Nov 2023 19:29:49 +0900
Masami Hiramatsu (Google) <mhiramat@kernel.org> wrote:
> On Tue, 07 Nov 2023 14:45:03 +0100
> Joel Granados via B4 Relay <devnull+j.granados.samsung.com@kernel.org> wrote:
>
> > From: Joel Granados <j.granados@samsung.com>
> >
> > 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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
> >
> > Remove sentinel elements from ftrace_sysctls and user_event_sysctls
> >
>
> Both looks good to me. (since register_sysctl_init() uses ARRAY_SIZE()
> macro to get the array size.)
>
> Acked-by: Masami Hiramatsu (Google) <mhiramat@kernel.org>
Agreed.
Acked-by: Steven Rostedt (Google) <rostedt@goodmis.org>
-- Steve
^ permalink raw reply [flat|nested] 51+ messages in thread
* Re: [PATCH 03/10] ftrace: Remove the now superfluous sentinel elements from ctl_table array
@ 2023-11-08 16:27 ` Steven Rostedt
0 siblings, 0 replies; 51+ messages in thread
From: Steven Rostedt @ 2023-11-08 16:27 UTC (permalink / raw)
To: Masami Hiramatsu (Google)
Cc: j.granados, Joel Granados via B4 Relay, Luis Chamberlain, willy,
josh, Kees Cook, Eric Biederman, Iurii Zaikin, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, Petr Mladek,
John Ogness, Sergey Senozhatsky, Naveen N. Rao,
Anil S Keshavamurthy, David S. Miller, Balbir Singh,
Alexei Starovoitov, Daniel Borkmann, John Fastabend,
Andrii Nakryiko, Martin KaFai Lau, Song Liu, Yonghong Song,
KP Singh, Stanislav Fomichev, Hao Luo, Jiri Olsa, linux-kernel,
kexec, linux-fsdevel, linux-trace-kernel, bpf
On Wed, 8 Nov 2023 19:29:49 +0900
Masami Hiramatsu (Google) <mhiramat@kernel.org> wrote:
> On Tue, 07 Nov 2023 14:45:03 +0100
> Joel Granados via B4 Relay <devnull+j.granados.samsung.com@kernel.org> wrote:
>
> > From: Joel Granados <j.granados@samsung.com>
> >
> > 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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
> >
> > Remove sentinel elements from ftrace_sysctls and user_event_sysctls
> >
>
> Both looks good to me. (since register_sysctl_init() uses ARRAY_SIZE()
> macro to get the array size.)
>
> Acked-by: Masami Hiramatsu (Google) <mhiramat@kernel.org>
Agreed.
Acked-by: Steven Rostedt (Google) <rostedt@goodmis.org>
-- Steve
_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec
^ permalink raw reply [flat|nested] 51+ messages in thread
* Re: [PATCH 07/10] printk: Remove the now superfluous sentinel elements from ctl_table array
2023-11-07 13:45 ` Joel Granados
@ 2023-11-28 14:07 ` Petr Mladek
-1 siblings, 0 replies; 51+ messages in thread
From: Petr Mladek @ 2023-11-28 14:07 UTC (permalink / raw)
To: j.granados
Cc: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, John Ogness,
Sergey Senozhatsky, Naveen N. Rao, Anil S Keshavamurthy,
David S. Miller, Balbir Singh, Alexei Starovoitov,
Daniel Borkmann, John Fastabend, Andrii Nakryiko,
Martin KaFai Lau, Song Liu, Yonghong Song, KP Singh,
Stanislav Fomichev, Hao Luo, Jiri Olsa, linux-kernel, kexec,
linux-fsdevel, linux-trace-kernel, bpf
On Tue 2023-11-07 14:45:07, Joel Granados via B4 Relay wrote:
> From: Joel Granados <j.granados@samsung.com>
>
> 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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
>
> rm sentinel element from printk_sysctls
>
> Signed-off-by: Joel Granados <j.granados@samsung.com>
I am a bit sceptical if the size and time reduction is worth the
effort. I feel that this change makes the access a bit less secure.
Well, almost all arrays are static so that it should just work.
The patch does what it says. Feel free to use:
Reviewed-by: Petr Mladek <pmladek@suse.com>
Best Regards,
Petr
^ permalink raw reply [flat|nested] 51+ messages in thread
* Re: [PATCH 07/10] printk: Remove the now superfluous sentinel elements from ctl_table array
@ 2023-11-28 14:07 ` Petr Mladek
0 siblings, 0 replies; 51+ messages in thread
From: Petr Mladek @ 2023-11-28 14:07 UTC (permalink / raw)
To: j.granados
Cc: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, John Ogness,
Sergey Senozhatsky, Naveen N. Rao, Anil S Keshavamurthy,
David S. Miller, Balbir Singh, Alexei Starovoitov,
Daniel Borkmann, John Fastabend, Andrii Nakryiko,
Martin KaFai Lau, Song Liu, Yonghong Song, KP Singh,
Stanislav Fomichev, Hao Luo, Jiri Olsa, linux-kernel, kexec,
linux-fsdevel, linux-trace-kernel, bpf
On Tue 2023-11-07 14:45:07, Joel Granados via B4 Relay wrote:
> From: Joel Granados <j.granados@samsung.com>
>
> 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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
>
> rm sentinel element from printk_sysctls
>
> Signed-off-by: Joel Granados <j.granados@samsung.com>
I am a bit sceptical if the size and time reduction is worth the
effort. I feel that this change makes the access a bit less secure.
Well, almost all arrays are static so that it should just work.
The patch does what it says. Feel free to use:
Reviewed-by: Petr Mladek <pmladek@suse.com>
Best Regards,
Petr
_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec
^ permalink raw reply [flat|nested] 51+ messages in thread
* Re: [PATCH 07/10] printk: Remove the now superfluous sentinel elements from ctl_table array
2023-11-28 14:07 ` Petr Mladek
@ 2023-12-04 8:56 ` Joel Granados
-1 siblings, 0 replies; 51+ messages in thread
From: Joel Granados @ 2023-12-04 8:56 UTC (permalink / raw)
To: Petr Mladek
Cc: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, John Ogness,
Sergey Senozhatsky, Naveen N. Rao, Anil S Keshavamurthy,
David S. Miller, Balbir Singh, Alexei Starovoitov,
Daniel Borkmann, John Fastabend, Andrii Nakryiko,
Martin KaFai Lau, Song Liu, Yonghong Song, KP Singh,
Stanislav Fomichev, Hao Luo, Jiri Olsa, linux-kernel, kexec,
linux-fsdevel, linux-trace-kernel, bpf
[-- Attachment #1: Type: text/plain, Size: 1279 bytes --]
Hey Petr
I missed this message somehow....
On Tue, Nov 28, 2023 at 03:07:43PM +0100, Petr Mladek wrote:
> On Tue 2023-11-07 14:45:07, Joel Granados via B4 Relay wrote:
> > From: Joel Granados <j.granados@samsung.com>
> >
> > 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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
> >
> > rm sentinel element from printk_sysctls
> >
> > Signed-off-by: Joel Granados <j.granados@samsung.com>
>
> I am a bit sceptical if the size and time reduction is worth the
> effort. I feel that this change makes the access a bit less secure.
In what way "less secure"? Can you expand on that?
Notice that if you pass a pointer to the register functions, you will
get a warning/error on compilation.
>
> Well, almost all arrays are static so that it should just work.
> The patch does what it says. Feel free to use:
Thx for the review. will do.
>
> Reviewed-by: Petr Mladek <pmladek@suse.com>
>
> Best Regards,
> Petr
--
Joel Granados
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
^ permalink raw reply [flat|nested] 51+ messages in thread
* Re: [PATCH 07/10] printk: Remove the now superfluous sentinel elements from ctl_table array
@ 2023-12-04 8:56 ` Joel Granados
0 siblings, 0 replies; 51+ messages in thread
From: Joel Granados @ 2023-12-04 8:56 UTC (permalink / raw)
To: Petr Mladek
Cc: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, John Ogness,
Sergey Senozhatsky, Naveen N. Rao, Anil S Keshavamurthy,
David S. Miller, Balbir Singh, Alexei Starovoitov,
Daniel Borkmann, John Fastabend, Andrii Nakryiko,
Martin KaFai Lau, Song Liu, Yonghong Song, KP Singh,
Stanislav Fomichev, Hao Luo, Jiri Olsa, linux-kernel, kexec,
linux-fsdevel, linux-trace-kernel, bpf
[-- Attachment #1.1: Type: text/plain, Size: 1279 bytes --]
Hey Petr
I missed this message somehow....
On Tue, Nov 28, 2023 at 03:07:43PM +0100, Petr Mladek wrote:
> On Tue 2023-11-07 14:45:07, Joel Granados via B4 Relay wrote:
> > From: Joel Granados <j.granados@samsung.com>
> >
> > 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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
> >
> > rm sentinel element from printk_sysctls
> >
> > Signed-off-by: Joel Granados <j.granados@samsung.com>
>
> I am a bit sceptical if the size and time reduction is worth the
> effort. I feel that this change makes the access a bit less secure.
In what way "less secure"? Can you expand on that?
Notice that if you pass a pointer to the register functions, you will
get a warning/error on compilation.
>
> Well, almost all arrays are static so that it should just work.
> The patch does what it says. Feel free to use:
Thx for the review. will do.
>
> Reviewed-by: Petr Mladek <pmladek@suse.com>
>
> Best Regards,
> Petr
--
Joel Granados
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
[-- Attachment #2: Type: text/plain, Size: 143 bytes --]
_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec
^ permalink raw reply [flat|nested] 51+ messages in thread
* Re: [PATCH 07/10] printk: Remove the now superfluous sentinel elements from ctl_table array
2023-12-04 8:56 ` Joel Granados
@ 2023-12-06 9:55 ` Petr Mladek
-1 siblings, 0 replies; 51+ messages in thread
From: Petr Mladek @ 2023-12-06 9:55 UTC (permalink / raw)
To: Joel Granados
Cc: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, John Ogness,
Sergey Senozhatsky, Naveen N. Rao, Anil S Keshavamurthy,
David S. Miller, Balbir Singh, Alexei Starovoitov,
Daniel Borkmann, John Fastabend, Andrii Nakryiko,
Martin KaFai Lau, Song Liu, Yonghong Song, KP Singh,
Stanislav Fomichev, Hao Luo, Jiri Olsa, linux-kernel, kexec,
linux-fsdevel, linux-trace-kernel, bpf
On Mon 2023-12-04 09:56:28, Joel Granados wrote:
> Hey Petr
>
> I missed this message somehow....
>
> On Tue, Nov 28, 2023 at 03:07:43PM +0100, Petr Mladek wrote:
> > On Tue 2023-11-07 14:45:07, Joel Granados via B4 Relay wrote:
> > > From: Joel Granados <j.granados@samsung.com>
> > >
> > > 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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
> > >
> > > rm sentinel element from printk_sysctls
> > >
> > > Signed-off-by: Joel Granados <j.granados@samsung.com>
> >
> > I am a bit sceptical if the size and time reduction is worth the
> > effort. I feel that this change makes the access a bit less secure.
> In what way "less secure"? Can you expand on that?
>
> Notice that if you pass a pointer to the register functions, you will
> get a warning/error on compilation.
I have vague memories that some arrays were not static or the length
has been somehow manipulated. But I might be wrong.
You are right that it should be safe with the static arrays.
And the NULL sentinel might be more error-prone after all.
Let's forget my mumbles.
Best Regards,
Petr
^ permalink raw reply [flat|nested] 51+ messages in thread
* Re: [PATCH 07/10] printk: Remove the now superfluous sentinel elements from ctl_table array
@ 2023-12-06 9:55 ` Petr Mladek
0 siblings, 0 replies; 51+ messages in thread
From: Petr Mladek @ 2023-12-06 9:55 UTC (permalink / raw)
To: Joel Granados
Cc: Luis Chamberlain, willy, josh, Kees Cook, Eric Biederman,
Iurii Zaikin, Steven Rostedt, Masami Hiramatsu, Mark Rutland,
Thomas Gleixner, John Stultz, Stephen Boyd, Andy Lutomirski,
Will Drewry, Ingo Molnar, Peter Zijlstra, Juri Lelli,
Vincent Guittot, Dietmar Eggemann, Ben Segall, Mel Gorman,
Daniel Bristot de Oliveira, Valentin Schneider, John Ogness,
Sergey Senozhatsky, Naveen N. Rao, Anil S Keshavamurthy,
David S. Miller, Balbir Singh, Alexei Starovoitov,
Daniel Borkmann, John Fastabend, Andrii Nakryiko,
Martin KaFai Lau, Song Liu, Yonghong Song, KP Singh,
Stanislav Fomichev, Hao Luo, Jiri Olsa, linux-kernel, kexec,
linux-fsdevel, linux-trace-kernel, bpf
On Mon 2023-12-04 09:56:28, Joel Granados wrote:
> Hey Petr
>
> I missed this message somehow....
>
> On Tue, Nov 28, 2023 at 03:07:43PM +0100, Petr Mladek wrote:
> > On Tue 2023-11-07 14:45:07, Joel Granados via B4 Relay wrote:
> > > From: Joel Granados <j.granados@samsung.com>
> > >
> > > 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/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/)
> > >
> > > rm sentinel element from printk_sysctls
> > >
> > > Signed-off-by: Joel Granados <j.granados@samsung.com>
> >
> > I am a bit sceptical if the size and time reduction is worth the
> > effort. I feel that this change makes the access a bit less secure.
> In what way "less secure"? Can you expand on that?
>
> Notice that if you pass a pointer to the register functions, you will
> get a warning/error on compilation.
I have vague memories that some arrays were not static or the length
has been somehow manipulated. But I might be wrong.
You are right that it should be safe with the static arrays.
And the NULL sentinel might be more error-prone after all.
Let's forget my mumbles.
Best Regards,
Petr
_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec
^ permalink raw reply [flat|nested] 51+ messages in thread
end of thread, other threads:[~2024-01-12 15:58 UTC | newest]
Thread overview: 51+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-11-07 13:45 [PATCH 00/10] sysctl: Remove sentinel elements from kernel dir Joel Granados via B4 Relay
2023-11-07 13:45 ` Joel Granados via B4 Relay
2023-11-07 13:45 ` Joel Granados
2023-11-07 13:45 ` [PATCH 01/10] kernel misc: Remove the now superfluous sentinel elements from ctl_table array Joel Granados via B4 Relay
2023-11-07 13:45 ` Joel Granados via B4 Relay
2023-11-07 13:45 ` Joel Granados
2023-11-07 13:45 ` [PATCH 02/10] umh: " Joel Granados via B4 Relay
2023-11-07 13:45 ` Joel Granados via B4 Relay
2023-11-07 13:45 ` Joel Granados
2023-11-07 13:45 ` [PATCH 03/10] ftrace: " Joel Granados via B4 Relay
2023-11-07 13:45 ` Joel Granados via B4 Relay
2023-11-07 13:45 ` Joel Granados
2023-11-08 10:29 ` Masami Hiramatsu
2023-11-08 10:29 ` Masami Hiramatsu
2023-11-08 16:27 ` Steven Rostedt
2023-11-08 16:27 ` Steven Rostedt
2023-11-07 13:45 ` [PATCH 04/10] timekeeping: " Joel Granados via B4 Relay
2023-11-07 13:45 ` Joel Granados via B4 Relay
2023-11-07 13:45 ` Joel Granados
2023-11-07 13:45 ` [PATCH 05/10] seccomp: " Joel Granados via B4 Relay
2023-11-07 13:45 ` Joel Granados via B4 Relay
2023-11-07 13:45 ` Joel Granados
2023-11-07 19:55 ` Kees Cook
2023-11-07 19:55 ` Kees Cook
2023-11-07 13:45 ` [PATCH 06/10] scheduler: " Joel Granados via B4 Relay
2023-11-07 13:45 ` Joel Granados via B4 Relay
2023-11-07 13:45 ` Joel Granados
2023-11-07 14:49 ` Peter Zijlstra
2023-11-07 14:49 ` Peter Zijlstra
2023-11-07 13:45 ` [PATCH 07/10] printk: " Joel Granados via B4 Relay
2023-11-07 13:45 ` Joel Granados via B4 Relay
2023-11-07 13:45 ` Joel Granados
2023-11-28 14:07 ` Petr Mladek
2023-11-28 14:07 ` Petr Mladek
2023-12-04 8:56 ` Joel Granados
2023-12-04 8:56 ` Joel Granados
2023-12-06 9:55 ` Petr Mladek
2023-12-06 9:55 ` Petr Mladek
2023-11-07 13:45 ` [PATCH 08/10] kprobes: " Joel Granados via B4 Relay
2023-11-07 13:45 ` Joel Granados via B4 Relay
2023-11-07 13:45 ` Joel Granados
2023-11-08 10:27 ` Masami Hiramatsu
2023-11-08 10:27 ` Masami Hiramatsu
2023-11-07 13:45 ` [PATCH 09/10] delayacct: " Joel Granados via B4 Relay
2023-11-07 13:45 ` Joel Granados via B4 Relay
2023-11-07 13:45 ` Joel Granados
2023-11-07 13:45 ` [PATCH 10/10] bpf: " Joel Granados via B4 Relay
2023-11-07 13:45 ` Joel Granados via B4 Relay
2023-11-07 13:45 ` Joel Granados
2023-11-07 18:15 ` Andrii Nakryiko
2023-11-07 18:15 ` Andrii Nakryiko
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.