]> granicus.if.org Git - apache/commitdiff
restore disable-nagle logic for connected sockets
authorJeff Trawick <trawick@apache.org>
Thu, 26 Mar 2009 15:43:46 +0000 (15:43 +0000)
committerJeff Trawick <trawick@apache.org>
Thu, 26 Mar 2009 15:43:46 +0000 (15:43 +0000)
Note: Prior to the MPM-as-DSO churn, the following MPMs did not enable this logic:

  WinNT       APR supports no-oping the setsockopt() based on inheriting Nagle from the
              listening socket on this platform, so the new APR call will be a no-op.
  simple      same as WinNT for at least some Unix-y platforms
  BeOS        shrug

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@758696 13f79535-47bb-0310-9956-ffa450edef68

server/core.c

index 6d9bfad108abab81cd2efdaad8596432bfc4898b..d363ee7f8802285efd77e7feefe66dd4a41231a3 100644 (file)
@@ -3860,7 +3860,6 @@ static int core_pre_connection(conn_rec *c, void *csd)
     core_net_rec *net = apr_palloc(c->pool, sizeof(*net));
     apr_status_t rv;
 
-#ifdef AP_MPM_DISABLE_NAGLE_ACCEPTED_SOCK
     /* The Nagle algorithm says that we should delay sending partial
      * packets in hopes of getting more data.  We don't want to do
      * this; we are not telnet.  There are bad interactions between
@@ -3876,7 +3875,6 @@ static int core_pre_connection(conn_rec *c, void *csd)
         ap_log_cerror(APLOG_MARK, APLOG_DEBUG, rv, c,
                       "apr_socket_opt_set(APR_TCP_NODELAY)");
     }
-#endif
 
     /* The core filter requires the timeout mode to be set, which
      * incidentally sets the socket to be nonblocking.  If this