cosmetics (before move to src/kernel/resource ??)

not needed anymore

do not use RouteCreationArgs outside of XML parsing

Resize(0) is redundant.

Missing includes.

Cleanup some unused and/or useless build config variables.

do not need ClusterCreationArgs anymore

refactor link characteristics management across Clustered Zones

Torus/Fat-Tree/Dragonfly: Aggregate callbacks

Put set callbacks in an object.
Less (or encapsulated) arguments for create_zones.
Remove create_cluster_zone (create_star_zone does its job for C++
interfaces).

Asan sometimes chokes on RUNNING_ON_VALGRIND. Workaround by changing the test.

Libunwind is useless for non-MC builds.

Moreover its presence causes unwanted false-positives with Asan.

Fix 32 bits now...

Ooops.. Fix CmakeFiles and MANIFEST

Cluster: use radicals from XML when creating resources

Use radicals from XML when creating hosts in sg_platf.xml.
Fix bug with limiter.
Add test.

do not throw signal from sg_platf

Use new API to create Torus/Dragonfly/Fat-Tree clusters from XML.

Get rid of ClusterCreationArgs in Torus/Dragonfly/Fat-tree.

NOTE: Routers aren't created anymore for Torus/Dragonfly/Fat-tree zones.
- This router was lost in the middle of topology connected through empty
routes (no links)
- In the past, the sg_platf created a router automatically for these
netzones since they shared the same sg_platf_new_cluster function.
- Impact on flatfier test: cluster_torus.xml.

FatTreeZone: Do the checks earlier

New: s4u::create_dragonfly_zone

Implements the builder method for Dragonfly zones.

New example: Dragonfly cluster of multi-cpu hosts(Star Zones)

Cosmetics

MC tuto: Review by Emma

Attribute noreturn for terminate handler.

We don't want to survive a ForcefulKillException!

The header <typeinfo> must be included before using typeid.

New example: Fat Tree cluster of multi-cpu hosts(Star Zones)

Adapt existing Torus example for different platforms.
Same idea: each leaf is a new StarZone netzone, emulating a multi-cpu
node.

New: s4u::create_fatTree_zone

Implements the builder method for Fat Tree zones.

Similar signature and behavior of create_torus_zone.

Fix FatTreeZone with limiter links

Are we sure we want this "feature" in complex clusters?
As for Torus, no test using it.

Segfault when communicating with limiter links.
Routers inside the FatTreeZone don't have limiters, even when user sets
the limiter on the XML.

Code was adding nullptr to the list of links in this case, generating a
segfault in communicate function.

Example:
  <zone id="world" routing="Full">
    <cluster id="bob_cluster"
     prefix="node-" radical="0-15" suffix=".simgrid.org"
     speed="1Gf" bw="125MBps" lat="50us"
             topology="FAT_TREE" topo_parameters="2;4,4;1,2;1,2"
     loopback_bw="100MBps" loopback_lat="0" limiter_link="100MBps" />

First version of a tutorial on Mc SimGrid

cosmetics

Add a pointer from the doc sources to the online one

add a (modern) wait_any for Io activities. Thx agier!

throw signal from seal() directly

Try to please sonar

Improvements in torus test

Use 2 links in Star Zone to simulate full-duplex communications.
Improv doc.

Initialize models together with netzone root

Remove static to initialize models once, anyway we have only one engine
in a simulation by now. Hope someday we'll be able to initialize the
models properly.

Remove workaround in UTs

New example: Torus cluster of multi-cpu hosts(Star Zones)

Add a new example showing how to build a torus cluster composed of
netzones instead of single hosts.

Each leaf is a StarZone, representing a host with 8 CPUs connected in a
star. CPU-0 is the gateway for each StarZone.

Runs a simple broadcast application where each CPU runs a specific
actor.

NetZone::get_netpoint: gets the netzone's netpoint

New method to get the netpoint used by netzones.
Usefull when describing routes using the netzones (add_route method)

Add some UT for TorusZone

Improve some checks.

New: s4u::create_torus_zone

Implements the builder method for Torus zones

NetZone* create_torus_zone(const std::string& name, const NetZone* parent, const std::vector<unsigned int>& dimensions,
                           double bandwidth, double latency, Link::SharingPolicy sharing_policy,
                           const std::function<TorusNetPointCb>& set_netpoint,
                           const std::function<TorusLinkCb>& set_loopback = {},
                           const std::function<TorusLinkCb>& set_limiter  = {});

This builder method allow us to keep hiding the internals of our
NetZones, exporting only a NetZone* to the end-user.

It uses callbacks functions to populate the leaves in the Torus, 3
callbacks can be used:
- set_netpoint: must return a pair NetPoint,Gateway. This allows us to
have complex netzones under each leaf
- set_loopback: The link used to loopback communications
- set_limiter: A limiter link for each leaf

The links inside the Torus are homogeneous and described by the
bandwidth, latency and sharing policy parameters.

As the creation of resources are linked to the netzones (create_host,
create_link functions), we have to add the parent parameter to the
builder function. Otherwise, the user cannot create the resources
associated with the leaves in the callbacks.

Obs.: By now, the sg_platf.cpp isn't using this new builder method.
It will be changed once we have implemented the builder for every
cluster type (fatree, dragonfly).

Rename VirtualMachine::get_impl() to not hide Host::get_impl().

Make SIMIX_simcall_name() use the type of observer_ if it exists.

It gives more information about the kind of simcall being involved,
since most simcalls are nowadays of type RUN_KERNEL or RUN_BLOCKING.

Use std::string::rfind to match string prefix.

Unused #include.

Raise exception for invalid parameters in StarZone

We've talked about using exceptions for invalid parameters in user'
calls.
Adjust for StarZone.
Bonus: We can test it in UT with catch

[sonar] Implicit casts should not lower precision.

Use type aid_t for actor ids.

Factor code for simcall observers with a result.

Cosmetics: make parameter name match with .hpp.

Factor Activity::wait() and wait_for() through CRTP.

Factor Activity::cancel() through CRTP.

Comm::test() is ready to be merged with Activity::test().

Define overridable Activity::complete() to be called on activity completion.

This adds some assignments of State::FINISHED, but it looks like they were
previously forgotten.  No test is broken.

Fix TorusZone with limiter links

Raises an std::out_of_range exception when it tries to add the limiter
link to routes.

Example:
    <cluster id="bob_cluster" topology="TORUS" topo_parameters="3,2,2"
     prefix="node-" radical="0-11" suffix=".simgrid.org"
     speed="1Gf" bw="125MBps" lat="50us"
     loopback_bw="100MBps" loopback_lat="0" limiter_link="100MBps"/>

Not sure if someone uses this limiter. We don't have any test that uses limiter_link tag inside a
TORUS cluster. It'll be added in with the new C++ interface.

Reduce scope for variable.

Global--.

Another bunch of int -> size_t conversion.

aid_t for src and dst in smpi::Request.

Convert some int -> size_t.

Prefer emplace to insert.

For Sonar.

More aid_t.

Avoid alternative keywords for boolean expressions.

Plus slight simplifications.

Reference for std::vector parameters.

Fix bound checking.

Combine identical conditions.

Use std algorithm.

Useless indirection.

Sonar const's...

Convert int -> size_t for TRACE_smpi_send (and below).

More int -> aid_t.

Use type aid_t for process id. in instr_smpi.

Adapt leak example to new display.
Add another leak and a size change between processes to see what it shows.

Group buffer leaks by origin as well.
Keep total size and min/max for each if possible, then display by largest total size.

Plug memory leak.

Const, once again.

Doc improvement [ci-skip]

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

Finally rename smpi::Group::actor_pid() back to actor().

Kill smpi::Group::rank(s4u::Actor*).

Kill smpi::Group::actor(int).

Change smpi::Group::set_mapping to take a pid.

Add assert's to ensure consistency between the two kinds of mapping.

Define new rank_to_pid_map_.

Lookup for parent process (like in Group::rank(s4u::Actor*).

In smpi::Group, 'index' is in fact a PID. Fix type and rename.

Kill a TODO (-1 is correct for an invalid PID).

MPI_SOURCE is a rank; src_traced should be a PID.

Fix comment.

Pass large parameter by const reference.

Minor cleanups (useless stuff).

also requalify mc version of the tesh

Group leak reports by type/file/line (for handles, will see for buffer later).
And split reporting.

Sonar polishing.

Pointer-to-const parameters in smpi::Group.

Reduce code duplication (and fix bug in Group::difference).

A for-loop with 'num' modified in the body is too complex according to Sonar.

Fix smpi::Group::intersection().

Using 'size2' for controlling the first loop was a bug (spotted by Sonar).

Fix Group::group_union().

Included ranks from group2 were not good.

Simplify smpi::Group::difference().

Review smpi::Group::*excl() and smpi::Group::*incl().

Mostly cosmetic changes.

Kill redundant member 'size_' from smpi::Group.