diff options
| author | Linus Torvalds <torvalds@linux-foundation.org> | 2013-05-03 15:22:00 -0700 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2013-05-04 11:24:21 -0700 |
| commit | 73b29505c36eeb4751eccad41f6aad78562521f8 (patch) | |
| tree | 228603617659029e082a0968b2e0a6490c121b0d /ipc/sem.c | |
| parent | 6d49dab8ae06c6d35a4d0967364a9ecbe8fdea2c (diff) | |
| download | lwn-73b29505c36eeb4751eccad41f6aad78562521f8.tar.gz lwn-73b29505c36eeb4751eccad41f6aad78562521f8.zip | |
ipc: sem_putref() does not need the semaphore lock any more
ipc_rcu_putref() uses atomics for the refcount, and the games to lock
and unlock the semaphore just to try to keep the reference counting
working are no longer useful.
Acked-by: Davidlohr Bueso <davidlohr.bueso@hp.com>
Cc: Rik van Riel <riel@redhat.com>
Cc: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'ipc/sem.c')
| -rw-r--r-- | ipc/sem.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/ipc/sem.c b/ipc/sem.c index 4b4139f6ad5c..5cf7b4c093b3 100644 --- a/ipc/sem.c +++ b/ipc/sem.c @@ -329,9 +329,7 @@ static inline void sem_lock_and_putref(struct sem_array *sma) static inline void sem_putref(struct sem_array *sma) { - sem_lock_and_putref(sma); - sem_unlock(sma, -1); - rcu_read_unlock(); + ipc_rcu_putref(sma); } static inline void sem_rmid(struct ipc_namespace *ns, struct sem_array *s) |