* [PATCH nf] netfilter: cttimeout: fix deadlock due to erroneous unlock/lock conversion
@ 2016-01-28 12:16 Florian Westphal
2016-02-01 17:39 ` Pablo Neira Ayuso
0 siblings, 1 reply; 2+ messages in thread
From: Florian Westphal @ 2016-01-28 12:16 UTC (permalink / raw)
To: netfilter-devel; +Cc: sasha.levin, Florian Westphal
The spin_unlock call should have been left as-is, revert.
Fixes: b16c29191dc89bd ("netfilter: nf_conntrack: use safer way to lock all buckets")
Reported-by: kernel test robot <fengguang.wu@intel.com>
Signed-off-by: Florian Westphal <fw@strlen.de>
---
net/netfilter/nfnetlink_cttimeout.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/net/netfilter/nfnetlink_cttimeout.c b/net/netfilter/nfnetlink_cttimeout.c
index 94837d2..2671b9d 100644
--- a/net/netfilter/nfnetlink_cttimeout.c
+++ b/net/netfilter/nfnetlink_cttimeout.c
@@ -312,7 +312,7 @@ static void ctnl_untimeout(struct net *net, struct ctnl_timeout *timeout)
hlist_nulls_for_each_entry(h, nn, &net->ct.hash[i], hnnode)
untimeout(h, timeout);
}
- nf_conntrack_lock(&nf_conntrack_locks[i % CONNTRACK_LOCKS]);
+ spin_unlock(&nf_conntrack_locks[i % CONNTRACK_LOCKS]);
}
local_bh_enable();
}
--
2.4.10
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH nf] netfilter: cttimeout: fix deadlock due to erroneous unlock/lock conversion
2016-01-28 12:16 [PATCH nf] netfilter: cttimeout: fix deadlock due to erroneous unlock/lock conversion Florian Westphal
@ 2016-02-01 17:39 ` Pablo Neira Ayuso
0 siblings, 0 replies; 2+ messages in thread
From: Pablo Neira Ayuso @ 2016-02-01 17:39 UTC (permalink / raw)
To: Florian Westphal; +Cc: netfilter-devel, sasha.levin
On Thu, Jan 28, 2016 at 01:16:59PM +0100, Florian Westphal wrote:
> The spin_unlock call should have been left as-is, revert.
>
> Fixes: b16c29191dc89bd ("netfilter: nf_conntrack: use safer way to lock all buckets")
> Reported-by: kernel test robot <fengguang.wu@intel.com>
> Signed-off-by: Florian Westphal <fw@strlen.de>
Florian, thanks for resolving this problems. Applied, thanks.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2016-02-01 17:39 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-01-28 12:16 [PATCH nf] netfilter: cttimeout: fix deadlock due to erroneous unlock/lock conversion Florian Westphal
2016-02-01 17:39 ` Pablo Neira Ayuso
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).