snake_case the signals in s4u::Link (no way to make a deprecation warning)

[simgrid.git] / src / surf / network_interface.cpp

diff --git a/src/surf/network_interface.cpp b/src/surf/network_interface.cpp
index 147747a..7fdfce0 100644 (file)
--- a/src/surf/network_interface.cpp
+++ b/src/surf/network_interface.cpp
@@ -5,7 +5,9 @@
 
 #include "network_interface.hpp"
 #include "simgrid/sg_config.hpp"
+#include "src/instr/instr_private.hpp" // TRACE_is_enabled(). FIXME: remove by subscribing tracing to the surf signals
 #include "src/surf/surf_interface.hpp"
+#include "surf/surf.hpp"
 
 #ifndef NETWORK_INTERFACE_CPP_
 #define NETWORK_INTERFACE_CPP_
@@ -70,7 +72,7 @@ namespace resource {
 
 /** @brief Command-line option 'network/TCP-gamma' -- see \ref options_model_network_gamma */
 simgrid::config::Flag<double> NetworkModel::cfg_tcp_gamma(
-    {"network/TCP-gamma", "network/TCP_gamma"},
+    "network/TCP-gamma", {"network/TCP_gamma"},
     "Size of the biggest TCP window (cat /proc/sys/net/ipv4/tcp_[rw]mem for recv/send window; "
     "Use the last given value, which is the max window size)",
     4194304.0);
@@ -143,7 +145,7 @@ void LinkImpl::destroy()
 {
   if (not currentlyDestroying_) {
     currentlyDestroying_ = true;
-    s4u::Link::onDestruction(this->piface_);
+    s4u::Link::on_destruction(this->piface_);
     delete this;
   }
 }
@@ -172,16 +174,26 @@ void LinkImpl::turn_on()
 {
   if (is_off()) {
     Resource::turn_on();
-    s4u::Link::onStateChange(this->piface_);
+    s4u::Link::on_state_change(this->piface_);
   }
 }
 void LinkImpl::turn_off()
 {
   if (is_on()) {
     Resource::turn_off();
-    s4u::Link::onStateChange(this->piface_);
+    s4u::Link::on_state_change(this->piface_);
   }
 }
+
+void LinkImpl::on_bandwidth_change()
+{
+  if (TRACE_categorized() || TRACE_uncategorized() || TRACE_platform())
+    instr::Container::by_name(get_cname())
+        ->get_variable("bandwidth")
+        ->set_event(surf_get_clock(), sg_bandwidth_factor * bandwidth_.scale * bandwidth_.peak);
+  s4u::Link::on_bandwidth_change(this->piface_);
+}
+
 void LinkImpl::setStateTrace(tmgr_trace_t trace)
 {
   xbt_assert(stateEvent_ == nullptr, "Cannot set a second state trace to Link %s", get_cname());
@@ -205,7 +217,7 @@ void LinkImpl::setLatencyTrace(tmgr_trace_t trace)
 void NetworkAction::set_state(Action::State state)
 {
   Action::set_state(state);
-  s4u::Link::onCommunicationStateChange(this);
+  s4u::Link::on_communication_state_change(this);
 }
 
 /** @brief returns a list of all Links that this action is using */