linux-brain/net/ncsi
Milton Miller bbbee59f4f net/ncsi: Avoid channel_monitor hrtimer deadlock
[ Upstream commit 03cb4d05b4ea9a3491674ca40952adb708d549fa ]

Calling ncsi_stop_channel_monitor from channel_monitor is a guaranteed
deadlock on SMP because stop calls del_timer_sync on the timer that
invoked channel_monitor as its timer function.

Recognise the inherent race of marking the monitor disabled before
deleting the timer by just returning if enable was cleared.  After
a timeout (the default case -- reset to START when response received)
just mark the monitor.enabled false.

If the channel has an entry on the channel_queue list, or if the
state is not ACTIVE or INACTIVE, then warn and mark the timer stopped
and don't restart, as the locking is broken somehow.

Fixes: 0795fb2021 ("net/ncsi: Stop monitor if channel times out or is inactive")
Signed-off-by: Milton Miller <miltonm@us.ibm.com>
Signed-off-by: Eddie James <eajames@linux.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Sasha Levin <sashal@kernel.org>
2021-04-14 08:24:15 +02:00
..
Kconfig treewide: Add SPDX license identifier - Makefile/Kconfig 2019-05-21 10:50:46 +02:00
Makefile treewide: Add SPDX license identifier - Makefile/Kconfig 2019-05-21 10:50:46 +02:00
internal.h net/ncsi: Disable global multicast filter 2019-09-19 18:04:40 -07:00
ncsi-aen.c treewide: Replace GPLv2 boilerplate/reference with SPDX - rule 152 2019-05-30 11:26:32 -07:00
ncsi-cmd.c net/ncsi: Fix the payload copying for the request coming from Netlink 2019-08-22 19:27:02 -07:00
ncsi-manage.c net/ncsi: Avoid channel_monitor hrtimer deadlock 2021-04-14 08:24:15 +02:00
ncsi-netlink.c net/ncsi: Fix netlink registration 2020-11-24 13:29:00 +01:00
ncsi-netlink.h net/ncsi: Fix netlink registration 2020-11-24 13:29:00 +01:00
ncsi-pkt.h net/ncsi: add response handlers for PLDM over NC-SI 2019-08-31 23:54:03 -07:00
ncsi-rsp.c net/ncsi: Use real net-device for response handler 2021-01-12 20:16:13 +01:00