* [PATCH net] sock: only define socket limit if mem cgroup configured
@ 2013-02-22 17:59 Stephen Hemminger
2013-02-22 20:14 ` David Miller
0 siblings, 1 reply; 2+ messages in thread
From: Stephen Hemminger @ 2013-02-22 17:59 UTC (permalink / raw)
To: David Miller; +Cc: netdev
The mem cgroup socket limit is only used if the config option is
enabled. Found with sparse
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
--- a/net/core/sock.c 2013-02-21 13:52:10.000000000 -0800
+++ b/net/core/sock.c 2013-02-21 21:03:06.075605811 -0800
@@ -186,8 +186,10 @@ void mem_cgroup_sockets_destroy(struct m
static struct lock_class_key af_family_keys[AF_MAX];
static struct lock_class_key af_family_slock_keys[AF_MAX];
+#if defined(CONFIG_MEMCG_KMEM)
struct static_key memcg_socket_limit_enabled;
EXPORT_SYMBOL(memcg_socket_limit_enabled);
+#endif
/*
* Make lock validator output more readable. (we pre-construct these
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH net] sock: only define socket limit if mem cgroup configured
2013-02-22 17:59 [PATCH net] sock: only define socket limit if mem cgroup configured Stephen Hemminger
@ 2013-02-22 20:14 ` David Miller
0 siblings, 0 replies; 2+ messages in thread
From: David Miller @ 2013-02-22 20:14 UTC (permalink / raw)
To: stephen; +Cc: netdev
From: Stephen Hemminger <stephen@networkplumber.org>
Date: Fri, 22 Feb 2013 09:59:10 -0800
> The mem cgroup socket limit is only used if the config option is
> enabled. Found with sparse
>
> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
Applied.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-02-22 20:14 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-02-22 17:59 [PATCH net] sock: only define socket limit if mem cgroup configured Stephen Hemminger
2013-02-22 20:14 ` David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).