diff options
| author | Florian Westphal <fw@strlen.de> | 2016-08-11 15:17:56 +0200 |
|---|---|---|
| committer | Steffen Klassert <steffen.klassert@secunet.com> | 2016-08-12 08:07:12 +0200 |
| commit | a7c44247f704e385c77579d65c6ee6d002832529 (patch) | |
| tree | 8937a83226c46ffe7ace1a3817d76b943b1f3c6b /net/xfrm | |
| parent | e37cc8ade5afaf082f804c6d18eb23377146bec4 (diff) | |
| download | linux-a7c44247f704e385c77579d65c6ee6d002832529.tar.gz linux-a7c44247f704e385c77579d65c6ee6d002832529.tar.bz2 linux-a7c44247f704e385c77579d65c6ee6d002832529.zip | |
xfrm: policy: make xfrm_policy_lookup_bytype lockless
side effect: no longer disables BH (should be fine).
Signed-off-by: Florian Westphal <fw@strlen.de>
Signed-off-by: Steffen Klassert <steffen.klassert@secunet.com>
Diffstat (limited to 'net/xfrm')
| -rw-r--r-- | net/xfrm/xfrm_policy.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/xfrm/xfrm_policy.c b/net/xfrm/xfrm_policy.c index 09f2e2b38246..9302647f20a0 100644 --- a/net/xfrm/xfrm_policy.c +++ b/net/xfrm/xfrm_policy.c @@ -1123,7 +1123,7 @@ static struct xfrm_policy *xfrm_policy_lookup_bytype(struct net *net, u8 type, if (unlikely(!daddr || !saddr)) return NULL; - read_lock_bh(&net->xfrm.xfrm_policy_lock); + rcu_read_lock(); retry: do { sequence = read_seqcount_begin(&xfrm_policy_hash_generation); @@ -1172,7 +1172,7 @@ static struct xfrm_policy *xfrm_policy_lookup_bytype(struct net *net, u8 type, if (ret && !xfrm_pol_hold_rcu(ret)) goto retry; fail: - read_unlock_bh(&net->xfrm.xfrm_policy_lock); + rcu_read_unlock(); return ret; } |