pml/ob1: Add checks arronud accelerator init/fini

There was previously no validation that accelerator
initialization succeeded and the finalize was entered
regardless.

Signed-off-by: William Zhang <wilzhang@amazon.com>

Author: wckzhang

ompi/mca/pml/ob1/pml_ob1.c

@@ -197,7 +197,7 @@ int mca_pml_ob1_enable(bool enable)
                           mca_pml_ob1.free_list_inc,
                           NULL, 0, NULL, NULL, NULL);
 
-    mca_pml_ob1_accelerator_init();
+    mca_pml_ob1.accelerator_enabled = (0 == mca_pml_ob1_accelerator_init()) ? true : false;
 
     mca_pml_ob1.enabled = true;
 

ompi/mca/pml/ob1/pml_ob1.h

@@ -86,6 +86,8 @@ struct mca_pml_ob1_t {
     char* allocator_name;
     mca_allocator_base_module_t* allocator;
     unsigned int unexpected_limit;
+    /* Accelerator support initialized */
+    bool accelerator_enabled;
 };
 typedef struct mca_pml_ob1_t mca_pml_ob1_t;
 

ompi/mca/pml/ob1/pml_ob1_component.c

@@ -363,7 +363,9 @@ int mca_pml_ob1_component_fini(void)
     OBJ_DESTRUCT(&mca_pml_ob1.lock);
     OBJ_DESTRUCT(&mca_pml_ob1.send_ranges);
 
-    mca_pml_ob1_accelerator_fini();
+    if (mca_pml_ob1.accelerator_enabled) {
+        mca_pml_ob1_accelerator_fini();
+    }
 
     if( NULL != mca_pml_ob1.allocator ) {
         (void)mca_pml_ob1.allocator->alc_finalize(mca_pml_ob1.allocator);