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 65FBDC3DA59 for ; Tue, 16 Jul 2024 15:26:23 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id DBDF66B0082; Tue, 16 Jul 2024 11:26:22 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id D6DA36B0085; Tue, 16 Jul 2024 11:26:22 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id C5C356B0088; Tue, 16 Jul 2024 11:26:22 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0014.hostedemail.com [216.40.44.14]) by kanga.kvack.org (Postfix) with ESMTP id A76256B0082 for ; Tue, 16 Jul 2024 11:26:22 -0400 (EDT) Received: from smtpin13.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay03.hostedemail.com (Postfix) with ESMTP id 3C9C4A066F for ; Tue, 16 Jul 2024 15:26:22 +0000 (UTC) X-FDA: 82345992204.13.EE9DF8B Received: from casper.infradead.org (casper.infradead.org [90.155.50.34]) by imf26.hostedemail.com (Postfix) with ESMTP id 5691114002D for ; Tue, 16 Jul 2024 15:26:20 +0000 (UTC) Authentication-Results: imf26.hostedemail.com; dkim=pass header.d=infradead.org header.s=casper.20170209 header.b=AVs3Died; spf=none (imf26.hostedemail.com: domain of willy@infradead.org has no SPF policy when checking 90.155.50.34) smtp.mailfrom=willy@infradead.org; dmarc=none ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1721143542; 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: in-reply-to:in-reply-to:references:references:dkim-signature; bh=+ZGcgOeaT+/JIj3aQHIikSw57lYTC0+Jg+f0BLjgrTQ=; b=AhtA8hYwFRK874Owr5IV2UrCAPNqMD6Eo/YWLN2Sh1OQEOly+XYDb4KXhOiSQsfnvKjyAX DqlmrBuNtjn8g+e2RKrPthjkbYAKXudKe2zbF4nys6KSx1hbH2RILGpl/YDXaxnq3Zyzut 4Ej6MCBEXpddQN2Xw7DDjr6L4dYCiNo= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1721143542; a=rsa-sha256; cv=none; b=Q7kWvx3JaKxusoBdK7IKNWUdllLizFCv/cUVYqxmz5LgnY2SzWXA4VQITWe2MZV9/tfE7U Xw+rVJ5FnX4AwUTQuy5pEi1SuQ+Syyk0ggU0rmfQ/MX5q6btdTi+ktF6lFuOjL5b847L2J h+pJFbgrqsqDJN68ZVG2yZRo8TqnOtY= ARC-Authentication-Results: i=1; imf26.hostedemail.com; dkim=pass header.d=infradead.org header.s=casper.20170209 header.b=AVs3Died; spf=none (imf26.hostedemail.com: domain of willy@infradead.org has no SPF policy when checking 90.155.50.34) smtp.mailfrom=willy@infradead.org; dmarc=none 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=+ZGcgOeaT+/JIj3aQHIikSw57lYTC0+Jg+f0BLjgrTQ=; b=AVs3Diedky0NxggnKuSLAKTYiv 37PBUzKjTOnVrXvjx2cQJZWT2ShGL28zhFNxFKNRyjyb4KRKv2vhYvUpHYH2h9ngIuGXSO+KeMfxK W70B5twuKz0TiAuSqKicQ7YLVSgdZ1sGhlg3R/zm+r2zcJWKyzii3qKT9cUMDrERzmuuox3knFkQG Vlcf5qFMRTusas2GujCUd1UKAPWHn1FU6swpyM/sdNF+PHnO04IAhG7jmbL8uOE75ItDFfXm1mP2d oy2MWQwz0DK0DJbRRVGQLaGi2Y46OvtvlydscVw0Ep+SsPXFVgGwElB8m0tbTVAF7mbpg6n6CTQrx MswkYg1Q==; Received: from willy by casper.infradead.org with local (Exim 4.97.1 #2 (Red Hat Linux)) id 1sTk4A-0000000HFLP-2Qjt; Tue, 16 Jul 2024 15:26:10 +0000 Date: Tue, 16 Jul 2024 16:26:10 +0100 From: Matthew Wilcox To: "Pankaj Raghav (Samsung)" Cc: david@fromorbit.com, chandan.babu@oracle.com, djwong@kernel.org, brauner@kernel.org, akpm@linux-foundation.org, linux-kernel@vger.kernel.org, yang@os.amperecomputing.com, linux-mm@kvack.org, john.g.garry@oracle.com, linux-fsdevel@vger.kernel.org, hare@suse.de, p.raghav@samsung.com, mcgrof@kernel.org, gost.dev@samsung.com, cl@os.amperecomputing.com, linux-xfs@vger.kernel.org, ryan.roberts@arm.com, hch@lst.de, Zi Yan Subject: Re: [PATCH v10 01/10] fs: Allow fine-grained control of folio sizes Message-ID: References: <20240715094457.452836-1-kernel@pankajraghav.com> <20240715094457.452836-2-kernel@pankajraghav.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20240715094457.452836-2-kernel@pankajraghav.com> X-Rspam-User: X-Rspamd-Server: rspam04 X-Rspamd-Queue-Id: 5691114002D X-Stat-Signature: 7kcnt6hf8sf35x1bkdwc3hp3iq6be5ps X-HE-Tag: 1721143580-3635 X-HE-Meta: U2FsdGVkX1/trR1ZVyw5XJjwA/6BCZ64MwGfcmPjgxBlb9xsTxh9boaLFLcEVS7sH+iSw2vH4qo6c232Il4JKoZml8YMWhqrpGAbeqX51HWRRvSQMBWQ0s/F4Mqffk+shund+6fEM35b0dCMYIuAy9NWs8PboYqTL9wIRcIIGvz6FCcDGdJqMzUlxsATnnleTNOuzQagEhXqbtffkyg2UNLyRWFBRHZ77NJUfRYg5Clc9kZ1qJuupcIgnz0AoW/TplqclEkYU8hm7+5q94JoQ0NU0pROJxuwNZPp7Bhp/oYX0AWhSe1RyqUKuINqhqNCVQeilvB+atcnCnANqUFMvp568xnJmRVAGNNfSjlD+4mXfJKmjaTG/8ch09JB5bse7+FtYHXzKrlGJZF4IBOsvqGEKDE5Z10tVnCS01gw/ksFcOM2gYWGLMdlTPng3vaYaNzq6Y1C9jcbZnYEiI6vVsGdTBoJH3Vu6vfL+fiHSlD3qIgu62iv9LIhmADTKRR5YocTC6XGH0mCSKY5ApmTKJ+V9U6TW+cq/td8YQRcMxljBDZ+gZBxaXgYvVEpvj0GJXnNxCkJT+1R+EMNvCtCamTpJ7r7BgTojPsZ35zwGByfEMLkMAy6W+SbQUg8qTkSiloABrqBVPLltHFd1Hf0mlUuIDLNLGi53Q27k8yKf+KjvJuSs0Yk/o5yX3CzSqGW+KShnb/qK/+eu2889palqc3HC08o78NdMLAqB8/QGXUSBM2jbkjZdWC+2b3nWMFmEzkPism8lysSTTu6W98pkbdetxFl0tOevMphP+9T3jtr7bqYyP7UPXOz6Phyph886+jimPgaHJy/xGrGyWd73UcLXjTViLPkPlDbzwT6XQSYeCDqjnlsxd+ViZ/zxumZoS5RpFRjTtH2pr3A5ik/oFFRTpQN7cHic3Uc1njFNTiEmxljRFM0Ez+N0S2700siOA5t2DPd/s+v2ao3T8O PZ5dyjxS o2ubRulTO1jMr0LEWvHxBKhA6GHLKM+TSoEjbc8JRSpWQgMz2jDcK9o/Nz7DNv1B3n9vCOckz7taFjEsGkEuAja3bpVxLGA1NLC/DRrOJf7sqHeCqy5xODraEMhoP5w9PuyBOfwb4FeBwXF16Sn2LFvRFw/2YqeecJjbEwNfmJf7RcRm5LTE8LryIbsvB8jcR0cYSh53uDRCS7mI= 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: List-Subscribe: List-Unsubscribe: On Mon, Jul 15, 2024 at 11:44:48AM +0200, Pankaj Raghav (Samsung) wrote: > +/* > + * mapping_max_folio_size_supported() - Check the max folio size supported > + * > + * The filesystem should call this function at mount time if there is a > + * requirement on the folio mapping size in the page cache. > + */ > +static inline size_t mapping_max_folio_size_supported(void) > +{ > + if (IS_ENABLED(CONFIG_TRANSPARENT_HUGEPAGE)) > + return 1U << (PAGE_SHIFT + MAX_PAGECACHE_ORDER); > + return PAGE_SIZE; > +} There's no need for this to be part of this patch. I've removed stuff from this patch before that's not needed, please stop adding unnecessary functions. This would logically be part of patch 10. > +static inline void mapping_set_folio_order_range(struct address_space *mapping, > + unsigned int min, > + unsigned int max) > +{ > + if (!IS_ENABLED(CONFIG_TRANSPARENT_HUGEPAGE)) > + return; > + > + if (min > MAX_PAGECACHE_ORDER) { > + VM_WARN_ONCE(1, > + "min order > MAX_PAGECACHE_ORDER. Setting min_order to MAX_PAGECACHE_ORDER"); > + min = MAX_PAGECACHE_ORDER; > + } This is really too much. It's something that will never happen. Just delete the message. > + if (max > MAX_PAGECACHE_ORDER) { > + VM_WARN_ONCE(1, > + "max order > MAX_PAGECACHE_ORDER. Setting max_order to MAX_PAGECACHE_ORDER"); > + max = MAX_PAGECACHE_ORDER; Absolutely not. If the filesystem declares it can support a block size of 4TB, then good for it. We just silently clamp it.