Correctly get the exit status for smpi processes.

Search in map only once.

Kill dead code.

process_data is empty a this point.

Revert "Revert "actually fix the memleak around smpi's process_data" and "try to plug a memleak""

This reverts commit c7c5def2870e604025be03b866967c0c5e293afe.

Revert "Finally fix memleak."

This reverts commit aad6442f8fd7d7610f49271010cb8a650f4772de.

Finally fix memleak.

Revert "actually fix the memleak around smpi's process_data" and "try to plug a memleak"

This reverts commit 3c40b1bfc4f4925fa401d3b5d0c74a4b4824b3d5
and commit d53a2108a1ea38b2f952abb48c39730ccc91c2a8.

inline a oneliner

plug another memleak

actually fix the memleak around smpi's process_data

- Call Actor::onDestruction from the right location
- Have SMPI only destroy the process data of its own processes (not of MSG ones)
- Make sure that the process created with MSG_process_attach() have
  the right refcount since we now use a smart pointer on it: we don't
  want it to get freed too early.

obey our coding style

try to plug a memleak

Pick the best available clang-format version

and also, update copyright info

[misc, doc] clang-format pre-commit hook cleanup

The previous script only worked on (old) debian.
This version should work with:
- more versions of clang-format
- more distribs (tested on arch)

Furthermore, I did some cleanup on the script and its doc

[#!] #! ./tesh -> #!/usr/bin/env tesh

find . \
    -type f \
    -not -iwholename '*.git*' \
    -exec sed -i -e 'sW#!\s*\./teshW#!/usr/bin/env teshWg' {} \;

[#!] #! tesh -> #!/usr/bin/env tesh

find . \
    -type f \
    -not -iwholename '*.git*' \
    -exec sed -i -e 'sW#!\s*teshW#!/usr/bin/env teshWg' {} \;

[#!] #!/bin/bash -> #!/usr/bin/env bash

find . \
    -type f \
    -not -iwholename '*.git*' \
    -exec sed -i -e 'sW#!\s*/bin/bashW#!/usr/bin/env bashWg' {} \;

[#!] #!/bin/sh -> #!/usr/bin/env sh

find . \
    -type f \
    -not -iwholename '*.git*' \
    -exec sed -i -e 'sW#!\s*/bin/shW#!/usr/bin/env shWg' {} \;

[misc] gitignore: ignore qtcreator main file

directly store the ns3Node, don't rely on a num ID to find it later on

ignore a new test

Factorize how we load NS3 libraries

try to ensure that link_energy plugin is inited in a timely manner

reduce the shotgun design pattern in the detection of NS3

make an interesting error message when mixing link_energy and NS3

cmake: more verbose msg about which NS3 is chosen

fix formatting issues (%u for unsigned)

fix make dist

ns3: much more debug info, and restore the SetCloseCallbacks that is actually needed

ns3: remove test on a platform that does not exist anymore

ns3: no segfault anymore, but a deadlock when using clusters

fix the creation of links in NS3

Cluster are still broken, sorry

Rewrite how we search for libunwind (hopefully correctly this time)

This time, I used the canonical approach: Instead of passing the
absolute path to the libraries to the linker, I use link_directory()
and -lname as I should.

I hope that this version will work for both java on all our
target architectures, and also for stampede.

If so, we should maybe contribute it to libunwind and/or cmake projects.

more informative outputs while searching for libunwind

Revert "sanitize our way to find and load libunwind"

This reverts commit 2a8f06bcb45d972f49b0661754fac0d51772f02f.

It was doing more harm (to java, to centos, etc) than good (possibly
to stampede, not even sure)

get ns3 to compile (but still fails)

cmake: fix a typo that does not break as it should on my machine

Gosh I hate unsafe scripting languages

Survive to when libunwind does not exist (hello again windows)

sanitize our way to find and load libunwind

The previous way of doing so was working OK on most machines, but not
on some weirdly configured supercomputers.

please sonar

improve a comment

Shorten this a bit, and do not dupplicate things that we can find with
a git blame.

Try new fix for rget_testall.

The test seems buggy, as the initialization part on one process may not be started when the other one reaches the second part, due to the way the locks are done.

Add a barrier to separate initialization from computation.

add another message useful for debugging here

Revert "[TESTS] SMPI/MPICH3: Fix failing rma test"

This reverts commit 1630b8c2917fce5760c0065b240764cda0a70eb4.

We decided to rather have a failing test than a test with
some logic disabled...

[TESTS] SMPI/MPICH3: Fix failing rma test

This test only occurs with compile time optimizations on. See inline
comment for more details.

[SMPI] Deprecate smpi_process_index() in v322 instead of v321

Merge branch 'master' of scm.gforge.inria.fr:/gitroot/simgrid/simgrid

[SMPI] smpi_win.cpp: Finish first local comms, as the DEBUG statement said.

There was a confusion between rank and rank_

[SMPI] smpi_win.cpp: Split debug message into two

[SMPI] Fix wrong usage of request->src() and dst() with ranks

We need to use the process id instead of the local rank, so
let's do that.

[SMPI] smpi_win.cpp: Convert target_rank for other communicators

I'm not sure that comm_ and recv_win->comm_ are the same, so I convert them.

[SMPI] Cosmetics: This fixes only '='-alignment issues for readability

[SMPI] Cosmetics: This fixes among others some really nasty indentation for smpi_win.cpp

[SMPI] Fix wrong src/dst calculation

In smpi_win.cpp, one needs to be very careful about which group / comm
is currently used: In some functions, a subset of the current comm
is passed which means one has to convert the ranks from that subgroup to
the ranks of the communicator's group.

[SMPI] Fix wrong smpi::Request instantiation for RMA's.

This converts the rank correctly to the pid

[SMPI] Added stupid comment to smpi_group.cpp

With all the different class members around, like
actor_to_rank_, rank_to_actor_, ..., I didn't realize that
this value was passed in as a param. Hope this comment will
save someone a few moments in the future.

[SMPI] Fix errors on SUSE: ActorPtr cannot be printed, use process_.get()

[SMPI] Replace Actor::self() with smpi::Process's process_

[SMPI] C++ify smpi::Process mailboxnames

[SMPI] Cosmetics in smpi_process.cpp: Correct indentation

The whole file was indented correctly except for this function...

[SMPI] smpi_process: Move instance_id_ from char* to std::string

[SMPI] Remove comment

[SMPI] Remove index_ and index() from smpi::Process

[SMPI] Rename parameters from index to process_id in helper function

[SMPI] smpi_process.cpp: Replace 'index' with 'Actor::getPid'

[SIMIX] Remove SMPI-specific segment_index notion from the Actor class

Privatization is now done via the process id and attached to the SMPI process.

[SMPI/CHANGELOG] Deprecate smpi_process_index()

[SMPI] smpi.h: Remove smpi_process_index() usage in macros

[SMPI] Search/Replace of smpi_process()->index() in src/smpi/

Via
sed -i -r -e 's@smpi_process\(\)\->index\(\)@simgrid::s4u::Actor::self\(\)\->getPid\(\)@' src/smpi/**/*.cpp

[SMPI] smpi_replay.cpp: Cosmetics as proposed by clang-format

[SMPI] Rename variables from 'rank' to 'my_proc_id' in smpi_replay.cpp

[SMPI] smpi_replay.cpp: Replace smpi_process()->index() calls

[CHANGELOG] Update changelog with SMPI's index + trace file changes

[SMPI] Cosmetics for smpi_pmpi_win.cpp as suggested by clang-format

[SMPI] Rename variable 'rank' to 'my_proc_id'

Because it's not the rank.

[SMPI] (LAST PATCH) Fix SMPI replay & smpi-replay tesh

This patch is the last and SMPI should build and run correctly.

[SMPI] (THIRD PATCH) Replace invalid uses of 'comm->rank()'

This patch may not build or may have failing tests. It's part of a series and
contains logical changes.

[SMPI] (SECOND PATCH) Fix destinations/origins for tracing *TIData objects

This patch will either not build or will have failing tests, as it belongs
to a series of patches.

[SMPI] (FIRST PATCH) Remove 'getPid() - 1' arithmetic from SMPI.

THIS COMMIT WILL NOT BUILD PROPERLY OR HAS FAILING TESTS. I decided to split up
a huge commit into several logical chunks in order to make things easier to read.

The 'getPid() - 1' expressions were the (temporary) replacement for smpi_process()->index() but it's better
to remove it completely: This is often confused for being an MPI rank
although it is not (ranks depend on the communicator in use).
Also, process id 0 is exclusively the maestro so we should not artificially
pretend we start counting from 0 when we're clearly not. So now we don't have to
say that 'process with id 1 has index 0'.

This patch substitutes almost exclusively 'getPid() - 1' computations with
just 'getPid()'. There are a few other replacements, especially in the smpi_win.cpp,
where former commits introduced logically wrong usage of 'comm_->rank()'.

The next few commits after this should also deal with a similar problematic.'

[TESH] Replace 'smpi_process_index()' with call to 'MPI_Comm_rank()'

[SMPI] Remove outdated comment

[SMPI] Cosmetics of smpi_pmpi_request.cpp as suggested by clang-format

[SMPI] Replace index() in smpi_pmpi_requests.cpp

[SMPI] Replace index() in smpi_requests.cpp

[SMPI] Remove index() from smpi_pmpi_win.cpp

[SMPI] Remove index() from smpi_win.cpp + cosmetics

Replace these calls with comm_->rank() instead.

The cosmetics were rather unwanted but it's just indentation.

[SMPI] Win::~Win(): Use class member rank_ instead of a local var

[SMPI] Remove usage of index() in smpi_pmpi_coll.cpp

We can just use comm->rank() instead. Note that usage of the index()
function was also incorrect: This would not have worked when the
communicator is not MPI_COMM_WORLD as the index() function always returned
Actor.pid-1 (as set in the Process::init() function)

[SMPI] Make Comm::rank() use Actors

[SMPI] Use actor.iface() instead of Actor::byId(actor.pid)

[SMPI] Move the pid calls to helper function that has more checks

[SMPI] Fix smpi-replay-multiple test

I moved the SMPI_init() to the right location and
changed the tesh.

[EXAMPLES] Update smpi_msg_masterslave -- move the SMPI_init()

The reason we moved this was that the processes were all set up before the hook
that sets up the smpi::Processes was even set up.

[SMPI] Update smpi_deployment_register_process for actors

[SMPI] Rename 'index' to 'my_proc_id' in smpi::Process::init()

[SMPI] Move process_data to map<Actor, smpi::Process>

[SMPI] Remove index notion from smpi::Group

This is a huge commit and removes the index() function from
smpi::Group and replaces it with an actor() function.

The index was basically just Actor::getPid()-1 and doesn't need
to be saved. It was also often used for the privatization segment
(to know where it was stored) but that is no longer required.

[SMPI] Program against s4u::Actor and not smx_actor_t

[SMPI] Move SMPI_init() call to other location

[SMPI] Next step for dynamic privatization

This commit breaks the following tests:
The following tests FAILED:
        181 - smpi-msg-masterslave-thread (Failed)
        182 - smpi-msg-masterslave-ucontext (Failed)
        183 - smpi-msg-masterslave-raw (Failed)

To fix this, many more changes will be required. This will be done in
the next commit. This commit is just there to make sure changes
remain easily understandable (by having small, easy-to-read commits).

I changed the following:

- Remove variable index_to_process_data
- Make process_data a map instead of an array.
- Remove the "index" variable for SMPI instances (no longer needed)

And cleanup the mess that comes with these changes (delete/free, ...)