Merge pull request #124 from mpoquet/master

[Energy platform parsing] assert + memory leak

Merge branch 'master' of https://github.com/mpoquet/simgrid

Energy: Fixed host parsing assertion + mem leak

When parsing the platform file with energy,
using the old watt description syntax
"watt_min:watt_max" instead of
"watt_idle:watt_min:watt_max" led to obscure
OOB error, the error message should now be
clearer.

Also found a little memory leak,
msg_idle was not freed.

fix lua build

move VM into their namespace, and greatly reduce the amount of sg_host_get_name (prefer C++)

inline yet another function in the VM

vm: kill some useless static functions

inline 4 more VM simcalls

I had to add some methods to s4u::VM, such as getRamsize()

VM: improve error messages, and prefer C++ over MSG

VM: keep inlining dummy functions

I could merge my commits, but these changes are really hard to review
so don't make it more complex than needed

VM: inline a function

VM: code simplification. Much more needed :(

give s4u::Host a cname() that returns a char*

kill commented code, anonymous blocs and reindent. No real change

VM: getters should not be simcalls

Removing a simcall changes a bit the output, but that's ok

The VirtualMachineImpl is stored in VirtualMachine->pimpl_vm_, not in Host->pimpl_

store the VMs by their interface, not by their implementation

don't let the impl mess up with the fields of the interface

inline all VM tracing functions

code simplification in the VM tracing code

the VM constructor does not take any storage as parameter anyway

the implementation should not mess with the fields of the interface

VM: move things from MSG to the C++ plugin

code simplification in the VM creation + inline a function

inline another VM function, and kill another one, unused

how many functions will I inline in the VMs????

Kill a VM-related function out of HostImpl

(hopefully) fix the memory issues in the VMs

- Create and populate an host extension, used by the VM plugin to
  check the total amount of memory of that host and whether this host
  allows overcommiting VM above this value.
- This is a great step toward the pluginization of the VMs
- Unfortunately, there is no way to change these values for an host
  yet :-|

don't use deprecated functions, and don't hide the corresponding warnings

avoid using C exceptions as valgrind don't like them too much

more VM functions inlining (plus, getters don't need to be a simcall)

inline 2 more VM functions, and kill a simcall

inline a bunch of VM functions

more verbose checks

cosmetics

test to suspend an exec too

fix copy/paste errors (found by sonar as dupplicated blocks ;)

finalize the traces in SD_exit also

Thanks to Alexey Nazarenko for the bug report and the fix.

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

kill useless simcall handlers

no need to know who's resuming an actor

kill 2 dumb functions in simix_vm

kill a bunch of brain-dead functions in surf

inline a stupid function

don't store the VM implem where the host stores its implem

inline a stupid function

move an enum to the right location

note for next time: try to not forget half of the commit when pushing.

Replace deque by boost:circular_buffer_space_optimized for memory reasons.
Deques are pre-allocated to a specific size and use ~1KB of memory, even when empty.
As we need 2 deques per mailbox, this was quite a lot in the end.

fix dist

kill two more useless functions in VM

kill 2 unused fields in MSG_Global o_O

remove unused functions in the VMs

move virtual machines to the directory src/plugins/vm

move the VM-related data out of MSG's private data for hosts

This is the beginning of turning the VMs into a separate plugin. I'm
sure that reconcentrating all VM code into a few files will greatly
simplify it.

This change is really messy because I did not want to be too intrusive
in msg_vm.c and had to expose much much of the VM internals for that.
It will be improved in the future (it cannot get any worse).

This reveals a little bug where the dirtyPages things of a VM that
runs on a host which turns down are not properly cleaned up. There is
nothing I can do right now as I don't have a onHostOff event or
something, so I simply spit a warning in this case. For now.

rename AsImpl::getRouteRecursive to AsImpl::getGlobalRoute (+doc improvment)

rename As::getRouteAndLatency into As::getLocalRoute

+ many small cleanups

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

various small cosmetics in the routing

reduce the visibility of some functions

reduce the visibility of one field

oops, add the previously documented option to the list.

remove another now useless function from RoutingPlat

I switched to NS 3.26 locally

add missing include (only for osx?)

that was useless.

code simplification: write the recursivity properly

Don't allocate 5KB per host created when it's not needed.
These were for VM and files only, which are optional. Instead they are now allocated when needed.

document new option

unXBTize a bit

cosmetics

let's do standard C++ here

cosmetics

reduce the amount of dynars created by getOneLinkRoutes()

less void*

kill a now useless function

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

empty RoutingPlatf: move the loopback link to the network_model(s)

Add a smpi/grow-injected-times option.
This is to allow triggering off the default behavior of injected timings growth in MPI_Iprobe and MPI_Test.
This behavior allows to simulate faster and avoid looping too much over these calls. But it could also hurt precision of the simulation.

some more tiny optims for the smpirun generation

Add a hint about a non trivial error.
mmap reports no memory left, while the error is actually the number of allowed mappings per process.

cosmetics

get rid of most of the awfully slow cut commands in smpirun.

Those for smpi_replay are still here, though, as the set solution only works one list at a time.
Let's hope this works everywhere.

less strdup/free

also take future releases of NS3

reindent, no change

the property holder does not take a dict in its constructor

don't give the properties to the Link constructor either

don't pass property set to the storage constructor, there is no need

Move some content out of HostImpl (into VirtualMachineImpl)

the AS can now create hosts.

useless cosmetics around the host creation

document my last changes

code simplification: we don't use non-LMM resources

there was a specific constructor for non-LMM resources that was not
given the lmm constraint, which leaded to many dupplicated code.

But actually, this is not needed: NetworkConstant does not create a
link resource anyway, and NS3 can easily pass nullptr to the
constructor of its ancestor.

less useless new and explicit mallocs

New function: MSG_parallel_task_execute_with_timeout

This is the fist time that executions can finish with a timeout, and
I'm not really proud of the resulting code. Sorry, that's just a quick
(fix #115) while I was fixing the platform generation.

At some point the whole activity thingy should be reworked, too.

Make a proper extension out of the Vivaldi coordinates

small cleanups in Java advised by eclipse

normalize a bit the inst callbacks

Spell check, again O:-)