Make smpi_switch_data_segment check if a switch is needed, and return true when it occurs.

Kill global SMPI_switch_data_segment.

Use existing function (also empties requests_ after waitall).

Improve debug messages and avoid calling finish_comms twice when for myself.

Initialize variable.

Use existing functions to finish comms (and fix Win::flush).

Review usage of rank/rank_/rank() is smpi_win.

Little simplifications in loops.

Some int -> bool conversions (+ use of existing macro).

Call rank() only once.

Ooops, fmt is second arg.

Prefer emplace_back.

Get rid of "%s" in second argument of function xbt_str_parse_*.

XBT_ATTRIB_PRINTF for vprintf-like functions.

Define class SmpiBenchGuard, and use RAII to handle smpi_bench_end()/smpi_bench_begin().

Add missing calls to smpi_bench_begin() on error paths.

Cannot set split-duplex through s4u intf.

This makes sense only in XML where it properly creates the
link-up/link-down.

Add fg#71 to changelog [ci-skip]

Missing include.

Coding style: no global "using namespace".

Prefer std algorithms.

Invert tests to reduce depth of nesting.

Useless tests for emptyness.

Add attribute(printf) to xbt::string_printf.

Pointer-to-const for Sonar.

Set shared variable *before* mutex unlock.

Die on unwanted function calls.

Parameter 'assert' is a bit field.

Fixes in UTs

Handle case where different groups are given to MPI_Win_start and MPI_Win_post on a seame process.

Handle duplicated datatypes within predefined MPI_Op.

Docker: try to get apt update to run despite the cache

A few more 'const'.

Replace redundant type with 'auto'.

Remove useless temporary shadowing outer variable.

Add some 'const' qualifiers.

This started with NetPoint::get_englobing_zone() and propagated quickly...

Unused exception parameter 'e'.

Explicitly delete the copy constructor and copy assignment operator (enforce rule-of-five).

No valgrind leak check for issue71.

xbt_replay: rethrow exception instead of xbt_die

Remove sg_bandwidth_factor from disks

No need for bw_factor here

remove a space to force update

Issue#71: add check in add_route for gw_src/gw_dst

When adding a NetZoneRoute, check if gw_src and gw_dst belongs to the
respective netzones.

We need to recursively search for the netpoint inside the netzone since
the netpoint can be from one of its children.

Fix refcount for Datatype_contents.

Error seen with Petsc test: vec_is_sf_tutorials-ex3_basic_dupped.

Allow null ranks for MPI_Group_incl when n == 0.

update draw_gantt.R script to work with pajengr

todo
- need to test with a recent version of S4U trace

update pajengr installation procedure in S4U tutorial

replace r-cran-dplyr by r-cran-devtools

update tuto-smpi Dockerfile to contain r-cran-tidyverse (replacing included R packages)

add additional necessary packages for pajengr installation

update pajengr R code and figure

details:
- note the addition of library(tidyverse)

use new option in random test.

document new option

Add flag to provide an optional barrier in MPI_Finalize.
This is meant to help for codes which can aggressively cleanup memory at finalization, while other processes still have to use it.
For example when attributes are attached to a local communicator, in SMPI this communicator may be cleaned up by another process in the end, and the attribute has to still be valid at this point.
This was an issue with PETSC, for example.

Use std::string for xbt_parse_units.

Combine parameters 'entity_kind' and 'name'.
Also rename surf_parse_* to xbt_parse_*.

Deprecate SIMIX_get_clock().

Use simgrid_get_clock() or Engine::get_clock().

Keyval should always exist.

Also please sonar by reducing depth for nested blocks.

leaks analysis: when -analyze is given to smpirun, set value for list-leaks to 50.

leaks analysis : activate if list-leaks is asked manually (without smpi/display-allocs or -analyze)

make simix_global->maestro_ private

Check types matching in MPI communications.
should work with basic ones and duplicated ones.
todo: derived types matching, which is more tricky.

mark these as derived datatypes, and don't divide by 0 if possible

keep track of which datatype was duplicated, to be able to match with it later

nothing old here, just the type ..

prepare transition of context_factory: make it private

Minor simplification in CM02::communicate

Don't need a function in NetworkModel for an "if" used only in CM02

Fix potential mem leak seen by scan-build.

Note that ret should not normally be different than MPI_SUCCESS in the other case.

Don't create mailbox before engine is turned on.

Lookup only once on mailbox creation.

Misc sonar smells.

vector::back() is good enough + assert for scan_build.

[cppcheck] Don't let fields uninitialized.

[sonar] Extract assignments from expressions.

Plug remaining leaks in mpich3-test/f77/attr.

Plug memleaks with Fortran bindings.

Review allocation of MPI attributes with Fortran bindings.

Fix refcounting of MPI keyvals when keyval_free is called twice.

Simplify a bit Keyval::attr_delete.

Don't increase refcount when existing elem is overwritten.

Save an indirection with smpi keyvals.

revalidate tesh

remove cruft [ci-skip]

Change way Mailboxes are create, stored, and destroyed

- keep the global map in EngineImpl (not as a static global in
  MailboxImpl.cpp)
- Delete mailboxes in ~EngineImpl() and get rid of SIMIX_mailbox_exit
- replace MailboxImpl::by_name_or_null and
  MailboxImpl::by_name_or_create by Engine::mailbox_by_name_or_create
  + better match with what is done for hosts, links, and actors
  + Mailbox::by_name cause two searchs in the map. One to check if
    name already points to a mailbox (by_name_or_null) and if not
    search again before creating a new mailbox. As there is no
    Mailbox::by_name_or_null, just keep the latter.
- Revalidate a bunch of tests (message ordering mostly)

delete Timer when shutting down EngineImpl

Continue to migrate contents of simix_global to EngineImpl

thx @agiersch for unblocking this commit with the usual magic!

Prefer using emplace() to insert() for smpi keyvals/attributes.

Make smpi::Keyval::attribute() return a reference.

No need to test for attributes()->empty().

Fix parameter names.

Coding style: ! -> 'not'.

Update filter line for recent versions of Asan.

Define variables outside of header file.

[sonar] Unused parameter.

[sonar] Const/explicit methods.

Please sonar

Adjust test timing

Minor changement in rounding up.
No real changement in timing (manually verified).

Single get_bandwidth_constraint function.

Unify behavior of get_bandwidth_constraint function.

Note: Possible side-effect in SMPI communications with user defined
rate. Do not scale the user rate with bandwidth factor.

No impact on tests

Refactor of NetworkCm02Model::communicate

Divide and conquer.
Reorganize code.
Cosmetics.

Dynamic factors: implement and test

Fully implements the dynamic factors through the NetworkModelIntf API.

Added an example showing its usage.

Added UT and doc.

NetworkModelIntf: Interface to dynamic factors

Adding API to set user callbacks for bandwidth and network factors.
Valid for CM02 models and its childrens (SMPI, IB, etc).

Not fully working yet.

Get netzones traversed by a route

Preparing field to implement dynamic bandwidth and latency factors