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 F3A03C25B7D for ; Fri, 24 May 2024 03:59:39 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 6F2A06B009D; Thu, 23 May 2024 23:59:39 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 6A2F36B00A1; Thu, 23 May 2024 23:59:39 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 590C46B00A3; Thu, 23 May 2024 23:59:39 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0016.hostedemail.com [216.40.44.16]) by kanga.kvack.org (Postfix) with ESMTP id 3CF126B009D for ; Thu, 23 May 2024 23:59:39 -0400 (EDT) Received: from smtpin12.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay04.hostedemail.com (Postfix) with ESMTP id E40E41A0639 for ; Fri, 24 May 2024 03:59:38 +0000 (UTC) X-FDA: 82151935236.12.D913560 Received: from casper.infradead.org (casper.infradead.org [90.155.50.34]) by imf19.hostedemail.com (Postfix) with ESMTP id CBA311A000F for ; Fri, 24 May 2024 03:59:35 +0000 (UTC) Authentication-Results: imf19.hostedemail.com; dkim=pass header.d=infradead.org header.s=casper.20170209 header.b=tuIL4tBU; spf=none (imf19.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-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1716523177; a=rsa-sha256; cv=none; b=sVtSvHxJjgcbXzUcFyfMgDT2uzqUSCoFTwx5VpYguQfLL/LUfqJY5mjPMkNY0ETL13ZLRy IQ45WrNhaltzXvNoGH9gn5uOZ9HevZOvTMbHbclGkAhbDD1g8rEbne0Ivs9wOUway3JENg e9VUCcaxrAJBooJyDjrxQsbDb97dlzs= ARC-Authentication-Results: i=1; imf19.hostedemail.com; dkim=pass header.d=infradead.org header.s=casper.20170209 header.b=tuIL4tBU; spf=none (imf19.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=1716523177; 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=TKUxDCU4Matxcb4X80T42SzbEohf2CEMtw7weq8/G6g=; b=muf595M50Bw2ZeS6Xg1NVVlazo8QgO3J/Jk1/c5F6ZOHugTbBWs0zXcgggQ9pxSWtanoa/ 6/kaJmfdg4DpokYpR827TzfgLNNaT72qz+FWpMWQqx5F7Qdt0Jd/8Qm3KMZb/QY1IpUAzB Ol1Gg8ZpwIG4YkcawDwiqOpjPtXme0M= 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=TKUxDCU4Matxcb4X80T42SzbEohf2CEMtw7weq8/G6g=; b=tuIL4tBUl3MYDxcWcCbgDxxvk9 laSyMsv8hX1myUUVW4emGzHles9mYf5jVvRT9JzaYsxRoEF9jXxPKgHs0+hzTR3SkvkO9t/ALthZv tTTj6SjQswBvGTfQ5cp5M1Png3GvEkC3COqBI9nI8c3XFafYXizbjpiXqQ05ecsvBdq9fj9KsPOB4 tQ7RBrRnwfrdDHH/4ayYIWYk5PRTpp6lhfKnQxsg6uTLB5sPjq3OdY7TJd2h+0n9vqsdgdl0JMBXH 3ts5KOvbthmldq19nDDI5IS7Whav5HCkVqPZ4yXH/CdE4ROKJDho2hj9hWA9o4BOjjNAek+2RDPfs onxUcnTg==; Received: from willy by casper.infradead.org with local (Exim 4.97.1 #2 (Red Hat Linux)) id 1sAM5b-00000002O0p-3GoC; Fri, 24 May 2024 03:59:32 +0000 Date: Fri, 24 May 2024 04:59:31 +0100 From: Matthew Wilcox To: Yosry Ahmed Cc: Andrew Morton , Johannes Weiner , Nhat Pham , Chengming Zhou , linux-mm@kvack.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 0/3] mm: zswap: trivial folio conversions Message-ID: References: <20240524033819.1953587-1-yosryahmed@google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20240524033819.1953587-1-yosryahmed@google.com> X-Rspam-User: X-Rspamd-Server: rspam06 X-Rspamd-Queue-Id: CBA311A000F X-Stat-Signature: zuaddowibycf6bizynxhhz7yn615bwcf X-HE-Tag: 1716523175-182700 X-HE-Meta: U2FsdGVkX19YUbFrwiLCYfAurzYz/lh9AAZax6oc+rR4JDgkROyyndGdRLHVj98PSdoXwvVGkPAiUV5fKkUVqRdpiX49r7mZPgvYim+E/NkxJl7/GhbAi1p+exNAt3km0UaIBQGUrrZy0V73xr2Q6ij/q5UbFdEc+l8Us/cQyDUG1Zj/JGJBxvLCWPc4WFDOX8hOsQW0WBKuoEg9798/Bkr8AAC3iypBDHidqWvP7Wa87GSF0mMLIHVo9EY1dKdjfH9KB4p3EflgC9A3zrZggdYVvH8yeiz24uOQvCdO/rB9otyxbi6CzbZRSf1g0zYg0h/rcuB+qMoPWyyIUbJqUYK3Xnoo8vGjua1r05lKMm63FDPvqrx+xzvz9QLWWVp/sTTm0kGPqMWI2yG6dHlHj6NZ/9hDmODOsLBvdi0C+DWdc76m3hy4f24X0Y30QIuhHfUETpDB7rT+cUXIaFTEVhx5IkOKVZFy0wyEdP82yaiJHr+Yi2P8+dn969ERpLAcCLnJMxu0FY7VBhBTp9JOt1EgM4i4XDLKEAThMBQdnEcFmM41gIQB4ZT2iyjpEvprbqbXm4R3zsU2L5YfVZb17ctlKiss1b3yUEBHQ/9k2A2u4DYEF/p5nu5EjXWOuoPae1Y6hXJw1UGgJWmwWDYJgvRv6VezRKdAyWPQJCS2JAV41UHeL5LcCiG4duRacpzt990mFzvqRlAOKYHIF8uI8UqPu9cs3xphLQHXNGrt1x4sVHtcLQUosxElPi5gQ88/yKQgGsuWwAwW8S7GGxETRyA8fHoFSEKIqasly8tEbuzvH00i4UgwMw9T5RSMIrvg6EPRoq5k0AGFmVeIUCCuUoWS77tOytHfzCdtqN1j8Lb5pizFlIqC9UxZGBcWuM8vFhfU6TVyVweWfV4scMELHJd1ROrmbOks/vIEYbWuourt7GjL37yDYZOvaIcqRy4Y0V4QbOjrOgWnN3W8Zym TPuA5ud9 NUpLNYUEEgPTJ/k55OgvpqleAQLHoKOmObte5wW/mgbgEAloi7K906iF7K5sbD8yvHKjUWC6GQq0ntG/yELii19H5LgASPBE7B4bpf5VfmCxi92engYXaDgaUjoFAfaTuDpLFI7A6DVd4FZs= 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 Fri, May 24, 2024 at 03:38:15AM +0000, Yosry Ahmed wrote: > Some trivial folio conversions in zswap code. The three patches themselves look good. > The mean reason I included a cover letter is that I wanted to get > feedback on what other trivial conversions can/should be done in > mm/zswap.c (keeping in mind that only order-0 folios are supported > anyway). These are the things I came across while searching for 'page' > in mm/zswap.c, and chose not to do anything about for now: I think there's a deeper question to answer before answering these questions, which is what we intend to do with large folios and zswap in the future. Do we intend to split them? Compress them as a large folio? Compress each page in a large folio separately? I can see an argument for choices 2 and 3, but I think choice 1 is going to be increasingly untenable. > 1. zswap_max_pages(), zswap_accept_thr_pages(), zswap_total_pages(): > - We can use 'size' instead of 'pages' and shift the return by > PAGE_SHIFT. This adds an unnecessary shift, but I doubt it matters > at all. The motivation is to get rid of 'page' to find things that > should be converted more easily. > > 2. Counters names: zswap_stored_pages, zswap_written_back_pages, etc. > > 3. Comments all over the place reference 'page' instead of 'folio'. > > 4. shrink_memcg_cb(), zswap_shrinker_scan(): > - Rename encountered_page_in_swap_cache to > encounterd_folio_in_swap_cache, or even better: folio_eexist or > hit_swap_cache. > > 5. entry_to_nid(): > - It's tempting to try to use folio_to_nid(virt_to_folio()), but I > think this adds an unnecessary call to compound_head(). It may not > matter in practice though because the page is always a head page. > > Yosry Ahmed (3): > mm: zswap: use sg_set_folio() in zswap_{compress/decompress}() > mm :zswap: use kmap_local_folio() in zswap_load() > mm: zswap: make same_filled functions folio-friendly > > mm/zswap.c | 41 ++++++++++++++++++----------------------- > 1 file changed, 18 insertions(+), 23 deletions(-) > > -- > 2.45.1.288.g0e0cd299f1-goog > >