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 mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 2882AC433F5 for ; Sat, 9 Oct 2021 20:17:48 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 6CE3760F6D for ; Sat, 9 Oct 2021 20:17:47 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 6CE3760F6D Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=infradead.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=kvack.org Received: by kanga.kvack.org (Postfix) id A3808900002; Sat, 9 Oct 2021 16:17:46 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 9E7D26B0073; Sat, 9 Oct 2021 16:17:46 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 92F0F6B006C; Sat, 9 Oct 2021 16:17:46 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0200.hostedemail.com [216.40.44.200]) by kanga.kvack.org (Postfix) with ESMTP id 844066B006C for ; Sat, 9 Oct 2021 16:17:46 -0400 (EDT) Received: from smtpin01.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay01.hostedemail.com (Postfix) with ESMTP id 39CD518116E12 for ; Sat, 9 Oct 2021 20:17:46 +0000 (UTC) X-FDA: 78678009732.01.B01EF45 Received: from casper.infradead.org (casper.infradead.org [90.155.50.34]) by imf06.hostedemail.com (Postfix) with ESMTP id BA9B5801BEC0 for ; Sat, 9 Oct 2021 20:17:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=casper.20170209; 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=QCahwkB1CqDtILURVeqMuorUmriTCbVy3djEQFfSyiI=; b=FZ/0xfnHz/A7L+ZVNvwuiGfM0y uL6tehn83mI4lK5C5+Z0le/6YHz3DicNN8TsgUkOXKSocLrkU6un1Fakb2/A3JKh5GEm1XudzRFwK rLsCUHfo9ae4C3eiivJOmpwy4xLQ4JTdUqVDDvS3fHvNH/4lVNU5piqyozccRvNdgdhg0hOBWvV/y tFgH1X3fRv3si0Y7RHTlmSBIGece5oa/lw7nl3EDNh3AgGE7rur2H+vFVuciGEr5zo5Ofr/iTVb5b RsAVUPdUfXMdTIsEBQuQvh6lM3Mb06SgUf1VOdFmZcc0mo1XrcQa/l/CJZBY1JNfgSfNxBksV7Mvf QYA7FWww==; Received: from willy by casper.infradead.org with local (Exim 4.94.2 #2 (Red Hat Linux)) id 1mZIkj-004NjC-8Q; Sat, 09 Oct 2021 20:15:44 +0000 Date: Sat, 9 Oct 2021 21:15:29 +0100 From: Matthew Wilcox To: John Hubbard Cc: Yunsheng Lin , davem@davemloft.net, kuba@kernel.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, linuxarm@openeuler.org, akpm@linux-foundation.org, hawk@kernel.org, ilias.apalodimas@linaro.org, peterz@infradead.org, yuzhao@google.com, will@kernel.org, jgg@ziepe.ca, mcroce@microsoft.com, willemb@google.com, cong.wang@bytedance.com, pabeni@redhat.com, haokexin@gmail.com, nogikh@google.com, elver@google.com, memxor@gmail.com, vvs@virtuozzo.com, linux-mm@kvack.org, edumazet@google.com, alexander.duyck@gmail.com, dsahern@gmail.com Subject: Re: [PATCH net-next -v5 3/4] mm: introduce __get_page() and __put_page() Message-ID: References: <20211009093724.10539-1-linyunsheng@huawei.com> <20211009093724.10539-4-linyunsheng@huawei.com> <62106771-7d2a-3897-c318-79578360a88a@nvidia.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <62106771-7d2a-3897-c318-79578360a88a@nvidia.com> X-Rspamd-Server: rspam05 X-Rspamd-Queue-Id: BA9B5801BEC0 X-Stat-Signature: y1xhhu3c1z5usg4jih8m3pe59nmkh837 Authentication-Results: imf06.hostedemail.com; dkim=pass header.d=infradead.org header.s=casper.20170209 header.b="FZ/0xfnH"; dmarc=none; spf=none (imf06.hostedemail.com: domain of willy@infradead.org has no SPF policy when checking 90.155.50.34) smtp.mailfrom=willy@infradead.org X-HE-Tag: 1633810665-189854 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 Sat, Oct 09, 2021 at 12:49:29PM -0700, John Hubbard wrote: > On 10/9/21 02:37, Yunsheng Lin wrote: > > Introduce __get_page() and __put_page() to operate on the > > base page or head of a compound page for the cases when a > > page is known to be a base page or head of a compound page. > > Hi, > > I wonder if you are aware of a much larger, 137-patch seriesto do that: > folio/pageset [1]? > > The naming you are proposing here does not really improve clarity. There > is nothing about __get_page() that makes it clear that it's meant only > for head/base pages, while get_page() tail pages as well. And the > well-known and widely used get_page() and put_page() get their meaning > shifted. > > This area is hard to get right, and that's why there have been 15 > versions, and a lot of contention associated with [1]. If you have an > alternate approach, I think it would be better in its own separate > series, with a cover letter that, at a minimum, explains how it compares > to folios/pagesets. I wasn't initially sure whether network pagepools should be part of struct folio or should be their own separate type. At this point, I think they should be a folio. But that's all kind of irrelevant until Linus decides whether he's going to take the folio patchset or not. Feel free to let him know your opinion when the inevitable argument blows up again around the next pull request.