Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
Minor simplification in CM02::communicate
[simgrid.git] / src / surf / network_cm02.cpp
1 /* Copyright (c) 2013-2021. 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/surf/network_cm02.hpp"
7 #include "simgrid/kernel/routing/NetZoneImpl.hpp"
8 #include "simgrid/s4u/Engine.hpp"
9 #include "simgrid/s4u/Host.hpp"
10 #include "simgrid/sg_config.hpp"
11 #include "src/kernel/EngineImpl.hpp"
12 #include "src/kernel/resource/profile/Event.hpp"
13 #include "src/surf/network_wifi.hpp"
14 #include "src/surf/surf_interface.hpp"
15 #include "surf/surf.hpp"
16
17 #include <algorithm>
18 #include <numeric>
19
20 XBT_LOG_EXTERNAL_DEFAULT_CATEGORY(res_network);
21
22 double sg_latency_factor     = 1.0; /* default value; can be set by model or from command line */
23 double sg_bandwidth_factor   = 1.0; /* default value; can be set by model or from command line */
24 double sg_weight_S_parameter = 0.0; /* default value; can be set by model or from command line */
25
26 /************************************************************************/
27 /* New model based on optimizations discussed during Pedro Velho's thesis*/
28 /************************************************************************/
29 /* @techreport{VELHO:2011:HAL-00646896:1, */
30 /*      url = {http://hal.inria.fr/hal-00646896/en/}, */
31 /*      title = {{Flow-level network models: have we reached the limits?}}, */
32 /*      author = {Velho, Pedro and Schnorr, Lucas and Casanova, Henri and Legrand, Arnaud}, */
33 /*      type = {Rapport de recherche}, */
34 /*      institution = {INRIA}, */
35 /*      number = {RR-7821}, */
36 /*      year = {2011}, */
37 /*      month = Nov, */
38 /*      pdf = {http://hal.inria.fr/hal-00646896/PDF/rr-validity.pdf}, */
39 /*  } */
40 void surf_network_model_init_LegrandVelho()
41 {
42   auto net_model = std::make_shared<simgrid::kernel::resource::NetworkCm02Model>("Network_LegrandVelho");
43   simgrid::kernel::EngineImpl::get_instance()->add_model(net_model);
44   simgrid::s4u::Engine::get_instance()->get_netzone_root()->get_impl()->set_network_model(net_model);
45
46   simgrid::config::set_default<double>("network/latency-factor", 13.01);
47   simgrid::config::set_default<double>("network/bandwidth-factor", 0.97);
48   simgrid::config::set_default<double>("network/weight-S", 20537);
49 }
50
51 /***************************************************************************/
52 /* The nice TCP sharing model designed by Loris Marchal and Henri Casanova */
53 /***************************************************************************/
54 /* @TechReport{      rr-lip2002-40, */
55 /*   author        = {Henri Casanova and Loris Marchal}, */
56 /*   institution   = {LIP}, */
57 /*   title         = {A Network Model for Simulation of Grid Application}, */
58 /*   number        = {2002-40}, */
59 /*   month         = {oct}, */
60 /*   year          = {2002} */
61 /* } */
62 void surf_network_model_init_CM02()
63 {
64   simgrid::config::set_default<double>("network/latency-factor", 1.0);
65   simgrid::config::set_default<double>("network/bandwidth-factor", 1.0);
66   simgrid::config::set_default<double>("network/weight-S", 0.0);
67
68   auto net_model = std::make_shared<simgrid::kernel::resource::NetworkCm02Model>("Network_CM02");
69   simgrid::kernel::EngineImpl::get_instance()->add_model(net_model);
70   simgrid::s4u::Engine::get_instance()->get_netzone_root()->get_impl()->set_network_model(net_model);
71 }
72
73 namespace simgrid {
74 namespace kernel {
75 namespace resource {
76
77 NetworkCm02Model::NetworkCm02Model(const std::string& name) : NetworkModel(name)
78 {
79   std::string optim = config::get_value<std::string>("network/optim");
80   bool select       = config::get_value<bool>("network/maxmin-selective-update");
81
82   if (optim == "Lazy") {
83     set_update_algorithm(Model::UpdateAlgo::LAZY);
84     xbt_assert(select || config::is_default("network/maxmin-selective-update"),
85                "You cannot disable network selective update when using the lazy update mechanism");
86     select = true;
87   }
88
89   set_maxmin_system(new lmm::System(select));
90   loopback_ = create_link("__loopback__", std::vector<double>{config::get_value<double>("network/loopback-bw")})
91                   ->set_sharing_policy(s4u::Link::SharingPolicy::FATPIPE)
92                   ->set_latency(config::get_value<double>("network/loopback-lat"));
93   loopback_->seal();
94 }
95
96 void NetworkCm02Model::check_lat_factor_cb()
97 {
98   if (not simgrid::config::is_default("network/latency-factor")) {
99     throw std::invalid_argument(
100         "NetworkModelIntf: Cannot mix network/latency-factor and callback configuration. Choose only one of them.");
101   }
102 }
103
104 void NetworkCm02Model::check_bw_factor_cb()
105 {
106   if (not simgrid::config::is_default("network/bandwidth-factor")) {
107     throw std::invalid_argument(
108         "NetworkModelIntf: Cannot mix network/bandwidth-factor and callback configuration. Choose only one of them.");
109   }
110 }
111
112 void NetworkCm02Model::set_lat_factor_cb(const std::function<NetworkFactorCb>& cb)
113 {
114   if (not cb)
115     throw std::invalid_argument("NetworkModelIntf: Invalid callback");
116   check_lat_factor_cb();
117
118   lat_factor_cb_ = cb;
119 }
120
121 void NetworkCm02Model::set_bw_factor_cb(const std::function<NetworkFactorCb>& cb)
122 {
123   if (not cb)
124     throw std::invalid_argument("NetworkModelIntf: Invalid callback");
125   check_bw_factor_cb();
126
127   bw_factor_cb_ = cb;
128 }
129
130 LinkImpl* NetworkCm02Model::create_link(const std::string& name, const std::vector<double>& bandwidths)
131 {
132   xbt_assert(bandwidths.size() == 1, "Non-WIFI links must use only 1 bandwidth.");
133   return (new NetworkCm02Link(name, bandwidths[0], get_maxmin_system()))->set_model(this);
134 }
135
136 LinkImpl* NetworkCm02Model::create_wifi_link(const std::string& name, const std::vector<double>& bandwidths)
137 {
138   return (new NetworkWifiLink(name, bandwidths, get_maxmin_system()))->set_model(this);
139 }
140
141 void NetworkCm02Model::update_actions_state_lazy(double now, double /*delta*/)
142 {
143   while (not get_action_heap().empty() && double_equals(get_action_heap().top_date(), now, sg_surf_precision)) {
144     auto* action = static_cast<NetworkCm02Action*>(get_action_heap().pop());
145     XBT_DEBUG("Something happened to action %p", action);
146
147     // if I am wearing a latency hat
148     if (action->get_type() == ActionHeap::Type::latency) {
149       XBT_DEBUG("Latency paid for action %p. Activating", action);
150       get_maxmin_system()->update_variable_penalty(action->get_variable(), action->sharing_penalty_);
151       get_action_heap().remove(action);
152       action->set_last_update();
153
154       // if I am wearing a max_duration or normal hat
155     } else if (action->get_type() == ActionHeap::Type::max_duration || action->get_type() == ActionHeap::Type::normal) {
156       // no need to communicate anymore
157       // assume that flows that reached max_duration have remaining of 0
158       XBT_DEBUG("Action %p finished", action);
159       action->finish(Action::State::FINISHED);
160       get_action_heap().remove(action);
161     }
162   }
163 }
164
165 void NetworkCm02Model::update_actions_state_full(double /*now*/, double delta)
166 {
167   for (auto it = std::begin(*get_started_action_set()); it != std::end(*get_started_action_set());) {
168     auto& action = static_cast<NetworkCm02Action&>(*it);
169     ++it; // increment iterator here since the following calls to action.finish() may invalidate it
170     XBT_DEBUG("Something happened to action %p", &action);
171     double deltap = delta;
172     if (action.latency_ > 0) {
173       if (action.latency_ > deltap) {
174         double_update(&action.latency_, deltap, sg_surf_precision);
175         deltap = 0.0;
176       } else {
177         double_update(&deltap, action.latency_, sg_surf_precision);
178         action.latency_ = 0.0;
179       }
180       if (action.latency_ <= 0.0 && not action.is_suspended())
181         get_maxmin_system()->update_variable_penalty(action.get_variable(), action.sharing_penalty_);
182     }
183
184     if (not action.get_variable()->get_number_of_constraint()) {
185       /* There is actually no link used, hence an infinite bandwidth. This happens often when using models like
186        * vivaldi. In such case, just make sure that the action completes immediately.
187        */
188       action.update_remains(action.get_remains());
189     }
190     action.update_remains(action.get_variable()->get_value() * delta);
191
192     if (action.get_max_duration() != NO_MAX_DURATION)
193       action.update_max_duration(delta);
194
195     if (((action.get_remains() <= 0) && (action.get_variable()->get_penalty() > 0)) ||
196         ((action.get_max_duration() != NO_MAX_DURATION) && (action.get_max_duration() <= 0))) {
197       action.finish(Action::State::FINISHED);
198     }
199   }
200 }
201
202 void NetworkCm02Model::comm_action_expand_constraints(const s4u::Host* src, const s4u::Host* dst,
203                                                       const NetworkCm02Action* action,
204                                                       const std::vector<LinkImpl*>& route,
205                                                       const std::vector<LinkImpl*>& back_route) const
206 {
207   /* expand route links constraints for route and back_route */
208   const NetworkWifiLink* src_wifi_link = nullptr;
209   const NetworkWifiLink* dst_wifi_link = nullptr;
210   if (not route.empty() && route.front()->get_sharing_policy() == s4u::Link::SharingPolicy::WIFI) {
211     src_wifi_link = static_cast<NetworkWifiLink*>(route.front());
212   }
213   if (route.size() > 1 && route.back()->get_sharing_policy() == s4u::Link::SharingPolicy::WIFI) {
214     dst_wifi_link = static_cast<NetworkWifiLink*>(route.back());
215   }
216
217   /* WI-FI links needs special treatment, do it here */
218   if (src_wifi_link != nullptr)
219     get_maxmin_system()->expand(src_wifi_link->get_constraint(), action->get_variable(),
220                                 1.0 / src_wifi_link->get_host_rate(src));
221   if (dst_wifi_link != nullptr)
222     get_maxmin_system()->expand(dst_wifi_link->get_constraint(), action->get_variable(),
223                                 1.0 / dst_wifi_link->get_host_rate(dst));
224
225   for (auto const* link : route) {
226     if (link->get_sharing_policy() != s4u::Link::SharingPolicy::WIFI)
227       get_maxmin_system()->expand(link->get_constraint(), action->get_variable(), 1.0);
228   }
229
230   if (cfg_crosstraffic) {
231     XBT_DEBUG("Crosstraffic active: adding backward flow using 5%% of the available bandwidth");
232     if (dst_wifi_link != nullptr)
233       get_maxmin_system()->expand(dst_wifi_link->get_constraint(), action->get_variable(),
234                                   .05 / dst_wifi_link->get_host_rate(dst));
235     if (src_wifi_link != nullptr)
236       get_maxmin_system()->expand(src_wifi_link->get_constraint(), action->get_variable(),
237                                   .05 / src_wifi_link->get_host_rate(src));
238
239     for (auto const* link : back_route) {
240       if (link->get_sharing_policy() != s4u::Link::SharingPolicy::WIFI)
241         get_maxmin_system()->expand(link->get_constraint(), action->get_variable(), .05);
242     }
243     // Change concurrency_share here, if you want that cross-traffic is included in the SURF concurrency
244     // (You would also have to change simgrid::kernel::lmm::Element::get_concurrency())
245     // action->getVariable()->set_concurrency_share(2)
246   }
247 }
248
249 NetworkCm02Action* NetworkCm02Model::comm_action_create(s4u::Host* src, s4u::Host* dst, double size,
250                                                         const std::vector<LinkImpl*>& route, bool failed)
251 {
252   NetworkWifiLink* src_wifi_link = nullptr;
253   NetworkWifiLink* dst_wifi_link = nullptr;
254   /* many checks related to Wi-Fi links */
255   if (not route.empty() && route.front()->get_sharing_policy() == s4u::Link::SharingPolicy::WIFI) {
256     src_wifi_link = static_cast<NetworkWifiLink*>(route.front());
257     xbt_assert(src_wifi_link->get_host_rate(src) != -1,
258                "The route from %s to %s begins with the WIFI link %s, but the host %s does not seem attached to that "
259                "WIFI link. Did you call link->set_host_rate()?",
260                src->get_cname(), dst->get_cname(), src_wifi_link->get_cname(), src->get_cname());
261   }
262   if (route.size() > 1 && route.back()->get_sharing_policy() == s4u::Link::SharingPolicy::WIFI) {
263     dst_wifi_link = static_cast<NetworkWifiLink*>(route.back());
264     xbt_assert(dst_wifi_link->get_host_rate(dst) != -1,
265                "The route from %s to %s ends with the WIFI link %s, but the host %s does not seem attached to that "
266                "WIFI link. Did you call link->set_host_rate()?",
267                src->get_cname(), dst->get_cname(), dst_wifi_link->get_cname(), dst->get_cname());
268   }
269   if (route.size() > 2)
270     for (unsigned i = 1; i < route.size() - 1; i++)
271       xbt_assert(route[i]->get_sharing_policy() != s4u::Link::SharingPolicy::WIFI,
272                  "Link '%s' is a WIFI link. It can only be at the beginning or the end of the route from '%s' to '%s', "
273                  "not in between (it is at position %u out of %zu). "
274                  "Did you declare an access_point in your WIFI zones?",
275                  route[i]->get_cname(), src->get_cname(), dst->get_cname(), i + 1, route.size());
276
277   for (auto const* link : route) {
278     if (link->get_sharing_policy() == s4u::Link::SharingPolicy::WIFI) {
279       xbt_assert(link == src_wifi_link || link == dst_wifi_link,
280                  "Wifi links can only occur at the beginning of the route (meaning that it's attached to the src) or "
281                  "at its end (meaning that it's attached to the dst");
282     }
283   }
284
285   /* create action and do some initializations */
286   NetworkCm02Action* action;
287   if (src_wifi_link == nullptr && dst_wifi_link == nullptr)
288     action = new NetworkCm02Action(this, *src, *dst, size, failed);
289   else
290     action = new NetworkWifiAction(this, *src, *dst, size, failed, src_wifi_link, dst_wifi_link);
291
292   if (is_update_lazy()) {
293     action->set_last_update();
294   }
295
296   return action;
297 }
298
299 bool NetworkCm02Model::comm_get_route_info(const s4u::Host* src, const s4u::Host* dst, double& latency,
300                                            std::vector<LinkImpl*>& route, std::vector<LinkImpl*>& back_route,
301                                            std::unordered_set<kernel::routing::NetZoneImpl*>& netzones) const
302 {
303   kernel::routing::NetZoneImpl::get_global_route_with_netzones(src->get_netpoint(), dst->get_netpoint(), route,
304                                                                &latency, netzones);
305
306   xbt_assert(not route.empty() || latency > 0,
307              "You're trying to send data from %s to %s but there is no connecting path between these two hosts.",
308              src->get_cname(), dst->get_cname());
309
310   bool failed = std::any_of(route.begin(), route.end(), [](const LinkImpl* link) { return not link->is_on(); });
311
312   if (cfg_crosstraffic) {
313     dst->route_to(src, back_route, nullptr);
314     if (not failed)
315       failed =
316           std::any_of(back_route.begin(), back_route.end(), [](const LinkImpl* link) { return not link->is_on(); });
317   }
318   return failed;
319 }
320
321 void NetworkCm02Model::comm_action_set_bounds(const s4u::Host* src, const s4u::Host* dst, double size,
322                                               NetworkCm02Action* action, const std::vector<LinkImpl*>& route,
323                                               const std::unordered_set<kernel::routing::NetZoneImpl*>& netzones,
324                                               double rate)
325 {
326   std::vector<s4u::Link*> s4u_route;
327   std::unordered_set<s4u::NetZone*> s4u_netzones;
328
329   /* transform data to user structures if necessary */
330   if (lat_factor_cb_ || bw_factor_cb_) {
331     std::for_each(route.begin(), route.end(), [&s4u_route](LinkImpl* l) { s4u_route.push_back(l->get_iface()); });
332     std::for_each(netzones.begin(), netzones.end(),
333                   [&s4u_netzones](kernel::routing::NetZoneImpl* n) { s4u_netzones.insert(n->get_iface()); });
334   }
335   double bw_factor;
336   if (bw_factor_cb_) {
337     bw_factor = bw_factor_cb_(size, src, dst, s4u_route, s4u_netzones);
338   } else {
339     bw_factor = get_bandwidth_factor(size);
340   }
341   /* get mininum bandwidth among links in the route and multiply by correct factor
342    * ignore wi-fi links, they're not considered for bw_factors */
343   double bandwidth_bound = -1.0;
344   for (const auto* l : route) {
345     if (l->get_sharing_policy() == s4u::Link::SharingPolicy::WIFI)
346       continue;
347     if (bandwidth_bound == -1.0 || l->get_bandwidth() < bandwidth_bound)
348       bandwidth_bound = l->get_bandwidth();
349   }
350   bandwidth_bound *= bw_factor;
351
352   /* the bandwidth is determined by the minimum between flow and user's defined rate */
353   if (rate >= 0 && rate < bandwidth_bound)
354     bandwidth_bound = rate;
355
356   action->set_user_bound(bandwidth_bound);
357
358   action->lat_current_ = action->latency_;
359   if (lat_factor_cb_) {
360     action->latency_ *= lat_factor_cb_(size, src, dst, s4u_route, s4u_netzones);
361   } else {
362     action->latency_ *= get_latency_factor(size);
363   }
364 }
365
366 void NetworkCm02Model::comm_action_set_variable(NetworkCm02Action* action, const std::vector<LinkImpl*>& route,
367                                                 const std::vector<LinkImpl*>& back_route)
368 {
369   size_t constraints_per_variable = route.size();
370   constraints_per_variable += back_route.size();
371
372   if (action->latency_ > 0) {
373     action->set_variable(get_maxmin_system()->variable_new(action, 0.0, -1.0, constraints_per_variable));
374     if (is_update_lazy()) {
375       // add to the heap the event when the latency is paid
376       double date = action->latency_ + action->get_last_update();
377
378       ActionHeap::Type type = route.empty() ? ActionHeap::Type::normal : ActionHeap::Type::latency;
379
380       XBT_DEBUG("Added action (%p) one latency event at date %f", action, date);
381       get_action_heap().insert(action, date, type);
382     }
383   } else
384     action->set_variable(get_maxmin_system()->variable_new(action, 1.0, -1.0, constraints_per_variable));
385
386   /* after setting the variable, update the bounds depending on user configuration */
387   if (action->get_user_bound() < 0) {
388     get_maxmin_system()->update_variable_bound(
389         action->get_variable(), (action->lat_current_ > 0) ? cfg_tcp_gamma / (2.0 * action->lat_current_) : -1.0);
390   } else {
391     get_maxmin_system()->update_variable_bound(
392         action->get_variable(), (action->lat_current_ > 0)
393                                     ? std::min(action->get_user_bound(), cfg_tcp_gamma / (2.0 * action->lat_current_))
394                                     : action->get_user_bound());
395   }
396 }
397
398 Action* NetworkCm02Model::communicate(s4u::Host* src, s4u::Host* dst, double size, double rate)
399 {
400   double latency = 0.0;
401   std::vector<LinkImpl*> back_route;
402   std::vector<LinkImpl*> route;
403   std::unordered_set<kernel::routing::NetZoneImpl*> netzones;
404
405   XBT_IN("(%s,%s,%g,%g)", src->get_cname(), dst->get_cname(), size, rate);
406
407   bool failed = comm_get_route_info(src, dst, latency, route, back_route, netzones);
408
409   NetworkCm02Action* action = comm_action_create(src, dst, size, route, failed);
410   action->sharing_penalty_  = latency;
411   action->latency_          = latency;
412
413   if (sg_weight_S_parameter > 0) {
414     action->sharing_penalty_ =
415         std::accumulate(route.begin(), route.end(), action->sharing_penalty_, [](double total, LinkImpl* const& link) {
416           return total + sg_weight_S_parameter / link->get_bandwidth();
417         });
418   }
419
420   /* setting bandwidth and latency bounds considering route and configured bw/lat factors */
421   comm_action_set_bounds(src, dst, size, action, route, netzones, rate);
422
423   /* creating the maxmin variable associated to this action */
424   comm_action_set_variable(action, route, back_route);
425
426   /* expand maxmin system to consider this communication in bw constraint for each link in route and back_route */
427   comm_action_expand_constraints(src, dst, action, route, back_route);
428   XBT_OUT();
429
430   simgrid::s4u::Link::on_communicate(*action);
431   return action;
432 }
433
434 /************
435  * Resource *
436  ************/
437 NetworkCm02Link::NetworkCm02Link(const std::string& name, double bandwidth, kernel::lmm::System* system)
438     : LinkImpl(name)
439 {
440   bandwidth_.scale = 1.0;
441   bandwidth_.peak  = bandwidth;
442   this->set_constraint(system->constraint_new(this, sg_bandwidth_factor * bandwidth));
443 }
444
445 void NetworkCm02Link::apply_event(kernel::profile::Event* triggered, double value)
446 {
447   /* Find out which of my iterators was triggered, and react accordingly */
448   if (triggered == bandwidth_.event) {
449     set_bandwidth(value);
450     tmgr_trace_event_unref(&bandwidth_.event);
451
452   } else if (triggered == latency_.event) {
453     set_latency(value);
454     tmgr_trace_event_unref(&latency_.event);
455
456   } else if (triggered == state_event_) {
457     if (value > 0)
458       turn_on();
459     else
460       turn_off();
461     tmgr_trace_event_unref(&state_event_);
462   } else {
463     xbt_die("Unknown event!\n");
464   }
465
466   XBT_DEBUG("There was a resource state event, need to update actions related to the constraint (%p)",
467             get_constraint());
468 }
469
470 void NetworkCm02Link::set_bandwidth(double value)
471 {
472   bandwidth_.peak = value;
473
474   get_model()->get_maxmin_system()->update_constraint_bound(get_constraint(),
475                                                             sg_bandwidth_factor * (bandwidth_.peak * bandwidth_.scale));
476
477   LinkImpl::on_bandwidth_change();
478
479   if (sg_weight_S_parameter > 0) {
480     double delta = sg_weight_S_parameter / value - sg_weight_S_parameter / (bandwidth_.peak * bandwidth_.scale);
481
482     const kernel::lmm::Element* elem     = nullptr;
483     const kernel::lmm::Element* nextelem = nullptr;
484     int numelem                          = 0;
485     while (const auto* var = get_constraint()->get_variable_safe(&elem, &nextelem, &numelem)) {
486       auto* action = static_cast<NetworkCm02Action*>(var->get_id());
487       action->sharing_penalty_ += delta;
488       if (not action->is_suspended())
489         get_model()->get_maxmin_system()->update_variable_penalty(action->get_variable(), action->sharing_penalty_);
490     }
491   }
492 }
493
494 LinkImpl* NetworkCm02Link::set_latency(double value)
495 {
496   latency_check(value);
497
498   double delta                         = value - latency_.peak;
499   const kernel::lmm::Element* elem     = nullptr;
500   const kernel::lmm::Element* nextelem = nullptr;
501   int numelem                          = 0;
502
503   latency_.scale = 1.0;
504   latency_.peak  = value;
505
506   while (const auto* var = get_constraint()->get_variable_safe(&elem, &nextelem, &numelem)) {
507     auto* action = static_cast<NetworkCm02Action*>(var->get_id());
508     action->lat_current_ += delta;
509     action->sharing_penalty_ += delta;
510     if (action->get_user_bound() < 0)
511       get_model()->get_maxmin_system()->update_variable_bound(action->get_variable(), NetworkModel::cfg_tcp_gamma /
512                                                                                           (2.0 * action->lat_current_));
513     else {
514       get_model()->get_maxmin_system()->update_variable_bound(
515           action->get_variable(),
516           std::min(action->get_user_bound(), NetworkModel::cfg_tcp_gamma / (2.0 * action->lat_current_)));
517
518       if (action->get_user_bound() < NetworkModel::cfg_tcp_gamma / (2.0 * action->lat_current_)) {
519         XBT_INFO("Flow is limited BYBANDWIDTH");
520       } else {
521         XBT_INFO("Flow is limited BYLATENCY, latency of flow is %f", action->lat_current_);
522       }
523     }
524     if (not action->is_suspended())
525       get_model()->get_maxmin_system()->update_variable_penalty(action->get_variable(), action->sharing_penalty_);
526   }
527   return this;
528 }
529
530 /**********
531  * Action *
532  **********/
533
534 void NetworkCm02Action::update_remains_lazy(double now)
535 {
536   if (not is_running())
537     return;
538
539   double delta = now - get_last_update();
540
541   if (get_remains_no_update() > 0) {
542     XBT_DEBUG("Updating action(%p): remains was %f, last_update was: %f", this, get_remains_no_update(),
543               get_last_update());
544     update_remains(get_last_value() * delta);
545
546     XBT_DEBUG("Updating action(%p): remains is now %f", this, get_remains_no_update());
547   }
548
549   update_max_duration(delta);
550
551   if ((get_remains_no_update() <= 0 && (get_variable()->get_penalty() > 0)) ||
552       ((get_max_duration() != NO_MAX_DURATION) && (get_max_duration() <= 0))) {
553     finish(Action::State::FINISHED);
554     get_model()->get_action_heap().remove(this);
555   }
556
557   set_last_update();
558   set_last_value(get_variable()->get_value());
559 }
560
561 } // namespace resource
562 } // namespace kernel
563 } // namespace simgrid