* rename the collectives sm bootstrap area to be consistent with other

shared memory segments
* make sure to properly unlink the collectives sm bootstrap area at
  shutdown
* Add missing / in the path for the mpool shared memory segment
* make sure to release the common_mmap structure in the SM btl
  after unlinking the file during shutdown

This commit was SVN r10886.

ompi/mca/btl/sm/btl_sm_component.c

@@ -203,6 +203,7 @@ int mca_btl_sm_component_close(void)
          * to call this, so that in an abnormal termination scenario,
          * this file will still get cleaned up */
         unlink(mca_btl_sm_component.mmap_file->map_path);
+        OBJ_RELEASE(mca_btl_sm_component.mmap_file);
     }
 
 #if OMPI_ENABLE_PROGRESS_THREADS == 1

ompi/mca/coll/sm/coll_sm_component.c

@@ -103,7 +103,7 @@ mca_coll_sm_component_t mca_coll_sm_component = {
     4096,
 
     /* (default) bootstrap filename */
-    "coll-sm-bootstrap",
+    "shared_mem_sm_bootstrap",
 
     /* (default) number of segments in bootstrap file */
     8,

ompi/mca/coll/sm/coll_sm_module.c

@@ -785,7 +785,10 @@ int mca_coll_sm_bootstrap_finalize(void)
            anything in here -- all data structures are referencing
            within the bootstrap area, so the one top-level unmap does
            it all) */
-        mca_common_sm_mmap_fini( meta );
+        if (OMPI_SUCCESS == mca_common_sm_mmap_fini( meta )) {
+            unlink(meta->map_path);
+        }
+        OBJ_RELEASE(meta);
     }
 
     return OMPI_SUCCESS;

ompi/mca/mpool/sm/mpool_sm_component.c

@@ -142,7 +142,7 @@ static mca_mpool_base_module_t* mca_mpool_sm_init(
     mca_mpool_sm_module_init(mpool_module); 
 
     /* create initial shared memory mapping */
-    len = asprintf( &file_name, "%sshared_mem_pool.%s",
+    len = asprintf( &file_name, "%s/shared_mem_pool.%s",
                     orte_process_info.job_session_dir,
                     orte_system_info.nodename );
     if ( 0 > len ) {