Now that 4.1 is out of the door I'd like to get this in, disabled by
default.

We can nuke it before release if required and gives people time to play
with it.

Comments?
From ecb0e37aaaa8e7d8e5ee60d9c9ef8d211ddc40f3 Mon Sep 17 00:00:00 2001
From: "Federico G. Schwindt" <[email protected]>
Date: Tue, 13 Oct 2015 17:11:13 +0100
Subject: [PATCH] Expose accept_filter parameter to Linux

Disabled by default.
---
 include/tbl/params.h | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/include/tbl/params.h b/include/tbl/params.h
index d3a2982..c7f1479 100644
--- a/include/tbl/params.h
+++ b/include/tbl/params.h
@@ -30,21 +30,23 @@
 
 /*lint -save -e525 -e539 */
 
-#ifdef HAVE_ACCEPT_FILTERS
 PARAM(
 	/* name */	accept_filter,
 	/* typ */	bool,
 	/* min */	NULL,
 	/* max */	NULL,
+#if defined(HAVE_ACCEPT_FILTERS) || defined(__linux)
 	/* default */	"on",
+#else
+	/* default */	"off",
+#endif /* HAVE_ACCEPT_FILTERS || __linux */
 	/* units */	"bool",
 	/* flags */	MUST_RESTART,
 	/* s-text */
-	"Enable kernel accept-filters, (if available in the kernel).",
+	"Enable kernel accept-filters (if available in the kernel).",
 	/* l-text */	NULL,
 	/* func */	NULL
 )
-#endif /* HAVE_ACCEPT_FILTERS */
 
 PARAM(
 	/* name */	acceptor_sleep_decay,
-- 
2.6.1

_______________________________________________
varnish-dev mailing list
[email protected]
https://www.varnish-cache.org/lists/mailman/listinfo/varnish-dev

Reply via email to