From 88448cf40a048886d95799398ca8c73b05242401 Mon Sep 17 00:00:00 2001 From: Brian Barrett Date: Sun, 26 Feb 2006 00:05:25 +0000 Subject: [PATCH] * fix setting of remote proc architecture integers. We were setting all procs architecture ints to the architecture of the last proc we got in the list from the gpr, which is probably not what we intended to do... This commit was SVN r9144. --- ompi/proc/proc.c | 15 +++++++-------- 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/ompi/proc/proc.c b/ompi/proc/proc.c index 9644785aa7..d46aefc960 100644 --- a/ompi/proc/proc.c +++ b/ompi/proc/proc.c @@ -519,16 +519,15 @@ static void callback(orte_gpr_notify_data_t *data, void *cbdata) proc != (ompi_proc_t*)opal_list_get_end(&ompi_proc_list); proc = (ompi_proc_t*)opal_list_get_next(proc)) { - /* if the nodename of this info is my local host, - * find the associated proc entry and set the local - * flag - */ - if (0 == strcmp(str, orte_system_info.nodename) && - 0 == orte_ns.compare(mask, &name, &proc->proc_name)) { + /* find the associated proc entry and update its + arch flag. If the nodename of this info is + my local host, also set the LOCAL flag. */ + if (0 == orte_ns.compare(mask, &name, &proc->proc_name)) { + proc->proc_arch = arch; + if (0 == strcmp(str, orte_system_info.nodename)) { proc->proc_flags |= OMPI_PROC_FLAG_LOCAL; + } } - /* set the architecture entry for this proc */ - proc->proc_arch = arch; } } }