cosmetics

fix Javadoc
prevented lib_in_jar build

[tesh] kill process -> kill whole process group

If the command launched by tesh executes some subprocesses
(which are not cleaned by the root process on SIGKILL)
the previous code let such subprocesses in a running state.

Now the root process is launched into a new session (man setsid)
and the whole session is killed by tesh. All subprocess should be killed
as long as the session is not changed in the process tree.

Traversing the process tree could also be done, but it may require
additional dependencies (i.e., psutil).

sonar don't like fields nor parameters ending with a _, so don't do it

Plus, inform sonar that SimGrid 3.18 was released since a while

Activate NS3 builds when it's installed through debian

But don't do it on MC builds

do not mix the enums (good catch, sonar)

sonar prefers dupplicated code to missing breaks

And clangs agrees on this one.

finish the s/FULLDUPLEX/SPLITDUPLEX/

s/FULLDUPLEX/SPLITDUPLEX/ in the documentation

hte doc should not speak of AS anymore

s/fullduplex/splitduplex/ in platform parsing (backward compatible, with a warning)

use SPLITDUPLEX internally instead of FULLDUPLEX

We will s/FULLDUPLEX/SPLITDUPLEX/ everywhere in SimGrid. Rational:

That sharing mode is modeled by spliting every link between two links:
one uplink used when traversing the link in a direction, and the
downlink when we traverse the same link in the other direction.

With SHARED links, the bandwidth is shared between all flows without
taking the flow direction into account.

So, SPLITDUPLEX is the way we model fullduplex links, but at the end,
each of the SimGrid link in a SPLITDUPLEX (either the up or down link)
can be used in one direction only. That's the right model, but using
FULLDUPLEX for the situation where you have 2 links was rather
disturbing.

And yes, users will soon notice, when they will try to use network
energy models.

Revert "switch default smpi privatization to mmap, as tsan is now allergic to RTLD_DEEPBIND"

This reverts commit c42ed4092ce78e88b5de1a4623c19d3d9d912d3d.

sonar: don't mask fields with parameters in Java

sonar: dont escape LF in string literals

sonar: don't use auto for pointers, but auto*

don't use deepbind in cases it's not needed (I think)

switch default smpi privatization to mmap, as tsan is now allergic to RTLD_DEEPBIND
This should not work for the -dlopen tests, but it should be better

woops, bash typo

And cleanup before installation

please MC builds, don't install your stuff

try to reduce the amount of warnings in StarPU compilation

[DVFS] Clean up the mess I made before; move to unique_ptr

[DVFS] Move plugin::HostDvfs to plugin::dvfs::HostDvfs

[DVFS] Make debug statement more verbose

[DVFS] Added 'override' specifier

[KERNEL] Simple spelling fix

[PLUGIN] Remove local variable and comment in host_load.cpp

[DVFS] Cosmetics as proposed by clang-format

[DVFS] Make governors individually configurable for each host

Add this to your host tag:
  - <prop id=plugin/dvfs/governor value=performance />

Valid values as of now are: performance, powersave, ondemand, conservative
It doesn't matter if you use uppercase or lowercase.

For the sampling rate, use this:

  - <prop id=plugin/dvfs/sampling_rate value=2 />

This will run the update() method of the specified governor every 2 seconds
on that host.

These properties can also be used within the <config> tag to configure
these values globally. Using them within the <host> will overwrite this
global configuration

[DVFS] Add/change debug statements for dvfs governors

[DVFS] Use local variable for pstate computation

[DVFS] Fix load calculation for Dvfs governors

The HostLoad plugin computes the load as <cores used>/<cores total>.
This means that governors sometimes decide to slow down even though
one or several cores were used because too many cores were unused.

[ENERGY] The energy now depends on the HostLoad plugin.

When a pstate is changed, the energy plugin will update a timestamp for that
host even if that host is never used; this means that we cannot rely on this
to determine if a host has been idle or not. The HostLoad plugin allows
us to do that.

This commit also updates several tesh files as some hosts were reported
as active in the past although they have not computed anything. These
hosts were hence idle and should be reported as such; a machine that
only boots or shuts down is in this case also regarded as unused.

[HostLoad] Load is currently 0 because of integer division. Fix that.

[HostLoad] Ignore VM's ...

[SURF] Move Action::suspended_ to new enum class SuspendedStates

jenkins: install everything after testing

cosmetics

rename two files in examples
Other files have the same name elsewhere, and this confuses coverage tool in sonar

Fix MC builds.

Kill unused function simgrid::xbt::three_way_partition().

Kill unused private function mmalloc_get_increment().

Kill unused private function MC_dwarf_at_linkage_name().

Kill unused private function MC_compare_frame_index_items().

Kill unused function surf_parse_warn().

Kill useless include files.

Kill unused function instr_platform_traced().

Kill unused function get_java_VM().

Kill unused function TRACE_surf_action().

Kill unused private variable surf_action_state_names[].

Function smpi_process_papi_counters() was renamed (untested).

Kill unused private functions instr_{pause,resume}_tracing().

FIXME: TRACE_smpi_sleeping_init() is never called.

Remove unused run-time parameter "tracing/onelink-only".

Kill unused private function SIMIX_process_get_by_name().

Kill unused private function SIMIX_process_exception_terminate().

Kill function declarations without definitions.

Kill unused private function MC_snapshot_memcmp().

HostAction was killed a long time ago.

more C++ around ns3

callbacks are actually meant for external users to use

Merge branch 'master' of scm.gforge.inria.fr:/gitroot/simgrid/simgrid

Add virtual destructor.

Remove semicolon (No, Sonar, it's not commented code).

EXIT_SUCCESS/EXIT_FAILURE are standard C defined in stdlib.h.

Please scan-build.

Inform scan-build that (bucket_id + i) cannot wrap around.

Remove unused variable.

[doc] fix link to Sonar dashboard

sonar don't like to have commented code, so hide it a bit

Fix distcheck.

Update ChangeLog.

Remove now unused smpi::Process::return_value.

Use explicit extern "C" in cpp file.

Make constructors explicit.

Parameter is not used.

Fix build with MC (simgrid::xbt::string).

fix build of dvfs plugin with warning enabled

Rescue cleanups lost with commit 8914cdf67bb2cdd7b64e6eeef5b06c29b24c3c96.

[DVFS] Cosmetics / indent as suggested by clang-format

[DVFS] Remove MSG from DVFS plugin and make it more flexible

The option plugin/dvfs/governor is now available and can be set on a per-host basis

[DVFS] Rephrase debug statements

[PLUGINS] Initial commit for the Dvfs plugin (frequency governors)

[SMPI] Fix issues with daemons by using isDaemon()

[S4U API] Expose the isDaemon() function to the API

[SMPI] Clarify in debug message that SMPI processes are meant, not SIMIX.

[CHANGELOG] Added brief description on the privatization changes

Cosmetics: get rid of stringstream.

Fix comment.

Disable unused copy constructors (and please cppcheck).

Correctly get the exit status for smpi processes.

Search in map only once.

Kill dead code.

process_data is empty a this point.

Revert "Revert "actually fix the memleak around smpi's process_data" and "try to plug a memleak""

This reverts commit c7c5def2870e604025be03b866967c0c5e293afe.

Revert "Finally fix memleak."

This reverts commit aad6442f8fd7d7610f49271010cb8a650f4772de.

Finally fix memleak.

Revert "actually fix the memleak around smpi's process_data" and "try to plug a memleak"

This reverts commit 3c40b1bfc4f4925fa401d3b5d0c74a4b4824b3d5
and commit d53a2108a1ea38b2f952abb48c39730ccc91c2a8.

inline a oneliner

plug another memleak

actually fix the memleak around smpi's process_data

- Call Actor::onDestruction from the right location
- Have SMPI only destroy the process data of its own processes (not of MSG ones)
- Make sure that the process created with MSG_process_attach() have
  the right refcount since we now use a smart pointer on it: we don't
  want it to get freed too early.