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 05D3FC77B75 for ; Tue, 18 Apr 2023 10:29:50 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 6274C6B0071; Tue, 18 Apr 2023 06:29:49 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 5D7566B0072; Tue, 18 Apr 2023 06:29:49 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 49EAA8E0001; Tue, 18 Apr 2023 06:29:49 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0013.hostedemail.com [216.40.44.13]) by kanga.kvack.org (Postfix) with ESMTP id 3BFE76B0071 for ; Tue, 18 Apr 2023 06:29:49 -0400 (EDT) Received: from smtpin13.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay01.hostedemail.com (Postfix) with ESMTP id F163E1C5E82 for ; Tue, 18 Apr 2023 10:29:48 +0000 (UTC) X-FDA: 80694140856.13.426B769 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by imf21.hostedemail.com (Postfix) with ESMTP id B20411C001B for ; Tue, 18 Apr 2023 10:29:46 +0000 (UTC) Authentication-Results: imf21.hostedemail.com; dkim=pass header.d=redhat.com header.s=mimecast20190719 header.b=DX5VujJb; dmarc=pass (policy=none) header.from=redhat.com; spf=pass (imf21.hostedemail.com: domain of david@redhat.com designates 170.10.129.124 as permitted sender) smtp.mailfrom=david@redhat.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1681813786; 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:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=/WVe/7/OKmghxefKS6nQkLQJNjNIEtm/g7km3sbxUMI=; b=G436W+2DQn52BN8nwxPBB13NSCYVONCTsJ+fJMe7ifdx/7R6viNOQVrQCDsCgNTB06Jvkg JXHUTdvNFfbVV+0tfcNVoGfT0q1Q8WbLpxo+aSm66S4dO9PhKqQGOff1u1A/my8BhzupMo ILeZbV1U22RJGEGXh3BcW6Jl+RIiW64= ARC-Authentication-Results: i=1; imf21.hostedemail.com; dkim=pass header.d=redhat.com header.s=mimecast20190719 header.b=DX5VujJb; dmarc=pass (policy=none) header.from=redhat.com; spf=pass (imf21.hostedemail.com: domain of david@redhat.com designates 170.10.129.124 as permitted sender) smtp.mailfrom=david@redhat.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1681813786; a=rsa-sha256; cv=none; b=2kjbT1fa+ZDqWZYoyOJj9aSr/YJVpYrHHU9eyBsjogEbX8nrJtGt/AGkOgLp8zZx8GG9f+ xCz4fjcf8lzBdwnAn/UgnWal+0L7iXuD1DxrCjSq81QavFMkO5R7leBcODPgIxKaPHmtFl hYxaHXX0qdIYYhLnaM4Bdt8NeFI8ZSA= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1681813786; h=from:from: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:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=/WVe/7/OKmghxefKS6nQkLQJNjNIEtm/g7km3sbxUMI=; b=DX5VujJb2hS5sYJCShP0nYNhCNsHjXcfvYuwnxIxhby9H7tRsZr3mW+FnC1gVFwnwzpCl/ yJAFOc5tzxy8yUKjK/h8dTNc9DSvBpADLslbd0r48gz3SRQn7UJijrRR2fgGLoPhAIOExm y7zTz+DJRDmw6jacHKcfwa1L9Vvzplg= Received: from mail-wr1-f70.google.com (mail-wr1-f70.google.com [209.85.221.70]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-99-Cs8R5aWTMFCpBEgjgqz4_A-1; Tue, 18 Apr 2023 06:29:44 -0400 X-MC-Unique: Cs8R5aWTMFCpBEgjgqz4_A-1 Received: by mail-wr1-f70.google.com with SMTP id ffacd0b85a97d-2f6bd453d64so1335010f8f.3 for ; Tue, 18 Apr 2023 03:29:44 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1681813783; x=1684405783; h=content-transfer-encoding:in-reply-to:organization:from:references :cc:to:content-language:subject:user-agent:mime-version:date :message-id:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=/WVe/7/OKmghxefKS6nQkLQJNjNIEtm/g7km3sbxUMI=; b=lVx3H3RAq7jfcS24TNmCmBwoptVcJxQRw1I/p3j/8BVuayrIIqg3HKDt7NlKC2p+iT tuq8e91DmsURZ4kR50JanpMMeTRLq0Hb7O6fkqxVqtupEjtF2w89aTaKWyVfz2IesPRv YQW1tQSLUT5Eln/BlEdV3zYTIT0nPC+R59xeOTuz5MvR6fkskywY24sISX9BT28n9lFx KY61rhGaBJzyvGtW/hh6yyBbYM987jaQaZ0qQ4j7djex/LX08hRZDfoZ7L8CTqZjkehf txdVmhIB8aLVI24LbqVJqicSuTnyS6Cp+2t/fgat/VL+/aqd1oywCo5X1ubLg+8tGWLw KgtA== X-Gm-Message-State: AAQBX9eSBAAnXjk+gpR2oIUuvtKgk98oZCk8JIpZBgCvB6FBfyupqcK0 +C+InNfebWz05nhPMU9OUB14t6dApHqkGA0BKthNKb5egh986P1FkGGLDdmAMZll/nI+sC1TK9B 7mKloeKcUU24= X-Received: by 2002:adf:f30d:0:b0:2f7:af3a:72fd with SMTP id i13-20020adff30d000000b002f7af3a72fdmr1467658wro.59.1681813783677; Tue, 18 Apr 2023 03:29:43 -0700 (PDT) X-Google-Smtp-Source: AKy350Zxg1+fvSmfIW+1kD2tgmPT7fUEB9YPbp5R/lSUC8reXHVDmWAT7newxKQPKKwoaA2tgk7thg== X-Received: by 2002:adf:f30d:0:b0:2f7:af3a:72fd with SMTP id i13-20020adff30d000000b002f7af3a72fdmr1467646wro.59.1681813783300; Tue, 18 Apr 2023 03:29:43 -0700 (PDT) Received: from ?IPV6:2003:cb:c715:3f00:7545:deb6:f2f4:27ef? (p200300cbc7153f007545deb6f2f427ef.dip0.t-ipconnect.de. [2003:cb:c715:3f00:7545:deb6:f2f4:27ef]) by smtp.gmail.com with ESMTPSA id j4-20020a5d5644000000b002f02df4c7a3sm12725811wrw.30.2023.04.18.03.29.41 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 18 Apr 2023 03:29:42 -0700 (PDT) Message-ID: Date: Tue, 18 Apr 2023 12:29:41 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.9.1 Subject: Re: [PATCH v3 0/7] Split a folio to any lower order folios To: Zi Yan Cc: Hugh Dickins , Andrew Morton , "Matthew Wilcox (Oracle)" , Yang Shi , Yu Zhao , linux-mm@kvack.org, "Kirill A . Shutemov" , Ryan Roberts , =?UTF-8?Q?Michal_Koutn=c3=bd?= , Roman Gushchin , Zach O'Keefe , linux-kernel@vger.kernel.org, cgroups@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-kselftest@vger.kernel.org References: <20230403201839.4097845-1-zi.yan@sent.com> <20230404144727.e613116684dbd65a4b4745c1@linux-foundation.org> <49ee481e-452f-61c7-2da5-28de2cf3de2@google.com> <8B784026-7C90-427D-91E2-2D551FA576D7@nvidia.com> From: David Hildenbrand Organization: Red Hat In-Reply-To: <8B784026-7C90-427D-91E2-2D551FA576D7@nvidia.com> X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Language: en-US Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Rspam-User: X-Rspamd-Server: rspam02 X-Rspamd-Queue-Id: B20411C001B X-Stat-Signature: jkkjdqja94ymetb7cndenzgpzx6re5sh X-HE-Tag: 1681813786-31960 X-HE-Meta: U2FsdGVkX18E83xy24nt1OzoJN3VZokgSGoRRZCVZjyle/Nu3RnH6aZnZCXHn0EEx5zC/ta+BOtAWRyJrPgoTv8zoIhpHHlkZLPRhWtU1j42AJyEQt0Fl4hMmaUwjGAXGOzXjU2ogTg6Xvv1zMMiTWpqjcV45XgbGzpK68d+r2IWgUlKbQ/gaa6JSmbahSs9h0aPQfXHHGXxFl085Xb+aohmPK1HLHLHGAdLAHKPlC6H59D1mXXDZMUN8Q0E2h0/uv+sbhIzSP0A9z155vlz4lfLj+ZbgDqchsskrVfaa/+HZbVXGaYPnmo9RFWsiowM9A7oOjFwL768uzY43j7d+oXaVqFs1Uzcpx+U7X8XX+vVVIEEZpdrKE80clLoWvIiE48QwJtrbib26Tu2hWGqroMIX/lYWhHnNNvWDo2OMDGRIweh0xT2s1IjQXKR20DeRLWQkbLkdqI8lmbJNXr1kj84Fdo8i6XxyVTHiaz1rWit7Nu94BpNIPP/I2/d3zhnIlWbB03GtyjlcI7sQyEgicetkm/ZETQdNNlHDyOlF1/GEoP9f2ZxIV8rSStMooaaarYLhNhPQ3jvOMZqlJK5gonvUgJ+UWKs0hXFR/v8+6TDC9X8ZN4a9psTdPUe9QYOQEvtUgV/jiypLRMb4h98mFUaTRKjyX2ZdqXKF6f4eDjKMauG/MeoA8sVp5/zTUp9s97owg4/YRUQWLDXhRlKPY1denZJFfJNvoj/CjJn2APv4VFFh1x+mE9g15LQt8hKvkd+lDCMtsL5GlKv78asxjs9Mo+FcWCEAGv4xsxO9a7M5oaFtESRGlJgSbFoqeVzKTYSTc8Xi/YfxgfdMukOfcPSGrgWKWB70OPownlMYu8lUelHS8Nz2WSp7H1DpRKmQI7/TU91oD7VT6DdQ2MjWKvfGIpo/bDdfu8KuGzPrRodYCZbE6U8C2chMDHD7a6RrNbCg4LPMZsDUP4+vTF KzLaS3pI vxRIX6mcNDdoo/x1v/BrcQe79aH0C43qGP2iI3Jy3J72ic7hrqZ5T7rKwjHxOCNZMsDaOOT58p5CUpttDKZ2WpTA+wamwItThUwGaXYO700RmN/Gy2e9P4LDoi+qJCXF03MgN/kiAbhfwA5D65Su1uV+Adr05P5fD2UnVDoS2YBrS9tqfNcjLbyq/2aos66o34UF9bBrAlcRP//Hu/SmG0ro7YHWjy5TxKgme44ngpr2GrnUV/8TtYocVWKJnTSIitsNNRfkSORqioitYDC4jW4vrk0ezrEYhD/T40sb7VEaCNncMHD2+hYABL7Lo1VITBVksBF6GA+aRUGedYQSPZ7pm4GMHD1s4qFT5oLaI8pav+MQLdAKxjtDMPft3yNS/sLYc6qJFWbqF3MFOCu2JxQyzDvZIqJ8iH5YgvO1JWo+jfK3BPHhMja5GMZH7wiD9zTLhNTo/vk7OD/q5KMu6KC3NxQ== 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 17.04.23 21:26, Zi Yan wrote: > On 17 Apr 2023, at 10:20, David Hildenbrand wrote: > >> On 16.04.23 20:11, Hugh Dickins wrote: >>> On Tue, 4 Apr 2023, Andrew Morton wrote: >>>> On Mon, 3 Apr 2023 16:18:32 -0400 Zi Yan wrote: >>>> >>>>> File folio supports any order and people would like to support flexible orders >>>>> for anonymous folio[1] too. Currently, split_huge_page() only splits a huge >>>>> page to order-0 pages, but splitting to orders higher than 0 is also useful. >>>>> This patchset adds support for splitting a huge page to any lower order pages >>>>> and uses it during file folio truncate operations. >>>> >>>> This series (and its v1 & v2) don't appear to have much in the way of >>>> detailed review. As it's at v3 and has been fairly stable I'll queue >>>> it up for some testing now, but I do ask that some reviewers go through >>>> it please. >>> >>> Andrew, please don't let this series drift into 6.4-rc1. >>> >>> I've seen a bug or two (I'll point out in response to those patches), >>> but overall I don't see what the justification for the series is: done >>> because it could be done, it seems to me, but liable to add surprises. >>> >>> The cover letter says "splitting to orders higher than 0 is also useful", >>> but it's not clear why; and the infrastructure provided seems unsuited >>> to the one use provided - I'll say more on that truncation patch. >> >> I agree. Maybe this patch set is something we want to have in the future once actual consumers that can benefit are in place, such that we can show actual performance numbers with/without. > > Ryan is working on large folio for anonymous pages and has shown promising performance > results[1]. This patchset would avoid getting base pages during split if possible. > Yes, I know. And it would be great to get some actual numbers with/without your patches to show that this optimization actually matters in practice. Unrelated, your cover letter mentions "file folio truncate operations.". Would it also apply to FALLOC_FL_PUNCH_HOLE, when partially zapping a THP? >> >> Until then, "365 insertions(+), 68 deletions(-)" certainly needs some reasonable motivation. > > Come on. 225 out of 365 insertions are self tests code. We need motivation to add > testing code? Well, if you add feature X and the tests target feature X, then certainly having the tests require the same motivation as feature X. But yeah, the actual kernel code change is smaller than it looks at first sight. -- Thanks, David / dhildenb