Re: [PATCH 50/51] net/core/flow.c: Fix CPU hotplug callback registration
From: David Miller <davem@davemloft.net>
Date: 2014-02-07 04:39:27
Also in:
lkml
From: David Miller <davem@davemloft.net>
Date: 2014-02-07 04:39:27
Also in:
lkml
From: "Srivatsa S. Bhat" <redacted> Date: Thu, 06 Feb 2014 03:43:46 +0530
Subsystems that want to register CPU hotplug callbacks, as well as perform initialization for the CPUs that are already online, often do it as shown below:
...
This is wrong, since it is prone to ABBA deadlocks involving the cpu_add_remove_lock and the cpu_hotplug.lock (when running concurrently with CPU hotplug operations). Instead, the correct and race-free way of performing the callback registration is:
...
Fix the code in net/core/flow.c by using this latter form of callback registration. Cc: "David S. Miller" <davem@davemloft.net> Cc: Li RongQing <redacted> Cc: Sasha Levin <redacted> Cc: Andrew Morton <akpm@linux-foundation.org> Cc: Chris Metcalf <redacted> Cc: netdev@vger.kernel.org Signed-off-by: Srivatsa S. Bhat <redacted>
Applied.