Relax atomic ordering on connection counter

Ordering not required. No other data depends on this counter
mullvad · Dec 19, 2023 · 47eb17c · 47eb17c
1 parent e26a70b
commit 47eb17c
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/statsd.rs b/src/statsd.rs
@@ -128,15 +128,15 @@ mod real {
         }
 
         pub fn incr_connections(&self) {
-            let num_connections = self.num_connections.fetch_add(1, Ordering::SeqCst) + 1;
+            let num_connections = self.num_connections.fetch_add(1, Ordering::Relaxed) + 1;
             log::debug!("Sending statsd num_connections = {num_connections}");
             if let Err(e) = self.client.gauge("num_connections", num_connections) {
                 log::error!("Failed to emit statsd num_connections: {e}");
             }
         }
 
         pub fn decr_connections(&self) {
-            let num_connections = self.num_connections.fetch_sub(1, Ordering::SeqCst) - 1;
+            let num_connections = self.num_connections.fetch_sub(1, Ordering::Relaxed) - 1;
             log::debug!("Sending statsd num_connections = {num_connections}");
             if let Err(e) = self.client.gauge("num_connections", num_connections) {
                 log::error!("Failed to emit statsd num_connections: {e}");