Skip to content

Commit

Permalink
Fix binding to multiple pe's
Browse files Browse the repository at this point in the history
Accept a package if it has enough pe's to satisfy
the binding - do not require that it have _more_
than required.

Thanks to Jakub Benda for the report and the fix!

Signed-off-by: Ralph Castain <rhc@pmix.org>
  • Loading branch information
rhc54 committed Aug 29, 2023
1 parent d4791d9 commit d659897
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/mca/rmaps/base/rmaps_base_binding.c
Original file line number Diff line number Diff line change
Expand Up @@ -346,7 +346,7 @@ static int bind_multiple(prte_job_t *jdata, prte_proc_t *proc,
#endif
hwloc_bitmap_and(prte_rmaps_base.available, prte_rmaps_base.available, node->available);
ncpus = hwloc_get_nbobjs_inside_cpuset_by_type(node->topology->topo, prte_rmaps_base.available, type);
if (ncpus > options->cpus_per_rank) {
if (ncpus >= options->cpus_per_rank) {
/* this is a good spot */
moveon = true;
break;
Expand Down

0 comments on commit d659897

Please sign in to comment.