diff options
author | Pierre Peiffer <pierre.peiffer@bull.net> | 2008-02-08 07:18:57 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2008-02-08 12:22:26 -0500 |
commit | 01b8b07a5d77d22e609267dcae74d15e3e9c5f13 (patch) | |
tree | dd53c51f75de9511da7fdf283c73080d6c2552a8 /ipc/namespace.c | |
parent | ed2ddbf88c0ddeeae4c78bb306a116dfd867c55c (diff) |
IPC: consolidate sem_exit_ns(), msg_exit_ns() and shm_exit_ns()
sem_exit_ns(), msg_exit_ns() and shm_exit_ns() are all called when an
ipc_namespace is released to free all ipcs of each type. But in fact, they
do the same thing: they loop around all ipcs to free them individually by
calling a specific routine.
This patch proposes to consolidate this by introducing a common function,
free_ipcs(), that do the job. The specific routine to call on each
individual ipcs is passed as parameter. For this, these ipc-specific
'free' routines are reworked to take a generic 'struct ipc_perm' as
parameter.
Signed-off-by: Pierre Peiffer <pierre.peiffer@bull.net>
Cc: Cedric Le Goater <clg@fr.ibm.com>
Cc: Pavel Emelyanov <xemul@openvz.org>
Cc: Nadia Derbey <Nadia.Derbey@bull.net>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'ipc/namespace.c')
-rw-r--r-- | ipc/namespace.c | 30 |
1 files changed, 30 insertions, 0 deletions
diff --git a/ipc/namespace.c b/ipc/namespace.c index 1fed8922d475..1b967655eb35 100644 --- a/ipc/namespace.c +++ b/ipc/namespace.c | |||
@@ -44,6 +44,36 @@ struct ipc_namespace *copy_ipcs(unsigned long flags, struct ipc_namespace *ns) | |||
44 | return new_ns; | 44 | return new_ns; |
45 | } | 45 | } |
46 | 46 | ||
47 | /* | ||
48 | * free_ipcs - free all ipcs of one type | ||
49 | * @ns: the namespace to remove the ipcs from | ||
50 | * @ids: the table of ipcs to free | ||
51 | * @free: the function called to free each individual ipc | ||
52 | * | ||
53 | * Called for each kind of ipc when an ipc_namespace exits. | ||
54 | */ | ||
55 | void free_ipcs(struct ipc_namespace *ns, struct ipc_ids *ids, | ||
56 | void (*free)(struct ipc_namespace *, struct kern_ipc_perm *)) | ||
57 | { | ||
58 | struct kern_ipc_perm *perm; | ||
59 | int next_id; | ||
60 | int total, in_use; | ||
61 | |||
62 | down_write(&ids->rw_mutex); | ||
63 | |||
64 | in_use = ids->in_use; | ||
65 | |||
66 | for (total = 0, next_id = 0; total < in_use; next_id++) { | ||
67 | perm = idr_find(&ids->ipcs_idr, next_id); | ||
68 | if (perm == NULL) | ||
69 | continue; | ||
70 | ipc_lock_by_ptr(perm); | ||
71 | free(ns, perm); | ||
72 | total++; | ||
73 | } | ||
74 | up_write(&ids->rw_mutex); | ||
75 | } | ||
76 | |||
47 | void free_ipc_ns(struct kref *kref) | 77 | void free_ipc_ns(struct kref *kref) |
48 | { | 78 | { |
49 | struct ipc_namespace *ns; | 79 | struct ipc_namespace *ns; |