Remove unnecessary qsort operation

We do not need to sort the procs passed to PMIx_Connect as that function will work with arbitrary ordering of the proc array between participants. Signed-off-by: Ralph Castain <rhc@pmix.org>

Author: rhc54

ompi/dpm/dpm.c

@@ -100,18 +100,6 @@ int ompi_dpm_init(void)
  return OMPI_SUCCESS;
 }
 
-static int compare_pmix_proc(const void *a, const void *b)
-{
- const pmix_proc_t *proc_a = (pmix_proc_t *)a;
- const pmix_proc_t *proc_b = (pmix_proc_t *)b;
-
- int nspace_dif = strncmp(proc_a->nspace, proc_b->nspace, PMIX_MAX_NSLEN);
- if (nspace_dif != 0)
- return nspace_dif;
-
- return proc_a->rank - proc_b->rank;
-}
-
 int ompi_dpm_connect_accept(ompi_communicator_t *comm, int root,
  const char *port_string, bool send_first,
  ompi_communicator_t **newcomm)
@@ -395,10 +383,6 @@ int ompi_dpm_connect_accept(ompi_communicator_t *comm, int root,
  PMIX_INFO_CONSTRUCT(&tinfo);
  PMIX_INFO_LOAD(&tinfo, PMIX_TIMEOUT, &ompi_pmix_connect_timeout, PMIX_UINT32);
 
- /*
- * sort procs so that all ranks call PMIx_Connect() with the processes in same order
- */
- qsort(procs, nprocs, sizeof(pmix_proc_t), compare_pmix_proc);
  pret = PMIx_Connect(procs, nprocs, &tinfo, 1);
  PMIX_INFO_DESTRUCT(&tinfo);
  PMIX_PROC_FREE(procs, nprocs);