commit 927666265dfc05d7edbf9f386f16872c5f80e3cb
author Igor Sysoev <igor@sysoev.ru> Thu Jan 11 18:59:17 2007 +0000
committer Igor Sysoev <igor@sysoev.ru> Thu Jan 11 18:59:17 2007 +0000
tree 0eabf2b7ae4403942c405622ed2072da7fcd3669
parent 7431938b90a79d44e7ee97860862c7d354d89fd7

optimize the SSL session cache allocations on 64-bit platforms

src/event/ngx_event_openssl.c

diff --git a/src/event/ngx_event_openssl.c b/src/event/ngx_event_openssl.c
index 69ea57a..2217bd2 100644
--- a/src/event/ngx_event_openssl.c
+++ b/src/event/ngx_event_openssl.c
@@ -1236,9 +1236,13 @@
  * between 1 and 32 bytes for SSLv3/TLSv1, typically 32 bytes.
  * It seems that the typical length of the external ASN1 representation
  * of a session is 118 or 119 bytes for SSLv3/TSLv1.
- * Thus we allocate separatly an rbtree node, a session id, and an ASN1
- * representation and on 32-bit platforms they take accordingly 64, 32, and
- * 128 bytes.
+ *
+ * Thus on 32-bit platforms we allocate separately an rbtree node,
+ * a session id, and an ASN1 representation, they take accordingly
+ * 64, 32, and 128 bytes.
+ *
+ * On 64-bit platforms we allocate separately an rbtree node + session_id,
+ * and an ASN1 representation, they take accordingly 128 and 128 bytes.
  *
  * OpenSSL's i2d_SSL_SESSION() and d2i_SSL_SESSION are slow,
  * so they are outside the code locked by shared pool mutex
@@ -1294,21 +1298,29 @@
         cached_sess = ngx_slab_alloc_locked(shpool, len);
 
         if (cached_sess == NULL) {
-            id = NULL;
+            sess_id = NULL;
             goto failed;
         }
     }
 
-    id = ngx_slab_alloc_locked(shpool, sess->session_id_length);
-    if (id == NULL) {
-        goto failed;
-    }
-
     sess_id = ngx_slab_alloc_locked(shpool, sizeof(ngx_ssl_sess_id_t));
     if (sess_id == NULL) {
         goto failed;
     }
 
+#if (NGX_PTR_SIZE == 8)
+
+    id = sess_id->sess_id;
+
+#else
+
+    id = ngx_slab_alloc_locked(shpool, sess->session_id_length);
+    if (id == NULL) {
+        goto failed;
+    }
+
+#endif
+
     ngx_memcpy(cached_sess, buf, len);
 
     ngx_memcpy(id, sess->session_id, sess->session_id_length);
@@ -1346,8 +1358,8 @@
         ngx_slab_free_locked(shpool, cached_sess);
     }
 
-    if (id) {
-        ngx_slab_free_locked(shpool, id);
+    if (sess_id) {
+        ngx_slab_free_locked(shpool, sess_id);
     }
 
     ngx_shmtx_unlock(&shpool->mutex);
@@ -1443,7 +1455,9 @@
                     ngx_rbtree_delete(cache->session_rbtree, node);
 
                     ngx_slab_free_locked(shpool, sess_id->session);
+#if (NGX_PTR_SIZE == 4)
                     ngx_slab_free_locked(shpool, sess_id->id);
+#endif
                     ngx_slab_free_locked(shpool, sess_id);
 
                     sess = NULL;
@@ -1523,7 +1537,9 @@
 		    ngx_rbtree_delete(cache->session_rbtree, node);
 
 		    ngx_slab_free_locked(shpool, sess_id->session);
+#if (NGX_PTR_SIZE == 4)
 		    ngx_slab_free_locked(shpool, sess_id->id);
+#endif
 		    ngx_slab_free_locked(shpool, sess_id);
 
 		    goto done;
@@ -1573,7 +1589,9 @@
                        "expire session: %08Xi", sess_id->node.key);
 
         ngx_slab_free_locked(shpool, sess_id->session);
+#if (NGX_PTR_SIZE == 4)
         ngx_slab_free_locked(shpool, sess_id->id);
+#endif
         ngx_slab_free_locked(shpool, sess_id);
     }
 }

src/event/ngx_event_openssl.h

diff --git a/src/event/ngx_event_openssl.h b/src/event/ngx_event_openssl.h
index 04cf3ad..9e8ce17 100644
--- a/src/event/ngx_event_openssl.h
+++ b/src/event/ngx_event_openssl.h
@@ -67,6 +67,10 @@
     ngx_ssl_sess_id_t          *prev;
     ngx_ssl_sess_id_t          *next;
     time_t                      expire;
+#if (NGX_PTR_SIZE == 8)
+    void                       *stub;
+    u_char                      sess_id[32];
+#endif
 };