207ca2d928
extravaganza.
= Short version =
This commit does several things, but the short version is that it
re-orients the error message creation of the ODLS default module to
generate error strings in the child process for errors that occur
after the fork but before the exec (such errors are ''usually''
related to paffinity). A show_help string is rendered in the child
and then IPC'ed up to the parent, who displays the string through
normal ORTE show_help aggregation mechanisms. We also broke up the
ginormous paffinity-setting logic into a few separate functions, both
to help us understand the code, and hopefully to ease future
maintenance.
The logic for the ODLS default binding should not have changed -- this
is mainly a code reshuffle and improvement on error reporting.
= Rationale =
The reasoning for this commit is complex. As mentioned above, it's
the first step in some paffinity cleanup. Here's the line of dominoes
that must fall (in this order):
1. Add hwloc paffinity component (already done).
1. While testing hwloc, we discovered that the error reporting from
the ODLS default module was abysmal. So we fixed it.
1. Further, we reorganized the code in the odsl_default_module.c a bit
to help our understanding of it.
1. We also discovered a few bugs in the original ODLS default module
logic that existed before this code shuffle; separate tickets
will be filed to fix them.
1. Next up will be some improvements to paffinity / odls default to
make the act of binding to a core ensure to bind to ''all''
hardware threads contained in that core (similar for sockets:
binding to a socket will bind to ''all'' hardware threads in that
socket).
1. Next will be improvements to paffinity to expose binding to
hardware threads through the paffinity framework API.
1. Finally, we'll expose these binding controls to the user (e.g.,
through mpirun command line arguments, MCA parameters, etc.).
This commit represents the first few bullets; the last 4 bullets are
being worked on right now, but there is no definite timeline for
completion.
= Miscelaneous =
A few points worth mentioning:
* We have tested this new code a bunch; we're pretty sure it behaves
just like the trunk -- but with better / more precise error
reporting. More testing is needed on a wider array of platforms,
however.
* A big comment at the top of odls_default_module.c explains the
(new) general scheme for the error reporting.
* The error reporting in the parent process is now really dumb;
almost all the intelligence about creating error messages is in the
child.
* The show_help file was renamed to be more consistent with other
help files (help-odls-default.txt -> help-orte-odls-default.txt)
* Removed the use of sched_yield() because of recent changes in the
Linux 2.6.3x kernels. We already had an #else clause for
select()'ing for 1us if we didn't have sched_yield() -- that is now
the only code path. This is not a performance-critical section of
the code, so this shouldn't be controversial.
* Replaced the macro-based error reporting with function-based
reporting. It's a bit more bulky, but it helped us understand the
code and saved us multiple times with compile-time parameter
checking, etc.
* Cleaned up the use of several show_help messages to ensure that
they mapped to real messages in help*.txt files.
This commit was SVN r23652.
76 строки
1.9 KiB
Plaintext
76 строки
1.9 KiB
Plaintext
# -*- text -*-
|
|
#
|
|
# Copyright (c) 2004-2007 The Trustees of Indiana University and Indiana
|
|
# University Research and Technology
|
|
# Corporation. All rights reserved.
|
|
# Copyright (c) 2010 Cisco Systems, Inc. All rights reserved.
|
|
# $COPYRIGHT$
|
|
#
|
|
# Additional copyrights may follow
|
|
#
|
|
# $HEADER$
|
|
#
|
|
# This is the US/English general help file for Open RTE's ODLS Framework
|
|
#
|
|
[orte-odls-base:could-not-preload-binary]
|
|
WARNING: Could not preload the binary file.
|
|
|
|
Binary: %s
|
|
|
|
Will continue attempting to launch the process.
|
|
[orte-odls-base:could-not-preload-files]
|
|
WARNING: Could not preload the files specified.
|
|
|
|
Fileset: %s
|
|
|
|
Will continue attempting to launch the process.
|
|
[orte-odls-base:could-not-preload]
|
|
WARNING: Could not preload the requested files and directories.
|
|
|
|
Binary : %s
|
|
Fileset: %s
|
|
|
|
Will continue attempting to launch the process.
|
|
|
|
#
|
|
[orte-odls-base:xterm-rank-out-of-bounds]
|
|
The xterm option was asked to display a rank that is larger
|
|
than the number of procs in the job:
|
|
|
|
Rank: %d
|
|
Num procs: %d
|
|
|
|
Note that ranks start with 0, not 1, and must be specified
|
|
accordingly.
|
|
#
|
|
[orte-odls-base:show-bindings]
|
|
System has detected external process binding to cores %04lx.
|
|
#
|
|
[warn not bound]
|
|
A request to bind the processes to a %s was made, but the operation
|
|
resulted in the processes being unbound. This was most likely caused
|
|
by the following:
|
|
|
|
%s
|
|
|
|
This is only a warning that can be suppressed in the future by
|
|
setting the odls_warn_if_not_bound MCA parameter to 0. Execution
|
|
will continue.
|
|
|
|
Local host: %s
|
|
Application name: %s
|
|
Action requested: %s %s
|
|
#
|
|
[error not bound]
|
|
A request to bind the processes to a %s was made, but the operation
|
|
resulted in the processes being unbound. This was most likely caused
|
|
by the following:
|
|
|
|
%s
|
|
|
|
This is an error; your job will now abort.
|
|
|
|
Local host: %s
|
|
Application name: %s
|
|
Action requested: %s %s
|