Skip to content

Commit

Permalink
broker: use libpmi-client
Browse files Browse the repository at this point in the history
Drop the somewhat contrived boot_pmi.c class from the broker,
and rewrite the PMI bootstrap code using pmi-client.h interfaces
directly.  I think this clarifies the code even though it is quite
verbose.

If PMI doesn't implement pmi_get_id(), derive the session-id from
the "appnum" (numerical jobid).

Don't attempt to call pmi_get_clique_ranks() unless epgm is enabled.

Neither pmi_get_id() nor pmi_get_clique_ranks() are implemented
in the "simple v1" PMI wire protocol, so allowing these functions
to be unimplemented enables Flux to be launched by mpiexec.hydra,
which addresses one goal of flux-framework#398.
  • Loading branch information
garlick committed Oct 5, 2015
1 parent 8d5ee42 commit 111495d
Show file tree
Hide file tree
Showing 4 changed files with 200 additions and 447 deletions.
2 changes: 0 additions & 2 deletions src/broker/Makefile.am
Original file line number Diff line number Diff line change
Expand Up @@ -18,8 +18,6 @@ flux_broker_SOURCES = \
module.h \
modservice.c \
modservice.h \
boot_pmi.c \
boot_pmi.h \
overlay.h \
overlay.c \
heartbeat.h \
Expand Down
384 changes: 0 additions & 384 deletions src/broker/boot_pmi.c

This file was deleted.

Loading

0 comments on commit 111495d

Please sign in to comment.