diff options
author | David S. Miller <davem@sunset.davemloft.net> | 2007-05-31 16:24:26 -0400 |
---|---|---|
committer | David S. Miller <davem@sunset.davemloft.net> | 2007-06-03 21:08:40 -0400 |
commit | 1c92b4e50ef926d1e26fcc056a520e4a7d12478c (patch) | |
tree | 5657e611aef30e1f994d018e23a3a8cffa38677e /include/net | |
parent | c1a13ff57ab1ce52a0aae9984594dbfcfbaf68c0 (diff) |
[AF_UNIX]: Make socket locking much less confusing.
The unix_state_*() locking macros imply that there is some
rwlock kind of thing going on, but the implementation is
actually a spinlock which makes the code more confusing than
it needs to be.
So use plain unix_state_lock and unix_state_unlock.
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/net')
-rw-r--r-- | include/net/af_unix.h | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/include/net/af_unix.h b/include/net/af_unix.h index c0398f5a8cb9..65f49fd7deff 100644 --- a/include/net/af_unix.h +++ b/include/net/af_unix.h | |||
@@ -62,13 +62,11 @@ struct unix_skb_parms { | |||
62 | #define UNIXCREDS(skb) (&UNIXCB((skb)).creds) | 62 | #define UNIXCREDS(skb) (&UNIXCB((skb)).creds) |
63 | #define UNIXSID(skb) (&UNIXCB((skb)).secid) | 63 | #define UNIXSID(skb) (&UNIXCB((skb)).secid) |
64 | 64 | ||
65 | #define unix_state_rlock(s) spin_lock(&unix_sk(s)->lock) | 65 | #define unix_state_lock(s) spin_lock(&unix_sk(s)->lock) |
66 | #define unix_state_runlock(s) spin_unlock(&unix_sk(s)->lock) | 66 | #define unix_state_unlock(s) spin_unlock(&unix_sk(s)->lock) |
67 | #define unix_state_wlock(s) spin_lock(&unix_sk(s)->lock) | 67 | #define unix_state_lock_nested(s) \ |
68 | #define unix_state_wlock_nested(s) \ | ||
69 | spin_lock_nested(&unix_sk(s)->lock, \ | 68 | spin_lock_nested(&unix_sk(s)->lock, \ |
70 | SINGLE_DEPTH_NESTING) | 69 | SINGLE_DEPTH_NESTING) |
71 | #define unix_state_wunlock(s) spin_unlock(&unix_sk(s)->lock) | ||
72 | 70 | ||
73 | #ifdef __KERNEL__ | 71 | #ifdef __KERNEL__ |
74 | /* The AF_UNIX socket */ | 72 | /* The AF_UNIX socket */ |