mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Joel Granados <joel.granados@kernel.org>
To: Kees Cook <kees@kernel.org>, Shuah Khan <shuah@kernel.org>,
	 Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	 "Rafael J. Wysocki" <rafael@kernel.org>,
	Danilo Krummrich <dakr@kernel.org>,
	 Andrew Morton <akpm@linux-foundation.org>
Cc: linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org,
	 linux-kselftest@vger.kernel.org,
	Joel Granados <joel.granados@kernel.org>
Subject: [PATCH 0/5] sysctl: Remove last two ctl_tables from the kern_table array
Date: Fri, 27 Jun 2025 11:27:24 +0200	[thread overview]
Message-ID: <20250627-jag-sysctl-v1-0-20dd9801420b@kernel.org> (raw)

This is the last series to relocate sysctl tables from kernel/sysctl.c
into their respective subsystems. After the move of two ctl_tables
(uevent_helper & overflow{uid,gid}), five remain. They either handle
variables defined within sysctl.c or serve as a common place for
variables that are defined in different architectures. These five will
not be moved. Note that this series includes two auxiliary changes:
Removal of an unused variable and Nix-based rework of sysctl.sh test
script

By decentralizing sysctl registrations, subsystem maintainers regain
control over their sysctl interfaces, improving maintainability and
reducing the likelihood of merge conflicts. All this is made possible by
the work done to reduce the ctl_table memory footprint in commit
d7a76ec87195 ("sysctl: Remove check for sentinel element in ctl_table
arrays").

A few comments on the process:
1. If you prefer to merge this through a non-sysctl tree, please let me
   know so I can avoid conflicts in linux-next.
2. Apologies if you were copied by mistake—let me know if you'd like to
   be removed.
3. This series builds on [1], so please rebase accordingly for clean
   application.
4. Testing done by running sysctl selftests on x86_64 and 0-day.

Comments/Suggestions greatly appreciated

[1] https://lore.kernel.org/20250509-jag-mv_ctltables_iter2-v1-0-d0ad83f5f4c3@kernel.org

Signed-off-by: Joel Granados <joel.granados@kernel.org>
---
Joel Granados (5):
      sysctl: Nixify sysctl.sh
      sysctl: Removed unused variable
      uevent: mv uevent_helper into kobject_uevent.c
      kernel/sys.c: Move overflow{uid,gid} sysctl into kernel/sys.c
      sysctl: rename kern_table -> sysctl_subsys_table

 include/linux/sysctl.h                   |  1 -
 kernel/sys.c                             | 29 +++++++++++++++++++
 kernel/sysctl.c                          | 49 +++++++-------------------------
 lib/kobject_uevent.c                     | 20 +++++++++++++
 tools/testing/selftests/sysctl/sysctl.sh |  2 +-
 5 files changed, 61 insertions(+), 40 deletions(-)
---
base-commit: 501dd0fbc76bcae57902ea000d9c6ccd9d5f226e
change-id: 20250627-jag-sysctl-823adf5732be

Best regards,
-- 
Joel Granados <joel.granados@kernel.org>



             reply	other threads:[~2025-06-27  9:29 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-06-27  9:27 Joel Granados [this message]
2025-06-27  9:27 ` [PATCH 1/5] sysctl: Nixify sysctl.sh Joel Granados
2025-06-27  9:27 ` [PATCH 2/5] sysctl: Removed unused variable Joel Granados
2025-06-27  9:27 ` [PATCH 3/5] uevent: mv uevent_helper into kobject_uevent.c Joel Granados
2025-06-27  9:27 ` [PATCH 4/5] kernel/sys.c: Move overflow{uid,gid} sysctl into kernel/sys.c Joel Granados
2025-06-27  9:27 ` [PATCH 5/5] sysctl: rename kern_table -> sysctl_subsys_table Joel Granados

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20250627-jag-sysctl-v1-0-20dd9801420b@kernel.org \
    --to=joel.granados@kernel.org \
    --cc=akpm@linux-foundation.org \
    --cc=dakr@kernel.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=kees@kernel.org \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-kselftest@vger.kernel.org \
    --cc=rafael@kernel.org \
    --cc=shuah@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®