diff options
| author | Joel Granados <joel.granados@kernel.org> | 2025-09-29 14:43:54 +0200 |
|---|---|---|
| committer | Joel Granados <joel.granados@kernel.org> | 2025-10-24 15:35:11 +0200 |
| commit | 18c4e028847092003c11f824796d1309bc01cd69 (patch) | |
| tree | 64c0629cf96df4de6deddc6f7aaff3db6c994590 | |
| parent | 3a8660878839faadb4f1a6dd72c3179c1df56787 (diff) | |
watchdog: move nmi_watchdog sysctl into .rodata
Move nmi_watchdog into the watchdog_sysctls array to prevent it from
unnecessary modification. This move effectively moves it inside the
.rodata section.
Initially moved out into its own non-const array in commit 9ec272c586b0
("watchdog/hardlockup: keep kernel.nmi_watchdog sysctl as 0444 if probe
fails"), which made it writable only when watchdog_hardlockup_available
was true. Moving it back to watchdog_sysctl keeps this behavior as
writing to nmi_watchdog still fails when watchdog_hardlockup_available
is false.
Reviewed-by: Petr Mladek <pmladek@suse.com>
Signed-off-by: Joel Granados <joel.granados@kernel.org>
| -rw-r--r-- | kernel/watchdog.c | 9 |
1 files changed, 1 insertions, 8 deletions
diff --git a/kernel/watchdog.c b/kernel/watchdog.c index 5b62d1002783..659f5844393c 100644 --- a/kernel/watchdog.c +++ b/kernel/watchdog.c @@ -1231,14 +1231,11 @@ static const struct ctl_table watchdog_sysctls[] = { }, #endif /* CONFIG_SMP */ #endif -}; - -static struct ctl_table watchdog_hardlockup_sysctl[] = { { .procname = "nmi_watchdog", .data = &watchdog_hardlockup_user_enabled, .maxlen = sizeof(int), - .mode = 0444, + .mode = 0644, .proc_handler = proc_nmi_watchdog, .extra1 = SYSCTL_ZERO, .extra2 = SYSCTL_ONE, @@ -1248,10 +1245,6 @@ static struct ctl_table watchdog_hardlockup_sysctl[] = { static void __init watchdog_sysctl_init(void) { register_sysctl_init("kernel", watchdog_sysctls); - - if (watchdog_hardlockup_available) - watchdog_hardlockup_sysctl[0].mode = 0644; - register_sysctl_init("kernel", watchdog_hardlockup_sysctl); } #else |