issue: 3586273 Use XLIO_DEFERRED_CLOSE by default

For incoming sockets - no change. For outgoing sockets - since outgoing sockets occupy a local port, we should release it on the socket destructor to prevent race from another socket to use the same port. This race might cause XLIO to hold 2 sockets with the same RFS object at the same time, and this is fatal (particularly for TCP). Signed-off-by: Iftah Levi <[email protected]>
Mellanox · Apr 14, 2024 · fc87b73 · fc87b73
1 parent 1e18c6a
commit fc87b73
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/core/util/sys_vars.h b/src/core/util/sys_vars.h
@@ -872,7 +872,7 @@ extern mce_sys_var &safe_mce_sys();
 #endif /* DEFINED_UTLS */
 
 #define MCE_DEFAULT_LRO                            (option_3::AUTO)
-#define MCE_DEFAULT_DEFERRED_CLOSE                 (false)
+#define MCE_DEFAULT_DEFERRED_CLOSE                 (true)
 #define MCE_DEFAULT_TCP_ABORT_ON_CLOSE             (false)
 #define MCE_DEFAULT_RX_POLL_ON_TX_TCP              (false)
 #define MCE_DEFAULT_TRIGGER_DUMMY_SEND_GETSOCKNAME (false)