]> git.baikalelectronics.ru Git - kernel.git/commit
fs: dlm: revert check required context while close
authorAlexander Aring <aahringo@redhat.com>
Mon, 29 May 2023 21:44:29 +0000 (17:44 -0400)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 23 Jul 2023 11:49:34 +0000 (13:49 +0200)
commit5dc507de0c8d89f79d84fbe4b638eb4e4cb089ca
tree74965388e50b6e78fb8420f2626f2452f5285ac7
parent95d49f79e94d4fa8105c880a266789609f3e791a
fs: dlm: revert check required context while close

commit c6b6d6dcc7f32767d57740e0552337c8de40610b upstream.

This patch reverts commit 2c3fa6ae4d52 ("dlm: check required context
while close"). The function dlm_midcomms_close(), which will call later
dlm_lowcomms_close(), is called when the cluster manager tells the node
got fenced which means on midcomms/lowcomms layer to disconnect the node
from the cluster communication. The node can rejoin the cluster later.
This patch was ensuring no new message were able to be triggered when we
are in the close() function context. This was done by checking if the
lockspace has been stopped. However there is a missing check that we
only need to check specific lockspaces where the fenced node is member
of. This is currently complicated because there is no way to easily
check if a node is part of a specific lockspace without stopping the
recovery. For now we just revert this commit as it is just a check to
finding possible leaks of stopping lockspaces before close() is called.

Cc: stable@vger.kernel.org
Fixes: 2c3fa6ae4d52 ("dlm: check required context while close")
Signed-off-by: Alexander Aring <aahringo@redhat.com>
Signed-off-by: David Teigland <teigland@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
fs/dlm/lockspace.c
fs/dlm/lockspace.h
fs/dlm/midcomms.c