r4163 - trunk/varnish-cache/lib/libvarnish

sky at projects.linpro.no sky at projects.linpro.no
Thu Jul 30 09:06:48 CEST 2009


Author: sky
Date: 2009-07-30 09:06:48 +0200 (Thu, 30 Jul 2009)
New Revision: 4163

Modified:
   trunk/varnish-cache/lib/libvarnish/tcp.c
Log:
revert r4160 and r4161 since apparently they do not fix the problem

Modified: trunk/varnish-cache/lib/libvarnish/tcp.c
===================================================================
--- trunk/varnish-cache/lib/libvarnish/tcp.c	2009-07-29 12:55:43 UTC (rev 4162)
+++ trunk/varnish-cache/lib/libvarnish/tcp.c	2009-07-30 07:06:48 UTC (rev 4163)
@@ -57,11 +57,6 @@
 
 #include "libvarnish.h"
 
-/* Solaris can't use the ioctl" */
-#ifdef __sun
-#include <fcntl.h>
-#endif
-
 /*--------------------------------------------------------------------*/
 
 void
@@ -132,9 +127,6 @@
  * us to do two syscalls, one to get and one to set, the latter of
  * which mucks about a bit before it ends up calling ioctl(FIONBIO),
  * at least on FreeBSD.
- *
- * On some platforms we need to use fcntl because the ioctl is unreliable
- * The one that we know this is the case for is solaris.
  */
 
 void
@@ -142,13 +134,8 @@
 {
 	int i;
 
-#ifdef __sun
-	i = fcntl (sock, F_GETFL,0);
-	fcntl(sock, F_SETFL, i & ~O_NONBLOCK);
-#else
 	i = 0;
 	AZ(ioctl(sock, FIONBIO, &i));
-#endif
 }
 
 void
@@ -156,13 +143,8 @@
 {
 	int i;
 
-#ifdef __sun
-	i = fcntl (sock, F_GETFL,0);
-	fcntl(sock, F_SETFL, i | O_NONBLOCK);
-#else
 	i = 1;
 	AZ(ioctl(sock, FIONBIO, &i));
-#endif
 }
 
 /*--------------------------------------------------------------------



More information about the varnish-commit mailing list