[Pvfs2-cvs] commit by slang in pvfs2/src/io/bmi: bmi.c

CVS commit program cvs at parl.clemson.edu
Fri Jun 1 16:18:55 EDT 2007


Update of /projects/cvsroot/pvfs2/src/io/bmi
In directory parlweb1:/tmp/cvs-serv31707/src/io/bmi

Modified Files:
	bmi.c 
Log Message:
fix id_gen_safe static allocation leakages.  Note that the job interface (to return op ids through the system interface) is really the only user of the id_gen_safe code.  The bmi layers call it as well, but only from _test and _testsome, which aren't ever used at presnet in PVFS.

Also increase the number of buckets in the id_gen_safe hashable.  67 gives a loading factor of about 50...we're probably going to want to allow more operations in-progress than that in todays world.



Index: bmi.c
===================================================================
RCS file: /projects/cvsroot/pvfs2/src/io/bmi/bmi.c,v
diff -p -u -r1.81 -r1.82
--- bmi.c	1 May 2007 16:30:58 -0000	1.81
+++ bmi.c	1 Jun 2007 20:18:55 -0000	1.82
@@ -124,6 +124,8 @@ int BMI_initialize(const char *method_li
     char *proto = NULL;
     int addr_count;
 
+    id_gen_safe_initialize();
+
     /* server must specify method list at startup, optional for client */
     if (flags & BMI_INIT_SERVER) {
 	if (!listen_addr || !method_list)
@@ -466,6 +468,8 @@ int BMI_finalize(void)
     /* destroy the reference list */
     /* (side effect: destroys all method addresses as well) */
     ref_list_cleanup(cur_ref_list);
+
+    id_gen_safe_finalize();
 
     return (0);
 }



More information about the Pvfs2-cvs mailing list