On Wed, Jan 15, 2025 at 09:53:53AM +0800, yukaixiong wrote:

On 2025/1/14 21:50, Joel Granados wrote:

quoted

On Sat, Jan 11, 2025 at 03:07:35PM +0800, Kaixiong Yu wrote:

quoted

This patch series moves sysctls of vm_table in kernel/sysctl.c to
places where they actually belong, and do some related code clean-ups.
After this patch series, all sysctls in vm_table have been moved into its
own files, meanwhile, delete vm_table.

All the modifications of this patch series base on
linux-next(tags/next-20250110). To test this patch series, the code was
compiled with both the CONFIG_SYSCTL enabled and disabled on arm64 and
x86_64 architectures. After this patch series is applied, all files
under /proc/sys/vm can be read or written normally.

It is looking good! Here is how I think we should move it upstream:

1. These should queued in for 6.15 instead of the next merge window.
    It is too late in the current cycle and if we put it in now, it will
    not properly tested in linux-next.

2. I am putting this in sysctl-testing with the expectation of pushing this
    up for the 6.15 merge window. Please tell me if you want this to go
    through some other tree.

Thx for the contribution

Best

Thank you! I don't want this to go through some other tree.

This was more for the mm, net and security maintainers :)

Best ...

quoted

quoted

my test steps as below listed:

Step 1: Set CONFIG_SYSCTL to 'n' and compile the Linux kernel on the
arm64 architecture. The kernel compiles successfully without any errors
or warnings.

...

quoted

  mm/swap.c                          |  16 ++-
  mm/swap.h                          |   1 +
  mm/util.c                          |  67 +++++++--
  mm/vmscan.c                        |  23 +++
  mm/vmstat.c                        |  44 +++++-
  net/sunrpc/auth.c                  |   2 +-
  security/min_addr.c                |  11 ++
  23 files changed, 336 insertions(+), 312 deletions(-)

-- 
2.34.1

-- 

Joel Granados