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 X-Spam-Level: X-Spam-Status: No, score=-3.5 required=3.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 4EF13C433B4 for ; Fri, 2 Apr 2021 14:49:27 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id E17DA61006 for ; Fri, 2 Apr 2021 14:49:26 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org E17DA61006 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=infradead.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 50DF96B0071; Fri, 2 Apr 2021 10:49:26 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 4BCFD6B0073; Fri, 2 Apr 2021 10:49:26 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 384416B0074; Fri, 2 Apr 2021 10:49:26 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0252.hostedemail.com [216.40.44.252]) by kanga.kvack.org (Postfix) with ESMTP id 1EE406B0071 for ; Fri, 2 Apr 2021 10:49:26 -0400 (EDT) Received: from smtpin15.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay03.hostedemail.com (Postfix) with ESMTP id D54078047A86 for ; Fri, 2 Apr 2021 14:49:25 +0000 (UTC) X-FDA: 77987710290.15.53895DD Received: from casper.infradead.org (casper.infradead.org [90.155.50.34]) by imf24.hostedemail.com (Postfix) with ESMTP id 9BB61A000395 for ; Fri, 2 Apr 2021 14:49:23 +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=fBsK7nTwBRh7zmZwlhR0hgVopltJEFGA41hXUoQ3g6Y=; b=HvUSmtVTxyIGtGp8LpJtiW6fHl e39Q6+s0X8R/zZdPF+774ETc5C3kK43KI5ZO/HVI5vkYqUCY8Ak7IWGlqnT1aPHe69hbIKzUpMYpF 5wkZg+M7GaiepYGmn89B6l4LamHDDq1tHPIzkTcSexsDgS7u8GSwbcm30HV/+0pQHcVL7V2bn1yfR kxSJCP5BFHzGw+abkUg22slEZxYS4D5Y99peIaVvL07aCaCTt1qLvvxkkyCBcB7VP3mUNxVv+uZkm zIoMJvmv0PSmHy7Wj+XwTTKFK0AiD2Xx+B97jGZoxxx56rtRh/qsJpYdPRu3o9HEMF1mLKOXeGxiN sAcEN/Xg==; Received: from willy by casper.infradead.org with local (Exim 4.94 #2 (Red Hat Linux)) id 1lSL6p-007l0E-W1; Fri, 02 Apr 2021 14:49:17 +0000 Date: Fri, 2 Apr 2021 15:49:15 +0100 From: Matthew Wilcox To: Christoph Hellwig Cc: linux-mm@kvack.org, linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-cachefs@redhat.com, linux-afs@lists.infradead.org Subject: Re: [PATCH v6 00/27] Memory Folios Message-ID: <20210402144915.GP351017@casper.infradead.org> References: <20210331184728.1188084-1-willy@infradead.org> <20210401070537.GB1363493@infradead.org> <20210401112656.GA351017@casper.infradead.org> <20210402143755.GA1843620@infradead.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210402143755.GA1843620@infradead.org> X-Rspamd-Queue-Id: 9BB61A000395 X-Stat-Signature: uu8mjzwxdhpoqzteyutskaz5r573qt5z X-Rspamd-Server: rspam02 Received-SPF: none (infradead.org>: No applicable sender policy available) receiver=imf24; identity=mailfrom; envelope-from=""; helo=casper.infradead.org; client-ip=90.155.50.34 X-HE-DKIM-Result: pass/pass X-HE-Tag: 1617374963-304848 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 Fri, Apr 02, 2021 at 03:37:55PM +0100, Christoph Hellwig wrote: > On Thu, Apr 01, 2021 at 12:26:56PM +0100, Matthew Wilcox wrote: > > On Thu, Apr 01, 2021 at 08:05:37AM +0100, Christoph Hellwig wrote: > > > On Wed, Mar 31, 2021 at 07:47:01PM +0100, Matthew Wilcox (Oracle) wrote: > > > > - Mirror members of struct page (for pagecache / anon) into struct folio, > > > > so (eg) you can use folio->mapping instead of folio->page.mapping > > > > > > Eww, why? > > > > So that eventually we can rename page->mapping to page->_mapping and > > prevent the bugs from people doing page->mapping on a tail page. eg > > https://lore.kernel.org/linux-mm/alpine.LSU.2.11.2103102214170.7159@eggly.anvils/ > > I'm not sure I like this. This whole concept of structures that do need > the same layout is very problematic, even with the safe guards you've > added. So if it was up to me I'd prefer the folio as a simple container > as it was in the previous revisions. At some point members should move > from the page to the folio, but I'd rather do that over a shorter period > an in targeted series. We need the basic to go in first. That was my original plan, but it'll be another round of churn, and I'm not sure there'll be the appetite for it. There's not a lot of appetite for this round, and this one has measurable performance gains!