rework SleepImpl (and save a cast)

let base class dtor deal with surf action

For CommImpl, it seems that unref() is called too late so keep the
current dtor

What ActivityImpl child should look like IMHO
* ctor does nothing more than create a class instance
* all fields can be set by chainable setters
* start() has no parameter

On the S4U side, we should have the same (with CRTP soon)
* ctor calls the Impl ctor and sets thing at user level
* start() does a simcall in which we call all the Impl setters to
  transfer all field values to the kernel and then call Impl::start()

Maybe we should also replace "new *Impl()" in S4U by a call to a
Impl::create() static method to be written.

cosmetic cleanups in S4U

Activate assert when debug log is active.

Use std::unordered_map + save an indirection.

Directly fill vectors at construction.

Simplification.

Remove unused type definitions.

don't kill the fools sourcing this script

comestics in src/kernel

cleanups in smpi, mostly cosmetic stuff and indent

Complete Asan suppressions.

Ensure field is initialized.

Clang is picky.

error: local variable value will be copied despite being returned by name [-Werror,-Wreturn-std-move]

Save valgrind_stack_id in a private field.

Only register stacks when running on Valgrind to avoid false alarms from Asan.

MailboxPtr looks like a smart pointer, but it's not. Kill it.

Remove unnecessary calls to c_str().

IIUC return std::move(...) is discouraged.

References++.

mv ActorImpl where it belongs

use consistent namespaces
Storage is a resource, it goes in simgrid::kernel::resource

don't use old fashioned simcall when you don't have to

properly split task/comm in msg files

objectify a few msg::Comm functions

mark old MSG functions as deprecated

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

Simplifications

Do not use an ugly global map when each smpi process can store its own
tracing category.

Replace void* when using std::thread.

Deprecated functions are still used by StarPU. Postpone their removal.

Pass std::string parameters by reference too.

Don't use pass-by-value for large parameters.

More references for parameters of type std::function.

Constify.

Bad cast.

Make ActorImpl::context_ a std::unique_ptr.

Make SwappedContext::parmap_ a std::unique_ptr.

Cosmetics around std::unique_ptr.

Fix xbt_assert with side effects.

Equality/assignment mismatch.

Release memory after last use only.

Put XBT_ATTRIB_DEPRECATED at proper place.

2 fixes in host-energy plugin

- When computing the current watts, consistently use the pstate that
  was set at the beginning of the previous period.
  (the pstate at the end of the period was used to determinate whether
  or not we are in boot/shutdown state, ie, whether speed==0)
- Do not mark as used hosts that are just turned on and off. This is
  to be consistent with the tesh files while fixing the other bug.

Specify bindir while launching tesh

Use references for callback parameters of on_exit().

Use references for parameters of type std::function.

Use references for parameters of type std::vector.

Fix mismatched alloc/free.

Remove duplicated but unused forward declarations.

Don't hide class member.

Fix null pointer dereference.

Switch s4u tuto to pajengr as well.

hopefully fix most issues. thx agier for spotting the main one

poor damage control for tonight only

revalidate another tesh

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

revalidate tesh

simplifications and fixes (forgot properties)

adapt smpi tuto to use pajengr (+ dockerfile)

use extension for user data in MSG

Split the behavior of ActorImpl::create
follow the init->set->start logic

Sometimes 'free' is not a member of 'std'.

Add exception specification for xbt_free_f.

Take the same exception specification as for std::free. Without that, Intel icc
fails to compile simgrid::xbt::demangle (backtrace.cpp) with:
error \#878: incompatible exception specifications

Missing include.

Use real pointer type for context wrappers.

[sonar] Use unsigned char* for context stacks.

[sonar] Use std::function.

Deprecate some functions.

Type consistency.

oops

WIP - SMPI tuto - add parts about sampling and memory sharing

TODO :
- collectives, and more ?

Try to help sonar.

Fix mc liveness examples.

Objectify MSG task send

Simplifications in MSG task execution

please sonar

remove or deprecate now useless code

please clang

s4u::Exec now has 2 child classes
s4u::ExecSeq and s4u::ExecPar

side effects:
+ no more C-style host list in HostImpl and ptaskLO7
+ Sequential execution can have a timeout
TODO:
+ get rid of ExecImplPtr in MSG
+ check and clean

Parmap status is simply a boolean.

Use std::function instead of function pointer.

Use proper type instead of void*.

Assert that the pointer is not null when dereferenced.

Field is initialized by constructor.

those pimpl are actually never null, stop fooling sonar

apply agier fixes for StarPU build

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

don't let sonar think that pimpl_ can be null

sonar: pass large parameters as const; don't name unused params

Sanitize the prototype of Actor::on_exit() callbacks

Changes on_exit(function<void(int failed, void*data)>, data)
into    on_exit(function<void(bool failed)>)

So, we are using bool for boolean values, and we don't force the user
to build a closure on void* data manually. If you want to pass a data
to the callback, just use the C++ formalism: [data](bool failed){...}

Other changes on the way (that excalated quickly)
- Document some recent changes to the internals in ChangeLog
- Document this_actor::on_exit() and Actor::on_exit() in doxygen
- Also sanitize the prototype of MSG_process_on_exit(). Was taking a
  (void*,void*) function; now takes a (int, void*) one. This was too
  weird to use void* to denote integers -- I actually doubt it ever
  worked the way it was written, but the tests are too poor to be
  sure: the only usage of this function ignores the return value.
- Also add a new test in activity-lifetime: test_sleep_restart_end
  Commented for now as it breaks miserably.

Move ForcefulKillException to the root namespace, along with the other exceptions

Rename StopRequest into ForcefulKillException to contribute to #325

further convert some bits of the documentation

snake case this bool

cosmetics

more factoring when sending MSG tasks

Add a comment to explain previous changes.

Fix memleak.

Fix use of invalidated iterator.

Use C++11 <random> for simcall mc_random.