Display leak origin in message, if -trace-call-location is used.
+ downgrade messages to INFO.

hide references to current_handle inside *.cpp file, to avoid breaking if SMPI is not compiled

appease 32 bits

Use a Remote<> for remote smpi::Request.

appease clang

change the way we display leak types, skipping demangling

try to improve handling of comm_self a bit.

fix for potentially empty names

Add the option to print origin of a failing handle.
This uses trace-call-location flag for smpicc, store the value at handle creation, and tries to return the most relevant one on a crash (that's actually a pain and will have to be checked)

add a name() to all F2C objects.
This avoids demangling need for leak check (and should fix tests, as freebsd has a broken demangler on our ci system).

oops

same for win, get_info needs a copy.

remove unwanted s4u include in all smpi files

get_info calls actually must return a copy.

generalize mark_as_deleted call

Don't add handles to the lookup for the sake of just removing them immediately...
if they're not there .. well they are not.

add two fortran declarations

Add message to help improve backtrace

\t removed

print backtrace when we exit due to an MPI failure

factor on_creation signal trigger in Host::seal()

prefer create_host than creating the creation args

cleanups

simplify kernel part of disk. It's really shallow now

cosmetics

Match Io and Exec creation

Save one assert and do not use shring policy to determine if WifiLink

cosmetics

Single way to create links in NetZone

With the refactoring, we don't need 2 methods in NetZoneImpl to create
new links.
The link type (wifi or wired) is determined by the NetZone type
(WifiZone).

Simplify NetZone's hierarchy

A NetZone is RoutingMode::base by default. When we add a child to it, it
becomes RoutingMode::recursive.

One method to set netzone's relationship

A netzone is child of its parent, set_parent() is enough to determine
the relationship parent/child of netzones.

Copy models from parent netzone

Allows user creating new netzones without explicitly setting models

Minor fix in doc

[appveyor] Stick to the stable branch of pybind11.

Missing space (?).

checks for PMPI_Type_create_subarray

check for alloc mem

check illegal in_place for gather/scatter
also stop calling pid rank, it's confusing.

check ranks for probes/iprobes

increase rank checking for RMA comms

this test is invalid with 1 process. Align with mpich and use 8.

group inputs validation

set keyval to invalid after free

MPI_Abort : try to kill other processes from the communicator.
No idea if this helps

Keyvals : handle MPI_KEYVAL_INVALID

Errhandler : inputs + cleanup

oops.

Abort : check input, and remove fixme... Actual implementations do the same

Init : dont just crash with asserts, return errors (they will crash if errhandler is configured to)

comm, input validation

use MPI_COMM_WORLD errhandler if MPI_COMM_NULL is given

Fix null dereference.

Sort the list of leaks before print.

avoid allocating comm_self if not needed

Tell when we have more leaked handles to display

Display the leaked handles in the tests (using no_loc to hide pointers)

Cosmetics in an error message

To ease the grepping of this message in a child project ;)

Merge branch 'master' of framagit.org:simgrid/simgrid

validate that MPI_Op can be applied at entrance of MPI call, instead of xbt_dying deep inside SMPI.
For RMA comms, somehow derived datatypes also are allowed. and it's too late for me to add this, so keep the previous checks for now.

Pack_size is meant to return the upper bound of the size.
Thanks agier

A bool is OK in boolean context.

Cosmetics.

change way WIFI are create + make sharing_policy optional

reindent

more simplifications

Fix count for SMPI leak report.

Function doesn't exist.

[mc] Enable finalize_app() in SafetyChecker.

Update mc-coll-allreduce-with-leaks.tesh accordingly.

[sonar] Const in s4u::Disk.

update test with latest version from mpich.
d5e44448d0 was a misguided fix, as MPI_SUM is forbidden on MPI_BYTE... And I can't remember what the issue with ubsan was, so let's see

MPI_BYTE has some specific requirements, and doesn't allow most operations on it

report the proper size on the receiver side, when its buffer was allocated too largely.
I'm sure it used to be done properly ...

select proper errhandler when MPI_COMM_WORLD is duplicated

recv should return MPI_ERR_TRUNCATE
we crash properly when needed, but don't return it properly if errhandler is meant not to crash

fail when trying to attach a buffer if one is already attached

when the standard said "return" here, it was not meant as return value.

we need to check comm is not null before trying for the rank, duh..

this tests actually does not conform to the standard and crashed now, but it's OK to say we are strict here.

Add a deleted flag to keep track of invalid but not actually deleted handles
We can then crash if someone tries to use it later on.
just do it for Comm for now, will extend later.

fix size of allocated buffer

Implement MPI_Comm_test_inter.
Answer is no, as we don't support intercommunicators.

add more Fortran definitions

Seal NetZone

We need to be able to seal the netzone too.
Sealing a netzone, seals its children and hosts.

Add protection to avoid sealing twice.

Constify list of links parameter

Simplified methods to add routes

Avoid using LinkImpl in s4u interface.
Add method that raises and exception if netpoint isn't found

Allows the implementation of a platform like small_platform.xml without
using "internal" structures.

simplify constant model a bit

snake casing IB

small simplifications and cleanups

simplification

Fix error handling.

Raise std::invalid_argument exception instead of dying.

Thanks for the review ;)

Avoid a local variable only used by xbt_assert.

Change "if(...) xbt_die(...)" to "xbt_assert(...)".

Some of them could certainly be improved to report the error to the user code,
for example with a proper exception, but it's another work.

Never disable xbt_assert anymore.

Make mc::Api::get_actor_name return a const&.

Sometimes avoids unnecessary copies.

Empty is when len == 0!.

Improve debug messages (code is still not functional).

FIXME++

Fix segfault with debug message.

Render mc::Api::get_actor_name() public.

Zero-initialize struct to avoid valgrind warnings.

Use memset, since there is no warranty that the padding bits are set to zero
with an empty initilizer-list {}