X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/16673e04b393c2d8c031d0d77ec318ac2c711f4e..7a955948edf1a0c37594d87693315dfc2dac9a8f:/src/simix/ActorImpl.cpp diff --git a/src/simix/ActorImpl.cpp b/src/simix/ActorImpl.cpp index 9bf3f1577d..2e2065bd8c 100644 --- a/src/simix/ActorImpl.cpp +++ b/src/simix/ActorImpl.cpp @@ -56,49 +56,8 @@ void SIMIX_process_cleanup(smx_actor_t process) XBT_DEBUG("Cleanup process %s (%p), waiting synchro %p", process->get_cname(), process, process->waiting_synchro.get()); - process->finished_ = true; - SIMIX_process_on_exit_runall(process); - - /* Unregister from the kill timer if any */ - if (process->kill_timer != nullptr) { - SIMIX_timer_remove(process->kill_timer); - process->kill_timer = nullptr; - } - simix_global->mutex.lock(); - /* cancel non-blocking communications */ - while (not process->comms.empty()) { - smx_activity_t synchro = process->comms.front(); - process->comms.pop_front(); - simgrid::kernel::activity::CommImplPtr comm = - boost::static_pointer_cast(synchro); - - /* make sure no one will finish the comm after this process is destroyed, - * because src_proc or dst_proc would be an invalid pointer */ - - if (comm->src_proc == process) { - XBT_DEBUG("Found an unfinished send comm %p (detached = %d), state %d, src = %p, dst = %p", comm.get(), - comm->detached, (int)comm->state_, comm->src_proc, comm->dst_proc); - comm->src_proc = nullptr; - - } else if (comm->dst_proc == process) { - XBT_DEBUG("Found an unfinished recv comm %p, state %d, src = %p, dst = %p", comm.get(), (int)comm->state_, - comm->src_proc, comm->dst_proc); - comm->dst_proc = nullptr; - - if (comm->detached && comm->src_proc != nullptr) { - /* the comm will be freed right now, remove it from the sender */ - comm->src_proc->comms.remove(comm); - } - } else { - xbt_die("Communication synchro %p is in my list but I'm not the sender nor the receiver", synchro.get()); - } - comm->cancel(); - } - - XBT_DEBUG("%s@%s(%ld) should not run anymore", process->get_cname(), process->iface()->get_host()->get_cname(), - process->pid_); simix_global->process_list.erase(process->pid_); if (process->host_ && process->host_process_list_hook.is_linked()) simgrid::xbt::intrusive_erase(process->host_->pimpl_->process_list_, *process); @@ -132,7 +91,6 @@ void SIMIX_process_empty_trash() } namespace simgrid { - namespace kernel { namespace actor { @@ -474,7 +432,7 @@ void SIMIX_process_kill(smx_actor_t actor, smx_actor_t issuer) // Forcefully kill the actor if its host is turned off. Not an HostFailureException because you should not survive that if (actor->host_->is_off()) - actor->throw_exception(std::make_exception_ptr(simgrid::kernel::context::Context::StopRequest("host failed"))); + actor->throw_exception(std::make_exception_ptr(simgrid::kernel::context::StopRequest("host failed"))); /* destroy the blocking synchro if any */ if (actor->waiting_synchro != nullptr) { @@ -721,7 +679,7 @@ void SIMIX_process_yield(smx_actor_t self) if (self->context_->iwannadie) { XBT_DEBUG("Process %s@%s is dead", self->get_cname(), self->host_->get_cname()); - // throw simgrid::kernel::context::Context::StopRequest(); Does not seem to properly kill the actor + // throw simgrid::kernel::context::StopRequest(); Does not seem to properly kill the actor self->context_->stop(); THROW_IMPOSSIBLE; } @@ -758,17 +716,6 @@ smx_actor_t SIMIX_process_from_PID(aid_t PID) return actor == simix_global->process_list.end() ? nullptr : actor->second; } -void SIMIX_process_on_exit_runall(smx_actor_t actor) -{ - simgrid::s4u::Actor::on_destruction(actor->iface()); - smx_process_exit_status_t exit_status = (actor->context_->iwannadie) ? SMX_EXIT_FAILURE : SMX_EXIT_SUCCESS; - while (not actor->on_exit.empty()) { - s_smx_process_exit_fun_t exit_fun = actor->on_exit.back(); - actor->on_exit.pop_back(); - (exit_fun.fun)(exit_status, exit_fun.arg); - } -} - void SIMIX_process_on_exit(smx_actor_t actor, int_f_pvoid_pvoid_t fun, void* data) { SIMIX_process_on_exit(actor, [fun](int a, void* b) { fun((void*)(intptr_t)a, b); }, data);