Sort Actor traits alphabetically + cleanups

Introduce ActorIDTrait to split ActorImpl apart

Simplify parameter passing between EngineImpl and ContextFactory

Indeed, it's easier to pass the object as a parameter instead of
having the called getting it afterward through a call back to the
caller.

Move the actor lifecycle markers from Context to ActorImpl

and reduce Context.hpp visibility

make the semaphore monkey more robust to deadlocks and to unanswered simcalls in RAII

document a function

ActorImpl::join: react right away if the other actor is already dead

fflush for all processes.

2nd try to make asan warnings ignored.
rank==0 is not always the last one to print.
[ci-skip]

[tesh] Remove ignored lines *before* trimming the output.

[ci-skip]

Adapt tests to avoid failures when spurious ASan warnings are interlaced.

Blind attempt at getting scoop working again on Azur windows

obey our coding conventions in ActorImpl::mailboxes, and free them once only

Simplify the Context::stop() and reduce duplication

Defuse a trap on refcounting activities by adding a comment

augment example to check sharing of a thread-execute

Make sure that actors killed by a timer are properly finished

ActorImpl::exit() does not schedule the victim for execution because
exit() usually denotes a suicide.

When ran from a timer, it's executed in maestro context, so the victim
needs to be scheduled to be properly freed.

It worked properly before commit 2cb8c87fc8 because the actors were
completely destroyed by Context::stop() which is still used here. Now,
the cleanup is split in 2 parts (in actor context and in maestro
context), that are not directly inter-linked.

Gosh, that was a long debug session for a small patch. I hope it's OK now

Cosmetics in ActorImpl

Merge branch 'add_remaining_comm_sync_bindings' into 'master'

Add remaining Comm synchronisation Python bindings

See merge request simgrid/simgrid!81

Minor improv in threads

Let the user_bound for bounds given by S4U user (if any).

For the requested_cores, the bound is already configured when creating
the action at CpuCas01Action.

Also, the current implementation for threads use the same mechanism used
by VMs, for a N-thread action:
- priority/sharing_penalty: 1/N to increase the amount of CPU given to
action.
- individual bound: speed < N*C: to limit the action speed to the number
of cores.

Comment back the unsuccessful try to use faster matrix solver.

Ideally we would like to use PartialPivLU most of the time. But in some
cases, as tested in the UTs, we need the complete pivoting and the
assurance given by FullPivLU.

Document why it didn't work.
Need another way to check that the solution is invalid that doesn't
include isNaN.

Add unit tests aiming at covering Comm::send and Comm::recv.

Kill dead code.

Unused since there is no future.

Use std::function for smpi_comm_copy_data_callback.

Use std::function for Extension deleters.

Use std::function for Comm callbacks.

easy sonar fixes

cosmetics in the release notes

Better split between the cleanups from self and the ones from the kernel (ie on maestro context)

This shall remove the need for OS synchronization when registering the
actors that should die. So that's hopefully another fix for the problem solved
by be00fecb16c89d44253052889f4815b35ab9abbe

Also, it protects more modifications to global datastructures from
concurrent modifications by doing them in maestro context. The
previous solution was only protecting a very small amount of them.

Add remaining Comm synchronisation Python bindings

Make our valgrind wrapper less verbose, so that it does not break tesh outputs when no error

further simpify the MC initialization on app side

Simplify the parameter passing while initializing the MC on the app side

Seal the platform only once from EngineImpl::run

Fix mingw warning.

redeclared without dllimport attribute: previous dllimport ignored [-Wattributes]

Kill useless target "make maintainer-clean".

It only tries to remove an non-existent file...

Don't modify variable twice in the same expression.

Define initial value, even if it's useless.

Provide constructor with parameters for lmm::Element.

Fix clang builds (unused lambda capture)

Cosmetics in the changelog and release notes

Improve the Actor::set_auto_restart doc by specifying which attributes are saved over reboots

Introduce a Trait to deal with autorestart matter in ActorImpl, and fix bugs

The daemonized behavior was not correctly saved, as noted by the
updated tesh-actor-autorestart test

use get_restart_count from the monkey-masterwork, for simplicity and to test it

new: Actor::get_restart_count(): Returns the number of reboots that this actor did

Fixes https://framagit.org/simgrid/simgrid/-/issues/11
(again)

Update an outdated comment

Rename mc::SafetyChecker to mc::DFSExplorer

Rename mc::CheckerAlgorithm to mc::ExplorationAlgorithm, because that's what it is

Update comment.

Move Simcall from namespace simix to kernel::actor.

Rename example, and update doc.

Configuration option simix/breakpoint was renamed some time ago.

Rename popping.cpp -> simcall.cpp to better reflect its content.

Make Simcall a real class.

Remove reference to dead files.

add an s to example name

Use existing XBT macro to declare enum class Simcall.

Mark all of simix.h as deprecated.

This include file will be empty after that anyway.

BMF: One pass in sonar issues

Missing include.

Define Comm::send and Comm::recv (replace simcall_comm_send/recv).

Mark last legacy simcalls as deprecated.

introduce thread_execute

mark 3 more simcalls as deprecated

Revert "Being optimistic for better performance"

This reverts commit aedc2aa6b82c69adc779ffc4bb8ece31ac423440.

Being optimistic for better performance

If the matrix isn't singular (which is true most of the cases) this
PartialPivLU is much faster than FullPivLU.

Let's be optimistic, what can go wrong? ;)

[jenkins] Avoid build errors on centos.

DISTRIB_ID is apparently now all lowercase on NixOS.

Fix install prefix for Eigen3 on appveyor.

Try to fix detection of eigen3.

blind new attempt for appveyor

Fix build

Remove boost container dependency and fix 32 bits build

new attempt for appveyor

Merge branch 'bmf' into 'master'

New model for parallel tasks: host/model:ptask_BMF

See merge request simgrid/simgrid!80

Guess for appveyor

Add eigen dependency to dockerfiles

Improve ChangeLog

Sonar here too.

* remove global variables
* delete unused copy constructor/assignment operator
* use const reference

Fix bug found at ptask experiments

The consumption of bounded variables exceed constraint capacity leading
to wrong behavior of the algorithm.

Added some UTs for special case with bounds.

ptask_BMF: High-level documentation of BMF and the algorithm

Add eigen in installation instructions
Add doc for ptask_BMF
Changelog

Adding test for subflows

BMF sharing penalty/priority

Fix error find by IO test.

Wrong sharing of saturated (but not selected resources)
UT for it

Remove single movement code

It seems better to be more aggressive to converge faster.

ptask_BMF: support for dynamic bounds (user's callback)

Allows the use of user callback to dynamically change the constraint
capacity.

Add a warning message for user since this approach is quite pessimist
when sharing results and may lead to inaccurate resource sharing

BMF: Fatpipe support

Adjust A' matrix and is_bmf functions accordingly.

Implements selective-update for bmf model (off by default).

Add a cfg parameter: "bmf/selective-update" to enable it

ptask_BMF: refactor code and loop scenarios

New cfg flag: --cfg=bmf/max-iterations:X. Configure the number maximum
of iterations done by BMF solver (default 1000).

Usually the algorithm converges much faster but in some cases it may be
necessary to increase it. However, can impact negatively on execution
time

Refactor code: new BmfSolver class to encapsulate matrix manipulation from system

Add a class to generate all possible allocations if necessary

Fix: default rate (bound) for comms is -1 not 0

Bugfix: ptask should use expand_add for exec tasks

The variable in the system is the same for ptasks, so use expand_add to
reuse the element if it exists in the system. Similar as it's done for
communications

Support for bounded actions in BMF solver

New model for parallel tasks: host/model:ptask_BMF

Implement a new solver for lmm::System based on BMF (Bottleneck max
fairness) objective.

BMF provides a more realistic sharing of heterogeneous resources as used
by parallel tasks.

Enable it using: --cfg=host/model:ptask_BMF instead of ptask_L07.

SimGrid compilation from source now requires a new library: Eigen3.

sonar

- Ignore MBI scripts in the coverage
- unused local variable was indeed a bug

sonar fixes and little cleanups

Use the sg4 namespace in all examples

Revert "Fix a bug in MBI generators: &com[j] is not &(com[j])"

This reverts commit c2d3a91e47e3fdc89e6818258993128fac8efd43.

Keep optimization (only_if_different) when cmake is recent enough.

[ci-skip]

Replace file(copy_file), which is supported since cmake 3.21 only.

[ci-skip]

Remove (last?) reference to umpire test suite.

[ci-skip]

Fix a bug in MBI generators: &com[j] is not &(com[j])

Add another MBI test generator, to ease its debugging

Use RAII to simplify the release of semaphores on exit.

Update .mailmap with new contributor.

Mark unused type aliases as deprecated.