For File, we can change the default error handler by specifying MPI_FILE_NULL as the fh argument

Unlike errors on communicators and windows, the default behavior for files is to have MPI_ERRORS_RETURN. ( End of advice to users.)
https://www.mpi-forum.org/docs/mpi-3.1/mpi31-report/node223.htm#Node223
because why not.

These tests really need zero buffering

switch to infty buffering for now as some tests are broken in zero-buffering mode

update changelog

This particular RMA test is filled with stupid calls... We send errors for most of them.
So let's put that to rest by setting MPI_ERRORS_RETURN for this one.

fix request and coll failing tests

topo was a bit too eager to return errors.

Don't return an error when a key is not found in an MPI_Info.

our coll-* tests include bad calls for coverage, they need MPI_ERRORS_RETURN

set default error handler to MPI_ERRORS_ARE_FATAL, as in a proper MPI implementation

disalign tags for collectives from their nonblocking counterparts, to correctly deadlock when both are entangled
as some blocking ones used internally are actually implement with nonblocking+wait, use the right blocking tag in this case.

add tests for errhandlers
Most of them actually need mpi_add_error features, which are not really implemented yet

Add support for MPI Errhandlers in Comm, File, Win.
Default remains MPI_ERRORS_RETURN for now, pending test fixes

Apply the default settings of 'smpi/buffering' too

Previously, we did obey to that option when given, but the default
value was ignored. This is because the handling was done in the value
verification callback, that is not used for the default value.

more informative error message when checking params of MPI_Iallgather

appveyor: don't build python as it was not tested anyway, and don't build anymore

MPI_Scatterv: sendcounts and displs params can be NULL on non-root ranks

more informative error messages on parameter checks of MPI_Iscatterv

This should be done for each and every collective, but not today...

Fix tesh for the new mc-sendsend test

- Test for all factories but thread (that is borken with MC)
- Don't run this test when Java, no matter what. Not sure how I came
  to such a stupid idea :)

fix make dist and python dist

add a test of MC detecting blocking send/send patterns

New MC option: smpi/buffering, to control MPI buffering

smpirun: add a -quiet argument, allowing failing tests in tesh

more informative message when setting inexistant config items

docs: fix borken links

PMPI_Cart_create: check that each dim is positive

Have SMPI fail on MPI_ERR_* in MC mode

The standard says that upon error, implementations should call the
current MPI error handler, which is MPI_ERRORS_ARE_FATAL by default
but could be changed to MPI_ERRORS_RETURN on need.

Since we don't implement MPI_Comm_set_errhandler() to switch between
modes, the simulation mode of SMPI is only issuing a warning on
errors (which is similar to MPI_ERRORS_RETURN).

This commit adds a MC_assert() stating that every MPI call succeed.
This will lead to a property failure (visible only in MC mode) when a
MPI_ERR_* is issued by the implementation (which is somehow similar to
MPI_ERRORS_ARE_FATAL).

fix non-MC builds

use assignment to non-trivial class rather than artificial trivialization and memset

plus some other small cleanups

Restore triviality of s_smx_simcall to please GCC

Field initialization make this struct non-trivial, making GCC to panic
when we memset it. Even if we initialize some fields to nullptr and
memset it to 0.

fix java build

small comments improvements around a complex code

now, kernel::actor::simcall_blocking can return a value

Let simcall.py produce valid code after recent renamings

code simplification + replace a FIXME with an assert

small simplifications around simcalls

- Use ActivityImpl::register_simcall() where possible
- Uniformity in ActivityImpl::post() methods
- Rename processes into actors

small logic simplification

That function is a bit long, but that's not a reason to test twice
whether we should kill the issuer because it runs on a dead host.

Introduce a class mc::SimcallInspector, that allows MC to learn about the ongoing simcalls

Not quite used yet.

MC: rename processes to actors

obey our coding standards, and snake_case some parts of MC

start to make generic simcalls observable from the MC

WIP, done in the easy parts for now.

Not finished in MC_state_get_request_for_process() yet. That's too bad
because this function is the core of the use of simcalls by the MC,
where the next simcall is picked.

Not done in the independence computing part either.

transparent cleanups around simcalls mechanism

move the simcall template to the kernel::actor namespace

protect accesses to cnst->cnst_light_

Port simcall_process_suspend to the modernity

Also simplify its logic by moving the s4u parts within the kernel

getters don't have to be simcalls

convert simcall_process_sleep to modernity

reduce the use of simcall_process_sleep() -> this_actor::sleep_for()

port a blocking simcall to the modernity

small simplification

introduce a simcall_blocking(), and improve the comments

Fix GCC+MC builds

convert SIMIX_simcall_answer() into ActorImpl::simcall_answer()

change SIMIX_simcall_handle() into ActorImpl::simcall_handle()

another assert to make one segfault more explicit

namespacify a global function of SIMIX

shy attempt at simplifying the simcall mechanism

forcefully kill exiting actors even if their host is not off

Without this, this_actor::exit() deadlocks on Thread factory because
the actor does not exit soon enough to release maestro that is joining
its thread.

improve debug msg

run Actor::on_destruction even if the actor was killed before starting, but not on maestro

It seems that the actors are sometimes killed after the simix_global
destruction... Maybe when the user code keeps a reference on them.

New signal: Actor::on_termination (when its code terminates)

activity-lifecycle: make tests shorter to ensure that they are really done when we test them so

Actor: make the refcount observable, and improve debug messages

smpicxx: obey the VERBOSE environment variable and display what we do

Actor::by_pid: also search through the dead actors

Finally, my trick works with it. MPI ranks are not garbage collected
as soon as they end because the MPI instance keeps a reference on
them. With this, by_pid() works properly, even on dead but not
collected actors.

But this only works until the trash is emptied, obviously. This seem
to be enough for the test I wanted to get running, so I will not fix
that point tonight.

ActorImpl: postpone the on_destroy signal to the destructor

This is to prevent that the SMPI extension gets destroyed too early.
The problem is also that this extension is not really an extension but
manually handled, but I'll not fix that tonight.

Restore the -no-privatize to the ampi example

fix ampi example by not cleaning the tracing submodule before the actor end

add a dependency of this test to mpi, as required on jenkins

SMPI: redesign the end of actors/ranks' lifetime

The problem is that we don't use enough of refcounting in SMPI, so we
should not let any rank finish before the others, because it may be
involved in a communication or something.

Previously, there were a barrier at the end of the user code, so that
every ranks finishes exactly at the same time.

Now, the MPI instance keeps a reference on every actor it contains,
and the actor terminates with no delay after its code. The terminating
actors unregister from their MPI instance, but they are still
referenced until the last actor unregisters from the MPI instance.
Once the MPI instance is empty, it unregisters all the actors,
allowing their collection by the refcounting.

This commit changes the ending time of many ranks in many examples, as
expected. The ranks now terminate as soon as they are done, they are
not waiting the others anymore.

It introduces a segfault in ampi that I fail to understand. It seems
that a container is used after being collected in this example, but I
fail to see the reason so far.

sort out the functions on MPI init/fini

small simplification of MPI initialization

MPI: we don't mess with argc/argv anymore nowadays

Previously, the rank and instance were added in argv, mandating a
specific handling of MPI_Init parameters. But now, they are passed as
properties, and the argv is left unmodified. So there is no need to
deal specifically with the MPI_Init parameters.

smpi_deployment: obey our coding rules

I'd like to turn process_data into a regular extension of Actor

SMPI: prefer xbt_assert to 'if () xbt_die'

MPI init: inline a function and various small cleanups

smpi::ActorExt: this constructor parameter is not used

MPI init: inline a function

SMPI init: stringify and rename a variable

kill smpi_process_count(), use smpi_get_universe_size() instead

process_count was probably the original name while universe_size was
added to implement MPI_Attr_get(MPI_UNIVERSE_SIZE). Merge both to sort
things out.

While I'm at it, move all of it to smpi_deployment to reduce the
amount of globals made visible to more than one module.

cmake: build smpi internals before the collectives

My changes break more often the internals than the collectives (that
are rarely modified)

MC: kill a useless function

smpi: some useless cleanups while I read this code

avoid mixing travis builders

docs: fix some typos reported by Gene

typo...

merge before pushing, next time

really build on mojave

try to fix broken detection of boost_stacktrace on mojave. Hope it does not break cold cmakes

add color for travis and appveyor builds

typo

add some comments on tests we don't pass

Partially revert 2b4dcee56 and activate a test.
mpi_type_create_x actually handle MPI_Aint, only mpi_type_x need conversion

activate a fortran test

standard says : "The names attached to opaque objects do not propagate when the object is duplicated or copied by MPI routines. "

appveyor's back

Make SG_BARRIER_SERIAL_THREAD visible from C files too

Fix https://framagit.org/simgrid/simgrid/issues/34 again, hopefully
properly this time.

safety: give some basic logging at verbose level already