diff options
| author | Eric Dumazet <eric.dumazet@gmail.com> | 2010-01-28 23:05:52 -0500 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2010-02-02 18:55:41 -0500 |
| commit | 28aecb9d7728dc26bf03ce7925fe622023a83a2a (patch) | |
| tree | b5adad3e5ba77bb00e13a81eea5823bc0d76fb82 /net/key | |
| parent | d1c9ae6d1e7b95cedc8e39e8949e795379a0669e (diff) | |
xfrm: avoid spinlock in get_acqseq()
Use atomic_inc_return() in get_acqseq() to avoid taking a spinlock
Signed-off-by: Eric Dumazet <eric.dumazet@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/key')
| -rw-r--r-- | net/key/af_key.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/net/key/af_key.c b/net/key/af_key.c index 4744b1f6372..e2aacf0ba01 100644 --- a/net/key/af_key.c +++ b/net/key/af_key.c | |||
| @@ -3019,12 +3019,11 @@ static int pfkey_send_policy_notify(struct xfrm_policy *xp, int dir, struct km_e | |||
| 3019 | static u32 get_acqseq(void) | 3019 | static u32 get_acqseq(void) |
| 3020 | { | 3020 | { |
| 3021 | u32 res; | 3021 | u32 res; |
| 3022 | static u32 acqseq; | 3022 | static atomic_t acqseq; |
| 3023 | static DEFINE_SPINLOCK(acqseq_lock); | ||
| 3024 | 3023 | ||
| 3025 | spin_lock_bh(&acqseq_lock); | 3024 | do { |
| 3026 | res = (++acqseq ? : ++acqseq); | 3025 | res = atomic_inc_return(&acqseq); |
| 3027 | spin_unlock_bh(&acqseq_lock); | 3026 | } while (!res); |
| 3028 | return res; | 3027 | return res; |
| 3029 | } | 3028 | } |
| 3030 | 3029 | ||
