[Cluster-devel] cluster/rgmanager ChangeLog src/daemons/groups ...

[lhh at sourceware.org]

CVSROOT:	/cvs/cluster
Module name:	cluster
Changes by:	lhh at sourceware.org	2007-07-10 18:25:27

Modified files:
	rgmanager      : ChangeLog 
	rgmanager/src/daemons: groups.c rg_thread.c 

Log message:
	Resolves: 247488

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/cluster/rgmanager/ChangeLog.diff?cvsroot=cluster&r1=1.50&r2=1.51
http://sourceware.org/cgi-bin/cvsweb.cgi/cluster/rgmanager/src/daemons/groups.c.diff?cvsroot=cluster&r1=1.35&r2=1.36
http://sourceware.org/cgi-bin/cvsweb.cgi/cluster/rgmanager/src/daemons/rg_thread.c.diff?cvsroot=cluster&r1=1.22&r2=1.23

--- cluster/rgmanager/ChangeLog	2007/07/02 15:15:00	1.50
+++ cluster/rgmanager/ChangeLog	2007/07/10 18:25:26	1.51
@@ -1,3 +1,7 @@
+2007-07-10 Lon Hohberger <lhh at redhat.com>
+	* src/daemons/rg_thread.c, groups.c: Make status queue max 1 instead
+	of unbounded (resolves: 247488)
+
 2007-06-29 Lon Hohberger <lhh at redhat.com>
 	* src/daemons/groups.c, rg_state.c: Make all test cases for #237144
 	work correctly
--- cluster/rgmanager/src/daemons/groups.c	2007/07/02 15:15:00	1.35
+++ cluster/rgmanager/src/daemons/groups.c	2007/07/10 18:25:26	1.36
@@ -1191,7 +1191,7 @@
 			continue;
 
 		rt_enqueue_request(rg, RG_STATUS,
-				   NULL, 0, 0, 0, 0);
+				   NULL, 1, 0, 0, 0);
 
 	} while (!list_done(&_tree, curr));
 
--- cluster/rgmanager/src/daemons/rg_thread.c	2007/06/27 14:03:51	1.22
+++ cluster/rgmanager/src/daemons/rg_thread.c	2007/07/10 18:25:26	1.23
@@ -670,7 +670,6 @@
 			} while (!list_done(resgroup->rt_queue, curr));
 	
 			if (count >= max) {
-				pthread_cond_broadcast(resgroup->rt_queue_cond);
 				pthread_mutex_unlock(resgroup->rt_queue_mutex);
 				pthread_mutex_unlock(&reslist_mutex);
 				/*