<html><head><meta name="color-scheme" content="light dark"></head><body><pre style="word-wrap: break-word; white-space: pre-wrap;">
From: David Teigland &lt;teigland@redhat.com&gt;

Fix potential race in lowcomms.

Signed-off-by: Patrick Caulfield &lt;pcaulfie@redhat.com&gt;
Signed-off-by: David Teigland &lt;teigland@redhat.com&gt;
Signed-off-by: Andrew Morton &lt;akpm@osdl.org&gt;
---

 drivers/dlm/lowcomms.c |    6 +-----
 1 files changed, 1 insertion(+), 5 deletions(-)

diff -puN drivers/dlm/lowcomms.c~dlm-communication-fix-lowcomms-race drivers/dlm/lowcomms.c
--- devel/drivers/dlm/lowcomms.c~dlm-communication-fix-lowcomms-race	2005-07-16 13:35:43.000000000 -0700
+++ devel-akpm/drivers/dlm/lowcomms.c	2005-07-16 13:35:43.000000000 -0700
@@ -1101,8 +1101,8 @@ static void process_output_queue(void)
 	list_for_each_safe(list, temp, &amp;write_nodes) {
 		struct nodeinfo *ni =
 		    list_entry(list, struct nodeinfo, write_list);
-		list_del(&amp;ni-&gt;write_list);
 		clear_bit(NI_WRITE_PENDING, &amp;ni-&gt;flags);
+		list_del(&amp;ni-&gt;write_list);
 
 		spin_unlock_bh(&amp;write_nodes_lock);
 
@@ -1271,11 +1271,7 @@ static int daemons_start(void)
 
 /*
  * This is quite likely to sleep...
- * Temporarily initialise the waitq head so that lowcomms_send_message
- * doesn't crash if it gets called before the thread is fully
- * initialised
  */
-
 int dlm_lowcomms_start(void)
 {
 	int error;
_
</pre></body></html>