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.

Minor changes in smpi::Group.

Remove unused option from example.

Simplify loop searching for a bypass route.

Prefer a do-while in this case, and rewrite without 'break'.

Use std::find_if when looking for a matching comm.

Replace C-style arrays and remove const_cast.

Define local class EngineWrapper to encapsulate common code.

Use const& for std::string parameters.

Pointer-to-const.

Fix build with older clang.

please clang take 2

please clang

Some code improvements.

Avoid unnecessary map lookups.

Thanks @agiersch for the review

add wrappers with scalars on the user side

simplification

cosmetics

cosmetics

s4u::NetZone::create_router.

Export create_router to s4u so we can use in C++ platforms.

Please sonar, just a little

Use rule-of-three, and hope to please sonar without breaking clang13 builds.

Revert "Delete the implicitely defined move constructor."

This reverts commit c26d236dedf486357690800a0d5527871ef45620.

Clang 13 fails with "The specified type does not meet the requirements of
Cpp17MoveInsertable"

AFAICT, the std::deque makes a noexcept constructor impossible.

Finally provide a noexcept move constructor.

Previous commit (f7cf88e9b8 Delete the implicitely defined move constructor.)
was causing clang 13 to fail with: "The specified type does not meet the
requirements of Cpp17MoveInsertable".

Use emplace_back to add a new property_set.

Sonar S6003.

Minor Sonar smells (const, etc).

Move a \n.

Delete the implicitely defined move constructor.

Fix for sonar issue S5018: Move and swap operations should be "noexcept"

Disable copy for actor::ProcessArg.

Also fix sonar issue S5018: Move and swap operations should be "noexcept"

Delete the implicitely defined move constructor.

Fix for sonar issue S5018: Move and swap operations should be "noexcept"

Anyway, AFAICT, the implicit move constructor here does nothing more than the copy-constructor.

Sonar requires a comment for empty functions.

Simplify add_route methods.

Leave only one add_route method, as it was initially.
Deprecate old add_route which uses LinkImpl*, s4u is the interface we want
with user.

Vivaldi on top of StarZone

Implement Vivaldi on top of StarZone.
This allows the easy usage of Vivaldi on C++ platforms (no need to
access specific methods from Vivaldi NetZone.

Platform creation from XML is unchanged.

Adjust add_route to accept route between netzones

StarZone: implement get_graph method

Add get_graph for StarZone. A top node named with the zone's name is
created to be the center of the star.

New netzone: Star Zone

Implements a netzone following a star topology.
Components inside the Star topology are connected by default (through no
links), users can add routes using the NetZone::add_route method:
- netpoint -> nullptr(*): links used in all outgoing communications
- nullptr(*) -> netpoint: links used in all incoming communication
- netpoint -> netpoint: loopback

Duplicated links in a route inside the Star Zone are removed.

Added UTs for StarZone, some tests are disabled since they check the
xbt_assert and would stop the test execution.

[ci-skip] Update lua tesh files with deprecation warning.

constify