[Pvfs2-cvs] commit by slang in pvfs2/src/client/sysint: finalize.c initialize.c

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


Update of /projects/cvsroot/pvfs2/src/client/sysint
In directory parlweb1:/tmp/cvs-serv31707/src/client/sysint

Modified Files:
	finalize.c initialize.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: finalize.c
===================================================================
RCS file: /projects/cvsroot/pvfs2/src/client/sysint/finalize.c,v
diff -p -u -r1.40 -r1.41
--- finalize.c	13 Apr 2007 05:14:16 -0000	1.40
+++ finalize.c	1 Jun 2007 20:18:54 -0000	1.41
@@ -35,6 +35,8 @@ extern PINT_smcb *g_smcb;
  */
 int PVFS_sys_finalize()
 {
+    id_gen_safe_finalize();
+
     PINT_util_digest_finalize();
     PINT_ncache_finalize();
     PINT_acache_finalize();

Index: initialize.c
===================================================================
RCS file: /projects/cvsroot/pvfs2/src/client/sysint/initialize.c,v
diff -p -u -r1.105 -r1.106
--- initialize.c	13 Apr 2007 05:14:16 -0000	1.105
+++ initialize.c	1 Jun 2007 20:18:54 -0000	1.106
@@ -212,9 +212,18 @@ int PVFS_sys_initialize(uint64_t default
 
     PINT_util_digest_init();
 
+    ret = id_gen_safe_initialize();
+    if(ret < 0)
+    {
+        gossip_lerr("Error initializing id_gen_safe\n");
+        goto error_exit;
+    }
+
     return 0;
 
   error_exit:
+
+    id_gen_safe_finalize();
 
     if (client_status_flag & CLIENT_CONFIG_MGR_INIT)
     {



More information about the Pvfs2-cvs mailing list