Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
factoring with an ActivityImpl::clean_action() method
[simgrid.git] / src / kernel / activity / CommImpl.cpp
1 /* Copyright (c) 2007-2019. The SimGrid Team. All rights reserved.          */
2
3 /* This program is free software; you can redistribute it and/or modify it
4  * under the terms of the license (GNU LGPL) which comes with this package. */
5
6 #include "src/kernel/activity/CommImpl.hpp"
7 #include "simgrid/Exception.hpp"
8 #include "simgrid/kernel/resource/Action.hpp"
9 #include "simgrid/modelchecker.h"
10 #include "simgrid/s4u/Host.hpp"
11 #include "src/kernel/activity/MailboxImpl.hpp"
12 #include "src/kernel/context/Context.hpp"
13 #include "src/mc/mc_replay.hpp"
14 #include "src/surf/cpu_interface.hpp"
15 #include "src/surf/network_interface.hpp"
16 #include "src/surf/surf_interface.hpp"
17
18 #include <boost/range/algorithm.hpp>
19 XBT_LOG_NEW_DEFAULT_SUBCATEGORY(simix_network, simix, "SIMIX network-related synchronization");
20
21 XBT_PRIVATE void simcall_HANDLER_comm_send(smx_simcall_t simcall, smx_actor_t src, smx_mailbox_t mbox, double task_size,
22                                            double rate, void* src_buff, size_t src_buff_size,
23                                            int (*match_fun)(void*, void*, simgrid::kernel::activity::CommImpl*),
24                                            void (*copy_data_fun)(simgrid::kernel::activity::CommImpl*, void*, size_t),
25                                            void* data, double timeout)
26 {
27   smx_activity_t comm = simcall_HANDLER_comm_isend(simcall, src, mbox, task_size, rate, src_buff, src_buff_size,
28                                                    match_fun, nullptr, copy_data_fun, data, 0);
29   SIMCALL_SET_MC_VALUE(simcall, 0);
30   simcall_HANDLER_comm_wait(simcall, static_cast<simgrid::kernel::activity::CommImpl*>(comm.get()), timeout);
31 }
32
33 XBT_PRIVATE smx_activity_t simcall_HANDLER_comm_isend(
34     smx_simcall_t /*simcall*/, smx_actor_t src_proc, smx_mailbox_t mbox, double task_size, double rate, void* src_buff,
35     size_t src_buff_size, int (*match_fun)(void*, void*, simgrid::kernel::activity::CommImpl*),
36     void (*clean_fun)(void*), // used to free the synchro in case of problem after a detached send
37     void (*copy_data_fun)(simgrid::kernel::activity::CommImpl*, void*, size_t), // used to copy data if not default one
38     void* data, bool detached)
39 {
40   XBT_DEBUG("send from mailbox %p", mbox);
41
42   /* Prepare a synchro describing us, so that it gets passed to the user-provided filter of other side */
43   simgrid::kernel::activity::CommImplPtr this_comm =
44       simgrid::kernel::activity::CommImplPtr(new simgrid::kernel::activity::CommImpl());
45   this_comm->set_type(simgrid::kernel::activity::CommImpl::Type::SEND);
46
47   /* Look for communication synchro matching our needs. We also provide a description of
48    * ourself so that the other side also gets a chance of choosing if it wants to match with us.
49    *
50    * If it is not found then push our communication into the rendez-vous point */
51   simgrid::kernel::activity::CommImplPtr other_comm =
52       mbox->find_matching_comm(simgrid::kernel::activity::CommImpl::Type::RECEIVE, match_fun, data, this_comm,
53                                /*done*/ false, /*remove_matching*/ true);
54
55   if (not other_comm) {
56     other_comm = std::move(this_comm);
57
58     if (mbox->permanent_receiver_ != nullptr) {
59       // this mailbox is for small messages, which have to be sent right now
60       other_comm->state_     = SIMIX_READY;
61       other_comm->dst_actor_ = mbox->permanent_receiver_.get();
62       mbox->done_comm_queue_.push_back(other_comm);
63       XBT_DEBUG("pushing a message into the permanent receive list %p, comm %p", mbox, other_comm.get());
64
65     } else {
66       mbox->push(other_comm);
67     }
68   } else {
69     XBT_DEBUG("Receive already pushed");
70
71     other_comm->state_ = SIMIX_READY;
72     other_comm->set_type(simgrid::kernel::activity::CommImpl::Type::READY);
73   }
74
75   if (detached) {
76     other_comm->detached_ = true;
77     other_comm->clean_fun = clean_fun;
78   } else {
79     other_comm->clean_fun = nullptr;
80     src_proc->comms.push_back(other_comm);
81   }
82
83   /* Setup the communication synchro */
84   other_comm->src_actor_     = src_proc;
85   other_comm->src_data_      = data;
86   (*other_comm).set_src_buff(src_buff, src_buff_size).set_size(task_size).set_rate(rate);
87
88   other_comm->match_fun     = match_fun;
89   other_comm->copy_data_fun = copy_data_fun;
90
91   if (MC_is_active() || MC_record_replay_is_active())
92     other_comm->state_ = SIMIX_RUNNING;
93   else
94     other_comm->start();
95
96   return (detached ? nullptr : other_comm);
97 }
98
99 XBT_PRIVATE void simcall_HANDLER_comm_recv(smx_simcall_t simcall, smx_actor_t receiver, smx_mailbox_t mbox,
100                                            void* dst_buff, size_t* dst_buff_size,
101                                            int (*match_fun)(void*, void*, simgrid::kernel::activity::CommImpl*),
102                                            void (*copy_data_fun)(simgrid::kernel::activity::CommImpl*, void*, size_t),
103                                            void* data, double timeout, double rate)
104 {
105   smx_activity_t comm = simcall_HANDLER_comm_irecv(simcall, receiver, mbox, dst_buff, dst_buff_size, match_fun,
106                                                    copy_data_fun, data, rate);
107   SIMCALL_SET_MC_VALUE(simcall, 0);
108   simcall_HANDLER_comm_wait(simcall, static_cast<simgrid::kernel::activity::CommImpl*>(comm.get()), timeout);
109 }
110
111 XBT_PRIVATE smx_activity_t simcall_HANDLER_comm_irecv(
112     smx_simcall_t /*simcall*/, smx_actor_t receiver, smx_mailbox_t mbox, void* dst_buff, size_t* dst_buff_size,
113     simix_match_func_t match_fun, void (*copy_data_fun)(simgrid::kernel::activity::CommImpl*, void*, size_t),
114     void* data, double rate)
115 {
116   simgrid::kernel::activity::CommImplPtr this_synchro =
117       simgrid::kernel::activity::CommImplPtr(new simgrid::kernel::activity::CommImpl());
118   this_synchro->set_type(simgrid::kernel::activity::CommImpl::Type::RECEIVE);
119   XBT_DEBUG("recv from mbox %p. this_synchro=%p", mbox, this_synchro.get());
120
121   simgrid::kernel::activity::CommImplPtr other_comm;
122   // communication already done, get it inside the list of completed comms
123   if (mbox->permanent_receiver_ != nullptr && not mbox->done_comm_queue_.empty()) {
124
125     XBT_DEBUG("We have a comm that has probably already been received, trying to match it, to skip the communication");
126     // find a match in the list of already received comms
127     other_comm = mbox->find_matching_comm(simgrid::kernel::activity::CommImpl::Type::SEND, match_fun, data,
128                                           this_synchro, /*done*/ true,
129                                           /*remove_matching*/ true);
130     // if not found, assume the receiver came first, register it to the mailbox in the classical way
131     if (not other_comm) {
132       XBT_DEBUG("We have messages in the permanent receive list, but not the one we are looking for, pushing request "
133                 "into list");
134       other_comm = std::move(this_synchro);
135       mbox->push(other_comm);
136     } else {
137       if (other_comm->surf_action_ && other_comm->get_remaining() < 1e-12) {
138         XBT_DEBUG("comm %p has been already sent, and is finished, destroy it", other_comm.get());
139         other_comm->state_ = SIMIX_DONE;
140         other_comm->set_type(simgrid::kernel::activity::CommImpl::Type::DONE);
141         other_comm->mbox   = nullptr;
142       }
143     }
144   } else {
145     /* Prepare a comm describing us, so that it gets passed to the user-provided filter of other side */
146
147     /* Look for communication activity matching our needs. We also provide a description of
148      * ourself so that the other side also gets a chance of choosing if it wants to match with us.
149      *
150      * If it is not found then push our communication into the rendez-vous point */
151     other_comm = mbox->find_matching_comm(simgrid::kernel::activity::CommImpl::Type::SEND, match_fun, data,
152                                           this_synchro, /*done*/ false,
153                                           /*remove_matching*/ true);
154
155     if (other_comm == nullptr) {
156       XBT_DEBUG("Receive pushed first (%zu comm enqueued so far)", mbox->comm_queue_.size());
157       other_comm = std::move(this_synchro);
158       mbox->push(other_comm);
159     } else {
160       XBT_DEBUG("Match my %p with the existing %p", this_synchro.get(), other_comm.get());
161
162       other_comm->state_ = SIMIX_READY;
163       other_comm->set_type(simgrid::kernel::activity::CommImpl::Type::READY);
164     }
165     receiver->comms.push_back(other_comm);
166   }
167
168   /* Setup communication synchro */
169   other_comm->dst_actor_     = receiver;
170   other_comm->dst_data_      = data;
171   other_comm->set_dst_buff(dst_buff, dst_buff_size);
172
173   if (rate > -1.0 && (other_comm->get_rate() < 0.0 || rate < other_comm->get_rate()))
174     other_comm->set_rate(rate);
175
176   other_comm->match_fun     = match_fun;
177   other_comm->copy_data_fun = copy_data_fun;
178
179   if (MC_is_active() || MC_record_replay_is_active()) {
180     other_comm->state_ = SIMIX_RUNNING;
181     return other_comm;
182   }
183   other_comm->start();
184   return other_comm;
185 }
186
187 void simcall_HANDLER_comm_wait(smx_simcall_t simcall, simgrid::kernel::activity::CommImpl* comm, double timeout)
188 {
189   /* Associate this simcall to the wait synchro */
190   XBT_DEBUG("simcall_HANDLER_comm_wait, %p", comm);
191
192   comm->simcalls_.push_back(simcall);
193   simcall->issuer->waiting_synchro = comm;
194
195   if (MC_is_active() || MC_record_replay_is_active()) {
196     int idx = SIMCALL_GET_MC_VALUE(simcall);
197     if (idx == 0) {
198       comm->state_ = SIMIX_DONE;
199     } else {
200       /* If we reached this point, the wait simcall must have a timeout */
201       /* Otherwise it shouldn't be enabled and executed by the MC */
202       if (timeout < 0.0)
203         THROW_IMPOSSIBLE;
204
205       if (comm->src_actor_ == simcall->issuer)
206         comm->state_ = SIMIX_SRC_TIMEOUT;
207       else
208         comm->state_ = SIMIX_DST_TIMEOUT;
209     }
210
211     comm->finish();
212     return;
213   }
214
215   /* If the synchro has already finish perform the error handling, */
216   /* otherwise set up a waiting timeout on the right side          */
217   if (comm->state_ != SIMIX_WAITING && comm->state_ != SIMIX_RUNNING) {
218     comm->finish();
219   } else { /* we need a sleep action (even when there is no timeout) to be notified of host failures */
220     simgrid::kernel::resource::Action* sleep = simcall->issuer->get_host()->pimpl_cpu->sleep(timeout);
221     sleep->set_activity(comm);
222
223     if (simcall->issuer == comm->src_actor_)
224       comm->src_timeout_ = sleep;
225     else
226       comm->dst_timeout_ = sleep;
227   }
228 }
229
230 void simcall_HANDLER_comm_test(smx_simcall_t simcall, simgrid::kernel::activity::CommImpl* comm)
231 {
232   int res;
233
234   if (MC_is_active() || MC_record_replay_is_active()) {
235     res = comm->src_actor_ && comm->dst_actor_;
236     if (res)
237       comm->state_ = SIMIX_DONE;
238   } else {
239     res = comm->state_ != SIMIX_WAITING && comm->state_ != SIMIX_RUNNING;
240   }
241
242   simcall_comm_test__set__result(simcall, res);
243   if (simcall_comm_test__get__result(simcall)) {
244     comm->simcalls_.push_back(simcall);
245     comm->finish();
246   } else {
247     SIMIX_simcall_answer(simcall);
248   }
249 }
250
251 void simcall_HANDLER_comm_testany(smx_simcall_t simcall, simgrid::kernel::activity::CommImpl* comms[], size_t count)
252 {
253   // The default result is -1 -- this means, "nothing is ready".
254   // It can be changed below, but only if something matches.
255   simcall_comm_testany__set__result(simcall, -1);
256
257   if (MC_is_active() || MC_record_replay_is_active()) {
258     int idx = SIMCALL_GET_MC_VALUE(simcall);
259     if (idx == -1) {
260       SIMIX_simcall_answer(simcall);
261     } else {
262       simgrid::kernel::activity::CommImpl* comm = comms[idx];
263       simcall_comm_testany__set__result(simcall, idx);
264       comm->simcalls_.push_back(simcall);
265       comm->state_ = SIMIX_DONE;
266       comm->finish();
267     }
268     return;
269   }
270
271   for (std::size_t i = 0; i != count; ++i) {
272     simgrid::kernel::activity::CommImpl* comm = comms[i];
273     if (comm->state_ != SIMIX_WAITING && comm->state_ != SIMIX_RUNNING) {
274       simcall_comm_testany__set__result(simcall, i);
275       comm->simcalls_.push_back(simcall);
276       comm->finish();
277       return;
278     }
279   }
280   SIMIX_simcall_answer(simcall);
281 }
282
283 static void SIMIX_waitany_remove_simcall_from_actions(smx_simcall_t simcall)
284 {
285   simgrid::kernel::activity::CommImpl** comms = simcall_comm_waitany__get__comms(simcall);
286   size_t count                                = simcall_comm_waitany__get__count(simcall);
287
288   for (size_t i = 0; i < count; i++) {
289     // Remove the first occurence of simcall:
290     auto* comm = comms[i];
291     auto j     = boost::range::find(comm->simcalls_, simcall);
292     if (j != comm->simcalls_.end())
293       comm->simcalls_.erase(j);
294   }
295 }
296 void simcall_HANDLER_comm_waitany(smx_simcall_t simcall, simgrid::kernel::activity::CommImpl* comms[], size_t count,
297                                   double timeout)
298 {
299   if (MC_is_active() || MC_record_replay_is_active()) {
300     if (timeout > 0.0)
301       xbt_die("Timeout not implemented for waitany in the model-checker");
302     int idx                 = SIMCALL_GET_MC_VALUE(simcall);
303     auto* comm              = comms[idx];
304     comm->simcalls_.push_back(simcall);
305     simcall_comm_waitany__set__result(simcall, idx);
306     comm->state_ = SIMIX_DONE;
307     comm->finish();
308     return;
309   }
310
311   if (timeout < 0.0) {
312     simcall->timer = NULL;
313   } else {
314     simcall->timer = simgrid::simix::Timer::set(SIMIX_get_clock() + timeout, [simcall]() {
315       SIMIX_waitany_remove_simcall_from_actions(simcall);
316       simcall_comm_waitany__set__result(simcall, -1);
317       SIMIX_simcall_answer(simcall);
318     });
319   }
320
321   for (size_t i = 0; i < count; i++) {
322     /* associate this simcall to the the synchro */
323     auto* comm = comms[i];
324     comm->simcalls_.push_back(simcall);
325
326     /* see if the synchro is already finished */
327     if (comm->state_ != SIMIX_WAITING && comm->state_ != SIMIX_RUNNING) {
328       comm->finish();
329       break;
330     }
331   }
332 }
333
334 /******************************************************************************/
335 /*                    SIMIX_comm_copy_data callbacks                       */
336 /******************************************************************************/
337 static void (*SIMIX_comm_copy_data_callback)(simgrid::kernel::activity::CommImpl*, void*,
338                                              size_t) = &SIMIX_comm_copy_pointer_callback;
339
340 void SIMIX_comm_copy_buffer_callback(simgrid::kernel::activity::CommImpl* comm, void* buff, size_t buff_size)
341 {
342   XBT_DEBUG("Copy the data over");
343   memcpy(comm->dst_buff_, buff, buff_size);
344   if (comm->detached_) { // if this is a detached send, the source buffer was duplicated by SMPI sender to make the
345                         // original buffer available to the application ASAP
346     xbt_free(buff);
347     comm->src_buff_ = nullptr;
348   }
349 }
350
351 void SIMIX_comm_set_copy_data_callback(void (*callback)(simgrid::kernel::activity::CommImpl*, void*, size_t))
352 {
353   SIMIX_comm_copy_data_callback = callback;
354 }
355
356 void SIMIX_comm_copy_pointer_callback(simgrid::kernel::activity::CommImpl* comm, void* buff, size_t buff_size)
357 {
358   xbt_assert((buff_size == sizeof(void*)), "Cannot copy %zu bytes: must be sizeof(void*)", buff_size);
359   *(void**)(comm->dst_buff_) = buff;
360 }
361
362 namespace simgrid {
363 namespace kernel {
364 namespace activity {
365
366 CommImpl& CommImpl::set_type(CommImpl::Type type)
367 {
368   type_ = type;
369   return *this;
370 }
371
372 CommImpl& CommImpl::set_size(double size)
373 {
374   size_ = size;
375   return *this;
376 }
377
378 CommImpl& CommImpl::set_rate(double rate)
379 {
380   rate_ = rate;
381   return *this;
382 }
383
384 CommImpl& CommImpl::set_src_buff(void* buff, size_t size)
385 {
386   src_buff_      = buff;
387   src_buff_size_ = size;
388   return *this;
389 }
390
391 CommImpl& CommImpl::set_dst_buff(void* buff, size_t* size)
392 {
393   dst_buff_      = buff;
394   dst_buff_size_ = size;
395   return *this;
396 }
397
398 CommImpl::~CommImpl()
399 {
400   XBT_DEBUG("Really free communication %p in state %d (detached = %d)", this, static_cast<int>(state_), detached_);
401
402   cleanupSurf();
403
404   if (detached_ && state_ != SIMIX_DONE) {
405     /* the communication has failed and was detached:
406      * we have to free the buffer */
407     if (clean_fun)
408       clean_fun(src_buff_);
409     src_buff_ = nullptr;
410   } else if (mbox) {
411     mbox->remove(this);
412   }
413 }
414
415 /**  @brief Starts the simulation of a communication synchro. */
416 CommImpl* CommImpl::start()
417 {
418   /* If both the sender and the receiver are already there, start the communication */
419   if (state_ == SIMIX_READY) {
420
421     s4u::Host* sender   = src_actor_->get_host();
422     s4u::Host* receiver = dst_actor_->get_host();
423
424     surf_action_ = surf_network_model->communicate(sender, receiver, size_, rate_);
425     surf_action_->set_activity(this);
426     surf_action_->set_category(get_tracing_category());
427     state_ = SIMIX_RUNNING;
428
429     XBT_DEBUG("Starting communication %p from '%s' to '%s' (surf_action: %p)", this, sender->get_cname(),
430               receiver->get_cname(), surf_action_);
431
432     /* If a link is failed, detect it immediately */
433     if (surf_action_->get_state() == resource::Action::State::FAILED) {
434       XBT_DEBUG("Communication from '%s' to '%s' failed to start because of a link failure", sender->get_cname(),
435                 receiver->get_cname());
436       state_ = SIMIX_LINK_FAILURE;
437       cleanupSurf();
438
439     } else if (src_actor_->is_suspended() || dst_actor_->is_suspended()) {
440       /* If any of the process is suspended, create the synchro but stop its execution,
441          it will be restarted when the sender process resume */
442       if (src_actor_->is_suspended())
443         XBT_DEBUG("The communication is suspended on startup because src (%s@%s) was suspended since it initiated the "
444                   "communication",
445                   src_actor_->get_cname(), src_actor_->get_host()->get_cname());
446       else
447         XBT_DEBUG("The communication is suspended on startup because dst (%s@%s) was suspended since it initiated the "
448                   "communication",
449                   dst_actor_->get_cname(), dst_actor_->get_host()->get_cname());
450
451       surf_action_->suspend();
452     }
453   }
454
455   return this;
456 }
457
458 /** @brief Copy the communication data from the sender's buffer to the receiver's one  */
459 void CommImpl::copy_data()
460 {
461   size_t buff_size = src_buff_size_;
462   /* If there is no data to copy then return */
463   if (not src_buff_ || not dst_buff_ || copied)
464     return;
465
466   XBT_DEBUG("Copying comm %p data from %s (%p) -> %s (%p) (%zu bytes)", this,
467             src_actor_ ? src_actor_->get_host()->get_cname() : "a finished process", src_buff_,
468             dst_actor_ ? dst_actor_->get_host()->get_cname() : "a finished process", dst_buff_, buff_size);
469
470   /* Copy at most dst_buff_size bytes of the message to receiver's buffer */
471   if (dst_buff_size_)
472     buff_size = std::min(buff_size, *(dst_buff_size_));
473
474   /* Update the receiver's buffer size to the copied amount */
475   if (dst_buff_size_)
476     *dst_buff_size_ = buff_size;
477
478   if (buff_size > 0) {
479     if (copy_data_fun)
480       copy_data_fun(this, src_buff_, buff_size);
481     else
482       SIMIX_comm_copy_data_callback(this, src_buff_, buff_size);
483   }
484
485   /* Set the copied flag so we copy data only once */
486   /* (this function might be called from both communication ends) */
487   copied = true;
488 }
489
490 void CommImpl::suspend()
491 {
492   /* FIXME: shall we suspend also the timeout synchro? */
493   if (surf_action_)
494     surf_action_->suspend();
495   /* if not created yet, the action will be suspended on creation, in CommImpl::start() */
496 }
497
498 void CommImpl::resume()
499 {
500   /*FIXME: check what happen with the timeouts */
501   if (surf_action_)
502     surf_action_->resume();
503   /* in the other case, the synchro were not really suspended yet, see CommImpl::suspend() and CommImpl::start() */
504 }
505
506 void CommImpl::cancel()
507 {
508   /* if the synchro is a waiting state means that it is still in a mbox so remove from it and delete it */
509   if (state_ == SIMIX_WAITING) {
510     if (not detached_) {
511       mbox->remove(this);
512       state_ = SIMIX_CANCELED;
513     }
514   } else if (not MC_is_active() /* when running the MC there are no surf actions */
515              && not MC_record_replay_is_active() && (state_ == SIMIX_READY || state_ == SIMIX_RUNNING)) {
516     surf_action_->cancel();
517   }
518 }
519
520 /** @brief This is part of the cleanup process, probably an internal command */
521 void CommImpl::cleanupSurf()
522 {
523   clean_action();
524
525   if (src_timeout_) {
526     src_timeout_->unref();
527     src_timeout_ = nullptr;
528   }
529
530   if (dst_timeout_) {
531     dst_timeout_->unref();
532     dst_timeout_ = nullptr;
533   }
534 }
535
536 void CommImpl::post()
537 {
538   /* Update synchro state */
539   if (src_timeout_ && src_timeout_->get_state() == resource::Action::State::FINISHED)
540     state_ = SIMIX_SRC_TIMEOUT;
541   else if (dst_timeout_ && dst_timeout_->get_state() == resource::Action::State::FINISHED)
542     state_ = SIMIX_DST_TIMEOUT;
543   else if (src_timeout_ && src_timeout_->get_state() == resource::Action::State::FAILED)
544     state_ = SIMIX_SRC_HOST_FAILURE;
545   else if (dst_timeout_ && dst_timeout_->get_state() == resource::Action::State::FAILED)
546     state_ = SIMIX_DST_HOST_FAILURE;
547   else if (surf_action_ && surf_action_->get_state() == resource::Action::State::FAILED) {
548     state_ = SIMIX_LINK_FAILURE;
549   } else
550     state_ = SIMIX_DONE;
551
552   XBT_DEBUG("SIMIX_post_comm: comm %p, state %d, src_proc %p, dst_proc %p, detached: %d", this, (int)state_,
553             src_actor_.get(), dst_actor_.get(), detached_);
554
555   /* destroy the surf actions associated with the Simix communication */
556   cleanupSurf();
557
558   /* if there are simcalls associated with the synchro, then answer them */
559   if (not simcalls_.empty()) {
560     finish();
561   }
562 }
563
564 void CommImpl::finish()
565 {
566   while (not simcalls_.empty()) {
567     smx_simcall_t simcall = simcalls_.front();
568     simcalls_.pop_front();
569
570     /* If a waitany simcall is waiting for this synchro to finish, then remove it from the other synchros in the waitany
571      * list. Afterwards, get the position of the actual synchro in the waitany list and return it as the result of the
572      * simcall */
573
574     if (simcall->call == SIMCALL_NONE) // FIXME: maybe a better way to handle this case
575       continue;                        // if process handling comm is killed
576     if (simcall->call == SIMCALL_COMM_WAITANY) {
577       SIMIX_waitany_remove_simcall_from_actions(simcall);
578       if (simcall->timer) {
579         simcall->timer->remove();
580         simcall->timer = nullptr;
581       }
582       if (not MC_is_active() && not MC_record_replay_is_active()) {
583         CommImpl** comms   = simcall_comm_waitany__get__comms(simcall);
584         size_t count       = simcall_comm_waitany__get__count(simcall);
585         CommImpl** element = std::find(comms, comms + count, this);
586         int rank           = (element != comms + count) ? element - comms : -1;
587         simcall_comm_waitany__set__result(simcall, rank);
588       }
589     }
590
591     /* If the synchro is still in a rendez-vous point then remove from it */
592     if (mbox)
593       mbox->remove(this);
594
595     XBT_DEBUG("CommImpl::finish(): synchro state = %d", static_cast<int>(state_));
596
597     /* Check out for errors */
598
599     if (not simcall->issuer->get_host()->is_on()) {
600       simcall->issuer->context_->iwannadie = true;
601     } else {
602       switch (state_) {
603
604         case SIMIX_DONE:
605           XBT_DEBUG("Communication %p complete!", this);
606           copy_data();
607           break;
608
609         case SIMIX_SRC_TIMEOUT:
610           simcall->issuer->exception_ = std::make_exception_ptr(
611               simgrid::TimeoutError(XBT_THROW_POINT, "Communication timeouted because of the sender"));
612           break;
613
614         case SIMIX_DST_TIMEOUT:
615           simcall->issuer->exception_ = std::make_exception_ptr(
616               simgrid::TimeoutError(XBT_THROW_POINT, "Communication timeouted because of the receiver"));
617           break;
618
619         case SIMIX_SRC_HOST_FAILURE:
620           if (simcall->issuer == src_actor_)
621             simcall->issuer->context_->iwannadie = true;
622           else
623             simcall->issuer->exception_ =
624                 std::make_exception_ptr(simgrid::NetworkFailureException(XBT_THROW_POINT, "Remote peer failed"));
625           break;
626
627         case SIMIX_DST_HOST_FAILURE:
628           if (simcall->issuer == dst_actor_)
629             simcall->issuer->context_->iwannadie = true;
630           else
631             simcall->issuer->exception_ =
632                 std::make_exception_ptr(simgrid::NetworkFailureException(XBT_THROW_POINT, "Remote peer failed"));
633           break;
634
635         case SIMIX_LINK_FAILURE:
636           XBT_DEBUG("Link failure in synchro %p between '%s' and '%s': posting an exception to the issuer: %s (%p) "
637                     "detached:%d",
638                     this, src_actor_ ? src_actor_->get_host()->get_cname() : nullptr,
639                     dst_actor_ ? dst_actor_->get_host()->get_cname() : nullptr, simcall->issuer->get_cname(),
640                     simcall->issuer, detached_);
641           if (src_actor_ == simcall->issuer) {
642             XBT_DEBUG("I'm source");
643           } else if (dst_actor_ == simcall->issuer) {
644             XBT_DEBUG("I'm dest");
645           } else {
646             XBT_DEBUG("I'm neither source nor dest");
647           }
648           simcall->issuer->throw_exception(
649               std::make_exception_ptr(simgrid::NetworkFailureException(XBT_THROW_POINT, "Link failure")));
650           break;
651
652         case SIMIX_CANCELED:
653           if (simcall->issuer == dst_actor_)
654             simcall->issuer->exception_ = std::make_exception_ptr(
655                 simgrid::CancelException(XBT_THROW_POINT, "Communication canceled by the sender"));
656           else
657             simcall->issuer->exception_ = std::make_exception_ptr(
658                 simgrid::CancelException(XBT_THROW_POINT, "Communication canceled by the receiver"));
659           break;
660
661         default:
662           xbt_die("Unexpected synchro state in CommImpl::finish: %d", static_cast<int>(state_));
663       }
664     }
665     /* if there is an exception during a waitany or a testany, indicate the position of the failed communication */
666     if (simcall->issuer->exception_ &&
667         (simcall->call == SIMCALL_COMM_WAITANY || simcall->call == SIMCALL_COMM_TESTANY)) {
668       // First retrieve the rank of our failing synchro
669       CommImpl** comms;
670       size_t count;
671       if (simcall->call == SIMCALL_COMM_WAITANY) {
672         comms = simcall_comm_waitany__get__comms(simcall);
673         count = simcall_comm_waitany__get__count(simcall);
674       } else {
675         /* simcall->call == SIMCALL_COMM_TESTANY */
676         comms = simcall_comm_testany__get__comms(simcall);
677         count = simcall_comm_testany__get__count(simcall);
678       }
679       CommImpl** element = std::find(comms, comms + count, this);
680       int rank           = (element != comms + count) ? element - comms : -1;
681
682       // In order to modify the exception we have to rethrow it:
683       try {
684         std::rethrow_exception(simcall->issuer->exception_);
685       } catch (simgrid::TimeoutError& e) {
686         e.value                     = rank;
687         simcall->issuer->exception_ = std::make_exception_ptr(e);
688       } catch (simgrid::NetworkFailureException& e) {
689         e.value                     = rank;
690         simcall->issuer->exception_ = std::make_exception_ptr(e);
691       } catch (simgrid::CancelException& e) {
692         e.value                     = rank;
693         simcall->issuer->exception_ = std::make_exception_ptr(e);
694       }
695     }
696
697     simcall->issuer->waiting_synchro = nullptr;
698     simcall->issuer->comms.remove(this);
699     if (detached_) {
700       if (simcall->issuer == src_actor_) {
701         if (dst_actor_)
702           dst_actor_->comms.remove(this);
703       } else if (simcall->issuer == dst_actor_) {
704         if (src_actor_)
705           src_actor_->comms.remove(this);
706       } else {
707         dst_actor_->comms.remove(this);
708         src_actor_->comms.remove(this);
709       }
710     }
711
712     if (simcall->issuer->get_host()->is_on())
713       SIMIX_simcall_answer(simcall);
714     else
715       simcall->issuer->context_->iwannadie = true;
716   }
717 }
718
719 } // namespace activity
720 } // namespace kernel
721 } // namespace simgrid