[experimental-ims] a343a0d Beef the gzip parameters up to the usual defaults: window=15 and memlevel 8.

Geoff Simmons geoff at varnish-cache.org
Fri Jul 8 11:47:48 CEST 2011


commit a343a0dad8bd4faa7ad46187f5914e5d4fefb852
Author: Poul-Henning Kamp <phk at FreeBSD.org>
Date:   Wed Jun 22 14:48:57 2011 +0000

    Beef the gzip parameters up to the usual defaults: window=15 and
    memlevel 8.
    
    Fix testcases which depend on older values.

diff --git a/bin/varnishd/mgt_param.c b/bin/varnishd/mgt_param.c
index 1b996e5..26fcb31 100644
--- a/bin/varnishd/mgt_param.c
+++ b/bin/varnishd/mgt_param.c
@@ -888,13 +888,15 @@ static const struct parspec input_parspec[] = {
 		0,
 		"6", ""},
 	{ "gzip_window", tweak_uint, &master.gzip_window, 8, 15,
-		"Gzip window size 8=least, 15=most compression",
+		"Gzip window size 8=least, 15=most compression.\n"
+		"Memory impact is 8=1k, 9=2k, ... 15=128k.",
 		0,
-		"8", ""},
-	{ "gzip_window", tweak_uint, &master.gzip_memlevel, 1, 9,
-		"Gzip memory level 1=least, 9=most compression",
+		"15", ""},
+	{ "gzip_memlevel", tweak_uint, &master.gzip_memlevel, 1, 9,
+		"Gzip memory level 1=slow/least, 9=fast/most compression.\n"
+		"Memory impact is 1=1k, 2=2k, ... 9=256k.",
 		0,
-		"1", ""},
+		"8", ""},
 	{ "gzip_stack_buffer", tweak_uint, &master.gzip_stack_buffer,
 	        2048, UINT_MAX,
 		"Size of stack buffer used for gzip processing.\n"
diff --git a/bin/varnishtest/tests/e00022.vtc b/bin/varnishtest/tests/e00022.vtc
index e402c5e..39302b4 100644
--- a/bin/varnishtest/tests/e00022.vtc
+++ b/bin/varnishtest/tests/e00022.vtc
@@ -26,6 +26,8 @@ varnish v1 -arg "-p sess_workspace=131072 -p thread_pool_stack=262144" -vcl+back
 varnish v1 -cliok "param.set esi_syntax 0xc"
 varnish v1 -cliok "param.set http_gzip_support true"
 varnish v1 -cliok "param.set gzip_tmp_space 1"
+varnish v1 -cliok "param.set gzip_window 8"
+varnish v1 -cliok "param.set gzip_memlevel 1"
 
 client c1 {
 	txreq  -hdr "Accept-Encoding: gzip"
diff --git a/bin/varnishtest/tests/g00002.vtc b/bin/varnishtest/tests/g00002.vtc
index 4d713c7..dba75ae 100644
--- a/bin/varnishtest/tests/g00002.vtc
+++ b/bin/varnishtest/tests/g00002.vtc
@@ -11,7 +11,10 @@ server s1 {
 	txresp -body {<H1><esi:include src="/foo"/></H1>}
 } -start
 
-varnish v1 -cliok "param.set http_gzip_support true" -vcl+backend {
+varnish v1 \
+	-cliok "param.set http_gzip_support true" \
+	-cliok "param.set gzip_memlevel 1" \
+	-vcl+backend {
 
 	sub vcl_fetch {
 		set beresp.do_esi = true;



More information about the varnish-commit mailing list