[PATCH 2/3] Optimize by memmory `empty_gif_size'

Kirill A. Korinskiy catap+nginx at catap.ru
Wed Mar 18 02:38:47 MSK 2009


From: Kirill A. Korinskiy <catap at catap.ru>

Now for send of very large empty gif did not allocated a lot of
memory, because I allocated buffer and used it multiple times. Size of
buffer is a first optional argument of `empty_gif_size' directive.
---
 src/http/modules/ngx_http_empty_gif_module.c |  109 +++++++++++++++++---------
 1 files changed, 71 insertions(+), 38 deletions(-)

diff --git a/src/http/modules/ngx_http_empty_gif_module.c b/src/http/modules/ngx_http_empty_gif_module.c
index 2abbc19ceb9991be85ebd2236d106c4800ae6508..17b46ac0efb06673232e808ca1d0b44d5994a1a2 100644
--- a/src/http/modules/ngx_http_empty_gif_module.c
+++ b/src/http/modules/ngx_http_empty_gif_module.c
@@ -9,7 +9,12 @@
 
 
 typedef struct {
-    ngx_str_t fake;
+    size_t     size;
+    ngx_str_t  str;
+} ngx_http_empty_gif_fake_t;
+
+typedef struct {
+    ngx_http_empty_gif_fake_t fake;
 } ngx_http_empty_gif_loc_conf_t;
 
 
@@ -34,7 +39,7 @@ static ngx_command_t  ngx_http_empty_gif_commands[] = {
       NULL },
 
     { ngx_string("empty_gif_size"),
-      NGX_HTTP_LOC_CONF|NGX_CONF_TAKE1,
+      NGX_HTTP_LOC_CONF|NGX_CONF_TAKE1|NGX_CONF_TAKE2,
       ngx_http_empty_gif_size,
       NGX_HTTP_LOC_CONF_OFFSET,
       offsetof(ngx_http_empty_gif_loc_conf_t, fake),
@@ -129,8 +134,9 @@ static ngx_int_t
 ngx_http_empty_gif_handler(ngx_http_request_t *r)
 {
     ngx_int_t                       rc;
-    ngx_buf_t                      *b;
-    ngx_chain_t                     out[2];
+    ngx_buf_t                      *b = NULL;
+    ngx_uint_t                      i, out_count;
+    ngx_chain_t                    *out;
     ngx_http_empty_gif_loc_conf_t  *conf;
 
     conf = ngx_http_get_module_loc_conf(r, ngx_http_empty_gif_module);
@@ -152,8 +158,8 @@ ngx_http_empty_gif_handler(ngx_http_request_t *r)
     if (r->method == NGX_HTTP_HEAD) {
         r->headers_out.status = NGX_HTTP_OK;
         r->headers_out.content_length_n = sizeof(ngx_empty_gif);
-        if (conf->fake.len) {
-            r->headers_out.content_length_n += conf->fake.len;
+        if (conf->fake.size) {
+            r->headers_out.content_length_n += conf->fake.size;
         }
         r->headers_out.last_modified_time = 23349600;
 
@@ -165,39 +171,50 @@ ngx_http_empty_gif_handler(ngx_http_request_t *r)
         return ngx_http_send_header(r);
     }
 
-    b = ngx_pcalloc(r->pool, sizeof(ngx_buf_t));
-    if (b == NULL) {
-        return NGX_HTTP_INTERNAL_SERVER_ERROR;
+    out_count = 1;
+    if (conf->fake.size) {
+        out_count += conf->fake.size / conf->fake.str.len;
+        if (conf->fake.size % conf->fake.str.len) {
+            out_count++;
+        }
     }
 
-    out[0].buf = b;
-    out[0].next = NULL;
-
-    b->pos = ngx_empty_gif;
-    b->last = ngx_empty_gif + sizeof(ngx_empty_gif);
-    b->memory = 1;
-
-    r->headers_out.status = NGX_HTTP_OK;
-    r->headers_out.content_length_n = sizeof(ngx_empty_gif);
-    r->headers_out.last_modified_time = 23349600;
-
-    if (conf->fake.len) {
-        out[0].next = &out[1];
-        r->headers_out.content_length_n += conf->fake.len;
+    out = ngx_palloc(r->pool, sizeof(ngx_chain_t) * out_count);
+    if (out == NULL) {
+        return NGX_HTTP_INTERNAL_SERVER_ERROR;
+    }
 
+    for (i = 0; i < out_count; i++) {
         b = ngx_pcalloc(r->pool, sizeof(ngx_buf_t));
         if (b == NULL) {
             return NGX_HTTP_INTERNAL_SERVER_ERROR;
         }
 
-        out[1].next = NULL;
-        out[1].buf = b;
+        out[i].buf = b;
+
+        if (0 == i) {
+            out[i].next = NULL;
+            b->pos = b->start = ngx_empty_gif;
+            b->last = b->end = ngx_empty_gif + sizeof(ngx_empty_gif);
+        } else if ((conf->fake.size / conf->fake.str.len) + 1 == i) {
+            out[i-1].next = &out[i];
+            out[i].next = NULL;
+            b->pos = b->start = conf->fake.str.data;
+            b->last = b->end = conf->fake.str.data + conf->fake.size % conf->fake.str.len;
+        } else {
+            out[i-1].next = &out[i];
+            out[i].next = NULL;
+            b->pos = b->start = conf->fake.str.data;
+            b->last = b->end = conf->fake.str.data + conf->fake.str.len;
+        }
 
-        b->pos = conf->fake.data;
-        b->last = conf->fake.data + conf->fake.len;
         b->memory = 1;
     }
 
+    r->headers_out.status = NGX_HTTP_OK;
+    r->headers_out.content_length_n = sizeof(ngx_empty_gif) + conf->fake.size;
+    r->headers_out.last_modified_time = 23349600;
+
     b->last_buf = 1;
 
     r->headers_out.etag_size = 40;
@@ -210,7 +227,7 @@ ngx_http_empty_gif_handler(ngx_http_request_t *r)
         return rc;
     }
 
-    return ngx_http_output_filter(r, &out[0]);
+    return ngx_http_output_filter(r, out);
 }
 
 
@@ -245,7 +262,7 @@ ngx_http_empty_gif_merge_local_conf(ngx_conf_t *cf, void *parent, void *child)
     ngx_http_empty_gif_loc_conf_t  *prev = parent;
     ngx_http_empty_gif_loc_conf_t  *conf = child;
 
-    if (prev->fake.len) {
+    if (prev->fake.size) {
         conf->fake = prev->fake;
     }
 
@@ -257,31 +274,47 @@ ngx_http_empty_gif_size(ngx_conf_t *cf, ngx_command_t *cmd, void *conf)
 {
     char  *p = conf;
 
-    ngx_str_t                 *field, *value;
+    ngx_str_t                 *value;
+    ngx_int_t                  i = 1;
     ngx_conf_post_t           *post;
+    ngx_http_empty_gif_fake_t *field;
 
-    field = (ngx_str_t *) (p + cmd->offset);
+    field = (ngx_http_empty_gif_fake_t *) (p + cmd->offset);
 
-    if (field->data) {
+    if (field->str.data) {
         return "is duplicate";
     }
 
     value = cf->args->elts;
 
-    field->len = ngx_parse_size(&value[1]);
+    if (cf->args->nelts == 3) {
+        field->str.len = ngx_parse_size(&value[i]);
+        if (field->str.len == (size_t) NGX_ERROR) {
+            return "invalid value";
+        }
+        i++;
+    } else {
+        field->str.len = 1024;
+    }
+
+    field->size = ngx_parse_size(&value[i]);
 
-    if (field->len == (size_t) NGX_ERROR) {
+    if (field->size == (size_t) NGX_ERROR) {
         return "invalid value";
     }
 
-    if (field->len < sizeof(ngx_empty_gif)) {
+    if (field->size < sizeof(ngx_empty_gif)) {
         return "is shortly";
     }
 
-    field->len -= sizeof(ngx_empty_gif);
+    field->size -= sizeof(ngx_empty_gif);
+
+    if (field->str.len > field->size) {
+        field->str.len = field->size;
+    }
 
-    field->data = ngx_pcalloc(cf->pool, field->len);
-    if (field->data == NULL) {
+    field->str.data = ngx_pcalloc(cf->pool, field->str.len);
+    if (field->str.data == NULL) {
         return NGX_CONF_ERROR;
     }
 
-- 
1.6.2






More information about the nginx-ru mailing list