]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
systemd-udevd: limit children-max by available memory (#8668)
authorMartin Wilck <mwilck@suse.com>
Sat, 7 Apr 2018 15:33:48 +0000 (17:33 +0200)
committerZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Sat, 7 Apr 2018 15:33:48 +0000 (17:33 +0200)
Udev workers consume typically 50-100MiB virtual memory.
On systems with lots of CPUs and relatively low memory, that may
easily cause workers to be OOM-killed.

This patch limits the number of workers to 8 per GiB memory.
But don't let the limit drop below the smallest value we had
without this patch (8 + 1 * 2 = 10); on small systems, udev's
memory footprint is likely lower.

src/udev/udevd.c

index 254a72556ef6167f63613eff2eb9370501e207b5..006cdd3501268628ed99abcde9b506d8bb383eec 100644 (file)
@@ -1673,12 +1673,16 @@ int main(int argc, char *argv[]) {
 
         if (arg_children_max == 0) {
                 cpu_set_t cpu_set;
+                unsigned long mem_limit;
 
                 arg_children_max = 8;
 
                 if (sched_getaffinity(0, sizeof(cpu_set), &cpu_set) == 0)
                         arg_children_max += CPU_COUNT(&cpu_set) * 2;
 
+                mem_limit = physical_memory() / (128LU*1024*1024);
+                arg_children_max = MAX(10U, MIN(arg_children_max, mem_limit));
+
                 log_debug("set children_max to %u", arg_children_max);
         }