Message ID | 20160824112847.GP3735@gauss.secunet.com |
---|---|
State | Awaiting Upstream, archived |
Delegated to: | David Miller |
Headers | show |
Steffen Klassert <steffen.klassert@secunet.com> wrote: > On Thu, Aug 11, 2016 at 03:17:59PM +0200, Florian Westphal wrote: > > @@ -972,14 +972,14 @@ int xfrm_policy_flush(struct net *net, u8 type, bool task_valid) > > if (pol->type != type) > > continue; > > __xfrm_policy_unlink(pol, dir); > > - write_unlock_bh(&net->xfrm.xfrm_policy_lock); > > + spin_unlock_bh(&net->xfrm.xfrm_policy_lock); > > cnt++; > > > > xfrm_audit_policy_delete(pol, 1, task_valid); > > > > xfrm_policy_kill(pol); > > > > - write_lock_bh(&net->xfrm.xfrm_policy_lock); > > + spin_unlock_bh(&net->xfrm.xfrm_policy_lock); > > I've just noticed that you accidentally replaced write_lock_bh > with spin_unlock_bh here. Sorry about this, thanks for fixing this up.
diff --git a/net/xfrm/xfrm_policy.c b/net/xfrm/xfrm_policy.c index dd01fd2..f7ce626 100644 --- a/net/xfrm/xfrm_policy.c +++ b/net/xfrm/xfrm_policy.c @@ -979,7 +979,7 @@ int xfrm_policy_flush(struct net *net, u8 type, bool task_valid) xfrm_policy_kill(pol); - spin_unlock_bh(&net->xfrm.xfrm_policy_lock); + spin_lock_bh(&net->xfrm.xfrm_policy_lock); goto again1; }