Disabling memcgs on Android from kernel command-line because important due to
new requirements for all vendors to share the same kernel config and because
some vendors use memcgs while others don't. The ones who don't, have to disable
memcgs via "cgroup_disable=memory" kernel command-line option and we would like
to minimize the cost of disabling memcgs this way vs disabling CONFIG_MEMCG.
This patchset is focused on minimizing performance costs of this option.
When running pft test with memcgs disabled via CONFIG_MEMCG=n vs
"cgroup_disable=memory" command-line option, we measured ~6% drop in the
average pagefault/sec rate with stddev of ~2%. The results were obtained by
running pft test 1500 times and averaging the results on an 8-core ARM64
Android device with system services stopped, performance governor and enabling
only Big or Little cores in one test to minimize the noise.
Using perf, a number of relatively high-cost areas were identified where extra
operations can be minimized. The patchset consists of a number of optimisations
gradually reducing this regression. Patches are applied incrementally while
testing and recording the impact for each one:
6.01% with vanilla cgroup_disable vs CONFIG_MEMCG=n
3.87% after patch #1 adding mem_cgroup_disabled checks vs CONFIG_MEMCG=n
3.49% after patch #2 inlining mem_cgroup_{charge/uncharge} vs CONFIG_MEMCG=n
2.48% After patch #3 inlining swap-related functions vs CONFIG_MEMCG=n
I kept them separate because they vary in their "impact vs readability cost"
and I'm not sure which ones pass the acceptable threashold.
Suren Baghdasaryan (3):
mm, memcg: add mem_cgroup_disabled checks in vmpressure and
swap-related functions
mm, memcg: inline mem_cgroup_{charge/uncharge} to improve disabled
memcg config
mm, memcg: inline swap-related functions to improve disabled memcg
config
include/linux/memcontrol.h | 54 ++++++++++++++++++++++++++++++++++----
include/linux/swap.h | 26 +++++++++++++++---
mm/memcontrol.c | 52 +++++-------------------------------
mm/swapfile.c | 2 +-
mm/vmpressure.c | 7 ++++-
5 files changed, 86 insertions(+), 55 deletions(-)
--
2.32.0.93.g670b81a890-goog
