diff options
| author | Alexander Aring <aahringo@redhat.com> | 2023-05-29 17:44:37 -0400 |
|---|---|---|
| committer | David Teigland <teigland@redhat.com> | 2023-06-14 10:17:33 -0500 |
| commit | 07ee38674a0b9071fa0bbec4dbda6aad1c5e4003 (patch) | |
| tree | 629a028e1041dbb71ad088577cd72b38301d9427 /fs/dlm/midcomms.c | |
| parent | 70cf2fecf87354be09c0ab4c233fccea36256d3c (diff) | |
| download | linux-07ee38674a0b9071fa0bbec4dbda6aad1c5e4003.tar.gz linux-07ee38674a0b9071fa0bbec4dbda6aad1c5e4003.tar.bz2 linux-07ee38674a0b9071fa0bbec4dbda6aad1c5e4003.zip | |
fs: dlm: filter ourself midcomms calls
It makes no sense to call midcomms/lowcomms functionality for the local
node as socket functionality is only required for remote nodes. This
patch filters those calls in the upper layer of lockspace membership
handling instead of doing it in midcomms/lowcomms layer as they should
never be aware of local nodeid.
Signed-off-by: Alexander Aring <aahringo@redhat.com>
Signed-off-by: David Teigland <teigland@redhat.com>
Diffstat (limited to 'fs/dlm/midcomms.c')
| -rw-r--r-- | fs/dlm/midcomms.c | 9 |
1 files changed, 0 insertions, 9 deletions
diff --git a/fs/dlm/midcomms.c b/fs/dlm/midcomms.c index 3df916a568ba..9c66cb853d17 100644 --- a/fs/dlm/midcomms.c +++ b/fs/dlm/midcomms.c @@ -1280,9 +1280,6 @@ void dlm_midcomms_add_member(int nodeid) struct midcomms_node *node; int idx; - if (nodeid == dlm_our_nodeid()) - return; - idx = srcu_read_lock(&nodes_srcu); node = nodeid2node(nodeid, GFP_NOFS); if (!node) { @@ -1328,9 +1325,6 @@ void dlm_midcomms_remove_member(int nodeid) struct midcomms_node *node; int idx; - if (nodeid == dlm_our_nodeid()) - return; - idx = srcu_read_lock(&nodes_srcu); node = nodeid2node(nodeid, 0); if (!node) { @@ -1487,9 +1481,6 @@ int dlm_midcomms_close(int nodeid) struct midcomms_node *node; int idx, ret; - if (nodeid == dlm_our_nodeid()) - return 0; - idx = srcu_read_lock(&nodes_srcu); /* Abort pending close/remove operation */ node = nodeid2node(nodeid, 0); |