Mercurial > hg > nginx

changeset 3967:83e41f6f6d96

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
The cache loader performs two tasks: inserting cache objects in inactivity list and evaluating total cache size. Reading just directory is enough for this purpose. Elimination of reading cache files saves at least one disk I/O operation per file. Preparation for elimination of reading cache files by cache loader: removing dependencies on the reading: *) cache node valid_sec and valid_msec are used only for caching errors; *) upstream buffer size can be used instead of cache node body_start.
author Igor Sysoev <igor@sysoev.ru>
date Sun, 24 Jul 2011 16:47:31 +0000
parents 9c425f22ea36
children d7ed8b8b0cdd
files src/http/ngx_http_file_cache.c
diffstat 1 files changed, 6 insertions(+), 11 deletions(-) [+]
line wrap: on
line diff
--- a/src/http/ngx_http_file_cache.c
+++ b/src/http/ngx_http_file_cache.c
@@ -533,7 +533,9 @@ ngx_http_file_cache_exists(ngx_http_file
         if (fcn->exists || fcn->uses >= c->min_uses) {
 
             c->exists = fcn->exists;
-            c->body_start = fcn->body_start;
+            if (c->body_start) {
+                c->body_start = fcn->body_start;
+            }
 
             rc = NGX_OK;
 
@@ -1365,9 +1367,6 @@ ngx_http_file_cache_add_file(ngx_tree_ct
 
     } else {
         c.uniq = ngx_file_uniq(&fi);
-        c.valid_sec = h.valid_sec;
-        c.valid_msec = h.valid_msec;
-        c.body_start = h.body_start;
         c.length = ngx_file_size(&fi);
         c.fs_size = (ngx_file_fs_size(&fi) + cache->bsize - 1) / cache->bsize;
     }
@@ -1377,10 +1376,6 @@ ngx_http_file_cache_add_file(ngx_tree_ct
                       ngx_close_file_n " \"%s\" failed", name->data);
     }
 
-    if (c.body_start == 0) {
-        return NGX_ERROR;
-    }
-
     p = &name->data[name->len - 2 * NGX_HTTP_CACHE_KEY_LEN];
 
     for (i = 0; i < NGX_HTTP_CACHE_KEY_LEN; i++) {
@@ -1426,14 +1421,14 @@ ngx_http_file_cache_add(ngx_http_file_ca
 
         fcn->uses = 1;
         fcn->count = 0;
-        fcn->valid_msec = c->valid_msec;
+        fcn->valid_msec = 0;
         fcn->error = 0;
         fcn->exists = 1;
         fcn->updating = 0;
         fcn->deleting = 0;
         fcn->uniq = c->uniq;
-        fcn->valid_sec = c->valid_sec;
-        fcn->body_start = c->body_start;
+        fcn->valid_sec = 0;
+        fcn->body_start = 0;
         fcn->fs_size = c->fs_size;
 
         cache->sh->size += c->fs_size;