I'm sorry, but this change is incorrect.
If you look in orte/mca/ess/base/ess_base_std_orted.c, you will see that
-all- orteds, regardless of how they are launched, open and select the PLM.

This change causes rsh launched daemons to
doubly-open/select the PLM, which is a very bad idea.

Would you please revert this change?

Thanks
Ralph


On Fri, Jun 19, 2009 at 12:50 PM, <c...@osl.iu.edu> wrote:

> Author: coti
> Date: 2009-06-19 14:50:06 EDT (Fri, 19 Jun 2009)
> New Revision: 21480
> URL: https://svn.open-mpi.org/trac/ompi/changeset/21480
>
> Log:
> Initialize the PML if we are a non-HNP daemon.
>
> If we do not initialize the PML, non-HNP daemons will not be able to use
> its functions. For example, RSH needs it when the tree_spawn mode is
> enabled: daemons call orte_pml.remote_spawn() function to spawn their
> children in the deployment tree.
>
>
> Text files modified:
>   trunk/orte/mca/ess/env/ess_env_module.c |    14 ++++++++++++++
>   1 files changed, 14 insertions(+), 0 deletions(-)
>
> Modified: trunk/orte/mca/ess/env/ess_env_module.c
>
> ==============================================================================
> --- trunk/orte/mca/ess/env/ess_env_module.c     (original)
> +++ trunk/orte/mca/ess/env/ess_env_module.c     2009-06-19 14:50:06 EDT
> (Fri, 19 Jun 2009)
> @@ -150,6 +150,20 @@
>             goto error;
>         }
>         opal_argv_free(hosts);
> +
> +        /* in case we are asked to spawn remote daemons... */
> +        if (ORTE_SUCCESS != (ret = orte_plm_base_open())) {
> +            ORTE_ERROR_LOG(ret);
> +            error = "orte_plm_base_open";
> +            goto error;
> +        }
> +
> +        if (ORTE_SUCCESS != (ret = orte_plm_base_select())) {
> +            ORTE_ERROR_LOG(ret);
> +            error = "orte_plm_base_select";
> +            goto error;
> +        }
> +
>         return ORTE_SUCCESS;
>     }
>
> _______________________________________________
> svn mailing list
> s...@open-mpi.org
> http://www.open-mpi.org/mailman/listinfo.cgi/svn
>

Reply via email to