1
1

Merge pull request #7895 from tkordenbrock/topic/v4.1.x/portals4.fix-inappropriate-use-of-abort

v4.1.x: portals4: fix inappropriate use of abort() in mtl-portals4 and coll-portals4 components
Этот коммит содержится в:
Jeff Squyres 2020-07-06 07:32:32 -04:00 коммит произвёл GitHub
родитель bc6587d3fa 20f9ed98f2
Коммит a878569386
Не найден ключ, соответствующий данной подписи
Идентификатор ключа GPG: 4AEE18F83AFDEB23
2 изменённых файлов: 6 добавлений и 6 удалений

Просмотреть файл

@ -754,7 +754,7 @@ portals4_progress(void)
} }
else if (PTL_EQ_DROPPED == ret) { else if (PTL_EQ_DROPPED == ret) {
opal_output(ompi_coll_base_framework.framework_output, "Flow control situation without recovery (EQ_DROPPED)\n"); opal_output(ompi_coll_base_framework.framework_output, "Flow control situation without recovery (EQ_DROPPED)\n");
abort(); ompi_rte_abort(ret, "coll-portals4: Flow control situation without recovery (EQ_DROPPED)");
} }
else { else {
opal_output(ompi_coll_base_framework.framework_output, "Error returned from PtlEQGet: %d", ret); opal_output(ompi_coll_base_framework.framework_output, "Error returned from PtlEQGet: %d", ret);

Просмотреть файл

@ -523,7 +523,7 @@ ompi_mtl_portals4_progress(void)
if (OMPI_SUCCESS != ret) { if (OMPI_SUCCESS != ret) {
opal_output(ompi_mtl_base_framework.framework_output, opal_output(ompi_mtl_base_framework.framework_output,
"Error returned from target event callback: %d", ret); "Error returned from target event callback: %d", ret);
abort(); ompi_rte_abort(ret, "mtl-portals4: Error returned from target event callback");
} }
} }
break; break;
@ -535,7 +535,7 @@ ompi_mtl_portals4_progress(void)
if (OMPI_SUCCESS != ret) { if (OMPI_SUCCESS != ret) {
opal_output(ompi_mtl_base_framework.framework_output, opal_output(ompi_mtl_base_framework.framework_output,
"Error returned from target event callback: %d", ret); "Error returned from target event callback: %d", ret);
abort(); ompi_rte_abort(ret, "mtl-portals4: Error returned from target event callback");
} }
} }
break; break;
@ -550,12 +550,12 @@ ompi_mtl_portals4_progress(void)
opal_output_verbose(1, ompi_mtl_base_framework.framework_output, opal_output_verbose(1, ompi_mtl_base_framework.framework_output,
"%s:%d: flowctl_trigger() failed: %d\n", "%s:%d: flowctl_trigger() failed: %d\n",
__FILE__, __LINE__, ret); __FILE__, __LINE__, ret);
abort(); ompi_rte_abort(ret, "mtl-portals4: Flow control failed (PT_DISABLED)");
} }
#else #else
opal_output(ompi_mtl_base_framework.framework_output, opal_output(ompi_mtl_base_framework.framework_output,
"Flow control situation without recovery (PT_DISABLED)"); "Flow control situation without recovery (PT_DISABLED)");
abort(); ompi_rte_abort(ret, "mtl-portals4: Flow control situation without recovery (PT_DISABLED)");
#endif #endif
break; break;
@ -573,7 +573,7 @@ ompi_mtl_portals4_progress(void)
opal_output(ompi_mtl_base_framework.framework_output, opal_output(ompi_mtl_base_framework.framework_output,
"Flow control situation without recovery (EQ_DROPPED): %d", "Flow control situation without recovery (EQ_DROPPED): %d",
which); which);
abort(); ompi_rte_abort(ret, "mtl-portals4: Flow control situation without recovery (EQ_DROPPED)");
} else { } else {
opal_output(ompi_mtl_base_framework.framework_output, opal_output(ompi_mtl_base_framework.framework_output,
"Error returned from PtlEQGet: %d", ret); "Error returned from PtlEQGet: %d", ret);