Received: by 2002:a05:6a11:4021:0:0:0:0 with SMTP id ky33csp239688pxb; Wed, 15 Sep 2021 00:31:50 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzlOgJ6Xb1drmEq4CW6InTSAPFeX0gp2oAXyLW6W4Bkve1Ngv5FOJ3B4ki+2a/ctbMw+hwM X-Received: by 2002:a05:6602:2c07:: with SMTP id w7mr16999352iov.122.1631691110342; Wed, 15 Sep 2021 00:31:50 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1631691110; cv=none; d=google.com; s=arc-20160816; b=WGGzUtbj1MNXyLa7ZhvP6Y6PWQ7K+LZo7MY/sWbJu1B6cStK8a9fj76YIcwNGdJWOb QlmUA0Ydc37qhfJqZWX3O8mucx+At1mjc284UYXA7xjpogEUV0REzKMViMJMjn2aONTc tD3VMfxwyzGFMMYMlcihBsGsKU6YJWGA1Xe2UhMPEUUtrX6EaVTrlcb0QMIIjKqaSEbW 6mQe17dE1fAD1+eZQu+lteMq4HUjWXCMdN38x0RVRAQ9fcpmdWsHHGzlxow+nvOfsaYS rPXrEiWSHqvK6neU4w8/w3Vd4IuGseo7+qhueazBmY92faiOYPxk5N1/YGwZF3U6cylk kEZg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:subject:message-id:date:from:in-reply-to :references:mime-version:dkim-signature; bh=ansDI2ALwiWUvo/BHMzSUNbG1zi7g1Z2RW2yT9Z/nQU=; b=M+yIIlMqGCCw4iCZQ3GmEb4wNIgremyTuNh4RqpVnlz54KEhXMrAtn2ggitdpqzuU4 ATanPkE/2LyW1i+d01mKQ58eYZy9iB7faNmXWDVY6IJVGtj+4xkolnpQKsTWf4yefnXv s8N1bSJxllVQlE02NnArl1Gnt59wb5foi0RJDkyPe7588KUyizPcIdPwq5xFA09I15VK M5N34o7RduHtnEVwQGPO3BiKoI0CC5wuum/PYWw4j6EBvptBeshb6iKfBVGX107tcelV v5yb/s8IEoQUIybmxC6/FyaRoS9/RSzfIb/qzf7gbGjwa0kD+GD/w1mqT1fuuOPKWns7 QabQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@bytedance-com.20210112.gappssmtp.com header.s=20210112 header.b=ORu8quCr; spf=pass (google.com: domain of linux-nfs-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-nfs-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=bytedance.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id q22si11795263jae.93.2021.09.15.00.30.58; Wed, 15 Sep 2021 00:31:50 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-nfs-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@bytedance-com.20210112.gappssmtp.com header.s=20210112 header.b=ORu8quCr; spf=pass (google.com: domain of linux-nfs-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-nfs-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=bytedance.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236514AbhIOHcJ (ORCPT + 99 others); Wed, 15 Sep 2021 03:32:09 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:47632 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231357AbhIOHcI (ORCPT ); Wed, 15 Sep 2021 03:32:08 -0400 Received: from mail-pg1-x52e.google.com (mail-pg1-x52e.google.com [IPv6:2607:f8b0:4864:20::52e]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6C205C061575 for ; Wed, 15 Sep 2021 00:30:50 -0700 (PDT) Received: by mail-pg1-x52e.google.com with SMTP id 17so1845447pgp.4 for ; Wed, 15 Sep 2021 00:30:50 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bytedance-com.20210112.gappssmtp.com; s=20210112; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=ansDI2ALwiWUvo/BHMzSUNbG1zi7g1Z2RW2yT9Z/nQU=; b=ORu8quCr4ulyj8peI8bv8LIWYdxUyRLHamjFT4YgcAyCXXSJdSEDIA5RUxfAmRN09J tY/7T5bYctY95S7e+fwttxX+NNE5jGnbTpmByrsUudiTX9yWUMLlenKQBt/TixdSRPsZ LrlLHUqNy1XM3U3BySYeH/YSkyih1AlxBf3CpwC7vZvdDR4Blb2RYh6a1Bq9HCVhkR1+ E0XG1SnRFmOSibxEc1rd4fVF8VCvFA6y+XiaAfVwVYGGx/EljBv00XVcINk3zbgbFI37 Na9ZrjmuFHRhntxtlqxBJbwI9WZHCcDwJ5mTZqO8dkI54BjWnhGuTajMW1ZdHWDAO0jc U2mQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=ansDI2ALwiWUvo/BHMzSUNbG1zi7g1Z2RW2yT9Z/nQU=; b=BI/t3bWEIKH5R5noqhW/eyxr9mPRVnx/YzL6DTDctXVkr7/KN77eebTNaLYwMhg6Qy cOKvXnMpzPg9c1C254dejJT4dAYSqkDUBSfxOn/MxUnzOC7vv6Ig9LqGKwRUjrxRimyw LPaQZfr6MrwjWerwZwxBjVRgbTz8o5W3UeHJswoqYRz0e2tYvHEOJZISdN1mt1KGGlEr 2UAH+KsRbg71lShtWZXhdGmqEEumCmnVYn4TZsJP5t0YilfGJv/isX2jVGXIyP7rl/Et Ebu3UK372gaxcD0XEKyZRjczFiuXA2SNqFPQt9EJyDRSmSLaJ9ASWRIZ4Ac9NIkyPD+f 9TeA== X-Gm-Message-State: AOAM532p/i4Hn2bnTJMgNbATovthFPyYKZ3Qtw6yfcBV998618WQ4e9S EBq1/yx7HI9v/ZSG90D/nUg9HX3nsiUp4wYSj/YUuw== X-Received: by 2002:aa7:9e8d:0:b0:406:be8:2413 with SMTP id p13-20020aa79e8d000000b004060be82413mr9157759pfq.66.1631691049890; Wed, 15 Sep 2021 00:30:49 -0700 (PDT) MIME-Version: 1.0 References: <20210914072938.6440-1-songmuchun@bytedance.com> In-Reply-To: From: Muchun Song Date: Wed, 15 Sep 2021 15:30:12 +0800 Message-ID: Subject: Re: [PATCH v3 00/76] Optimize list lru memory consumption To: "Theodore Ts'o" Cc: Matthew Wilcox , Andrew Morton , Johannes Weiner , Michal Hocko , Vladimir Davydov , Shakeel Butt , Roman Gushchin , Yang Shi , Alex Shi , Wei Yang , Dave Chinner , trond.myklebust@hammerspace.com, anna.schumaker@netapp.com, linux-fsdevel , LKML , Linux Memory Management List , linux-nfs@vger.kernel.org, Qi Zheng , Xiongchun duan , fam.zheng@bytedance.com, Muchun Song Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-nfs@vger.kernel.org On Wed, Sep 15, 2021 at 4:23 AM Theodore Ts'o wrote: > > On Tue, Sep 14, 2021 at 03:28:22PM +0800, Muchun Song wrote: > > So we have to convert to new API for all filesystems, which is done in > > one patch. Some filesystems are easy to convert (just replace > > kmem_cache_alloc() to alloc_inode_sb()), while other filesystems need to > > do more work. > > From what I can tell, three are 54 file systems for which it was a > trivial one-line change, and two (f2fs and nfs42) that were a tad bit > more complex. Definitely right. Thanks for your clarification. > > > In order to make it easy for maintainers of different > > filesystems to review their own maintained part, I split the patch into > > patches which are per-filesystem in this version. I am not sure if this > > is a good idea, because there is going to be more commits. > > What I'd actually suggest is that you combine all of the trivial file > system changes into a single commit, and keep the two more complex > changes for f2fs and nfs42 in separate commits. Got it. Will do in the next version. > > Acked-by: Theodore Ts'o Thanks. > > ... for the ext4 related change. > > - Ted >