[4.0] b853437 Double allocated varnishlog buffer space.

Lasse Karstensen lkarsten at varnish-software.com
Mon Sep 22 16:38:25 CEST 2014


commit b85343734ce803d97eef60d1ef3a1f25a7f83acf
Author: Lasse Karstensen <lkarsten at varnish-software.com>
Date:   Wed Sep 3 17:18:08 2014 +0200

    Double allocated varnishlog buffer space.
    
    Extensive test cases often lead to a lot of log output,
    which at times tripped the following assert:
    
    Assert error in vtc_log_emit(), vtc_log.c line 122:
      Condition(vtclog_left > l) not true.
    
    Double the amount of buffer space allowed to avoid this. Another 256KB
    memory use per concurrent running test should be fine.

diff --git a/bin/varnishtest/vtc_main.c b/bin/varnishtest/vtc_main.c
index 823a0a2..6fdfb6f 100644
--- a/bin/varnishtest/vtc_main.c
+++ b/bin/varnishtest/vtc_main.c
@@ -260,7 +260,7 @@ start_test(void)
 	ALLOC_OBJ(jp, JOB_MAGIC);
 	AN(jp);
 
-	jp->bufsiz = 256*1024;		/* XXX */
+	jp->bufsiz = 512*1024;		/* XXX */
 
 	jp->buf = mmap(NULL, jp->bufsiz, PROT_READ|PROT_WRITE,
 	    MAP_ANON | MAP_SHARED, -1, 0);



More information about the varnish-commit mailing list