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 13F23C433F5 for ; Sun, 13 Feb 2022 15:34:49 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 11A546B0072; Sun, 13 Feb 2022 10:34:49 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 0A35B6B0073; Sun, 13 Feb 2022 10:34:49 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id E5E266B0078; Sun, 13 Feb 2022 10:34:48 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0121.hostedemail.com [216.40.44.121]) by kanga.kvack.org (Postfix) with ESMTP id D7A926B0073 for ; Sun, 13 Feb 2022 10:34:48 -0500 (EST) Received: from smtpin30.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay04.hostedemail.com (Postfix) with ESMTP id 8DB34886C7 for ; Sun, 13 Feb 2022 15:34:48 +0000 (UTC) X-FDA: 79138154256.30.F9B81EA Received: from new2-smtp.messagingengine.com (new2-smtp.messagingengine.com [66.111.4.224]) by imf19.hostedemail.com (Postfix) with ESMTP id BF9961A0006; Sun, 13 Feb 2022 15:34:47 +0000 (UTC) Received: from compute5.internal (compute5.nyi.internal [10.202.2.45]) by mailnew.nyi.internal (Postfix) with ESMTP id 21C5E58012C; Sun, 13 Feb 2022 10:34:47 -0500 (EST) Received: from mailfrontend2 ([10.202.2.163]) by compute5.internal (MEProxy); Sun, 13 Feb 2022 10:34:47 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-type:date:date:from:from :in-reply-to:in-reply-to:message-id:mime-version:references :reply-to:sender:subject:subject:to:to:x-me-proxy:x-me-proxy :x-me-sender:x-me-sender:x-sasl-enc; s=fm2; bh=UA8T6JiS2f3ocAFcD gjHyjxICxgUmp2Ip3lgkBm5b9k=; b=ahRWvieaYNmVdj3m2nCPuqkay7sl2L4ut BuMLw7cTlreuR8lIZh7gi4o/Fmre4X8afwjL79FXQdgh67v+JJYY+blDijP1fZsT PELmblx7syqCsp281VitTT0I5Lv/GtxiJJFgBjgkv43tJzDzznDxg954cJf/aRI6 P930bt4poGLa+M0D8ioQT0wb8fDLA3ZAaUy6F4wnoffegOe9q9IGK3sn3cBFdm8q BfV43tR3WENkOWn4I5Nh8WFd58Ikn/O+o429UdrsG5zrh/CzZEIizHhFUO3tIK3b rePocJilix9hHlmqoijz1XesiGLMpZt6xchTOJDSJCpCOuAms/gwg== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvvddrjedtgdejfecutefuodetggdotefrodftvf curfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfghnecu uegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenuc fjughrpeffhffvuffkfhggtggujgesthdtredttddtvdenucfhrhhomhepkfguohcuufgt hhhimhhmvghluceoihguohhstghhsehiughoshgthhdrohhrgheqnecuggftrfgrthhtvg hrnheptdffkeekfeduffevgeeujeffjefhtefgueeugfevtdeiheduueeukefhudehleet necuvehluhhsthgvrhfuihiivgeptdenucfrrghrrghmpehmrghilhhfrhhomhepihguoh hstghhsehiughoshgthhdrohhrgh X-ME-Proxy: Received: by mail.messagingengine.com (Postfix) with ESMTPA; Sun, 13 Feb 2022 10:34:42 -0500 (EST) Date: Sun, 13 Feb 2022 17:34:38 +0200 From: Ido Schimmel To: Domenico Andreoli Cc: Luis Chamberlain , Tong Zhang , "Eric W. Biederman" , David Airlie , Andrew Morton , amir73il@gmail.com, Andy Shevchenko , Arnd Bergmann , bcrl@kvack.org, benh@kernel.crashing.org, clemens@ladisch.de, crope@iki.fi, dgilbert@interlog.com, Greg KH , jack@suse.cz, jani.nikula@intel.com, jani.nikula@linux.intel.com, "James E.J. Bottomley" , jlbec@evilplan.org, john.ogness@linutronix.de, joonas.lahtinen@linux.intel.com, Joseph Qi , julia.lawall@inria.fr, Kees Cook , kernel@tuxforce.de, Linux Memory Management List , mark@fasheh.com, "Martin K. Petersen" , mm-commits@vger.kernel.org, nixiaoming@huawei.com, penguin-kernel@i-love.sakura.ne.jp, peterz@infradead.org, phil@philpotter.co.uk, pjt@google.com, pmladek@suse.com, rafael@kernel.org, rodrigo.vivi@intel.com, rostedt@goodmis.org, senozhatsky@chromium.org, sre@kernel.org, steve@sk2.org, surenb@google.com, torvalds@linux-foundation.org, tytso@mit.edu, Al Viro , wangqing@vivo.com, Iurii Zaikin , open list , regressions@lists.linux.dev Subject: Re: [PATCH v2] Fix regression due to "fs: move binfmt_misc sysctl to its own file" Message-ID: References: <20220121221021.60533b009c357d660791476e@linux-foundation.org> <20220122061228.nmuo75sDn%akpm@linux-foundation.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Stat-Signature: ooo9k7nyj1qmhy6fmtir7edqs55gsxne X-Rspamd-Server: rspam02 X-Rspamd-Queue-Id: BF9961A0006 Authentication-Results: imf19.hostedemail.com; dkim=pass header.d=messagingengine.com header.s=fm2 header.b=ahRWviea; dmarc=none; spf=none (imf19.hostedemail.com: domain of idosch@idosch.org has no SPF policy when checking 66.111.4.224) smtp.mailfrom=idosch@idosch.org X-Rspam-User: X-HE-Tag: 1644766487-373216 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: On Wed, Feb 09, 2022 at 08:49:20AM +0100, Domenico Andreoli wrote: > fs/binfmt_misc.c | 6 +----- > fs/file_table.c | 2 ++ > 2 files changed, 3 insertions(+), 5 deletions(-) > > Index: b/fs/binfmt_misc.c > =================================================================== > --- a/fs/binfmt_misc.c > +++ b/fs/binfmt_misc.c > @@ -817,20 +817,16 @@ static struct file_system_type bm_fs_typ > }; > MODULE_ALIAS_FS("binfmt_misc"); > > -static struct ctl_table_header *binfmt_misc_header; > - > static int __init init_misc_binfmt(void) > { > int err = register_filesystem(&bm_fs_type); > if (!err) > insert_binfmt(&misc_format); > - binfmt_misc_header = register_sysctl_mount_point("fs/binfmt_misc"); > - return 0; > + return err; > } > > static void __exit exit_misc_binfmt(void) > { > - unregister_sysctl_table(binfmt_misc_header); > unregister_binfmt(&misc_format); > unregister_filesystem(&bm_fs_type); > } > Index: b/fs/file_table.c > =================================================================== > --- a/fs/file_table.c > +++ b/fs/file_table.c > @@ -119,6 +119,8 @@ static struct ctl_table fs_stat_sysctls[ > static int __init init_fs_stat_sysctls(void) > { > register_sysctl_init("fs", fs_stat_sysctls); > + if (IS_ENABLED(CONFIG_BINFMT_MISC)) > + register_sysctl_mount_point("fs/binfmt_misc"); Hi, kmemleak complains about this: # cat /sys/kernel/debug/kmemleak unreferenced object 0xffff8881045fea88 (size 96): comm "swapper/0", pid 1, jiffies 4294669355 (age 167.804s) hex dump (first 32 bytes): e0 c8 07 88 ff ff ff ff 00 00 00 00 01 00 00 00 ................ 01 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ................ backtrace: [] __register_sysctl_table+0x117/0x1150 [] init_fs_stat_sysctls+0x30/0x33 [] do_one_initcall+0x108/0x690 [] kernel_init_freeable+0x45a/0x4de [] kernel_init+0x1f/0x220 [] ret_from_fork+0x1f/0x30 > return 0; > } > fs_initcall(init_fs_stat_sysctls); >