use previous buffer check feature in MPI checks, to crash when a buffer overflow is detected and provide good information.

Count all intercepted allocations/deallocations in order to build a small leak-checker.
This allows to check for leaks, but also to provide debug information on the involved buffers (file, line, size) when a crash happens
todo: evaluate memory cost and slowdown for real apps (disabled by default, needs --cfg=smpi/display-allocs or -analyze flag to be activated)

Use directly std::string instead of c_str.

s4u::Host::create_disk, human-friendly string version

s4u::Host: moving create_disk impl to HostImpl

s4u::Disk allow chaining set_property

s4u: allows full single-line initialization

seal() returns a pointer to the s4u object.
Allows users to use the object after its seal, without having to do it
in another line

Make fields private, and static functions class-member.

Useless forward declaration.

Redundant code (ctor is already inherited).

Prefer nullptr.

Kill erroneous constant.

Const/override for member functions.

Inherited constructor.

Pointer-to-const parameter.

continue to mess with platform creation

Kill empty statement.

Make BacktraceImpl::resolve a bit faster.

Print the full frame information all at once.

Replace void* used for RemoteProcess pointers.

Use a RemotePtr<> for RemoteProcess::heap_address.

Stop using void* for xbt_mheap_t.

Fix cloud.xml: symmetrical routes are implicit.

Add file to basic_parsing_test so that it doesn't remain untested..

Make NetworkConstantAction "final": virtual set_state() called from constructor.

Make virtual functions called from constructor "final".

Sonar S1699: Constructors and destructors should only use defined methods and fields

please sonar

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