chiark / gitweb /
pid1: turn memory accounting on by default now
authorLennart Poettering <lennart@poettering.net>
Fri, 9 Feb 2018 18:59:53 +0000 (19:59 +0100)
committerSven Eden <yamakuzure@gmx.net>
Wed, 30 May 2018 05:58:56 +0000 (07:58 +0200)
After discussions with @htejun it appears it's OK now to enable memory
accounting by default for all units without affecting system performance
too badly. facebook has made good experiences with deploying memory
accounting across their infrastructure.

This hence turns MemoryAccounting= from opt-in to opt-out, similar to
how TasksAccounting= is already handled. The other accounting options
remain off, their performance impact is too big still.

NEWS

diff --git a/NEWS b/NEWS
index 7f22e475bf581ffa04b33a3ceff178fae78de80a..df552f5f8a2bde091d4de3f9d258519ebb36059b 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -1,5 +1,21 @@
 systemd System and Service Manager
 
+CHANGES WITH 238 in spe:
+
+        * The MemoryAccounting= unit property now defaults to on. After
+          discussions with the upstream control group maintainers we learnt
+          that the negative impact of cgroup memory accounting on current
+          kernels is finally relatively minimal, so that it should be safe to
+          enable this by default without affecting system performance too
+          much. Besides memory accounting only tasks accounting is turned on by
+          default, all other forms of resource accounting (CPU, IO, IP) remain
+          off for now, because it's not clear yet that their impact is small
+          enough to move from opt-in to opt-out for them, too. We recommend
+          downstreams to leave memory accounting on by default, however in some
+          situations it might be wise to revert this change of defaults, in
+          particular on very resource constrained systems or when support for
+          old kernels is a necessity.
+
 CHANGES WITH 237:
 
         * Some keyboards come with a zoom see-saw or rocker which until now got