diff options
| author | Alexey Dobriyan <adobriyan@gmail.com> | 2009-06-17 19:27:56 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-06-18 16:03:56 -0400 |
| commit | 90af90d7d3a7411db64860c9d6e5798ff87cad08 (patch) | |
| tree | 84f9c68edb231d6b64d2b24add4d7bedcb5a4aa8 /kernel | |
| parent | 612ce478fac2729ad564ec3f5d3c551674b8e9c2 (diff) | |
nsproxy: extract create_nsproxy()
clone_nsproxy() does useless copying of old nsproxy -- every pointer will
be rewritten to new ns or to old ns. Remove copying, rename
clone_nsproxy(), create_nsproxy() will be used by C/R code to create fresh
nsproxy on restart.
Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
Acked-by: Serge Hallyn <serue@us.ibm.com>
Cc: Pavel Emelyanov <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>
Diffstat (limited to 'kernel')
| -rw-r--r-- | kernel/nsproxy.c | 19 |
1 files changed, 7 insertions, 12 deletions
diff --git a/kernel/nsproxy.c b/kernel/nsproxy.c index 63598dca2d0c..09b4ff9711b2 100644 --- a/kernel/nsproxy.c +++ b/kernel/nsproxy.c | |||
| @@ -26,19 +26,14 @@ static struct kmem_cache *nsproxy_cachep; | |||
| 26 | 26 | ||
| 27 | struct nsproxy init_nsproxy = INIT_NSPROXY(init_nsproxy); | 27 | struct nsproxy init_nsproxy = INIT_NSPROXY(init_nsproxy); |
| 28 | 28 | ||
| 29 | /* | 29 | static inline struct nsproxy *create_nsproxy(void) |
| 30 | * creates a copy of "orig" with refcount 1. | ||
| 31 | */ | ||
| 32 | static inline struct nsproxy *clone_nsproxy(struct nsproxy *orig) | ||
| 33 | { | 30 | { |
| 34 | struct nsproxy *ns; | 31 | struct nsproxy *nsproxy; |
| 35 | 32 | ||
| 36 | ns = kmem_cache_alloc(nsproxy_cachep, GFP_KERNEL); | 33 | nsproxy = kmem_cache_alloc(nsproxy_cachep, GFP_KERNEL); |
| 37 | if (ns) { | 34 | if (nsproxy) |
| 38 | memcpy(ns, orig, sizeof(struct nsproxy)); | 35 | atomic_set(&nsproxy->count, 1); |
| 39 | atomic_set(&ns->count, 1); | 36 | return nsproxy; |
| 40 | } | ||
| 41 | return ns; | ||
| 42 | } | 37 | } |
| 43 | 38 | ||
| 44 | /* | 39 | /* |
| @@ -52,7 +47,7 @@ static struct nsproxy *create_new_namespaces(unsigned long flags, | |||
| 52 | struct nsproxy *new_nsp; | 47 | struct nsproxy *new_nsp; |
| 53 | int err; | 48 | int err; |
| 54 | 49 | ||
| 55 | new_nsp = clone_nsproxy(tsk->nsproxy); | 50 | new_nsp = create_nsproxy(); |
| 56 | if (!new_nsp) | 51 | if (!new_nsp) |
| 57 | return ERR_PTR(-ENOMEM); | 52 | return ERR_PTR(-ENOMEM); |
| 58 | 53 | ||
