diff options
author | Cedric Le Goater <clg@fr.ibm.com> | 2007-07-16 02:41:07 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-07-16 12:05:47 -0400 |
commit | 98c0d07cbf2a8582a0341b05ad564247e608f6f9 (patch) | |
tree | 9c72ff1a024f472a9186cbb62f244e1e5829e639 | |
parent | 467e9f4b5086a60a5cb2e032ccaf4a31abadc4c2 (diff) |
add a kmem_cache for nsproxy objects
It should improve performance in some scenarii where a lot of
these nsproxy objects are created by unsharing namespaces. This is
a typical use of virtual servers that are being created or entered.
This is also a good tool to find leaks and gather statistics on
namespace usage.
Signed-off-by: Cedric Le Goater <clg@fr.ibm.com>
Cc: Herbert Poetzl <herbert@13thfloor.at>
Cc: Pavel Emelianov <xemul@openvz.org>
Cc: "Eric W. Biederman" <ebiederm@xmission.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | kernel/nsproxy.c | 21 |
1 files changed, 17 insertions, 4 deletions
diff --git a/kernel/nsproxy.c b/kernel/nsproxy.c index 15a6015a849d..4b7fcc8f9a48 100644 --- a/kernel/nsproxy.c +++ b/kernel/nsproxy.c | |||
@@ -21,6 +21,8 @@ | |||
21 | #include <linux/utsname.h> | 21 | #include <linux/utsname.h> |
22 | #include <linux/pid_namespace.h> | 22 | #include <linux/pid_namespace.h> |
23 | 23 | ||
24 | static struct kmem_cache *nsproxy_cachep; | ||
25 | |||
24 | struct nsproxy init_nsproxy = INIT_NSPROXY(init_nsproxy); | 26 | struct nsproxy init_nsproxy = INIT_NSPROXY(init_nsproxy); |
25 | 27 | ||
26 | static inline void get_nsproxy(struct nsproxy *ns) | 28 | static inline void get_nsproxy(struct nsproxy *ns) |
@@ -43,9 +45,11 @@ static inline struct nsproxy *clone_nsproxy(struct nsproxy *orig) | |||
43 | { | 45 | { |
44 | struct nsproxy *ns; | 46 | struct nsproxy *ns; |
45 | 47 | ||
46 | ns = kmemdup(orig, sizeof(struct nsproxy), GFP_KERNEL); | 48 | ns = kmem_cache_alloc(nsproxy_cachep, GFP_KERNEL); |
47 | if (ns) | 49 | if (ns) { |
50 | memcpy(ns, orig, sizeof(struct nsproxy)); | ||
48 | atomic_set(&ns->count, 1); | 51 | atomic_set(&ns->count, 1); |
52 | } | ||
49 | return ns; | 53 | return ns; |
50 | } | 54 | } |
51 | 55 | ||
@@ -109,7 +113,7 @@ out_uts: | |||
109 | if (new_nsp->mnt_ns) | 113 | if (new_nsp->mnt_ns) |
110 | put_mnt_ns(new_nsp->mnt_ns); | 114 | put_mnt_ns(new_nsp->mnt_ns); |
111 | out_ns: | 115 | out_ns: |
112 | kfree(new_nsp); | 116 | kmem_cache_free(nsproxy_cachep, new_nsp); |
113 | return ERR_PTR(err); | 117 | return ERR_PTR(err); |
114 | } | 118 | } |
115 | 119 | ||
@@ -160,7 +164,7 @@ void free_nsproxy(struct nsproxy *ns) | |||
160 | put_pid_ns(ns->pid_ns); | 164 | put_pid_ns(ns->pid_ns); |
161 | if (ns->user_ns) | 165 | if (ns->user_ns) |
162 | put_user_ns(ns->user_ns); | 166 | put_user_ns(ns->user_ns); |
163 | kfree(ns); | 167 | kmem_cache_free(nsproxy_cachep, ns); |
164 | } | 168 | } |
165 | 169 | ||
166 | /* | 170 | /* |
@@ -185,3 +189,12 @@ int unshare_nsproxy_namespaces(unsigned long unshare_flags, | |||
185 | err = PTR_ERR(*new_nsp); | 189 | err = PTR_ERR(*new_nsp); |
186 | return err; | 190 | return err; |
187 | } | 191 | } |
192 | |||
193 | static int __init nsproxy_cache_init(void) | ||
194 | { | ||
195 | nsproxy_cachep = kmem_cache_create("nsproxy", sizeof(struct nsproxy), | ||
196 | 0, SLAB_PANIC, NULL, NULL); | ||
197 | return 0; | ||
198 | } | ||
199 | |||
200 | module_init(nsproxy_cache_init); | ||