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 E6AE1C072A2 for ; Mon, 20 Nov 2023 02:41:41 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 021AC6B0331; Sun, 19 Nov 2023 21:41:41 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id F12A46B0332; Sun, 19 Nov 2023 21:41:40 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id E01746B0333; Sun, 19 Nov 2023 21:41:40 -0500 (EST) 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 D28496B0331 for ; Sun, 19 Nov 2023 21:41:40 -0500 (EST) Received: from smtpin17.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay04.hostedemail.com (Postfix) with ESMTP id 63FB11A0248 for ; Mon, 20 Nov 2023 02:41:39 +0000 (UTC) X-FDA: 81476781918.17.3E75FA8 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by imf19.hostedemail.com (Postfix) with ESMTP id A6BD51A000E for ; Mon, 20 Nov 2023 02:41:37 +0000 (UTC) Authentication-Results: imf19.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b="o/0Z/1/j"; spf=pass (imf19.hostedemail.com: domain of chrisl@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=chrisl@kernel.org; dmarc=pass (policy=none) header.from=kernel.org ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1700448097; a=rsa-sha256; cv=none; b=FIWukf1Qn8ZhJt/M3BDDFX/cO+VygSSVHy5sLBcNbXn38kY5XDv9niYS5yx+1f1gib873F Wzu9uscLtDcVi/m25qjTSo1PEAujCbNqNgeHfudIR8nqhK4HXtV1Jth7R+9OvLyFNphr44 VYyFMdDKBKI//ejI4d65tkkQ3AbZvO8= ARC-Authentication-Results: i=1; imf19.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b="o/0Z/1/j"; spf=pass (imf19.hostedemail.com: domain of chrisl@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=chrisl@kernel.org; dmarc=pass (policy=none) header.from=kernel.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1700448097; 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:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=lCG9ZawC3ztul0nCGCI4RHneLck6XEIyYbizhx+EprY=; b=fRVp1TOsxMdGwM4g5XCMFwAZOHDuZDglOHAq3U/B64FCIOhB9AlfbMFEaqkpMDsC9k+M8w t1r04PJRlG70fObeQY96bYNQ/nguOWvx3m9wuQLsY1WXDysbZ0DP6DDmuES/q/sdU3FiWt 6PKuqq8iCzr5y0+NTvFPVVLreLFZOCo= Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id DB73660DF4; Mon, 20 Nov 2023 02:41:35 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 0AFE6C433C8; Mon, 20 Nov 2023 02:41:33 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1700448094; bh=UiWuaKeTiSFKQxVccuysbinvCkmLWJ1QXqtmU4c9M9o=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=o/0Z/1/jlmTimJm9937305yEJETPIoA66Qdh+9bcGh4iO1jC7uwn5sAd36iHbSjfF y5073eOz1lEnL5m3wFSz+1ILlybe2uy2Rr7g2ANq8FuQVZjZySqzgVBrnuTj55kuec W/BBC+TgPpbkDkulF6xyej5O57aBSgYOqfSM8dL4QcR5dWPS3Ol+IfXsSKi2L3qPC8 slYmHD0Q0xTpui580WF/xxTH3hnDRteip0YpBaJ4l9c/EObCSkSXsUcMtvI4iMgYcO rQEGHHyGoNic200lCdcB8PDVuKrs2sR1Ccje+ypBCoXPRrypbL+y3FwSgKbK9M5gr3 Wvs4axuOecFJw== Date: Sun, 19 Nov 2023 18:41:32 -0800 From: Chris Li To: Nhat Pham Cc: Yosry Ahmed , Andrew Morton , tj@kernel.org, lizefan.x@bytedance.com, Johannes Weiner , Domenico Cerasuolo , Seth Jennings , Dan Streetman , Vitaly Wool , Michal Hocko , Roman Gushchin , Shakeel Butt , Muchun Song , Hugh Dickins , corbet@lwn.net, Konrad Rzeszutek Wilk , senozhatsky@chromium.org, rppt@kernel.org, linux-mm , kernel-team@meta.com, LKML , linux-doc@vger.kernel.org, david@ixit.cz Subject: Re: [PATCH v5] zswap: memcontrol: implement zswap writeback disabling Message-ID: References: <20231115172344.4155593-1-nphamcs@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: X-Rspamd-Server: rspam08 X-Rspamd-Queue-Id: A6BD51A000E X-Stat-Signature: fkd1ukw7bisw3wi444c1o5os3r47i8of X-Rspam-User: X-HE-Tag: 1700448097-837332 X-HE-Meta: U2FsdGVkX1+RLO7l3CK9WMBKE8g5XHrulHSxQKdy9kyjYSOMPN7n97atIgGoB41YGRxs3ZTHLzObp3k4GYoKVStRhDqSq9DKprzbO8gfixNhD0zt8hvfSIOTF58ycuX0gaKcjzRo9LK8UT7AwxogI+IRBJseDFMxkGM0hBPbk0JpvFo01xC0IS+FZ70lXG0TztIeQRzej0skTESub2fQ21ALyrnBugs8lczaBMWNDDv3KT/bbiModvWze3B29W/nDbTUAlW6rUvymZhz4A7ZLtAvMPU0zaUGoKxzK+SmyQX9D4Lz2kHndVNy+uinqhei3Sdws0tjHkjAD1+1B68t/1ukrkqQwy/VCVadPeg23xEjXFhP5s1YmcW4tfXSiR4wzvMnB32CL7nilXeEPokcRkwCjBFmFqTFIZ5oHBd0wbskyAIDVC3nGP9ayOV17rH57uLPKV8eCZmsV6XXMywXlUx9z4pL7p8F757MN5GcfwxjtXSIhVBbOgF7PyDbiqXW5MJQCe/tw+5blxAOsalcHLKR1mecuZNhrP9PqW7AibvMxVYdGPKq58boYThox8WcPzxkLnl53bKCZdi9IG6ZzD/22QPjLhcCGpxmVvPsubKEadyi3Ky01O+QbhDh5CplLBRFUryDPO1x4M/9GUJXT/2yMdJQevQHktWAgGcVu1zfZqYtxkgAuN6rDWr6D84Dl6634hgurpNdSxKiNWDE5hgpTMJ5/1/cLADbpU6jF042Q/6C9kX9lxiFQ9WY8Jew2+sXnUYmBKl2EeuMKR4W6KhCCyo6ha3CzJlabC5n7sVJBz8YkGWjchTf6jP/6edaS30kbxSaK3OQEX9Kged4H/BTEhaNDG9pwPQaja/tYvv2vtHIkeuycXseaJ7gaeXP2VwlzgnwKf9luKOuqs4HW3g5x1pbEJPG8P/g+tZGksbNpsB6I7d5xjcH4I+hZcUVVySGmmP2bAS2yuLlFrQ 5aWkNPAh VhZnIgn27VAs1mYMO+/Iujw52EF4BQbuPkf+gtaB6ntK+E2S1cLYw2es8jKKrYONIWymbIv2em6Zuzduz7T6cbSZwj/hJ0HNrESEl74nXaIK5lNml04q9NESLUudOBPDq+CucheL20HEsDzNHuzaKrbhS4N708QuTbcsUK5EjUhUiz6M= 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 Nhat, On Sun, Nov 19, 2023 at 01:50:17PM -0800, Chris Li wrote: > On Sun, Nov 19, 2023 at 11:08 AM Nhat Pham wrote: > > I don't have any major argument against this. It just seems a bit > > heavyweight for what we need at the moment (only disabling > > swap-to-disk usage). > > The first milestone we just implement the reserved keywords without > the custom swap tier list. > That should be very similar to "zswap.writeback". Instead of writing 0 > to "zswap.writeback". > You write "zswap" to "swap.tiers". Writing "none" will disable all > swap. Writing "all" will allow all swap devices. > I consider this conceptually cleaner than the "zswap.writeback" == 0 > will also disable other swap types behavior. "disabled zswap writeback > == disable all swap" feels less natural. I implement a minimal version of the "swap.tiers" to replace the "zswap.writeback". It only implements the ABI level. Under the hook it is using the writeback bool. This patch builds on top of your V5 patch. implement memory.swap.tiers on top of memory.zswap.writeback. "memory.swap.tiers" supports two key words for now: all: all swap swap tiers are considered. (previously zswap.writback == 1) zswap: only zswap tier are considered. (previously zswap.writeback == 0) Index: linux/mm/memcontrol.c =================================================================== --- linux.orig/mm/memcontrol.c +++ linux/mm/memcontrol.c @@ -7992,6 +7992,32 @@ static int swap_events_show(struct seq_f return 0; } +static int swap_tiers_show(struct seq_file *m, void *v) +{ + struct mem_cgroup *memcg = mem_cgroup_from_seq(m); + + seq_printf(m, "%s\n", READ_ONCE(memcg->zswap_writeback) ? "all" : "zswap"); + return 0; +} + +static ssize_t swap_tiers_write(struct kernfs_open_file *of, + char *buf, size_t nbytes, loff_t off) +{ + struct mem_cgroup *memcg = mem_cgroup_from_css(of_css(of)); + int zswap_writeback; + + buf = strstrip(buf); + if (!strcmp(buf, "all")) + zswap_writeback = 1; + else if (!strcmp(buf, "zswap")) + zswap_writeback = 0; + else + return -EINVAL; + + WRITE_ONCE(memcg->zswap_writeback, zswap_writeback); + return nbytes; +} + static struct cftype swap_files[] = { { .name = "swap.current", @@ -8021,6 +8047,12 @@ static struct cftype swap_files[] = { .file_offset = offsetof(struct mem_cgroup, swap_events_file), .seq_show = swap_events_show, }, + { + .name = "swap.tiers", + .seq_show = swap_tiers_show, + .write = swap_tiers_write, + }, + { } /* terminate */ }; @@ -8183,31 +8215,6 @@ static ssize_t zswap_max_write(struct ke return nbytes; } -static int zswap_writeback_show(struct seq_file *m, void *v) -{ - struct mem_cgroup *memcg = mem_cgroup_from_seq(m); - - seq_printf(m, "%d\n", READ_ONCE(memcg->zswap_writeback)); - return 0; -} - -static ssize_t zswap_writeback_write(struct kernfs_open_file *of, - char *buf, size_t nbytes, loff_t off) -{ - struct mem_cgroup *memcg = mem_cgroup_from_css(of_css(of)); - int zswap_writeback; - ssize_t parse_ret = kstrtoint(strstrip(buf), 0, &zswap_writeback); - - if (parse_ret) - return parse_ret; - - if (zswap_writeback != 0 && zswap_writeback != 1) - return -EINVAL; - - WRITE_ONCE(memcg->zswap_writeback, zswap_writeback); - return nbytes; -} - static struct cftype zswap_files[] = { { .name = "zswap.current", @@ -8220,11 +8227,6 @@ static struct cftype zswap_files[] = { .seq_show = zswap_max_show, .write = zswap_max_write, }, - { - .name = "zswap.writeback", - .seq_show = zswap_writeback_show, - .write = zswap_writeback_write, - }, { } /* terminate */ }; #endif /* CONFIG_MEMCG_KMEM && CONFIG_ZSWAP */