Add BUGS and review TODO.

[loba.git] / loba_fairstrategy.cpp

diff --git a/loba_fairstrategy.cpp b/loba_fairstrategy.cpp
index ba5a7d9f12720f90050a5fa0e0517bd8ba6e05b9..4ad0bff2be874cb0ce515858c68d11c109e88988 100644 (file)
--- a/loba_fairstrategy.cpp
+++ b/loba_fairstrategy.cpp
@@ -11,39 +11,30 @@ XBT_LOG_EXTERNAL_DEFAULT_CATEGORY(loba);
  */
 
 class compare {
-public : 
-    bool operator()(const neighbor*a, const neighbor*b)        {
-        return a->get_load()>b->get_load();
+public:
+    bool operator()(const neighbor*a, const neighbor*b) {
+        return a->get_load() > b->get_load();
     }
 };
 
-double loba_fairstrategy::load_balance(double my_load)
+void loba_fairstrategy::load_balance()
 {
     std::sort(pneigh.begin(), pneigh.end(), compare());
 
-    print_loads_p();
+    print_loads_p(false, xbt_log_priority_debug);
 
-    double sum_sent=0;
-    bool found=true;
-    
-    while(found) {
-        found=false;
+    bool found = true;
+
+    while (found) {
+        found = false;
         for (unsigned i = 0 ; i < pneigh.size() ; ++i) {
-            double l = pneigh[i]->get_load();
-            if (l >= my_load)
-                continue;
-            if (l < my_load+2) {
-                found=true;
-                pneigh[i]->set_load(l+1);
-                pneigh[i]->add_to_send(1);
-                INFO1("sent to %s",pneigh[i]->get_name());
-                my_load--;
-                sum_sent++;
+            if (pneigh[i]->get_load() <= get_load() - 2) {
+                found = true;
+                send(pneigh[i], 1);
+                DEBUG1("sent to %s", pneigh[i]->get_name());
             }
         }
     }
-
-    return sum_sent;
 }
 
 // Local variables: