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 {}

[ci-skip] save some space on docker images (without breaking stuff hopefully)

snake_case some surviving camels

Renaming, snake_case

Friendly set_latency, accepts strings.

Human-friendly version for set_latency function.
Accepts strings such as "1ms".

Add create_link in s4u::NetZone

With SharingPolicy as parameter since it's mandatory to decide if we
must create a WiFi link or not.

Setting default as SHARED according to DTD

Simplyfying create_host and exporting it to s4u::NetZone

Adding seal() for Host

Allow changing Host settings before seal

Factory functions for all Netzones

Deobfuscate error checking.

Move declarations for xbt_abort and xbt_die to xbt/asserts.h.

Adjust some #include consequently.

Make xbt_die use the root log category.

Using xbt_* functions in vasprintf() does not look like a good idea.

Moving the models initialization to NetZoneImpl.

By moving the models init to NetZoneImpl, we don't need the xml to initialize the
models.

This should enable C++ platform creation.

sg_exit never allowed to reload a platform afterward, so reduce the visibility of a variable

At least it never worked properly

typo (reported by lintian)

Actor's ID should be a long, and should be named AID

add libfmt-dev as a pajeng dependency [noci]