Method is redundant with base class.

Mark methods as "override".

Capitalize constants.

Replace #define with constexpr declarations.

Kill unused constants.

fix (#220) ?

If an actor ends right after a dsend, the Comm should not be canceled
to let the receiver a chance to catch it.

It was removed in Comm::cancel(), now it's not, and the destructor has
to be a bit more careful.

As the comm was canceled, if the receive is posted later, it nevers
find its matching send and may stall forever or timeout.

This changed the output of the chord example (using dsend), but I
suspect it was not so correct before with some dsend being canceled
and the corresponding receive ending on timeout.

This also highlighted an issue in Java. The cleanup function for dsend
is now called at the end of the simulation and get_current_thread_env
crashed (because the current thread is maestro).

Hopefully, this commit does not cause new leaks.

make MSG_process_create die if host is off.
This was requested by adsein long time ago.

fix typos in warning

Change color of "barrier" in trace. It was too close to "computing"

Add simple gemm example for SMPI with sampling macros.
Adapted from the proxy app from Codevault.
This could be a good basis for the tuto

Yet again change the way SMPI_SAMPLE_MACRO works
There is no need anymore to have the body of the loop inside the macro. Kudos to Arnaud L for the ideas.

now usage is just turning
for(x;y;z) into SMPI_SAMPLE_GLOBAL(x,y,z,iters,thres)

Time injection will be done only once for all the section, speeding up the simulation.

TODO : iterator value will not be correct at the end of the loop for now.
Not important if iterator is local to the loop, but may be annoying if not.

fix (#324)
Throw a HostFailureException when one tries to create (or attach) an
actor an a host that is not on.
If this happens during the parsing of the deployment file, just let it
go.
for MSG, we could die if this happens (we don't for now)

Build still broken with ld 2.31.1.  Check only version for gcc.

[sonar] Improve Cognitive Complexity.

NO_MAX_DURATION is a special value, use (in)equality for testing.

Move common test to Action::update_max_duration.

Kill now unused functions SIMIX_mutex_{un}ref.

Convert last call to SIMIX_mutex_unref.

fix last issues
+ counter_ is used by heaps, use id_ instead
+ fix dumb find/replace

end objectification of MSG tasks

mv counter from msg_task_t to simgrid::msg::Task
this was the last field, MSG tasks are now objects
have to remove the useless layer now!

mv data from msg_task_t to simgrid::msg::Task

simplify these tests

mv category from msg_task_t to simgrid::msg::Task

move name from msg_task_t to simgrid::msg::Task

do not promote bad practices: use "accessor" to get task name

objectify simdata_task_t

priority and bound of MSG tasks can only be set BEFORE execution

this is the rule for S4U, should be the same for MSG.
BTW, the behavior of the LMM is likely to be confused by such dynamic
changes during execution.

Side effect: we should remove the setters in ExecImpl, but there are
used by two simcalls that are tagged as deprected with v3.25

source is sender's host. One field is enough

play with (parallel) execs. still not satisfying

SIMIX_mutex_{un}ref become MutexImpl::{un}ref

dont call HANDLER

please sonar

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

make destructor public

Fix build error with mingw.

do not use xbt_mutex_t in SMPI

convert maestro-set to S4U

dumb me.

deprecate SIMIX_process_{a,de}tach
* replaced by ActorImpl::{a,de}tach
* add sg_actor_{a,de}tach to the C API of actors
* MSG_process_{a,de}tach go to msg_legacy

forgot that one

stop using old simcalls in our own code

snake casing

trim instr_msg code

use TASK_CANCELED rather than HOST_FAILURE for CancelExeption

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

Throw exception when comm was canceled.

Handle CancelException in MSG_{comm,test} too.

MSG is a C API and should not throw.

remove doublons caused by extra pop

remove useless code
this is handled by callbacks in instr-platform now

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

use more s4u calls in MSG
One more step to transform MSG into a S4U wrapper \o/

revalidate tesh file

fix some issues with Cancel
* Comm::cancel can be called when there is no pimpl
* if we wait for a canceled activity we can return not let the
impossible happen.

Parameter is a boolean.

Don't strip last component of cwd.

damned picky clang

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

a few simplifications

replace another old simcall in MSG

save a cast

modernize a simcall in MSG

(Hope to) fix number of cores for VMs.

The bug was apparently introduced by commit 5a469c9dca9ce25cee1a52682d7eb7bbd736a49b.

cleanups and uniformization in Activity headers

This is a CommImpl*.

move a function to a righter place

prevent segfault when DEBUG is activated

Change the syntax of SMPI_SAMPLE_* macros.
To handle the end of the computing loop when sampling is still on, we need to wrap it.

This replaces this syntax
for(k = 1; k <= np; k++) { SMPI_SAMPLE_GLOBAL(0.25 * np, 0.03) {
where the macro had no idea of the loop conditions

by this
SMPI_SAMPLE_GLOBAL(k = 1, k <= np, k++, 0.25 * np, 0.03,
 {...}
)
syntax, where the body of loop is itself a parameter. This is not perfect, and may change soon. But now we can exit gracefully the sampling whenever the upper loop exits.

Define and use optimized variants for simcalls comm_testany and comm_waitany.

Specialize parameter for simcall comm_waitany.

Specialize parameter for simcall comm_testany.

Specialize parameter for simcall comm_wait.

Specialize parameter for simcall comm_test.

Specialize parameter for simcall execution_wait.

Specialize parameter for simcall execution_test.

Specialize parameter for simcall io_wait.

Kill now unused mc_xbt.

Fix MC build after changes for waitany.

Change dynar parameter for waitany to a raw array (mimic testany).

Remove superfluous refcounting.

Simcall waitany really takes a dynar of ActivityImpl*.

Use CommImpl* for comm_copy callbacks.

It's in fact already used for match_fun.

Cosmetics: adjust method names in debug messages.

Make cleanupSurf() private.

tiny doc improvement

sonar: point to framagit instead of github

[SMPI] Change sampling behavior.
Before, in sampling mode, each sampled loop was executed once, and the process would inject the time it took immediately.
This lead to a simcall being executed, which would then yield execution and possibly give the hand to another process.
This was destroying the caches, and resulted in over-estimated times.

This modification holds the injected time until the end of the sampling phase, trying to do the most of it on one process, without swapping contextes in between.

TODO :
- check what happens if num sampling iters < num iters/process (mandatory swapping)
Issues might be triggered if process leaves sampling before reaching the given bounds first.

remove doxygen css and py tools scripts from codacy

reduce size of message sent ..
For collectives we have to allocate fake buffers, and these ones may be a bit too large for some small systems (arm)

All activities have their own finish method \o/
The only sparks were of joy ...

SIMIX_io_finish becomes IoImp::finish

get rid of smx_io*
the simcall_handlers will go at the beginning of the
kernel/activity/*Impl.cpp

add a StorageFailureException

_SIMIX_cond_wait becomes ConditionVariable::wait

SIMIX_synchro_wait becomes RawImpl::start

objectify simix timers.
public C API remains for now. Might be deprecated or removed (?)

Avoid some casts and copies of CommImplPtr.

late modification wasn't tested, former was better

SIMIX_process_cleanup becomes ActorImpl::Cleanup
There is no more need to pass the callback on an external function
when creating context.

week-end cleanups in ActorImpl
That excalated quickly ...