diff options
| author | Ingo Molnar <mingo@elte.hu> | 2006-07-03 03:24:53 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-07-03 18:27:04 -0400 |
| commit | 4ea2176dfa714882e88180b474e4cbcd888b70af (patch) | |
| tree | 7ff3810f6b8750c226234887bb3063d91e1d71c3 /include/linux | |
| parent | a8f24a3978c5f82419e1c90dc90460731204f46f (diff) | |
[PATCH] lockdep: prove rwsem locking correctness
Use the lock validator framework to prove rwsem locking correctness.
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Signed-off-by: Arjan van de Ven <arjan@linux.intel.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'include/linux')
| -rw-r--r-- | include/linux/rwsem-spinlock.h | 23 | ||||
| -rw-r--r-- | include/linux/rwsem.h | 59 |
2 files changed, 46 insertions, 36 deletions
diff --git a/include/linux/rwsem-spinlock.h b/include/linux/rwsem-spinlock.h index d68afcc36ac9..ae1fcadd598e 100644 --- a/include/linux/rwsem-spinlock.h +++ b/include/linux/rwsem-spinlock.h | |||
| @@ -32,18 +32,37 @@ struct rw_semaphore { | |||
| 32 | __s32 activity; | 32 | __s32 activity; |
| 33 | spinlock_t wait_lock; | 33 | spinlock_t wait_lock; |
| 34 | struct list_head wait_list; | 34 | struct list_head wait_list; |
| 35 | #ifdef CONFIG_DEBUG_LOCK_ALLOC | ||
| 36 | struct lockdep_map dep_map; | ||
| 37 | #endif | ||
| 35 | }; | 38 | }; |
| 36 | 39 | ||
| 40 | #ifdef CONFIG_DEBUG_LOCK_ALLOC | ||
| 41 | # define __RWSEM_DEP_MAP_INIT(lockname) , .dep_map = { .name = #lockname } | ||
| 42 | #else | ||
| 43 | # define __RWSEM_DEP_MAP_INIT(lockname) | ||
| 44 | #endif | ||
| 45 | |||
| 37 | #define __RWSEM_INITIALIZER(name) \ | 46 | #define __RWSEM_INITIALIZER(name) \ |
| 38 | { 0, SPIN_LOCK_UNLOCKED, LIST_HEAD_INIT((name).wait_list) } | 47 | { 0, SPIN_LOCK_UNLOCKED, LIST_HEAD_INIT((name).wait_list) __RWSEM_DEP_MAP_INIT(name) } |
| 39 | 48 | ||
| 40 | #define DECLARE_RWSEM(name) \ | 49 | #define DECLARE_RWSEM(name) \ |
| 41 | struct rw_semaphore name = __RWSEM_INITIALIZER(name) | 50 | struct rw_semaphore name = __RWSEM_INITIALIZER(name) |
| 42 | 51 | ||
| 43 | extern void FASTCALL(init_rwsem(struct rw_semaphore *sem)); | 52 | extern void __init_rwsem(struct rw_semaphore *sem, const char *name, |
| 53 | struct lock_class_key *key); | ||
| 54 | |||
| 55 | #define init_rwsem(sem) \ | ||
| 56 | do { \ | ||
| 57 | static struct lock_class_key __key; \ | ||
| 58 | \ | ||
| 59 | __init_rwsem((sem), #sem, &__key); \ | ||
| 60 | } while (0) | ||
| 61 | |||
| 44 | extern void FASTCALL(__down_read(struct rw_semaphore *sem)); | 62 | extern void FASTCALL(__down_read(struct rw_semaphore *sem)); |
| 45 | extern int FASTCALL(__down_read_trylock(struct rw_semaphore *sem)); | 63 | extern int FASTCALL(__down_read_trylock(struct rw_semaphore *sem)); |
| 46 | extern void FASTCALL(__down_write(struct rw_semaphore *sem)); | 64 | extern void FASTCALL(__down_write(struct rw_semaphore *sem)); |
| 65 | extern void FASTCALL(__down_write_nested(struct rw_semaphore *sem, int subclass)); | ||
| 47 | extern int FASTCALL(__down_write_trylock(struct rw_semaphore *sem)); | 66 | extern int FASTCALL(__down_write_trylock(struct rw_semaphore *sem)); |
| 48 | extern void FASTCALL(__up_read(struct rw_semaphore *sem)); | 67 | extern void FASTCALL(__up_read(struct rw_semaphore *sem)); |
| 49 | extern void FASTCALL(__up_write(struct rw_semaphore *sem)); | 68 | extern void FASTCALL(__up_write(struct rw_semaphore *sem)); |
diff --git a/include/linux/rwsem.h b/include/linux/rwsem.h index 93581534b915..658afb37c3f5 100644 --- a/include/linux/rwsem.h +++ b/include/linux/rwsem.h | |||
| @@ -27,64 +27,55 @@ struct rw_semaphore; | |||
| 27 | /* | 27 | /* |
| 28 | * lock for reading | 28 | * lock for reading |
| 29 | */ | 29 | */ |
| 30 | static inline void down_read(struct rw_semaphore *sem) | 30 | extern void down_read(struct rw_semaphore *sem); |
| 31 | { | ||
| 32 | might_sleep(); | ||
| 33 | __down_read(sem); | ||
| 34 | } | ||
| 35 | 31 | ||
| 36 | /* | 32 | /* |
| 37 | * trylock for reading -- returns 1 if successful, 0 if contention | 33 | * trylock for reading -- returns 1 if successful, 0 if contention |
| 38 | */ | 34 | */ |
| 39 | static inline int down_read_trylock(struct rw_semaphore *sem) | 35 | extern int down_read_trylock(struct rw_semaphore *sem); |
| 40 | { | ||
| 41 | int ret; | ||
| 42 | ret = __down_read_trylock(sem); | ||
| 43 | return ret; | ||
| 44 | } | ||
| 45 | 36 | ||
| 46 | /* | 37 | /* |
| 47 | * lock for writing | 38 | * lock for writing |
| 48 | */ | 39 | */ |
| 49 | static inline void down_write(struct rw_semaphore *sem) | 40 | extern void down_write(struct rw_semaphore *sem); |
| 50 | { | ||
| 51 | might_sleep(); | ||
| 52 | __down_write(sem); | ||
| 53 | } | ||
| 54 | 41 | ||
| 55 | /* | 42 | /* |
| 56 | * trylock for writing -- returns 1 if successful, 0 if contention | 43 | * trylock for writing -- returns 1 if successful, 0 if contention |
| 57 | */ | 44 | */ |
| 58 | static inline int down_write_trylock(struct rw_semaphore *sem) | 45 | extern int down_write_trylock(struct rw_semaphore *sem); |
| 59 | { | ||
| 60 | int ret; | ||
| 61 | ret = __down_write_trylock(sem); | ||
| 62 | return ret; | ||
| 63 | } | ||
| 64 | 46 | ||
| 65 | /* | 47 | /* |
| 66 | * release a read lock | 48 | * release a read lock |
| 67 | */ | 49 | */ |
| 68 | static inline void up_read(struct rw_semaphore *sem) | 50 | extern void up_read(struct rw_semaphore *sem); |
| 69 | { | ||
| 70 | __up_read(sem); | ||
| 71 | } | ||
| 72 | 51 | ||
| 73 | /* | 52 | /* |
| 74 | * release a write lock | 53 | * release a write lock |
| 75 | */ | 54 | */ |
| 76 | static inline void up_write(struct rw_semaphore *sem) | 55 | extern void up_write(struct rw_semaphore *sem); |
| 77 | { | ||
| 78 | __up_write(sem); | ||
| 79 | } | ||
| 80 | 56 | ||
| 81 | /* | 57 | /* |
| 82 | * downgrade write lock to read lock | 58 | * downgrade write lock to read lock |
| 83 | */ | 59 | */ |
| 84 | static inline void downgrade_write(struct rw_semaphore *sem) | 60 | extern void downgrade_write(struct rw_semaphore *sem); |
| 85 | { | 61 | |
| 86 | __downgrade_write(sem); | 62 | #ifdef CONFIG_DEBUG_LOCK_ALLOC |
| 87 | } | 63 | /* |
| 64 | * nested locking: | ||
| 65 | */ | ||
| 66 | extern void down_read_nested(struct rw_semaphore *sem, int subclass); | ||
| 67 | extern void down_write_nested(struct rw_semaphore *sem, int subclass); | ||
| 68 | /* | ||
| 69 | * Take/release a lock when not the owner will release it: | ||
| 70 | */ | ||
| 71 | extern void down_read_non_owner(struct rw_semaphore *sem); | ||
| 72 | extern void up_read_non_owner(struct rw_semaphore *sem); | ||
| 73 | #else | ||
| 74 | # define down_read_nested(sem, subclass) down_read(sem) | ||
| 75 | # define down_write_nested(sem, subclass) down_write(sem) | ||
| 76 | # define down_read_non_owner(sem) down_read(sem) | ||
| 77 | # define up_read_non_owner(sem) up_read(sem) | ||
| 78 | #endif | ||
| 88 | 79 | ||
| 89 | #endif /* __KERNEL__ */ | 80 | #endif /* __KERNEL__ */ |
| 90 | #endif /* _LINUX_RWSEM_H */ | 81 | #endif /* _LINUX_RWSEM_H */ |
