btl/ugni: always check local SMSG CQ

This commit removes the local operation count check from the local SMSG
completion queue. This check was leading to hangs due to an undocumented
feature of the ugni library. The local SMSG CQ is used to send credit
return messages back to the sender. The ugni library never checks for
the completion itself but relying on the SMSG user to periodically
check the CQ.

Signed-off-by: Nathan Hjelm <[email protected]>

Author: hjelmn

opal/mca/btl/ugni/btl_ugni_smsg.h

@@ -42,11 +42,6 @@ static inline int mca_btl_ugni_progress_local_smsg (mca_btl_ugni_module_t *ugni_
     gni_cq_entry_t event_data;
     gni_return_t grc;
 
-    /* nothing to do */
-    if (0 == device->dev_smsg_local_cq.active_operations) {
-        return OPAL_SUCCESS;
-    }
-
     grc = mca_btl_ugni_cq_get_event (device, &device->dev_smsg_local_cq, &event_data);
     if (GNI_RC_NOT_DONE == grc) {
         return OPAL_SUCCESS;