Merge pull request pmodels#7069 from hzhou/2407_hydra_bind

hydra: check return code from hwloc_set_cpubind Approved-by: Ken Raffenetti
raffenet · Aug 7, 2024 · 38385fb · 38385fb
2 parents dc5e618 + b3f77b4
commit 38385fb
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/src/pm/hydra/lib/tools/topo/hwloc/topo_hwloc.c b/src/pm/hydra/lib/tools/topo/hwloc/topo_hwloc.c
@@ -693,9 +693,14 @@ HYD_status HYDT_topo_hwloc_bind(int idx)
             HYDU_dump_noprefix(stdout, "process %d binding: %s\n", idx, binding);
             MPL_free(binding);
         }
-        hwloc_set_cpubind(topology, HYDT_topo_hwloc_info.bitmap[id], 0);
-        hwloc_set_membind(topology, HYDT_topo_hwloc_info.bitmap[id],
-                          HYDT_topo_hwloc_info.membind, 0);
+        int rc;
+        rc = hwloc_set_cpubind(topology, HYDT_topo_hwloc_info.bitmap[id], 0);
+        HYDU_ERR_CHKANDJUMP(status, rc, HYD_INTERNAL_ERROR,
+                            "hwloc_set_cpubind failed, rc = %d\n", rc);
+        rc = hwloc_set_membind(topology, HYDT_topo_hwloc_info.bitmap[id],
+                               HYDT_topo_hwloc_info.membind, 0);
+        HYDU_ERR_CHKANDJUMP(status, rc, HYD_INTERNAL_ERROR,
+                            "hwloc_set_membind failed, rc = %d\n", rc);
     }