dlm: be sure we reset all nodes at forced shutdown

Alexander Aring · teigland · commit e759eb3e27e5 · 2023-10-12T15:20:48.000-05:00
In case we running in a force shutdown in either midcomms or lowcomms implementation we will make sure we reset all per midcomms node information. Fixes: 63e711b ("fs: dlm: create midcomms nodes when configure") Signed-off-by: Alexander Aring <aahringo@redhat.com> Signed-off-by: David Teigland <teigland@redhat.com>
diff --git a/fs/dlm/midcomms.c b/fs/dlm/midcomms.c
@@ -1405,10 +1405,16 @@ void dlm_midcomms_shutdown(void)
 			midcomms_shutdown(node);
 		}
 	}
-	srcu_read_unlock(&nodes_srcu, idx);
-	mutex_unlock(&close_lock);
 
 	dlm_lowcomms_shutdown();
+
+	for (i = 0; i < CONN_HASH_SIZE; i++) {
+		hlist_for_each_entry_rcu(node, &node_hash[i], hlist) {
+			midcomms_node_reset(node);
+		}
+	}
+	srcu_read_unlock(&nodes_srcu, idx);
+	mutex_unlock(&close_lock);
 }
 
 int dlm_midcomms_close(int nodeid)

Original file line number	Diff line number	Diff line change
`@@ -1405,10 +1405,16 @@ void dlm_midcomms_shutdown(void)`
`1405`	`1405`	`midcomms_shutdown(node);`
`1406`	`1406`	`}`
`1407`	`1407`	`}`
`1408`		`- srcu_read_unlock(&nodes_srcu, idx);`
`1409`		`- mutex_unlock(&close_lock);`
`1410`	`1408`
`1411`	`1409`	`dlm_lowcomms_shutdown();`
	`1410`	`+`
	`1411`	`+ for (i = 0; i < CONN_HASH_SIZE; i++) {`
	`1412`	`+ hlist_for_each_entry_rcu(node, &node_hash[i], hlist) {`
	`1413`	`+ midcomms_node_reset(node);`
	`1414`	`+ }`
	`1415`	`+ }`
	`1416`	`+ srcu_read_unlock(&nodes_srcu, idx);`
	`1417`	`+ mutex_unlock(&close_lock);`
`1412`	`1418`	`}`
`1413`	`1419`
`1414`	`1420`	`int dlm_midcomms_close(int nodeid)`