bug 822456 - workaround syn_stream id issue for backport with larger buffers r=honzab a=akeybl
authorPatrick McManus <mcmanus@ducksong.com>
Wed, 19 Dec 2012 16:03:34 -0500 (2012-12-19)
changeset 118993 7de2ac1d8d3d3e6af140de0eccaadbd7ff05a269
parent 118992 5ce9870231003615970aaad2aeb9f91e15a35906
child 118994 8d77162e26134bb119c31d780d02e6b7ade572b8
push id3014
push usermcmanus@ducksong.com
push dateWed, 19 Dec 2012 21:06:15 +0000 (2012-12-19)
treeherdermozilla-aurora@7de2ac1d8d3d [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewershonzab, akeybl
bugs822456
milestone19.0a2
bug 822456 - workaround syn_stream id issue for backport with larger buffers r=honzab a=akeybl
netwerk/protocol/http/SpdySession2.h
netwerk/protocol/http/SpdySession3.h
--- a/netwerk/protocol/http/SpdySession2.h
+++ b/netwerk/protocol/http/SpdySession2.h
@@ -112,18 +112,18 @@ public:
 
   // This should be big enough to hold all of your control packets,
   // but if it needs to grow for huge headers it can do so dynamically.
   // About 1% of requests to SPDY google services seem to be > 1000
   // with all less than 2000.
   const static uint32_t kDefaultBufferSize = 2048;
 
   // kDefaultQueueSize must be >= other queue size constants
-  const static uint32_t kDefaultQueueSize =  16384;
-  const static uint32_t kQueueMinimumCleanup = 8192;
+  const static uint32_t kDefaultQueueSize =  32768;
+  const static uint32_t kQueueMinimumCleanup = 24576;
   const static uint32_t kQueueTailRoom    =  4096;
   const static uint32_t kQueueReserved    =  1024;
 
   const static uint32_t kDefaultMaxConcurrent = 100;
   const static uint32_t kMaxStreamID = 0x7800000;
   
   // This is a sentinel for a deleted stream. It is not a valid
   // 31 bit stream ID.
--- a/netwerk/protocol/http/SpdySession3.h
+++ b/netwerk/protocol/http/SpdySession3.h
@@ -111,18 +111,18 @@ public:
 
   // This should be big enough to hold all of your control packets,
   // but if it needs to grow for huge headers it can do so dynamically.
   // About 1% of requests to SPDY google services seem to be > 1000
   // with all less than 2000.
   const static uint32_t kDefaultBufferSize = 2048;
 
   // kDefaultQueueSize must be >= other queue size constants
-  const static uint32_t kDefaultQueueSize =  16384;
-  const static uint32_t kQueueMinimumCleanup = 8192;
+  const static uint32_t kDefaultQueueSize =  32768;
+  const static uint32_t kQueueMinimumCleanup = 24576;
   const static uint32_t kQueueTailRoom    =  4096;
   const static uint32_t kQueueReserved    =  1024;
 
   const static uint32_t kDefaultMaxConcurrent = 100;
   const static uint32_t kMaxStreamID = 0x7800000;
 
   // This is a sentinel for a deleted stream. It is not a valid
   // 31 bit stream ID.