Re: [PATCH v5 -next 00/16] sysctl: move sysctls from vm_table into its own files

[yukaixiong <yukaixiong@huawei.com>]

On 2025/1/14 21:50, Joel Granados wrote:
> On Sat, Jan 11, 2025 at 03:07:35PM +0800, Kaixiong Yu wrote:
>> 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.

Best ...
>> 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.
>>
> ...
>>   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
>>