[106/212] memcg: enable accounting for posix_timers_cache slab

Commit Message

Andrew Morton Sept. 2, 2021, 9:55 p.m. UTC

From: Vasily Averin <[e-mail]>
Subject: memcg: enable accounting for posix_timers_cache slab

A program may create multiple interval timers using timer_create().  For
each timer the kernel preallocates a "queued real-time signal",
Consequently, the number of timers is limited by the RLIMIT_SIGPENDING
resource limit.  The allocated object is quite small, ~250 bytes, but even
the default signal limits allow to consume up to 100 megabytes per user.

It makes sense to account for them to limit the host's memory consumption
from inside the memcg-limited container.

Link: https://lkml.kernel.org/r/[e-mail]
Signed-off-by: Vasily Averin <[e-mail]>
Reviewed-by: Thomas Gleixner <[e-mail]>
Reviewed-by: Shakeel Butt <[e-mail]>
Cc: Alexander Viro <[e-mail]>
Cc: Alexey Dobriyan <[e-mail]>
Cc: Andrei Vagin <[e-mail]>
Cc: Borislav Petkov <[e-mail]>
Cc: Borislav Petkov <[e-mail]>
Cc: Christian Brauner <[e-mail]>
Cc: Dmitry Safonov <[e-mail]>
Cc: "Eric W. Biederman" <[e-mail]>
Cc: Greg Kroah-Hartman <[e-mail]>
Cc: "H. Peter Anvin" <[e-mail]>
Cc: Ingo Molnar <[e-mail]>
Cc: "J. Bruce Fields" <[e-mail]>
Cc: Jeff Layton <[e-mail]>
Cc: Jens Axboe <[e-mail]>
Cc: Jiri Slaby <[e-mail]>
Cc: Johannes Weiner <[e-mail]>
Cc: Kirill Tkhai <[e-mail]>
Cc: Michal Hocko <[e-mail]>
Cc: Oleg Nesterov <[e-mail]>
Cc: Roman Gushchin <[e-mail]>
Cc: Serge Hallyn <[e-mail]>
Cc: Tejun Heo <[e-mail]>
Cc: Vladimir Davydov <[e-mail]>
Cc: Yutian Yang <[e-mail]>
Cc: Zefan Li <[e-mail]>
Signed-off-by: Andrew Morton <[e-mail]>
---

 kernel/time/posix-timers.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Patch

--- a/kernel/time/posix-timers.c~memcg-enable-accounting-for-posix_timers_cache-slab
+++ a/kernel/time/posix-timers.c
@@ -273,8 +273,8 @@  static int posix_get_hrtimer_res(clockid
 static __init int init_posix_timers(void)
 {
 	posix_timers_cache = kmem_cache_create("posix_timers_cache",
-					sizeof (struct k_itimer), 0, SLAB_PANIC,
-					NULL);
+					sizeof(struct k_itimer), 0,
+					SLAB_PANIC | SLAB_ACCOUNT, NULL);
 	return 0;
 }
 __initcall(init_posix_timers);