Also cleanup the fields of s_sg_platf_link_cbarg_t

This one is not just a renaming:
 - kill the unused V_policy_initial_link
 - make V_link_sharing_policy (now 'policy') a
   e_surf_link_sharing_policy_t (which thus become public)
This should be enough for field renaming for now. But the cleanup is
definitly not finished :-/

also cleanup the field names of s_sg_platf_router_cbarg_t

Still manual, so atomic commits to avoid issues

Parser cleanup: simplify structure's fields' names

Damn, my eclipse fail that refactoring! That's manual, pity me...

Reimplement the lua console on top of sg_platf

I admit it was too early to do that change since sg_platf is not quite
fully separated from the XML parsing yet, but the cruft accumulates
from everywhere, so I have to break stuff a bit to fix the overall
picture. Yeah, the lua tests are not fixed yet.

kill more empty function chaining in lua console

Slight cleanups in the lua console

- kill some empty functions chaining
- allow for the creation of routers from lua
(the console is still broken, more profund cleanups will be needed)

Kill some further empty function chaining.

This breaks the lua console for a while, but this is to be redone with
the new sg_platf anyway.

Properly integrate the link creation into sg_platf

This structure was not necessary

Lua: don't garbage collect a task after sending it (the receiver will)

Also clean the code of Lua tasks. Show an appropriate error message when
the sender tries to access a task already sent to someone else.

Lua: add a function sglua_copy_value

Add comments about who should or shouldn't call MSG_task_destroy

After a send, the sender should never access the task anymore.

Since processes are in different Lua worlds, the receiver can garbage
collect the task.

further parser cleanups

Properly integrate the begin and end of platform definition in sg_platf.

New public functions:
 - sg_platf_open()
 - sg_platf_close()
New internal callback:
 - sg_platf_postparse_add_cb(void_f_void_t)
 (nobody cares about sg_platf_open(), it's just here for cosmetics in
  user code)

also deprecate ETag_surfxml_router

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

Further parser cleanups

- Kill useless function chaining
- Deprecate the ETag_host in XML now that it's useless

Don't use a fix value of host.

Add a source file for model vivaldi. Write vivaldi_get_route.
Update vivaldi platforms (examples/platforms/syscoord/

Remove some bprintf, fix memleaks.

fix the last broken tests by sanitizing the way properties are handled during the parse

Resolved merge "merge changes. Strange, it seems to ate pierre commit" and
 redo the missing part. Set the struct for get_route of tag cluster.

Remove unused function.

Further parser cleanups

Simply rename a bunch of symbols to make the module logic clear

integrate properly the router creation in sg_platf module

plug memleaks: surfxml does not allocate memory, sg_platf (and callbacks) do it on need

Make the sg_plat_new_host() function public

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

Continue parser cleanups

Prepare to split the interface of sg_platf in two: one public
interface (include/simgrid/platf.h) will contain the function
available to the user code to create elements while another interface
(src/include/simgrid/platf_interface.h) will contain what internal
modules need to declare callbacks on the element creation events.

For now, the files are still very empty, but I try to not break stuff
even further, so I commit soon.

The parser's behavior has changed, revalidate the output of msg-trace

Add some _XBT_GNUC_UNUSED to avoid compilation warnings with NDEBUG

This include is not needed anymore.

Remove useless code obfuscation.

Fix handling of the backtrace log formatters %b and %B.

The first backtrace entry (_xbt_log_event_log) is not displayed.

Cosmetics: fix comment.

A message changed its category.

Improve xbt_log_layout_format_doit().

Make it simpler and faster.

Not sure about the %b formatter, but who cares?

Improve xbt_log_layout_simple_doit().

Make it simpler and slightly faster.

Avoid (almost) duplicated code in xbt_log_layout_*.

Remove the dynamic log layout function.

The layout functions now try to build the log string in a buffer of given
length, and return success or failure.

The first try is with a buffer of fixed size.
In case of failure, the buffer is expanded, until it is large enough.

With this change, the code is much simpler, and the only penalty is with
very large log strings.

Use fputs() instead of fprintf("%s") in file log appender.

It is more natural, and should be faster.

Remove old debugging cruft.

Rationale: xbt_backtrace_display_current() uses fprintf(stderr) instead
of the logging facility.

Fix the fixme.

And don't short-circuit the logging facility.

using pointer to scalar is a strange way to return a value...

Nobody will ever need to override the behavior of config parsing, so simplify the code

Add a new module allowing to interact with the platform directly

* For now, it's super sparse, but the ongoing cleanup in the parser aims
  precisely at populating it.
* Also, kill some oneline brain-dead functions in the lua console

Call surf_parse_models_setup() from the right level

This function is in charge of initializing the models after all
configs have been parsed. It used to be called in the XML parser
callback, it is now called from the platform creation API, which is
intended to be called either from the XML parser or from the lua
console, or directly from the user code.

Register the generic <AS> callback soon enough

- It is generic because it calls routing_AS_init() with the arguments
  from AX_*
- soon enough is right at the beginning of the parsing process, not
  deep after the models are inited

fixing lua console (brain dead) damages to the parser

this function should definitely not be called from lua, but from within the parsing logic

Clean callbacks for the host parsing.

- do not rely on global field, but pass the parameter to the callbacks
- do not show the dynar containing the callbacks to the public, but
  provide a specific function to add a callback not needing to see the
  dynar

This breaks some tests, but I need to split my changes to see when I
improve the situation, and revert stuff when I degrade the situation.
I start to fix the tests right away.

putain, ca compile pas cette merde

merge changes. Strange, it seems to ate pierre commit

yet another psychotic cleanup

regenerate with flexml 1.9.2 (fix to my recent changes by ... agier ;)

Create a structure to put link up and down for model cluster. Factorize the tag cluster.

normalize some type names

remove useless empty parsing callbacks

kill a bunch of warnings that prevented me from compiling

Without ! is better sorry...

Add V_cluster_availability_file and V_cluster_state_file to struct for cluster.

Do the free into the right file.

Get rid of uselessly duplicated code.

no need to reset a struct that will be freed

Replace stupid char* by doubles in the cluster struct to avoid tons of useless printf/scanf.

Add new model for cluster tag.

ensure that the create_resource functions do call strdup by themselves

Also, ensure that the CPU models use surf_resource_new() instead of
dupplicating its code

useless cosmetics

damn. Make sure that the models are inited only once when parsing the same file twice, while ensuring that it still work when parsing after a SD_exit()/SD_init() reset (we have vicious test cases, of which 10 are still broken)

this output changed back, so revert my change of today

fix a bug that I recently introduced that prevented the parser to reload a new platf after a  new SD_exit() / SD_init() cycle

use the right flag to get the pthread compilation schema

Fix warnings about clobbered variables in gras/rpc example.

Fix warnings about clobbered variables in gras/pmm example.

Fix warnings about clobbered variables in gras/ping example.

Fix warnings about clobbered variables in gras/mmrpc example.

Fix warnings about clobbered variables in gras/console/ping example.

Fix warnings about clobbered variables in amok/bandwidth example.

Remove spurious semicolon.

Do not hide useful warnings: unused-but-set-variable.

Mark them as no-error to avoid to break builds.
For example, some versions of valgrind.h cause a "variable unused but set"
warning in gcc 4.6.

Fix path for out-of-source builds.

Enable clobbered variable warnings again and fix the last one (for me)

integrate initialization functions within the parsing main function to clean the calling code

cleanup the way surf_parse_models_setup is called after the last <config> and before the first <AS>-like

split raw parsing functions from platform parsing, which is somehow similar to the deployment parsing done in simix

privatize this function that must be called at some point of the parsing process (thus clearly not for the users)

oops, this should have been in a previous commit (a4f777f30924d7890b9e1da29f9d1a637035531b)

stop messing with the surf internals from the test suite, so that these internals can be modified

Fix the mallocator of SimDag tasks.

Tasks were uninitialized too late (and with duplicated code), in
SD_task_recycle_f() and SD_task_free_f(). The uninitialization code
is now back in SD_task_destroy().
This fixes simdag-minmin-test.

xbt_swag_foreach_safe is actually needed here.

This fixes tests tesh-simdag-basic6 and simdag-minmin-test.

Nobody cares about this argument since maybe 4 years (when we managed to parse the platf in one pass only), so kill the extra complexity

finish to kill a unused function

keep_track should be set only when lmm_solve is called from network_im

Make xbt_swag_insert_at_head/tail uniform and check an error

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

improve the documentation of the parser (getting used to that code before using my haxe)

Optimize swags and save calls from Surf

This constant may be too large for type long

This useless test makes a warning when assertions are disabled

Fix gcc 4.6 warnings about clobbered variables with optimizations

Implement direct context switching also for raw contexts