Skip to content

Commit 0c85d55

Browse files
shakeelbkelsey-steele
authored andcommitted
memcg: introduce per-memcg reclaim interface
This patch series adds a memory.reclaim proactive reclaim interface. The rationale behind the interface and how it works are in the first patch. This patch (of 4): Introduce a memcg interface to trigger memory reclaim on a memory cgroup. Use case: Proactive Reclaim --------------------------- A userspace proactive reclaimer can continuously probe the memcg to reclaim a small amount of memory. This gives more accurate and up-to-date workingset estimation as the LRUs are continuously sorted and can potentially provide more deterministic memory overcommit behavior. The memory overcommit controller can provide more proactive response to the changing behavior of the running applications instead of being reactive. A userspace reclaimer's purpose in this case is not a complete replacement for kswapd or direct reclaim, it is to proactively identify memory savings opportunities and reclaim some amount of cold pages set by the policy to free up the memory for more demanding jobs or scheduling new jobs. A user space proactive reclaimer is used in Google data centers. Additionally, Meta's TMO paper recently referenced a very similar interface used for user space proactive reclaim: https://dl.acm.org/doi/pdf/10.1145/3503222.3507731 Benefits of a user space reclaimer: ----------------------------------- 1) More flexible on who should be charged for the cpu of the memory reclaim. For proactive reclaim, it makes more sense to be centralized. 2) More flexible on dedicating the resources (like cpu). The memory overcommit controller can balance the cost between the cpu usage and the memory reclaimed. 3) Provides a way to the applications to keep their LRUs sorted, so, under memory pressure better reclaim candidates are selected. This also gives more accurate and uptodate notion of working set for an application. Why memory.high is not enough? ------------------------------ - memory.high can be used to trigger reclaim in a memcg and can potentially be used for proactive reclaim. However there is a big downside in using memory.high. It can potentially introduce high reclaim stalls in the target application as the allocations from the processes or the threads of the application can hit the temporary memory.high limit. - Userspace proactive reclaimers usually use feedback loops to decide how much memory to proactively reclaim from a workload. The metrics used for this are usually either refaults or PSI, and these metrics will become messy if the application gets throttled by hitting the high limit. - memory.high is a stateful interface, if the userspace proactive reclaimer crashes for any reason while triggering reclaim it can leave the application in a bad state. - If a workload is rapidly expanding, setting memory.high to proactively reclaim memory can result in actually reclaiming more memory than intended. The benefits of such interface and shortcomings of existing interface were further discussed in this RFC thread: https://lore.kernel.org/linux-mm/[email protected]/ Interface: ---------- Introducing a very simple memcg interface 'echo 10M > memory.reclaim' to trigger reclaim in the target memory cgroup. The interface is introduced as a nested-keyed file to allow for future optional arguments to be easily added to configure the behavior of reclaim. Possible Extensions: -------------------- - This interface can be extended with an additional parameter or flags to allow specifying one or more types of memory to reclaim from (e.g. file, anon, ..). - The interface can also be extended with a node mask to reclaim from specific nodes. This has use cases for reclaim-based demotion in memory tiering systens. - A similar per-node interface can also be added to support proactive reclaim and reclaim-based demotion in systems without memcg. - Add a timeout parameter to make it easier for user space to call the interface without worrying about being blocked for an undefined amount of time. For now, let's keep things simple by adding the basic functionality. [[email protected]: worked on versions v2 onwards, refreshed to current master, updated commit message based on recent discussions and use cases] Link: https://lkml.kernel.org/r/[email protected] Link: https://lkml.kernel.org/r/[email protected] Signed-off-by: Shakeel Butt <[email protected]> Co-developed-by: Yosry Ahmed <[email protected]> Signed-off-by: Yosry Ahmed <[email protected]> Acked-by: Johannes Weiner <[email protected]> Acked-by: Michal Hocko <[email protected]> Acked-by: Wei Xu <[email protected]> Acked-by: Roman Gushchin <[email protected]> Acked-by: David Rientjes <[email protected]> Cc: Tejun Heo <[email protected]> Cc: Zefan Li <[email protected]> Cc: Jonathan Corbet <[email protected]> Cc: Shuah Khan <[email protected]> Cc: Yu Zhao <[email protected]> Cc: Dave Hansen <[email protected]> Cc: Greg Thelen <[email protected]> Cc: Chen Wandun <[email protected]> Cc: Vaibhav Jain <[email protected]> Cc: "Michal Koutn" <[email protected]> Cc: Tim Chen <[email protected]> Signed-off-by: Andrew Morton <[email protected]> [kms: backport to 5.15] Signed-off-by: Kelsey Steele <[email protected]>
1 parent 8bb7eca commit 0c85d55

File tree

2 files changed

+66
-0
lines changed

2 files changed

+66
-0
lines changed

Documentation/admin-guide/cgroup-v2.rst

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1200,6 +1200,27 @@ PAGE_SIZE multiple when read back.
12001200
high limit is used and monitored properly, this limit's
12011201
utility is limited to providing the final safety net.
12021202

1203+
memory.reclaim
1204+
A write-only nested-keyed file which exists for all cgroups.
1205+
1206+
This is a simple interface to trigger memory reclaim in the
1207+
target cgroup.
1208+
1209+
This file accepts a single key, the number of bytes to reclaim.
1210+
No nested keys are currently supported.
1211+
1212+
Example::
1213+
1214+
echo "1G" > memory.reclaim
1215+
1216+
The interface can be later extended with nested keys to
1217+
configure the reclaim behavior. For example, specify the
1218+
type of memory to reclaim from (anon, file, ..).
1219+
1220+
Please note that the kernel can over or under reclaim from
1221+
the target cgroup. If less bytes are reclaimed than the
1222+
specified amount, -EAGAIN is returned.
1223+
12031224
memory.oom.group
12041225
A read-write single value file which exists on non-root
12051226
cgroups. The default value is "0".

mm/memcontrol.c

Lines changed: 45 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6429,6 +6429,46 @@ static ssize_t memory_oom_group_write(struct kernfs_open_file *of,
64296429
return nbytes;
64306430
}
64316431

6432+
static ssize_t memory_reclaim(struct kernfs_open_file *of, char *buf,
6433+
size_t nbytes, loff_t off)
6434+
{
6435+
struct mem_cgroup *memcg = mem_cgroup_from_css(of_css(of));
6436+
unsigned int nr_retries = MAX_RECLAIM_RETRIES;
6437+
unsigned long nr_to_reclaim, nr_reclaimed = 0;
6438+
int err;
6439+
6440+
buf = strstrip(buf);
6441+
err = page_counter_memparse(buf, "", &nr_to_reclaim);
6442+
if (err)
6443+
return err;
6444+
6445+
while (nr_reclaimed < nr_to_reclaim) {
6446+
unsigned long reclaimed;
6447+
6448+
if (signal_pending(current))
6449+
return -EINTR;
6450+
6451+
/*
6452+
* This is the final attempt, drain percpu lru caches in the
6453+
* hope of introducing more evictable pages for
6454+
* try_to_free_mem_cgroup_pages().
6455+
*/
6456+
if (!nr_retries)
6457+
lru_add_drain_all();
6458+
6459+
reclaimed = try_to_free_mem_cgroup_pages(memcg,
6460+
nr_to_reclaim - nr_reclaimed,
6461+
GFP_KERNEL, true);
6462+
6463+
if (!reclaimed && !nr_retries--)
6464+
return -EAGAIN;
6465+
6466+
nr_reclaimed += reclaimed;
6467+
}
6468+
6469+
return nbytes;
6470+
}
6471+
64326472
static struct cftype memory_files[] = {
64336473
{
64346474
.name = "current",
@@ -6487,6 +6527,11 @@ static struct cftype memory_files[] = {
64876527
.seq_show = memory_oom_group_show,
64886528
.write = memory_oom_group_write,
64896529
},
6530+
{
6531+
.name = "reclaim",
6532+
.flags = CFTYPE_NS_DELEGATABLE,
6533+
.write = memory_reclaim,
6534+
},
64906535
{ } /* terminate */
64916536
};
64926537

0 commit comments

Comments
 (0)