From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=infradead.org header.i=@infradead.org header.b="3ZJHg4Ex" Received: from bombadil.infradead.org (bombadil.infradead.org [IPv6:2607:7c80:54:3::133]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6BD16BE for ; Mon, 11 Dec 2023 23:44:33 -0800 (PST) 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=0m/BbEGfqGhU9IZLa6Rvr7Xy/aZcNJznuBiNVsxcVfA=; b=3ZJHg4ExrXicJFtaAfbVIWgMPY Hl96lApw4s3U7jYW2g9nKrjf/stEbSYDzlOwcpaWK5Xzd2SQ/9TLkslT2nz+X23dOFLLKR2Un7nsn zuoGYUNI1lRwMfLhKKiTyyQWmg9dwWht0I8ImEi0xje06V3M1M7DI+S5xpaF23StSoIwmHGXurbD6 6BMbppd2+XkR+cXuNhkN4eQfKk2PCBqRBL2EY+MQZtn2Iy/UgFGaXEb6vRuIDNA2Vaks2TF8KNnar xfBDh/wYOmtls3cFFWqUeXiyU204ELLi/OxiUMyiJtOHseCovYzajEuhFa9dhTXNjAaYGyBdBJMCl ncraZqGg==; Received: from hch by bombadil.infradead.org with local (Exim 4.96 #2 (Red Hat Linux)) id 1rCxRR-00Awy2-0l; Tue, 12 Dec 2023 07:44:33 +0000 Date: Mon, 11 Dec 2023 23:44:33 -0800 From: Christoph Hellwig To: Matthew Wilcox Cc: Christoph Hellwig , linux-fsdevel@vger.kernel.org Subject: Re: [PATCH] freevxfs: Convert vxfs_immed_read_folio to the new folio APIs Message-ID: References: <20231206204629.771797-1-willy@infradead.org> Precedence: bulk X-Mailing-List: linux-fsdevel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-SRS-Rewrite: SMTP reverse-path rewritten from by bombadil.infradead.org. See http://www.infradead.org/rpr.html On Thu, Dec 07, 2023 at 01:22:03PM +0000, Matthew Wilcox wrote: > On Wed, Dec 06, 2023 at 09:24:03PM -0800, Christoph Hellwig wrote: > > On Wed, Dec 06, 2023 at 08:46:29PM +0000, Matthew Wilcox (Oracle) wrote: > > > Use folio_fill_tail() and folio_end_read() instead of open-coding them. > > > Add a sanity check in case a folio is allocated above index 0. > > > > Where do these helpers come from? Can't find them in Linus' tree. > > Is your tree out of date? > > 0b237047d5a7 for folio_end_read() > $ git describe --contains 0b237047d5a7 > v6.7-rc1~90^2~161 > > folio_fill_tail() appears to only be in akpm's tree for now. Looks like > it's still in the 'unstable' part for now, so no sha1 for that. Heh, I only looked for the first one. But adding what tree you're sending a patch against not only makes reviewing possible, but also helps with the destination.