From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 220D4E77188 for ; Sat, 28 Dec 2024 12:15:10 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 4595F6B007B; Sat, 28 Dec 2024 07:15:10 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 413A36B0082; Sat, 28 Dec 2024 07:15:10 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 2D03A6B0083; Sat, 28 Dec 2024 07:15:10 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0014.hostedemail.com [216.40.44.14]) by kanga.kvack.org (Postfix) with ESMTP id 10A616B007B for ; Sat, 28 Dec 2024 07:15:10 -0500 (EST) Received: from smtpin05.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay10.hostedemail.com (Postfix) with ESMTP id AEA1DC1702 for ; Sat, 28 Dec 2024 12:15:09 +0000 (UTC) X-FDA: 82944261372.05.79F596B Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by imf10.hostedemail.com (Postfix) with ESMTP id B05D4C0002 for ; Sat, 28 Dec 2024 12:14:48 +0000 (UTC) Authentication-Results: imf10.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=IPmKL7Ab; spf=pass (imf10.hostedemail.com: domain of joel.granados@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=joel.granados@kernel.org; dmarc=pass (policy=quarantine) header.from=kernel.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1735388074; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=Sg8VDOzaEvm76mvj9ZwmMnJm27s65Kxdefo7y/lZMEA=; b=lF6AJoJetO2i/JVEDe1tuglKMGa2/F/L+MQR/9cw5BdWHGPcseULMUvfGhdgUBY2itllRy +9l5Qw8YAOiaRw069rWDVFfZ7lfozvuMFBtBS0MFMaVvDtNVYgovFLn+H9Sj8KNOPofMAB qqobIBR91SBx22kwNyX78vBJ8V9ZkoM= ARC-Authentication-Results: i=1; imf10.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=IPmKL7Ab; spf=pass (imf10.hostedemail.com: domain of joel.granados@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=joel.granados@kernel.org; dmarc=pass (policy=quarantine) header.from=kernel.org ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1735388074; a=rsa-sha256; cv=none; b=JleMzR9raZpUN9HNbhESf5HH9TCNPJ2eQ2+DIobBfCmr5ZRiw91XWK78TuogXrGR9wKUug ieWnpaJuYp6wmWVFRKE8jfdhNnBdi86vkB80DYLqwQcaITtb4XDul3e2MQln5h8+fZsCkT +iTvkwZOysDreXGA/Aa2Ou9rxR9b0bI= Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id B9D325C4AFF; Sat, 28 Dec 2024 12:14:24 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 62852C4CECD; Sat, 28 Dec 2024 12:15:05 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1735388106; bh=97/5224KtfCVtGhVMHx6krsVPkUDF0EY+TYp3O6MChU=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=IPmKL7AbU1Kqx31Ft9P9mrdb3gcll7S8m0Qfed5FXR1ytUSAQIbGF9qdOnyxed7Hq jstjuYa3DV3wH/C2qaXbWwCo/rqhF7/nvV1cKQ9tLz3K5J8GyOFNzDsy2QpIjPO5xq NepbzrZht8wPww1EphkIlzQverVOpEFgxAm3DtnjPK27lBBvnHBlq01mMQxJhMWBHC mDunzRZm3w41CX0jK5/YcHTXirHiHIkYNa42270CjE/FXV0DcFfEox0AwYPYRdsOGR zGnazmXNZEQwfDCiGXjJeQA7MvBYV412juQ3OdflPcnvQ6PuOuIfBBRCY1sTwmdRRv pZvEd0sww2NqQ== Date: Sat, 28 Dec 2024 13:15:00 +0100 From: Joel Granados To: Kaixiong Yu Cc: akpm@linux-foundation.org, mcgrof@kernel.org, ysato@users.sourceforge.jp, dalias@libc.org, glaubitz@physik.fu-berlin.de, luto@kernel.org, tglx@linutronix.de, mingo@redhat.com, bp@alien8.de, dave.hansen@linux.intel.com, hpa@zytor.com, viro@zeniv.linux.org.uk, brauner@kernel.org, jack@suse.cz, kees@kernel.org, j.granados@samsung.com, willy@infradead.org, Liam.Howlett@oracle.com, vbabka@suse.cz, lorenzo.stoakes@oracle.com, trondmy@kernel.org, anna@kernel.org, chuck.lever@oracle.com, jlayton@kernel.org, neilb@suse.de, okorniev@redhat.com, Dai.Ngo@oracle.com, tom@talpey.com, davem@davemloft.net, edumazet@google.com, kuba@kernel.org, pabeni@redhat.com, paul@paul-moore.com, jmorris@namei.org, linux-sh@vger.kernel.org, linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-mm@kvack.org, linux-nfs@vger.kernel.org, netdev@vger.kernel.org, linux-security-module@vger.kernel.org, dhowells@redhat.com, haifeng.xu@shopee.com, baolin.wang@linux.alibaba.com, shikemeng@huaweicloud.com, dchinner@redhat.com, bfoster@redhat.com, souravpanda@google.com, hannes@cmpxchg.org, rientjes@google.com, pasha.tatashin@soleen.com, david@redhat.com, ryan.roberts@arm.com, ying.huang@intel.com, yang@os.amperecomputing.com, zev@bewilderbeest.net, serge@hallyn.com, vegard.nossum@oracle.com, wangkefeng.wang@huawei.com Subject: Re: [PATCH v4 -next 00/15] sysctl: move sysctls from vm_table into its own files Message-ID: <42tsyuvdvym6i3j4ppsluvx7kejxjzbma5z4jjgccni6kuwtj7@rhuklbyko7yf> References: <20241223141550.638616-1-yukaixiong@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20241223141550.638616-1-yukaixiong@huawei.com> X-Rspamd-Queue-Id: B05D4C0002 X-Rspamd-Server: rspam12 X-Stat-Signature: wsf3iuag8gbt8mfcomh867by5ih4pkdj X-Rspam-User: X-HE-Tag: 1735388088-838808 X-HE-Meta: U2FsdGVkX1+fv2+ilpm9mISJHRWRNuCG6ES6BYuJOF6qr5q7FEvNhxW67GTYWgGrLYek/7MAKbOJSbygz5NFy528LjhvcPugt4zNz+9HDYHGmStGwsN8AC3Q2G4KCpEdszq7r09v67oBkj3qrAdRYmOukkVRNXJ5INcIfdtblRlRTWngdI/eaqJEcRw9mHov5JpKFScoc+bKD5pcu12ifEuz/IfiAm5oqgRhdTtypp8BdkRnqoesZbMF+cWdQDYqiWgLxGv0V4Aro7VHmBF7WTWkUsJIDm+Ts2bAVH3g0NYnObQZIQvwPhZMG1tg+jl5u0BVC6tSkMtNYne15JIUcONDcEArWTD8Y9xb4dYmk0b/pBckRsgiuvOoiO/gx2sgxSG1w/v29uywZxfLn8rlLI5/RAftsksFi3rA51St7gwdO6sANYrO6ieytRSO/Muw5k4u6U85AfLwC0fVLi5eQHMfLRxssWE93ag17dRj05e+/EyAj0lhDy6E0bz2JK1S4DzTzH+aCFl/GXlDp5psIXB4Hg/HywrYylYqx2brsOIxkIEFflIC/rw4Wud6wj67aIIeI0A8Z+l6AbLGMxcpbnwhsdM5vde0fCjR+iUx0X7rGddjay3t5i/YdKYRpKWcEh3XRIdjqbwYTY0fZWExX5xnW5LU9QF+4pwUx4MfcemHVJItcd18cgDiXTnzoHvhd4shIfYaSLHkpdrvv3x0ZeA/fiWF+0L1xlNZJyu5E5lZnN4aZFREEfXCCyFF9qcAHroyB5MY/2BcHygsOC7iyUCS+BUdRNVbTJoUQ63vfI7aOeTFj49LKQiBYwc/iBlICx7ZKh7A7Kxn34c28pS/PjqvAG4kAiWO1GPUmJZQMmFh41sTYb7KqkJBZKzCxG0um6JyUuIYFHhF6g/Rko2h3/5XIB/kQkOSKhCyj85XUEPDktFXdn5HcylBmFEvAzdAZa7hwL/4SAim2GTnMoa 6IqB6jyx TmoOxvvZMa4MouHtrLCf3MBizOeakNbv8obBoUMrJe6puVl0OVcGmdz7jxE/zjJWWxOpSMppTF9fijD1Nt59ZDan8GCtJFl0vsJEoiYsePH8WCJom7wK0c/Xd2FZ39QC2B7jvY1BHZIpraOl+mq6kyZORxk5x/ncws+ihG0IODdYjuoB12HiWxssGFKbF/c1BB0n31ZVR/+lsFkD1tnadnusZujMdbvLjLotDsaU0gs0/38QWqTqymSLV30oHaJZKEvyeqnci7WOOFfJYWtw1iwj8xraUhz1fSiq1MZUkntSm/IFlR4Tm7Xc0tsrwdxFr2yBwFYS15AaLwBWjGPwWWigDqJLtuw2wd1rV X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: On Mon, Dec 23, 2024 at 10:15:19PM +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-20241219). 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. > > Changes in v4: > - change all "static struct ctl_table" type into > "static const struct ctl_table" type in patch1~10,12,13,14 > - simplify result of rpcauth_cache_shrink_count() in patch11 > > Changes in v3: > - change patch1~10, patch14 title suggested by Joel Granados > - change sysctl_stat_interval to static type in patch1 > - add acked-by from Paul Moore in patch7 > - change dirtytime_expire_interval to static type in patch9 > - add acked-by from Anna Schumaker in patch11 > > Changes in v2: > - fix sysctl_max_map_count undeclared issue in mm/nommu.c for patch6 > - update changelog for patch7/12, suggested by Kees/Paul > - fix patch8, sorry for wrong changes and forget to built with NOMMU > - add reviewed-by from Kees except patch8 since patch8 is wrong in v1 > - add reviewed-by from Jan Kara, Christian Brauner in patch12 > > Kaixiong Yu (15): > mm: vmstat: move sysctls to mm/vmstat.c > mm: filemap: move sysctl to mm/filemap.c > mm: swap: move sysctl to mm/swap.c > mm: vmscan: move vmscan sysctls to mm/vmscan.c > mm: util: move sysctls to mm/util.c > mm: mmap: move sysctl to mm/mmap.c > security: min_addr: move sysctl to security/min_addr.c > mm: nommu: move sysctl to mm/nommu.c > fs: fs-writeback: move sysctl to fs/fs-writeback.c > fs: drop_caches: move sysctl to fs/drop_caches.c > sunrpc: simplify rpcauth_cache_shrink_count() > fs: dcache: move the sysctl to fs/dcache.c > x86: vdso: move the sysctl to arch/x86/entry/vdso/vdso32-setup.c > sh: vdso: move the sysctl to arch/sh/kernel/vsyscall/vsyscall.c > sysctl: remove unneeded include This patchset looks strange. There seems to be 15 patches, but there are 30 e-mails in the thread? You can also see this when you look at it in lore [1]. And they are different repeated e-mails (mutt does not de-duplicate them). Also `b4 shazam ...` does not work. What happened? Did you send it twice with the same mail ID? Am I the only one seeing this? I would suggest the following (hopefully you are using b4): 1. Check to see how things will be sent with b4. `b4 send --resend -o OUTPUT_DIR` If you see 30 emails in that dir from your patchset then something is still wrong. 2. After you make sure that everything is in order. Do the resend without bumping the version up (leave it at version 4) Best [1] : https://lore.kernel.org/all/20241223141550.638616-1-yukaixiong@huawei.com/ > > arch/sh/kernel/vsyscall/vsyscall.c | 14 ++ > arch/x86/entry/vdso/vdso32-setup.c | 16 ++- > fs/dcache.c | 21 ++- > fs/drop_caches.c | 23 ++- > fs/fs-writeback.c | 30 ++-- > include/linux/dcache.h | 7 +- > include/linux/mm.h | 23 --- > include/linux/mman.h | 2 - > include/linux/swap.h | 9 -- > include/linux/vmstat.h | 11 -- > include/linux/writeback.h | 4 - > kernel/sysctl.c | 221 ----------------------------- > mm/filemap.c | 18 ++- > mm/internal.h | 10 ++ > mm/mmap.c | 54 +++++++ > mm/nommu.c | 15 +- > 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, 330 insertions(+), 312 deletions(-) > > -- > 2.34.1 > -- Joel Granados