diff options
author | Alexey Dobriyan <adobriyan@gmail.com> | 2008-11-04 03:11:29 (GMT) |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-11-04 03:11:29 (GMT) |
commit | bbb770e7ab9a436752babfc8765e422d7481be1f (patch) | |
tree | ed0531a408dd7889912b9c8d2db71c973f021529 /net/xfrm/xfrm_policy.c | |
parent | 19ecb6ba800765743bb4525c66562f0d30993f8d (diff) | |
download | linux-fsl-qoriq-bbb770e7ab9a436752babfc8765e422d7481be1f.tar.xz |
xfrm: Fix xfrm_policy_gc_lock handling.
From: Alexey Dobriyan <adobriyan@gmail.com>
Based upon a lockdep trace by Simon Arlott.
xfrm_policy_kill() can be called from both BH and
non-BH contexts, so we have to grab xfrm_policy_gc_lock
with BH disabling.
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/xfrm/xfrm_policy.c')
-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 2587274..058f04f 100644 --- a/net/xfrm/xfrm_policy.c +++ b/net/xfrm/xfrm_policy.c @@ -315,9 +315,9 @@ static void xfrm_policy_kill(struct xfrm_policy *policy) return; } - spin_lock(&xfrm_policy_gc_lock); + spin_lock_bh(&xfrm_policy_gc_lock); hlist_add_head(&policy->bydst, &xfrm_policy_gc_list); - spin_unlock(&xfrm_policy_gc_lock); + spin_unlock_bh(&xfrm_policy_gc_lock); schedule_work(&xfrm_policy_gc_work); } |