[master] 87072ee Replay SVN r5769

Poul-Henning Kamp phk at project.varnish-software.com
Fri Jan 21 12:17:08 CET 2011


commit 87072eedab8d96f29db2c456a3251cda4ff85afd
Author: Poul-Henning Kamp <phk at FreeBSD.org>
Date:   Fri Jan 21 11:06:43 2011 +0000

    Replay SVN r5769
    
    Add a comment with our worst case gzip scenario

diff --git a/bin/varnishd/cache_gzip.c b/bin/varnishd/cache_gzip.c
index 4529364..d3925ad 100644
--- a/bin/varnishd/cache_gzip.c
+++ b/bin/varnishd/cache_gzip.c
@@ -33,6 +33,35 @@
  *
  * The API defined by this file, will also insulate the rest of the code,
  * should we find a better gzip library at a later date.
+ *
+ * The absolutely worst case gzip processing path, once we have pipe-lining,
+ * will be the following, so we need to be a bit careful with the scratch
+ * space we use:
+ *
+ * 	Backend		Tmp	Input	Output
+ *         |		----------------------
+ *	   v
+ *	 gunzip		wrk	stack	?
+ *         |
+ *	   v
+ *	  esi
+ *         |
+ *	   v
+ *	  gzip		wrk	?	storage
+ *         |
+ *	   v
+ *	  cache
+ *         |
+ *	   v
+ *	 gunzip		wrk	storage	stack
+ *         |
+ *	   v
+ *	 client
+ *
+ * XXXX: The two '?' are obviously the same memory, but I have yet to decide
+ * where it goes.   As usual we try to avoid the session->ws if we can but
+ * I may have to use that.
+ *
  */
 
 #include "config.h"



More information about the varnish-commit mailing list