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 977F9C7619A for ; Wed, 22 Mar 2023 08:36:01 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229902AbjCVIgA (ORCPT ); Wed, 22 Mar 2023 04:36:00 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:38710 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229670AbjCVIfp (ORCPT ); Wed, 22 Mar 2023 04:35:45 -0400 Received: from bombadil.infradead.org (bombadil.infradead.org [IPv6:2607:7c80:54:3::133]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 9FD205DC89; Wed, 22 Mar 2023 01:35:13 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=bombadil.20210309; h=In-Reply-To:Content-Type:MIME-Version :References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=scGQwlZSm2am6XcIFMQR5FAyHspPLWT/pphajCsVLAM=; b=NZpopG0ww/9PG2arH/g8svd+su MkHAA6xnjEz8GvbL4kxv0uBXV0VhiVpROSp+2jUDYy5e+EwUwe9ENwghNvrohKZJ1ENHudSNjOg9I C2OuaWzfhP2vZXkpmvZKizy5PzN+oVn2FTnWfjsZFSmJ8Zga/lt5TTQVzbwhL2lkGEkbNnVYdhUc2 +dxz5m+UDuzxc4Tmclsmt2cWltp7KHWlX9PUs4nw6fHGXJBYsulB8xOUs1RAqsmZYR90pMBI2423/ 8aWjWrMsbbcM6WyQFTRBEFNxMZmzgrF1EaBSz7sM66vEUK+BnYsPh6IDMv/gjFx5LH8V93T9LHbeZ flUJ7Dug==; Received: from hch by bombadil.infradead.org with local (Exim 4.96 #2 (Red Hat Linux)) id 1petw1-00FCw8-03; Wed, 22 Mar 2023 08:35:05 +0000 Date: Wed, 22 Mar 2023 01:35:04 -0700 From: Christoph Hellwig To: ye.xingchen@zte.com.cn Cc: mcgrof@kernel.org, keescook@chromium.org, yzaikin@google.com, akpm@linux-foundation.org, linmiaohe@huawei.com, chi.minghao@zte.com.cn, linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-mm@kvack.org Subject: Re: [PATCH V5 1/2] mm: compaction: move compaction sysctl to its own file Message-ID: References: <202303221046286197958@zte.com.cn> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <202303221046286197958@zte.com.cn> X-SRS-Rewrite: SMTP reverse-path rewritten from by bombadil.infradead.org. See http://www.infradead.org/rpr.html Precedence: bulk List-ID: X-Mailing-List: linux-fsdevel@vger.kernel.org On Wed, Mar 22, 2023 at 10:46:28AM +0800, ye.xingchen@zte.com.cn wrote: > From: Minghao Chi > > This moves all compaction sysctls to its own file. So there's a whole lot of these 'move sysctrls to their own file' patches, but no actual explanation of why that is desirable. Please explain why we'd want to split code that is closely related, and now requires marking symbols non-static just to create a new tiny source file.