Author: andre
Date: Sat Oct 20 10:51:32 2012
New Revision: 241779
URL: http://svn.freebsd.org/changeset/base/241779

Log:
  Tidy up somaxconn (accept queue limit) and related functions
  and move it together into one place.

Modified:
  head/sys/kern/uipc_socket.c

Modified: head/sys/kern/uipc_socket.c
==============================================================================
--- head/sys/kern/uipc_socket.c Sat Oct 20 10:34:55 2012        (r241778)
+++ head/sys/kern/uipc_socket.c Sat Oct 20 10:51:32 2012        (r241779)
@@ -182,15 +182,37 @@ MALLOC_DEFINE(M_PCB, "pcb", "protocol co
        VNET_ASSERT(curvnet != NULL,                                    \
            ("%s:%d curvnet is NULL, so=%p", __func__, __LINE__, (so)));
 
+/*
+ * Limit on the number of connections in the listen queue waiting
+ * for accept(2).
+ */
 static int somaxconn = SOMAXCONN;
-static int sysctl_somaxconn(SYSCTL_HANDLER_ARGS);
-/* XXX: we dont have SYSCTL_USHORT */
+
+static int
+sysctl_somaxconn(SYSCTL_HANDLER_ARGS)
+{
+       int error;
+       int val;
+
+       val = somaxconn;
+       error = sysctl_handle_int(oidp, &val, 0, req);
+       if (error || !req->newptr )
+               return (error);
+
+       if (val < 1 || val > USHRT_MAX)
+               return (EINVAL);
+
+       somaxconn = val;
+       return (0);
+}
 SYSCTL_PROC(_kern_ipc, KIPC_SOMAXCONN, somaxconn, CTLTYPE_UINT | CTLFLAG_RW,
-    0, sizeof(int), sysctl_somaxconn, "I", "Maximum pending socket connection "
-    "queue size");
+    0, sizeof(int), sysctl_somaxconn, "I",
+    "Maximum listen socket pending connection accept queue size");
+
 static int numopensockets;
 SYSCTL_INT(_kern_ipc, OID_AUTO, numopensockets, CTLFLAG_RD,
     &numopensockets, 0, "Number of open sockets");
+
 #ifdef ZERO_COPY_SOCKETS
 /* These aren't static because they're used in other files. */
 int so_zero_copy_send = 1;
@@ -3269,24 +3291,6 @@ socheckuid(struct socket *so, uid_t uid)
        return (0);
 }
 
-static int
-sysctl_somaxconn(SYSCTL_HANDLER_ARGS)
-{
-       int error;
-       int val;
-
-       val = somaxconn;
-       error = sysctl_handle_int(oidp, &val, 0, req);
-       if (error || !req->newptr )
-               return (error);
-
-       if (val < 1 || val > USHRT_MAX)
-               return (EINVAL);
-
-       somaxconn = val;
-       return (0);
-}
-
 /*
  * These functions are used by protocols to notify the socket layer (and its
  * consumers) of state changes in the sockets driven by protocol-side events.
_______________________________________________
svn-src-head@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to