From patchwork Mon Sep 30 22:12:14 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Kanchana P Sridhar X-Patchwork-Id: 13817143 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 8A75BCEB2C9 for ; Mon, 30 Sep 2024 22:12:28 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id DCB04280031; Mon, 30 Sep 2024 18:12:27 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id DA11B28002D; Mon, 30 Sep 2024 18:12:27 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id C3E8C280031; Mon, 30 Sep 2024 18:12:27 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0010.hostedemail.com [216.40.44.10]) by kanga.kvack.org (Postfix) with ESMTP id 9C9C628002D for ; Mon, 30 Sep 2024 18:12:27 -0400 (EDT) Received: from smtpin27.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay04.hostedemail.com (Postfix) with ESMTP id 343A21A0C1B for ; Mon, 30 Sep 2024 22:12:27 +0000 (UTC) X-FDA: 82622804334.27.707E3D3 Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.9]) by imf16.hostedemail.com (Postfix) with ESMTP id D2674180008 for ; Mon, 30 Sep 2024 22:12:23 +0000 (UTC) Authentication-Results: imf16.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b=cSm9pWYB; spf=pass (imf16.hostedemail.com: domain of kanchana.p.sridhar@intel.com designates 198.175.65.9 as permitted sender) smtp.mailfrom=kanchana.p.sridhar@intel.com; dmarc=pass (policy=none) header.from=intel.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1727734177; 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-transfer-encoding:content-transfer-encoding: in-reply-to:references:dkim-signature; bh=FGLZ/ltAXDmqJjBUlOqaPcWi0sE8JrynJTftm/49AjM=; b=IGvot9ttPMGgs4VKke7NpMUTbvYrpKz0I6PUX9+OAeBWe+0AtMBaxAmevZo5dssbCGinWo bw4qaBHaCSK1PZ9CkAQBE0Gol5tog1YGIzf6sCQMFmBFyBomisb0G518DR/fZdQZjEYTU+ KUyWd5D7XMgDeyPpOHAQe6vjMIhbeGw= ARC-Authentication-Results: i=1; imf16.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b=cSm9pWYB; spf=pass (imf16.hostedemail.com: domain of kanchana.p.sridhar@intel.com designates 198.175.65.9 as permitted sender) smtp.mailfrom=kanchana.p.sridhar@intel.com; dmarc=pass (policy=none) header.from=intel.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1727734177; a=rsa-sha256; cv=none; b=NS8yRyV7MezyLeKcl5XiaLm5sOx5IHeFaNXG3C9WzTFe/cpzlzZnX+Hmo2y7R0gxEmjYC7 vbkWvQjNlKcZkPLodWSL8pIYXzSc/Rf1bPTDb2dQDuzmrXDXAnDf0JDyxCJqRerFdWVOhp HksFa2l29F5rUTkDUzqFSN6APzxiNik= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1727734344; x=1759270344; h=from:to:cc:subject:date:message-id:mime-version: content-transfer-encoding; bh=r1Tp+YxdHCql6xZQeNEyCDJjGfHKi22y7efld9iTwJ0=; b=cSm9pWYBIZj9E+GwfxwlQE5FkEY42LuBdSI9LOCyFWQk3v6AMpmt1Dvf 7ghGNbcAdJeOxDwVGWCGdgmpBWpZDB3KUx1/0nVnClmMB8RidfHSPkdgC MkETCkIQchzdvErjE5dsvpxdXdQQVTeNJBhSQd4npzWC4gSKMvah37JvJ xQ5PXy/HB2vC8LHB56NZ/tQZKIqzMdI19AWQeTplIOlsit5Gt4k+AJSTZ olJjPr5ygXeZAhYp7Q2snTv8aTgo63dwYNh4nJQIFz22JESmu/PXxHvsX eqHabWNuQ8b4Iq0r76w7L4R+8IoVyddrqC9vk44iN5hSxckMXdEqgKjfR g==; X-CSE-ConnectionGUID: be0k0TJoRdOgkUV6A2P1cg== X-CSE-MsgGUID: ZtanP4xmTqqGypM14lZp7w== X-IronPort-AV: E=McAfee;i="6700,10204,11211"; a="49368391" X-IronPort-AV: E=Sophos;i="6.11,166,1725346800"; d="scan'208";a="49368391" Received: from fmviesa004.fm.intel.com ([10.60.135.144]) by orvoesa101.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 30 Sep 2024 15:12:23 -0700 X-CSE-ConnectionGUID: rUqxE1wxSQ+cflvb0iCUNw== X-CSE-MsgGUID: 7SB+6fYfRoKiAzai6x8P/g== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.11,166,1725346800"; d="scan'208";a="77985569" Received: from jf5300-b11a338t.jf.intel.com ([10.242.51.6]) by fmviesa004.fm.intel.com with ESMTP; 30 Sep 2024 15:12:22 -0700 From: Kanchana P Sridhar To: linux-kernel@vger.kernel.org, linux-mm@kvack.org, hannes@cmpxchg.org, yosryahmed@google.com, nphamcs@gmail.com, chengming.zhou@linux.dev, usamaarif642@gmail.com, shakeel.butt@linux.dev, ryan.roberts@arm.com, ying.huang@intel.com, 21cnbao@gmail.com, akpm@linux-foundation.org, willy@infradead.org Cc: nanhai.zou@intel.com, wajdi.k.feghali@intel.com, vinodh.gopal@intel.com, kanchana.p.sridhar@intel.com Subject: [PATCH v9 0/7] mm: zswap swap-out of large folios Date: Mon, 30 Sep 2024 15:12:14 -0700 Message-Id: <20240930221221.6981-1-kanchana.p.sridhar@intel.com> X-Mailer: git-send-email 2.27.0 MIME-Version: 1.0 X-Rspamd-Server: rspam06 X-Rspamd-Queue-Id: D2674180008 X-Stat-Signature: 1c4sozzi3dnfgu3c1hj7xeohbzajep3i X-Rspam-User: X-HE-Tag: 1727734343-875929 X-HE-Meta: U2FsdGVkX1/lqUbsLpxBIV9Ls+4VnyVTEAlXpWQlb6Ithg3wyqL1LPZ1+v75RxdO2uSnxrg8GoPnj/WvVbiP3+eFiZHweOXyUm+XpljGn3tjk4/QRoivQ53SZry8vtd+fyU7zEWQBww6awjKJLXaofqW0qRikW7PJrRCqHNwyosYCGlTD9QyWSmx1BeeXiQkC8nribypzO/EAuzYCbnIvxMsImWANdTzKthTA42IYvoNBYE0328kHRx+8MXa3mO/XmM0fpoCKvH3ILoiaJ8IGUXw6oDXLs5/46e1FqipGO7Pkj/dYQa/HSUgFvwoxLCE1xJdeZKcReygeH2OOqVNXBQLI+TDSs9LH2bn8DwP6aw6k+SWkYdv4UvD19lpGQjuLDitA5ZWfDWFjGYefO6rLaSpRlp/mwPqw74xMiC7HKEhwLfQzR08ejWRfNEmD1nS1vLq2hToVtXcSJEWYFRE69BwxzNmnyo/QKo0GHF5Va0RjHB45P1v7EYe2L3ec9ye7E2oARXdeY9CJ7AQ4+Wd2dcv6IBbTjeZ4rH6NrFlT52iTI7pVOda427Hc3cMQyyaRjVH7iNrTMioq3jqiJjdnTBA5LkmGvUfqqHN4QBBCqCqoNQN8u/XoWBTcwtbGE/wb1i272el3aC0b54ot2iMycVhBi1xeW0hivvI0ThiWccn2Qcw0oKOvPXbtOXjqa3TD45I3E4t8P+Mu/gcujowQ4jpZ8xpvMxA916umViepVrW9tom0i/lx+bya1PomKrnhbu3fCU8b4PCpChJlGqFTvjFz3wp0hr/EELX9QiCB0iRWQa2b1iZ8nam2jHnvDzwLevq342mhUHPQBfZpGhLjmMvMjs28x28M4m2wen4uxAUCvgRnYxXSpOkDsl+WJUtEumsIUD8SLdtKWryWcfPuKVe4SQQWPERhO7mNTv0JfSM7zA3MR8R7Na/12Ve9ydqMWL/foTPoGQFlss1fGU SHYCgjDG c4AGaitY1G9ckDpuhQ6rDR6mdhyOMgo80m5z+FoSbposJ/hV+gvPriHZ2wMvv/jOA7gzK0aCXKyS1xM0M1arZemueZb9C+CeFFu9XFPaj3bAjI2DxD8Rt8kOQd53s4sCmjetPSdtv2od2veE4sgVOP0pB2y+QVlJ3OOoOgwd51LHfvX6AUrf/lz3DlW/IsxkzH/uZm+K07SwLJT7yuwIleIUjkmp/a1v2nmqnBy76Fiabsm0J1q6jqYzLt2frHNBiarSf 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: Hi All, This patch-series enables zswap_store() to accept and store large folios. The most significant contribution in this series is from the earlier RFC submitted by Ryan Roberts [1]. Ryan's original RFC has been migrated to mm-unstable as of 9-27-2024 in patch 6 of this series, and adapted based on code review comments received for v7 of the current patch-series. [1]: [RFC PATCH v1] mm: zswap: Store large folios without splitting https://lore.kernel.org/linux-mm/20231019110543.3284654-1-ryan.roberts@arm.com/T/#u The first few patches do the prep work for supporting large folios in zswap_store. Patch 6 provides the main functionality to swap-out large folios in zswap. Patch 7 adds sysfs per-order hugepages "zswpout" counters that get incremented upon successful zswap_store of large folios: /sys/kernel/mm/transparent_hugepage/hugepages-*kB/stats/zswpout Patch 8 updates the documentation for the new sysfs "zswpout" counters. This patch-series is a pre-requisite for zswap compress batching of large folio swap-out and decompress batching of swap-ins based on swapin_readahead(), using Intel IAA hardware acceleration, which we would like to submit in subsequent patch-series, with performance improvement data. Thanks to Ying Huang for pre-posting review feedback and suggestions! Thanks also to Nhat, Yosry, Johannes, Barry, Chengming, Usama, Ying and Matthew for their helpful feedback, code/data reviews and suggestions! Co-development signoff request: =============================== I would like to thank Ryan Roberts for his original RFC [1] and request his co-developer signoff on patch 6 in this series. Thanks Ryan! System setup for testing: ========================= Testing of this patch-series was done with mm-unstable as of 9-27-2024, commit de2fbaa6d9c3576ec7133ed02a370ec9376bf000 (without this patch-series) and mm-unstable 9-30-2024 commit 66af62407e82647ec5b44462dc29d50ba03fdb22 (with this patch-series). Data was gathered on an Intel Sapphire Rapids server, dual-socket 56 cores per socket, 4 IAA devices per socket, 503 GiB RAM and 525G SSD disk partition swap. Core frequency was fixed at 2500MHz. The vm-scalability "usemem" test was run in a cgroup whose memory.high was fixed at 150G. The is no swap limit set for the cgroup. 30 usemem processes were run, each allocating and writing 10G of memory, and sleeping for 10 sec before exiting: usemem --init-time -w -O -s 10 -n 30 10g Other kernel configuration parameters: zswap compressors : zstd, deflate-iaa zswap allocator : zsmalloc vm.page-cluster : 2 In the experiments where "deflate-iaa" is used as the zswap compressor, IAA "compression verification" is enabled by default (cat /sys/bus/dsa/drivers/crypto/verify_compress). Hence each IAA compression will be decompressed internally by the "iaa_crypto" driver, the crc-s returned by the hardware will be compared and errors reported in case of mismatches. Thus "deflate-iaa" helps ensure better data integrity as compared to the software compressors, and the experimental data listed below is with verify_compress set to "1". Metrics reporting methodology: ============================== Total and average throughput are derived from the individual 30 processes' throughputs reported by usemem. elapsed/sys times are measured with perf. All percentage changes are "new" vs. "old"; hence a positive value denotes an increase in the metric, whether it is throughput or latency, and a negative value denotes a reduction in the metric. Positive throughput change percentages and negative latency change percentages denote improvements. The vm stats and sysfs hugepages stats included with the performance data provide details on the swapout activity to zswap/swap device. Testing labels used in data summaries: ====================================== The data refers to these test configurations and the before/after comparisons that they do: before-case1: ------------- mm-unstable 9-27-2024, CONFIG_THP_SWAP=N (compares zswap 4K vs. zswap 64K) In this scenario, CONFIG_THP_SWAP=N results in 64K/2M folios to be split into 4K folios that get processed by zswap. before-case2: ------------- mm-unstable 9-27-2024, CONFIG_THP_SWAP=Y (compares SSD swap large folios vs. zswap large folios) In this scenario, CONFIG_THP_SWAP=Y results in zswap rejecting large folios, which will then be stored by the SSD swap device. after: ------ v9 of this patch-series, CONFIG_THP_SWAP=Y The "after" is CONFIG_THP_SWAP=Y and v9 of this patch-series, that results in 64K/2M folios to not be split, and to be processed by zswap_store. Regression Testing: =================== I ran vm-scalability usemem without large folios, i.e., only 4K folios with mm-unstable and this patch-series. The main goal was to make sure that there is no functional or performance regression wrt the earlier zswap behavior for 4K folios, now that 4K folios will be processed by the new zswap_store() code. The data indicates there is no significant regression. ------------------------------------------------------------------------------- 4K folios: ========== zswap compressor zstd zstd zstd zstd v9 zstd v9 before-case1 before-case2 after vs. vs. case1 case2 ------------------------------------------------------------------------------- Total throughput (KB/s) 4,793,363 4,880,978 4,850,564 1% -1% Average throughput (KB/s) 159,778 162,699 161,685 1% -1% elapsed time (sec) 130.14 123.17 125.90 -3% 2% sys time (sec) 3,135.53 2,985.64 3,078.31 -2% 3% memcg_high 446,826 444,626 443,925 memcg_swap_fail 0 0 0 zswpout 48,932,107 48,931,971 48,932,186 zswpin 383 386 392 pswpout 0 0 0 pswpin 0 0 0 thp_swpout 0 0 0 thp_swpout_fallback 0 0 0 64kB-mthp_swpout_fallback 0 0 0 pgmajfault 3,063 3,077 3,075 swap_ra 93 94 98 swap_ra_hit 47 47 48 ZSWPOUT-64kB n/a n/a 0 SWPOUT-64kB 0 0 0 ------------------------------------------------------------------------------- Performance Testing: ==================== We list the data for 64K folios with before/after data per-compressor, followed by the same for 2M pmd-mappable folios. ------------------------------------------------------------------------------- 64K folios: zstd: ================= zswap compressor zstd zstd zstd zstd v9 before-case1 before-case2 after vs. vs. case1 case2 ------------------------------------------------------------------------------- Total throughput (KB/s) 5,222,213 1,076,611 6,284,884 20% 484% Average throughput (KB/s) 174,073 35,887 209,496 20% 484% elapsed time (sec) 120.50 347.16 109.12 -9% -69% sys time (sec) 2,930.33 248.16 2,603.12 -11% 949% memcg_high 416,773 552,200 481,632 memcg_swap_fail 3,192,906 1,293 749 zswpout 48,931,583 20,903 48,932,292 zswpin 384 363 395 pswpout 0 40,778,448 0 pswpin 0 16 0 thp_swpout 0 0 0 thp_swpout_fallback 0 0 0 64kB-mthp_swpout_fallback 3,192,906 1,293 749 pgmajfault 3,452 3,072 3,048 swap_ra 90 87 100 swap_ra_hit 42 43 53 ZSWPOUT-64kB n/a n/a 3,057,504 SWPOUT-64kB 0 2,548,653 0 ------------------------------------------------------------------------------- ------------------------------------------------------------------------------- 64K folios: deflate-iaa: ======================== zswap compressor deflate-iaa deflate-iaa deflate-iaa deflate-iaa v9 before-case1 before-case2 after vs. vs. case1 case2 ------------------------------------------------------------------------------- Total throughput (KB/s) 5,652,608 1,089,180 7,046,527 25% 547% Average throughput (KB/s) 188,420 36,306 234,884 25% 547% elapsed time (sec) 102.90 343.35 88.29 -14% -74% sys time (sec) 2,246.86 213.53 1,869.78 -17% 776% memcg_high 576,104 502,907 657,505 memcg_swap_fail 4,016,117 1,407 1,618 zswpout 61,163,423 22,444 56,358,604 zswpin 401 368 447 pswpout 0 40,862,080 0 pswpin 0 20 0 thp_swpout 0 0 0 thp_swpout_fallback 0 0 0 64kB-mthp_swpout_fallback 4,016,117 1,407 1,618 pgmajfault 3,063 3,153 3,145 swap_ra 96 93 132 swap_ra_hit 46 45 57 ZSWPOUT-64kB n/a n/a 3,520,777 SWPOUT-64kB 0 2,553,880 0 ------------------------------------------------------------------------------- ------------------------------------------------------------------------------- 2M folios: zstd: ================ zswap compressor zstd zstd zstd zstd v9 before-case1 before-case2 after vs. vs. case1 case2 ------------------------------------------------------------------------------- Total throughput (KB/s) 5,895,500 1,109,694 6,684,792 13% 502% Average throughput (KB/s) 196,516 36,989 222,826 13% 502% elapsed time (sec) 108.77 334.28 103.47 -5% -69% sys time (sec) 2,657.14 94.88 2,334.70 -12% 2361% memcg_high 64,200 66,316 61,944 memcg_swap_fail 101,182 70 39 zswpout 48,931,499 36,507 48,785,119 zswpin 380 379 405 pswpout 0 40,166,400 0 pswpin 0 0 0 thp_swpout 0 78,450 0 thp_swpout_fallback 101,182 70 39 2MB-mthp_swpout_fallback 0 0 39 pgmajfault 3,067 3,417 5,513 swap_ra 91 90 6,228 swap_ra_hit 45 45 6,174 ZSWPOUT-2MB n/a n/a 95,234 SWPOUT-2MB 0 78,450 0 ------------------------------------------------------------------------------- ------------------------------------------------------------------------------- 2M folios: deflate-iaa: ======================= zswap compressor deflate-iaa deflate-iaa deflate-iaa deflate-iaa v9 before-case1 before-case2 after vs. vs. case1 case2 ------------------------------------------------------------------------------- Total throughput (KB/s) 6,286,587 1,126,785 7,372,721 17% 554% Average throughput (KB/s) 209,552 37,559 245,757 17% 554% elapsed time (sec) 96.19 333.03 84.72 -12% -75% sys time (sec) 2,141.44 99.96 1,748.87 -18% 1650% memcg_high 99,253 64,666 82,564 memcg_swap_fail 129,074 53 107 zswpout 61,312,794 28,321 60,323,493 zswpin 383 406 426 pswpout 0 40,048,128 0 pswpin 0 0 0 thp_swpout 0 78,219 0 thp_swpout_fallback 129,074 53 107 2MB-mthp_swpout_fallback 0 0 107 pgmajfault 3,430 3,077 9,077 swap_ra 91 103 17,725 swap_ra_hit 47 46 17,658 ZSWPOUT-2MB n/a n/a 117,670 SWPOUT-2MB 0 78,219 0 ------------------------------------------------------------------------------- And finally, this is a comparison of deflate-iaa vs. zstd with v9 of this patch-series: --------------------------------------------- zswap_store large folios v9 Impr w/ deflate-iaa vs. zstd 64K folios 2M folios --------------------------------------------- Throughput (KB/s) 12% 10% elapsed time (sec) -19% -18% sys time (sec) -28% -25% --------------------------------------------- Conclusions based on the performance results: ============================================= v9 wrt before-case1: -------------------- We see significant improvements in throughput, elapsed and sys time for zstd and deflate-iaa, when comparing before-case1 (THP_SWAP=N) vs. after (THP_SWAP=Y) with zswap_store large folios. v9 wrt before-case2: -------------------- We see even more significant improvements in throughput and elapsed time for zstd and deflate-iaa, when comparing before-case2 (large-folio-SSD) vs. after (large-folio-zswap). The sys time increases with large-folio-zswap as expected, due to the CPU compression time vs. asynchronous disk write times, as pointed out by Ying and Yosry. In before-case2, when zswap does not store large folios, only allocations and cgroup charging due to 4K folio zswap stores count towards the cgroup memory limit. However, in the after scenario, with the introduction of zswap_store() of large folios, there is an added component of the zswap compressed pool usage from large folio stores from potentially all 30 processes, that gets counted towards the memory limit. As a result, we see higher swapout activity in the "after" data. Summary: ======== The v9 data presented above shows that zswap_store of large folios demonstrates good throughput/performance improvements compared to conventional SSD swap of large folios with a sufficiently large 525G SSD swap device. Hence, it seems reasonable for zswap_store to support large folios, so that further performance improvements can be implemented. In the experimental setup used in this patchset, we have enabled IAA compress verification to ensure additional hardware data integrity CRC checks not currently done by the software compressors. We see good throughput/latency improvements with deflate-iaa vs. zstd with zswap_store of large folios. Some of the ideas for further reducing latency that have shown promise in our experiments, are: 1) IAA compress/decompress batching. 2) Distributing compress jobs across all IAA devices on the socket. The tests run for this patchset are using only 1 IAA device per core, that avails of 2 compress engines on the device. In our experiments with IAA batching, we distribute compress jobs from all cores to the 8 compress engines available per socket. We further compress the pages in each folio in parallel in the accelerator. As a result, we improve compress latency and reclaim throughput. In decompress batching, we use swapin_readahead to generate a prefetch batch of 4K folios that we decompress in parallel in IAA. ------------------------------------------------------------------------------ IAA compress/decompress batching Further improvements wrt v9 zswap_store Sequential subpage store using "deflate-iaa": "deflate-iaa" Batching "deflate-iaa-canned" [2] Batching Additional Impr Additional Impr 64K folios 2M folios 64K folios 2M folios ------------------------------------------------------------------------------ Throughput (KB/s) 21% 37% 29% 48% elapsed time (sec) -6% -13% -11% -20% sys time (sec) 3% -3% -5% -14% ------------------------------------------------------------------------------ With zswap IAA compress/decompress batching, we are able to demonstrate significant performance improvements and memory savings in server scalability experiments in highly contended system scenarios under significant memory pressure; as compared to software compressors. We hope to submit this work in subsequent patch series. The current patch-series is a prequisite for these future submissions. Thanks, Kanchana [1] https://lore.kernel.org/linux-mm/20231019110543.3284654-1-ryan.roberts@arm.com/T/#u [2] https://patchwork.kernel.org/project/linux-crypto/cover/cover.1710969449.git.andre.glover@linux.intel.com/ Changes since v8: ================= 1) Rebased to mm-unstable as of 9-30-2024, commit 66af62407e82647ec5b44462dc29d50ba03fdb22. 2) Changed count_objcg_event() to become count_objcg_events() that accepts a "count" parameter. Modified existing calls to count_objcg_event() to instead call count_objcg_events() with a "count" of 1. Thanks Yosry for this suggestion! 3) Avoid the check for "zswap_pool_reached_full" if zswap is disabled, as pointed out by Yosry. Thanks Yosry! 4) Modified zswap_store_page() to take a page as input, and to derive the swp_entry_t for the page by calling page_swap_entry() as suggested by Johannes. Thanks Johannes! 5) Code cleanup and code comments clarifications suggested by Yosry in zswap_store_page() and zswap_store(). Also, more concise commit log for patch 6 as suggested by Yosry. Thanks Yosry! 6) Squashed the commits related to the sysfs hugepage zswpout stats addition and the documentation update into a single commit, as suggested by Nhat. Thanks Nhat! 7) Modified the latency metric change reporting convention to be "new" vs. "old", as per Yosry's suggestion. 8) Changes in the cover-letter suggested by Yosry. Thanks Yosry! 9) Added "Acked-by" and "Reviewed-by" from Johannes, Yosry, Nhat and Chengming. Thanks to all! Changes since v7: ================= 1) Rebased to mm-unstable as of 9-27-2024, commit de2fbaa6d9c3576ec7133ed02a370ec9376bf000. 2) Added Nhat's 'Reviewed-by' to patches 1 and 2. Thanks Nhat! 3) Implemented one-time obj_cgroup_may_zswap and zswap_check_limits at the start of zswap_store. Implemented one-time batch updates to cgroup zswap charging (with total compressed bytes), zswap_stored_pages and the memcg/vm zswpout event stats (with folio_nr_pages()) only for successful stores at the end of zswap_store. Thanks Yosry and Johannes for guidance on this! 4) Changed the existing zswap_pool_get() to zswap_pool_tryget(). Modified zswap_pool_current_get() and zswap_pool_find_get() to call zswap_pool_tryget(). Furthermore, zswap_store() obtains a reference to a valid zswap_pool upfront by calling zswap_pool_tryget(), and errors out if the tryget fails. Added a new zswap_pool_get() that calls "percpu_ref_get(&pool->ref)" and is called in zswap_store_page(), as suggested by Johannes & Yosry. Thanks both! 5) Provided a new count_objcg_events() API for batch event updates. 6) Changed "zswap_stored_pages" to atomic_long_t to support adding folio_nr_pages() to it once a large folio is stored successfully. 7) Deleted the refactoring done in v7 for the xarray updates in zswap_store_page(); and unwinding of stored offsets in zswap_store() in case of errors, as suggested by Johannes. 8) Deleted the CONFIG_ZSWAP_STORE_THP_DEFAULT_ON config option and "zswap_mthp_enabled" tunable, as recommended by Yosry, Johannes and Nhat. 9) Replaced references to "mTHP" with "large folios"; organized before/after data per-compressor for easier visual comparisons; incorporated Nhat's feedback in the documentation updates; moved changelog to the end. Thanks Johannes, Yosry and Nhat! 10) Moved the usemem testing configuration to 30 processes, each allocating 10G within a 150G memory-limit constrained cgroup, maintaining the allocated memory for 10 sec before exiting. Thanks Ying for this suggestion! Changes since v6: ================= 1) Rebased to mm-unstable as of 9-23-2024, commit acfabf7e197f7a5bedf4749dac1f39551417b049. 2) Refactored into smaller commits, as suggested by Yosry and Chengming. Thanks both! 3) Reworded the commit log for patches 5 and 6 as per Yosry's suggestion. Thanks Yosry! 4) Gathered data on a Sapphire Rapids server that has 823GiB SSD swap disk partition. Also, all experiments are run with usemem --sleep 10, so that the memory allocated by the 70 processes remains in memory longer. Posted elapsed and sys times. Thanks to Yosry, Nhat and Ying for their help with refining the performance characterization methodology. 5) Updated Documentation/admin-guide/mm/transhuge.rst as suggested by Nhat. Thanks Nhat! Changes since v5: ================= 1) Rebased to mm-unstable as of 8/29/2024, commit 9287e4adbc6ab8fa04d25eb82e097fed877a4642. 2) Added CONFIG_ZSWAP_STORE_THP_DEFAULT_ON (off by default) to enable/disable zswap_store() of mTHP folios. Thanks Nhat for the suggestion to add a knob by which users can enable/disable this change. Nhat, I hope this is along the lines of what you were thinking. 3) Added vm-scalability usemem data with 4K folios with CONFIG_ZSWAP_STORE_THP_DEFAULT_ON off, that I gathered to make sure there is no regression with this change. 4) Added data with usemem with 64K and 2M THP for an alternate view of before/after, as suggested by Yosry, so we can understand the impact of when mTHPs are split into 4K folios in shrink_folio_list() (CONFIG_THP_SWAP off) vs. not split (CONFIG_THP_SWAP on) and stored in zswap. Thanks Yosry for this suggestion. Changes since v4: ================= 1) Published before/after data with zstd, as suggested by Nhat (Thanks Nhat for the data reviews!). 2) Rebased to mm-unstable from 8/27/2024, commit b659edec079c90012cf8d05624e312d1062b8b87. 3) Incorporated the change in memcontrol.h that defines obj_cgroup_get() if CONFIG_MEMCG is not defined, to resolve build errors reported by kernel robot; as per Nhat's and Michal's suggestion to not require a separate patch to fix the build errors (thanks both!). 4) Deleted all same-filled folio processing in zswap_store() of mTHP, as suggested by Yosry (Thanks Yosry!). 5) Squashed the commits that define new mthp zswpout stat counters, and invoke count_mthp_stat() after successful zswap_store()s; into a single commit. Thanks Yosry for this suggestion! Changes since v3: ================= 1) Rebased to mm-unstable commit 8c0b4f7b65fd1ca7af01267f491e815a40d77444. Thanks to Barry for suggesting aligning with Ryan Roberts' latest changes to count_mthp_stat() so that it's always defined, even when THP is disabled. Barry, I have also made one other change in page_io.c where count_mthp_stat() is called by count_swpout_vm_event(). I would appreciate it if you can review this. Thanks! Hopefully this should resolve the kernel robot build errors. Changes since v2: ================= 1) Gathered usemem data using SSD as the backing swap device for zswap, as suggested by Ying Huang. Ying, I would appreciate it if you can review the latest data. Thanks! 2) Generated the base commit info in the patches to attempt to address the kernel test robot build errors. 3) No code changes to the individual patches themselves. Changes since RFC v1: ===================== 1) Use sysfs for zswpout mTHP stats, as per Barry Song's suggestion. Thanks Barry! 2) Addressed some of the code review comments that Nhat Pham provided in Ryan's initial RFC [1]: - Added a comment about the cgroup zswap limit checks occuring once per folio at the beginning of zswap_store(). Nhat, Ryan, please do let me know if the comments convey the summary from the RFC discussion. Thanks! - Posted data on running the cgroup suite's zswap kselftest. 3) Rebased to v6.11-rc3. 4) Gathered performance data with usemem and the rebased patch-series. Kanchana P Sridhar (7): mm: Define obj_cgroup_get() if CONFIG_MEMCG is not defined. mm: zswap: Modify zswap_compress() to accept a page instead of a folio. mm: zswap: Rename zswap_pool_get() to zswap_pool_tryget(). mm: Change count_objcg_event() to count_objcg_events() for batch event updates. mm: zswap: Modify zswap_stored_pages to be atomic_long_t. mm: zswap: Support large folios in zswap_store(). mm: swap: Count successful large folio zswap stores in hugepage zswpout stats. Documentation/admin-guide/mm/transhuge.rst | 8 +- fs/proc/meminfo.c | 2 +- include/linux/huge_mm.h | 1 + include/linux/memcontrol.h | 16 +- include/linux/zswap.h | 2 +- mm/huge_memory.c | 3 + mm/page_io.c | 1 + mm/zswap.c | 251 ++++++++++++++------- 8 files changed, 196 insertions(+), 88 deletions(-) base-commit: 66af62407e82647ec5b44462dc29d50ba03fdb22