firedancer: don't warn about rmem_max, wmem_max sysctls

firedancer-io · Sep 16, 2024 · 4bd6749 · 4bd6749
1 parent 7c9b60a
commit 4bd6749
Showing 1 changed file with 12 additions and 10 deletions.
diff --git a/core/src/system_monitor_service.rs b/core/src/system_monitor_service.rs
@@ -401,16 +401,18 @@ enum InterestingLimit {
 
 #[cfg(target_os = "linux")]
 const INTERESTING_LIMITS: &[(&str, InterestingLimit)] = &[
-    ("net.core.rmem_max", InterestingLimit::Recommend(134217728)),
-    (
-        "net.core.rmem_default",
-        InterestingLimit::Recommend(134217728),
-    ),
-    ("net.core.wmem_max", InterestingLimit::Recommend(134217728)),
-    (
-        "net.core.wmem_default",
-        InterestingLimit::Recommend(134217728),
-    ),
+    // FIREDANCER: Limits are no longer enforced, since we use our own
+    // XDP networking stack.
+    // ("net.core.rmem_max", InterestingLimit::Recommend(134217728)),
+    // (
+    //     "net.core.rmem_default",
+    //     InterestingLimit::Recommend(134217728),
+    // ),
+    // ("net.core.wmem_max", InterestingLimit::Recommend(134217728)),
+    // (
+    //     "net.core.wmem_default",
+    //     InterestingLimit::Recommend(134217728),
+    // ),
     ("vm.max_map_count", InterestingLimit::Recommend(1000000)),
     ("net.core.optmem_max", InterestingLimit::QueryOnly),
     ("net.core.netdev_max_backlog", InterestingLimit::QueryOnly),