NetZoneImpl: simpler constructor

NetZoneImpl(std::string& name);
One step towards a simple contructor and set() idiom.
Removed father and network model from constructor.

Collateral effect:

NetPoint:
- removed NetZoneImpl* from its constructor
- added method set_englobing_zone(NetZoneImpl*)

In the past, we could create the NetPoint in NetZoneImpl with the
pointer to the father. But now, the father is only set latter at
set_father method.

NetZoneImpl: rework seal

Create a do_seal that can be rewritten by derived classes

Assure that flag sealed_ is sealed properly when derived classes
override the seal() method.

Merge pull request #374 from adegomme/master

try to handle inconsistencies in github environments.

sometimes, it's still there...

[sonar] Move explaining comment inside body of empty function.

[sonar] Use '=default'.

[sonar] Declare functions 'const'.

[sonar] Use :: to call static member functions.

[sonar] Pointer to const.

[sonar] Favor using-declarations.

Merge pull request #373 from adegomme/master

fix jar creation github action, and add weekly build

add weekly build at 6:00pm sundays

try to put back boost on windows

https://github.com/actions/virtual-environments/issues/2667

temporary attempt to workaround scoop bug

https://github.com/ScoopInstall er/Main/issues/1752

Remove useless default destructor definitions

Two small sonar fixes

- dead stores
- default destructor should not even be written

Document direct communications, deprecate Host::sendto() because Comm::sendto() is better for that

implement MPI_File_set_size

Merge branch 's4u/model_list' into 'master'

s4u Engine: expose get_all_models, get_model_list

See merge request simgrid/simgrid!53

Merge branch 'unify_models' into 'master'

Unify treatment of models in the EngineImpl

See merge request simgrid/simgrid!52

Uniform treatment in EngineImpl for all models.

Override the update_action_state function in CPUL07 and LinkL07, they
are not necessary since the actions are updated by the HostL07Model, but
they allow to make the same treatment for all models.

Improv comment

Const for some get functions

s4u Engine: expose get_all_models, get_model_list

Before e22da6010c6499813ff88c76041cf499ffbf2b67 the list of models
could be accessed using the global `all_existing_models`

This patch exposes two methods to get the list of models in the
s4u::Engine, they wrap those of `EngineImpl`.

Sonar prefers nullptr.

Cosmetics around #include.

More error messages.

Prefer xbt_assert here.

Update .mailmap.

Merge oberver classes MutexTrylockSimcall and MutexLockSimcall.

Modernize simcall mutex_lock.

Move check from checker side to app. side.

Define (and use) a callback for RawImpl::finish.

Make scan-build ignore volontary null-pointer dereference.

Merge branch 'bruno.donassolo/simgrid-add_model_shared' into next

also test with several cores. Let's see how this goes

Engine*::add_model: from unique to shared_ptr

Merge branch 'hotfix_add_model_api' into 'master'

Hotfix: expose add_model method in s4u::Engine

See merge request simgrid/simgrid!50

Hotfix: expose add_model method in s4u::Engine

Expose the add_model API to external users.

namespacing

A canceled Activity is a completed Activity too

namespacing

be more uniform on action cleaning accross activities

Do the same for IoImpl as for ExecImpl when timeout on completion

user bound is now stored at Action level and not modified by LMM
Then it can be passed from old to new action when migrating the Exec

Allow failure detection for parallel exec
change nothing when hosts has size 1. use std::any_of for multiple hosts

Merge branch 'multi_models_no_globals' into 'master'

Refactoring: remove surf_*_model globals

See merge request simgrid/simgrid!49

Move globals to EngineImpl

A step forwards encapsulation.
Still not good for ptask model and its particularities.

Say goodbye to last global: surf_host_model

- By now, add the host_model in netZone too
- Probably need some check when running parallel_execute in a list of
hosts (all belongs to same model/netzone?)

clang-format: ExecImpl.cpp and sd_task.cpp

surf_disk_model: remove it.

- Add disk_model to NetZone as done for network and cpu models
- Add create_disk method in NetZone too: keep interface uniform

s4u_Host.cpp: clang-format

Adjust code after rebase.

surf_network_model doesn't exist anymore

surf_cpu_model_pm: remove global

Get rid of surf_cpu_model_vm.

- Add cpu_model_vm to NetZoneImpl. By now, it gets the global pointer in
models_by_type if available
- Separate CPU in CPU_PM and CPU_VM.
- Move next_occurring_event call to CPU_VM models to surf_solve. One step
closer to centralizing the models management

NetZoneImpl::network_model_: get() and private object

Getting rid off surf_network_model global

Adapt network model initialization to remove surf_network_model
occurrences.

Remove surf_network_model from host_clm03.

surf_solve concentrates all the next_occurring_event calls.
Get network_model from zone of source host in communications

Remove last entry from surf_solve to surf_network_model.

Improv doc.

Modify iteration over models in surf_solve

Added a map with the different models initialized to iterate in
surf_solve function.

Extra: clang-format

Avoid using surf_network_model global

Get network model from NetZoneImpl instead of using the global.
First step to remove global model variables.

Extra: clang-format modified files.

clang-format before starting changing files

propagate the user bound down to the action

mv NetworkAction::rate_ to Action::user_bound_

cosmetics

extend example to include a ptask with timeout

Split the {TEST,WAIT}ANY translation away from MC_state_choose_request_for_process

Prepare to split the {TEST,WAIT}ANY translation away from MC_state_choose_request_for_process

fix GH#343

- An exec that ends right when the timeout expires is actually done
- when the timeout_detector finished, change the state of the action of
the ExecImpl that owns this detector to FAILED. This will trigger
on_state_change for the right action and solve the issue on energy
logging of this issue.
- Side effect, more instrumentation is displayed in another example of
a ptask with timeout.

trigger Host::on_creation a bit earlier

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

Modernize simcall mutex_trylock.

Fix deprecation message.

Sonar wants one declaration per line.

Don't adjust variable name each time Api::get_maxpid() is called.

Cache result of Api::get_maxpid() when used multiple times.

Minor reindent.

Typo.

Useless braces.

Add 'explicit'.

Remove useless declaration of default destructor.

Remove public destructors defined '=default' at declaration, unless
they are virtual and the enclosing class is derived somewhere.

That was a nasty optimization :-/

new example with ptasks on multi-core hosts

MC dependency: do the easy test before the harder one

Use less of state->internal_req_ and more of state->executed_req_

The idea is to soon kill internal_req and have the dependency checking
do the conversion WAITANY -> WAIT and TESTANY -> TEST on its own.

continue to fluentify Cpu creation. Fix FG37 on the way

MAIS C'ETAIT SUR, EN FAIT ! C'ETAIT SUR !

fix make distcheck, as usual.

MC: Extend the random-bug example to test what's going on when the app is segfaulting

Use default parameter value instead of having an overload just for that

doc: further cosmetics in the TOC and in the pages' URLs

doc: put every info related to models on the same page

The summary contains too many top level entries.

doc: suppress more useless warnings

Doc: Move the blurb about the default TCP model to the right location

move MPI_COMM_WORLD to smpi_comm.cpp.
It's better here.

generalize previous change to other MPI predefined globals.
Except MPI_COMM_WORLD because of its specificities (has to be a variable)

sonar is right here

change the way MPI datatypes are declared.
Our previous solution did not let everything seen as const, and it was preventing some codes to compile (initializer element is not constant).
This allows to compile  https://github.com/open-mpi/mpi-test-suite.git without changint the code (will be added in proxy apps) in tst_types.c
todo : other types (op and co)

debug message --

Use flags to handle cancellation instead of a tri-state int

Change handling of MPI_PROC_NULL as a src or a dest.
We used to just skip the requests, but it was not possible to set status->MPI_SOURCE later in this case.