UCP/DEVICE: Skip error logging for UCS_ERR_NOT_CONNECTED (#10978)

Author: michal-shalev

src/ucp/api/device/ucp_host.h

@@ -149,6 +149,8 @@ typedef struct ucp_device_mem_list_params {
  * @param [out] handle    Created descriptor list handle.
  *
  * @return Error code as defined by @ref ucs_status_t.
+ * @retval UCS_ERR_NOT_CONNECTED if the endpoint is not connected yet.
+ *         The caller should retry after calling @ref ucp_worker_progress.
  */
 ucs_status_t
 ucp_device_mem_list_create(ucp_ep_h ep,

src/ucp/core/ucp_device.c

@@ -419,6 +419,11 @@ ucp_device_mem_list_create(ucp_ep_h ep,
     uct_allocated_memory_t mem;
 
     if (!(ep->flags & UCP_EP_FLAG_REMOTE_CONNECTED)) {
+        /*
+         * Do not log error here because UCS_ERR_NOT_CONNECTED is expected
+         * during connection establishment. Applications are expected to retry
+         * with progress.
+         */
         return UCS_ERR_NOT_CONNECTED;
     }
 
@@ -457,7 +462,14 @@ ucp_device_mem_list_create(ucp_ep_h ep,
     status = ucp_device_mem_list_create_handle(ep, local_sys_dev, params, lanes,
                                                ep_config, mem_type, &mem);
     if (status != UCS_OK) {
-        ucs_error("failed to create handle: %s", ucs_status_string(status));
+        /*
+         * Do not log error for UCS_ERR_NOT_CONNECTED because it is expected
+         * during connection establishment. Applications are expected to retry
+         * with progress.
+         */
+        if (status != UCS_ERR_NOT_CONNECTED) {
+            ucs_error("failed to create handle: %s", ucs_status_string(status));
+        }
         return status;
     }