BUGS

   1 ========================================================================
   2 -- Wed, Feb 29 16:31:56 2012 +0100
   3
   4 Les fonctions MSG_get_host{number,table} n'existent plus dans les
   5 dernières versions de SimGrid.  Utiliser MSG_hosts_as_dynar à la place.
   6
   7 -- Tue, 06 Mar 2012 17:02:02 +0100
   8
   9 Le patch suivant corrige le problème :
  10 --- a/hostdata.cpp
  11 +++ b/hostdata.cpp
  12 @@ -19,8 +19,9 @@ void hostdata::create()
  13      using std::placeholders::_1;
  14      using std::placeholders::_2;
  15
  16 -    int nhosts = MSG_get_host_number();
  17 -    m_host_t* host_list = MSG_get_host_table();
  18 +    xbt_dynar_t host_dynar = MSG_hosts_as_dynar();
  19 +    int nhosts = xbt_dynar_length(host_dynar);
  20 +    m_host_t* host_list = static_cast<m_host_t*>(xbt_dynar_to_array(host_dynar));
  21      // only sort hosts for automatically created deployment
  22      if (opt::auto_depl::enabled)
  23          std::sort(host_list, host_list + nhosts,
  24
  25 ========================================================================
  26 -- Wed, Feb 29 16:16:45 2012 +0100
  27
  28 Les exécutions parallèles donnent des résultats différents.
  29 Pourquoi ?
  30
  31 ========================================================================
  32 ##### MOSTLY RESOLVED BUGS COME AFTER THIS #############################
  33 ========================================================================
  34 -- Mon, Feb 27 13:26:08 2012 +0100
  35
  36 Les variables globales process::total_load_* ne sont pas protégées
  37 contre les accès concurrents.  Il n'est donc pas possible actuellement
  38 d'exécuter les simulations en parallèle (--cfg=contexts/nthreads).
  39
  40 -- Wed, Feb 29 15:54:31 2012 +0100
  41
  42 Corrigé en partie.  Il reste quelques "fixme: get locked?" à régler
  43 (ou pas).
  44
  45 ========================================================================
  46 -- Fri, May 20 17:01:33 2011 +0200
  47
  48 Il faut réviser l'utilisation du mutex entre le thread d'équilibrage
  49 et le thread de calcul.  Il semble gardé beaucoup trop longtemps.
  50
  51 -- Tue, May 24 14:55:59 2011 +0200
  52
  53 Bon, une partie du problème est rectifiée par le commit
  54 48de954 Stop locking the mutex on data_receive.
  55
  56 Pour le reste, je pense maintenant que ça ne gêne pas, au moins dans
  57 le simulateur.  Pour faire bien, il faudrait plus séparer les deux
  58 threads d'équilibrage et de calcul, et faire en sorte que chacun garde
  59 un cache des données globales partagées.  Il suffirait alors de
  60 synchroniser ces caches à chaque itération.
  61
  62 Les données partagées sont essentiellement les données des voisins :
  63 load, to_send et debt.
  64
  65 ========================================================================
  66 -- Tue, Apr 19 17:48:39 2011 +0200
  67
  68 Valgrind détecte une fuite de mémoire liée à un appel à backtrace.
  69
  70 Le problème semble être indépendant de SimGrid et peut être reproduit
  71 avec le code suivant (NB: l'équivalent, compilé avec gcc ne génère pas
  72 d'erreur).
  73 ,----
  74 | #include <execinfo.h>
  75 | #include <iostream>
  76 | int main()
  77 | {
  78 |     void *buffer[64];
  79 |     int size = -1;
  80 |     size = backtrace(buffer, sizeof buffer / sizeof buffer[0]);
  81 |     std::cerr << "backtrace() returned " << size << "\n";
  82 | }
  83 `----
  84 ==532== HEAP SUMMARY:
  85 ==532==     in use at exit: 56 bytes in 1 blocks
  86 ==532==     ...
  87 ==532== LEAK SUMMARY:
  88 ==532==    ...
  89 ==532==    still reachable: 56 bytes in 1 blocks
  90
  91 ========================================================================