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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id BB31AC77B6F for ; Thu, 13 Apr 2023 12:06:36 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229636AbjDMMGf (ORCPT ); Thu, 13 Apr 2023 08:06:35 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:60892 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229530AbjDMMGd (ORCPT ); Thu, 13 Apr 2023 08:06:33 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 17BA2269E; Thu, 13 Apr 2023 05:06:32 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id A2CF76151A; Thu, 13 Apr 2023 12:06:31 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 32A6EC433EF; Thu, 13 Apr 2023 12:06:29 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1681387591; bh=hz2w5nal7eVLlA7Xa6WHUXAtrQ6t9Pp72sNtrEuy3o4=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=snY5lLk1fn1oIcjN4BkdeAWylnRoDON8emgSmmuYmNxxElnJsyCBtnSZiUAGCib0t /xGfIoo/VpGtAA/ViWp1ZFxy848h9vALpU1euR51Elhk2OSLOppmaJHLUGZHnbxwBT rB9C7TBUQV1qK2/YABH3aWxqFFHVLsAsLYsckrsI1KbWm+WYeK3zw3m0WUsouucsQc 8Z7jM4J30zrwcNomyHfhb8I8yDVF2DzKu2bHpCeFyAPkeL+nEjMLWxO6PTjoyf4CkX KiQ0Ncjbpvhoznd2ltc/pQ70gx8stRVL7WtYFxPtekaHRUxxTeE1DJ6eHDe6xHyHH4 ILXUOQ+aVkmkg== Date: Thu, 13 Apr 2023 14:06:24 +0200 From: Christian Brauner To: Kefeng Wang Cc: Luis Chamberlain , linux-fsdevel@vger.kernel.org, Andrew Morton , linux-kernel@vger.kernel.org, Alexander Viro Subject: Re: [PATCH] fs: fix sysctls.c built Message-ID: <20230413-gepfiffen-hochphase-e98047bd16bd@brauner> References: <20230331084502.155284-1-wangkefeng.wang@huawei.com> <66c0e8b6-64d1-5be6-cd4d-9700d84e1b84@huawei.com> <20230412-sympathie-haltbar-da2d2183067b@brauner> <20230413-kufen-infekt-02c7eb2a9adc@brauner> <3c98309c-eb56-1aee-5a6a-8f0a4c84d378@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <3c98309c-eb56-1aee-5a6a-8f0a4c84d378@huawei.com> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Apr 13, 2023 at 05:45:20PM +0800, Kefeng Wang wrote: > > > On 2023/4/13 16:35, Christian Brauner wrote: > > On Thu, Apr 13, 2023 at 09:34:54AM +0800, Kefeng Wang wrote: > > > > > > > > > On 2023/4/12 17:19, Christian Brauner wrote: > > > > On Tue, Apr 11, 2023 at 12:14:44PM +0800, Kefeng Wang wrote: > > > > > /proc/sys/fs/overflowuid and overflowgid will be lost without > > > > > building this file, kindly ping, any comments, thanks. > > > > > > > > > > > > > > > On 2023/3/31 16:45, Kefeng Wang wrote: > > > > > > 'obj-$(CONFIG_SYSCTL) += sysctls.o' must be moved after "obj-y :=", > > > > > > or it won't be built as it is overwrited. > > > > > > ... > > > > > > > Given the description in > > > > ab171b952c6e ("fs: move namespace sysctls and declare fs base directory") > > > > you probably want to move this earlier. > > > > > > Oh, missing that part, but the /proc/sys/fs/overflowuid and overflowgid are > > > lost after it, is it expected? Luis, could you take a look? thanks. > > > > No, /proc/sys/fs/overflow{g,u}id need to be there of course. What I mean > > is something like the following (similar to how net/core/ does it): > > > > Thanks for your explanation, > > > UNTESTED > > Will try, > > > --- > > fs/Makefile | 24 +++++++++++++----------- > > 1 file changed, 13 insertions(+), 11 deletions(-) > > > > diff --git a/fs/Makefile b/fs/Makefile > > index 05f89b5c962f..dfaea8a28d92 100644 > > --- a/fs/Makefile > > +++ b/fs/Makefile > > @@ -6,17 +6,19 @@ > > # Rewritten to use lists instead of if-statements. > > # > > > > -obj-$(CONFIG_SYSCTL) += sysctls.o > > - > > -obj-y := open.o read_write.o file_table.o super.o \ > > - char_dev.o stat.o exec.o pipe.o namei.o fcntl.o \ > > - ioctl.o readdir.o select.o dcache.o inode.o \ > > - attr.o bad_inode.o file.o filesystems.o namespace.o \ > > - seq_file.o xattr.o libfs.o fs-writeback.o \ > > - pnode.o splice.o sync.o utimes.o d_path.o \ > > - stack.o fs_struct.o statfs.o fs_pin.o nsfs.o \ > > - fs_types.o fs_context.o fs_parser.o fsopen.o init.o \ > > - kernel_read_file.o mnt_idmapping.o remap_range.o > > +obj-y := fs_types.o > > +obj-$(CONFIG_SYSCTL) += sysctls.o > > +obj-y += open.o read_write.o file_table.o super.o \ > > + char_dev.o stat.o exec.o pipe.o namei.o \ > > + fcntl.o ioctl.o readdir.o select.o dcache.o \ > > + inode.o attr.o bad_inode.o file.o \ > > + filesystems.o namespace.o seq_file.o \ > > + xattr.o libfs.o fs-writeback.o pnode.o \ > > + splice.o sync.o utimes.o d_path.o stack.o \ > > + fs_struct.o statfs.o fs_pin.o nsfs.o \ > > + fs_types.o fs_context.o fs_parser.o \ > > drop this fs_types.o ? Yes.