diff options
author | Al Viro <viro@zeniv.linux.org.uk> | 2013-10-03 13:28:06 -0400 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2013-10-24 23:43:29 -0400 |
commit | 1adfcb03e31ba0d6be5fddf773da4357d0792cbb (patch) | |
tree | c5c89b58403938c263e81c115fc864ad9aabc782 /kernel | |
parent | 1dcddd4abd2c6df7f28928ad5cafa4a1cd20030b (diff) |
pid_namespace: make freeing struct pid_namespace rcu-delayed
makes procfs ->premission() instances safety in RCU mode independent
from vfsmount_lock.
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'kernel')
-rw-r--r-- | kernel/pid_namespace.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/kernel/pid_namespace.c b/kernel/pid_namespace.c index 42086551a24a..06c62de9c711 100644 --- a/kernel/pid_namespace.c +++ b/kernel/pid_namespace.c | |||
@@ -132,6 +132,12 @@ out: | |||
132 | return ERR_PTR(err); | 132 | return ERR_PTR(err); |
133 | } | 133 | } |
134 | 134 | ||
135 | static void delayed_free_pidns(struct rcu_head *p) | ||
136 | { | ||
137 | kmem_cache_free(pid_ns_cachep, | ||
138 | container_of(p, struct pid_namespace, rcu)); | ||
139 | } | ||
140 | |||
135 | static void destroy_pid_namespace(struct pid_namespace *ns) | 141 | static void destroy_pid_namespace(struct pid_namespace *ns) |
136 | { | 142 | { |
137 | int i; | 143 | int i; |
@@ -140,7 +146,7 @@ static void destroy_pid_namespace(struct pid_namespace *ns) | |||
140 | for (i = 0; i < PIDMAP_ENTRIES; i++) | 146 | for (i = 0; i < PIDMAP_ENTRIES; i++) |
141 | kfree(ns->pidmap[i].page); | 147 | kfree(ns->pidmap[i].page); |
142 | put_user_ns(ns->user_ns); | 148 | put_user_ns(ns->user_ns); |
143 | kmem_cache_free(pid_ns_cachep, ns); | 149 | call_rcu(&ns->rcu, delayed_free_pidns); |
144 | } | 150 | } |
145 | 151 | ||
146 | struct pid_namespace *copy_pid_ns(unsigned long flags, | 152 | struct pid_namespace *copy_pid_ns(unsigned long flags, |