From: Arnaud Giersch <arnaud.giersch@iut-bm.univ-fcomte.fr>
Date: Mon, 17 Sep 2012 08:08:45 +0000 (+0200)
Subject: Move instruction at proper place.
X-Git-Tag: v3_8~146^2~50
X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/commitdiff_plain/7c408ffb3954d4624ef1cef162348f811c5e5405?hp=9c8be72b2f1b45b0312d03af65e386bae29a1f8e

Move instruction at proper place.

This instruction was wrongly displaced by commit 98897032e770b9a1437d6aa5820fb448e71bd224
(add an immediate asynchronous send possibility for messages [...]), leading some
simulations to failure.

One such example is "chord" with large platforms (cluster with 10000+ hosts).
---

diff --git a/src/simix/smx_network.c b/src/simix/smx_network.c
index d71158af3a..a247f086b1 100644
--- a/src/simix/smx_network.c
+++ b/src/simix/smx_network.c
@@ -415,12 +415,10 @@ smx_action_t SIMIX_comm_irecv(smx_process_t dst_proc, smx_rdv_t rdv,
       --smx_total_comms; // this creation was a pure waste
       other_action->state = SIMIX_READY;
       other_action->comm.type = SIMIX_COMM_READY;
-      xbt_fifo_push(dst_proc->comms, other_action);
-
     }
+    xbt_fifo_push(dst_proc->comms, other_action);
   }
 
-
   /* Setup communication action */
   other_action->comm.dst_proc = dst_proc;
   other_action->comm.dst_buff = dst_buff;