[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