diff options
| author | Andrew Morton <akpm@linux-foundation.org> | 2009-04-29 18:59:58 -0400 |
|---|---|---|
| committer | Ingo Molnar <mingo@elte.hu> | 2009-04-30 03:01:34 -0400 |
| commit | a511e3f968c462a55ef58697257f5347c73d306e (patch) | |
| tree | 7b331c694e0149f4f9e692eaec1726d28cedac8d /kernel | |
| parent | 23b94b967f118bef941369238f33c8140be46539 (diff) | |
mutex: add atomic_dec_and_mutex_lock(), fix
include/linux/mutex.h:136: warning: 'mutex_lock' declared inline after being called
include/linux/mutex.h:136: warning: previous declaration of 'mutex_lock' was here
uninline it.
[ Impact: clean up and uninline, address compiler warning ]
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Cc: Al Viro <viro@zeniv.linux.org.uk>
Cc: Christoph Hellwig <hch@lst.de>
Cc: Eric Paris <eparis@redhat.com>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
LKML-Reference: <200904292318.n3TNIsi6028340@imap1.linux-foundation.org>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel')
| -rw-r--r-- | kernel/mutex.c | 25 |
1 files changed, 24 insertions, 1 deletions
diff --git a/kernel/mutex.c b/kernel/mutex.c index 507cf2b5e9f1..e2d25e9e62ae 100644 --- a/kernel/mutex.c +++ b/kernel/mutex.c | |||
| @@ -471,5 +471,28 @@ int __sched mutex_trylock(struct mutex *lock) | |||
| 471 | 471 | ||
| 472 | return ret; | 472 | return ret; |
| 473 | } | 473 | } |
| 474 | |||
| 475 | EXPORT_SYMBOL(mutex_trylock); | 474 | EXPORT_SYMBOL(mutex_trylock); |
| 475 | |||
| 476 | /** | ||
| 477 | * atomic_dec_and_mutex_lock - return holding mutex if we dec to 0 | ||
| 478 | * @cnt: the atomic which we are to dec | ||
| 479 | * @lock: the mutex to return holding if we dec to 0 | ||
| 480 | * | ||
| 481 | * return true and hold lock if we dec to 0, return false otherwise | ||
| 482 | */ | ||
| 483 | int atomic_dec_and_mutex_lock(atomic_t *cnt, struct mutex *lock) | ||
| 484 | { | ||
| 485 | /* dec if we can't possibly hit 0 */ | ||
| 486 | if (atomic_add_unless(cnt, -1, 1)) | ||
| 487 | return 0; | ||
| 488 | /* we might hit 0, so take the lock */ | ||
| 489 | mutex_lock(lock); | ||
| 490 | if (!atomic_dec_and_test(cnt)) { | ||
| 491 | /* when we actually did the dec, we didn't hit 0 */ | ||
| 492 | mutex_unlock(lock); | ||
| 493 | return 0; | ||
| 494 | } | ||
| 495 | /* we hit 0, and we hold the lock */ | ||
| 496 | return 1; | ||
| 497 | } | ||
| 498 | EXPORT_SYMBOL(atomic_dec_and_mutex_lock); | ||
