Mercurial > hg > nginx
annotate src/core/ngx_open_file_cache.c @ 2782:4bd7825fab80
uniform ngx_file_info() interface with ngx_fd_info()
| author | Igor Sysoev <igor@sysoev.ru> |
|---|---|
| date | Wed, 29 Apr 2009 19:28:52 +0000 |
| parents | 09cab3f8d92e |
| children | 87c088e6956a |
| rev | line source |
|---|---|
| 1453 | 1 |
| 2 /* | |
| 3 * Copyright (C) Igor Sysoev | |
| 4 */ | |
| 5 | |
| 6 | |
| 7 #include <ngx_config.h> | |
| 8 #include <ngx_core.h> | |
| 9 #include <ngx_event.h> | |
| 10 | |
| 11 | |
| 12 /* | |
| 13 * open file cache caches | |
| 14 * open file handles with stat() info; | |
| 15 * directories stat() info; | |
| 16 * files and directories errors: not found, access denied, etc. | |
| 17 */ | |
| 18 | |
| 19 | |
| 20 static void ngx_open_file_cache_cleanup(void *data); | |
| 1775 | 21 static ngx_int_t ngx_open_and_stat_file(u_char *name, ngx_open_file_info_t *of, |
| 22 ngx_log_t *log); | |
| 23 static void ngx_open_file_add_event(ngx_open_file_cache_t *cache, | |
| 24 ngx_cached_open_file_t *file, ngx_open_file_info_t *of, ngx_log_t *log); | |
| 1453 | 25 static void ngx_open_file_cleanup(void *data); |
| 26 static void ngx_close_cached_file(ngx_open_file_cache_t *cache, | |
| 1772 | 27 ngx_cached_open_file_t *file, ngx_uint_t min_uses, ngx_log_t *log); |
| 1775 | 28 static void ngx_open_file_del_event(ngx_cached_open_file_t *file); |
| 1453 | 29 static void ngx_expire_old_cached_files(ngx_open_file_cache_t *cache, |
| 30 ngx_uint_t n, ngx_log_t *log); | |
| 31 static void ngx_open_file_cache_rbtree_insert_value(ngx_rbtree_node_t *temp, | |
| 32 ngx_rbtree_node_t *node, ngx_rbtree_node_t *sentinel); | |
| 1775 | 33 static ngx_cached_open_file_t * |
| 34 ngx_open_file_lookup(ngx_open_file_cache_t *cache, ngx_str_t *name, | |
| 35 uint32_t hash); | |
| 1453 | 36 static void ngx_open_file_cache_remove(ngx_event_t *ev); |
| 37 | |
| 38 | |
| 39 ngx_open_file_cache_t * | |
| 40 ngx_open_file_cache_init(ngx_pool_t *pool, ngx_uint_t max, time_t inactive) | |
| 41 { | |
| 42 ngx_pool_cleanup_t *cln; | |
| 43 ngx_open_file_cache_t *cache; | |
| 44 | |
| 45 cache = ngx_palloc(pool, sizeof(ngx_open_file_cache_t)); | |
| 46 if (cache == NULL) { | |
| 47 return NULL; | |
| 48 } | |
| 49 | |
| 1761 | 50 ngx_rbtree_init(&cache->rbtree, &cache->sentinel, |
|
1743
4fc402c3ec73
optimize rbtree initialization and insert
Igor Sysoev <igor@sysoev.ru>
parents:
1458
diff
changeset
|
51 ngx_open_file_cache_rbtree_insert_value); |
| 1453 | 52 |
| 1765 | 53 ngx_queue_init(&cache->expire_queue); |
| 54 | |
| 1453 | 55 cache->current = 0; |
| 56 cache->max = max; | |
| 57 cache->inactive = inactive; | |
| 58 | |
| 59 cln = ngx_pool_cleanup_add(pool, 0); | |
| 60 if (cln == NULL) { | |
| 61 return NULL; | |
| 62 } | |
| 63 | |
| 64 cln->handler = ngx_open_file_cache_cleanup; | |
| 65 cln->data = cache; | |
| 66 | |
| 67 return cache; | |
| 68 } | |
| 69 | |
| 70 | |
| 71 static void | |
| 72 ngx_open_file_cache_cleanup(void *data) | |
| 73 { | |
| 74 ngx_open_file_cache_t *cache = data; | |
| 75 | |
| 1765 | 76 ngx_queue_t *q; |
| 1453 | 77 ngx_cached_open_file_t *file; |
| 78 | |
| 79 ngx_log_debug0(NGX_LOG_DEBUG_CORE, ngx_cycle->log, 0, | |
| 80 "open file cache cleanup"); | |
| 81 | |
| 82 for ( ;; ) { | |
| 83 | |
| 1765 | 84 if (ngx_queue_empty(&cache->expire_queue)) { |
| 1766 | 85 break; |
| 1453 | 86 } |
| 87 | |
| 1765 | 88 q = ngx_queue_last(&cache->expire_queue); |
| 89 | |
| 90 file = ngx_queue_data(q, ngx_cached_open_file_t, queue); | |
| 91 | |
| 92 ngx_queue_remove(q); | |
| 1453 | 93 |
| 94 ngx_rbtree_delete(&cache->rbtree, &file->node); | |
| 95 | |
| 96 cache->current--; | |
| 97 | |
| 98 ngx_log_debug1(NGX_LOG_DEBUG_CORE, ngx_cycle->log, 0, | |
| 99 "delete cached open file: %s", file->name); | |
| 100 | |
| 101 if (!file->err && !file->is_dir) { | |
| 102 file->close = 1; | |
| 103 file->count = 0; | |
| 1772 | 104 ngx_close_cached_file(cache, file, 0, ngx_cycle->log); |
| 1453 | 105 |
| 106 } else { | |
| 107 ngx_free(file->name); | |
| 108 ngx_free(file); | |
| 109 } | |
| 110 } | |
| 111 | |
| 112 if (cache->current) { | |
| 113 ngx_log_error(NGX_LOG_ALERT, ngx_cycle->log, 0, | |
| 114 "%d items still leave in open file cache", | |
| 115 cache->current); | |
| 116 } | |
| 117 | |
| 118 if (cache->rbtree.root != cache->rbtree.sentinel) { | |
| 119 ngx_log_error(NGX_LOG_ALERT, ngx_cycle->log, 0, | |
| 120 "rbtree still is not empty in open file cache"); | |
| 121 | |
| 122 } | |
| 123 } | |
| 124 | |
| 125 | |
| 126 ngx_int_t | |
| 127 ngx_open_cached_file(ngx_open_file_cache_t *cache, ngx_str_t *name, | |
| 128 ngx_open_file_info_t *of, ngx_pool_t *pool) | |
| 129 { | |
| 130 time_t now; | |
| 131 uint32_t hash; | |
| 132 ngx_int_t rc; | |
|
2756
09cab3f8d92e
*) of.test_only to not open file if only stat() is enough
Igor Sysoev <igor@sysoev.ru>
parents:
2629
diff
changeset
|
133 ngx_file_info_t fi; |
| 1453 | 134 ngx_pool_cleanup_t *cln; |
| 135 ngx_cached_open_file_t *file; | |
| 136 ngx_pool_cleanup_file_t *clnf; | |
| 137 ngx_open_file_cache_cleanup_t *ofcln; | |
| 138 | |
|
2069
23930ccd2642
use ngx_file_info() and test uniq if file is already open
Igor Sysoev <igor@sysoev.ru>
parents:
2063
diff
changeset
|
139 of->fd = NGX_INVALID_FILE; |
| 1453 | 140 of->err = 0; |
| 141 | |
| 142 if (cache == NULL) { | |
| 143 | |
|
2756
09cab3f8d92e
*) of.test_only to not open file if only stat() is enough
Igor Sysoev <igor@sysoev.ru>
parents:
2629
diff
changeset
|
144 if (of->test_only) { |
|
09cab3f8d92e
*) of.test_only to not open file if only stat() is enough
Igor Sysoev <igor@sysoev.ru>
parents:
2629
diff
changeset
|
145 |
|
2782
4bd7825fab80
uniform ngx_file_info() interface with ngx_fd_info()
Igor Sysoev <igor@sysoev.ru>
parents:
2756
diff
changeset
|
146 if (ngx_file_info(name->data, &fi) == NGX_FILE_ERROR) { |
|
2756
09cab3f8d92e
*) of.test_only to not open file if only stat() is enough
Igor Sysoev <igor@sysoev.ru>
parents:
2629
diff
changeset
|
147 of->err = ngx_errno; |
|
09cab3f8d92e
*) of.test_only to not open file if only stat() is enough
Igor Sysoev <igor@sysoev.ru>
parents:
2629
diff
changeset
|
148 of->failed = ngx_file_info_n; |
|
09cab3f8d92e
*) of.test_only to not open file if only stat() is enough
Igor Sysoev <igor@sysoev.ru>
parents:
2629
diff
changeset
|
149 return NGX_ERROR; |
|
09cab3f8d92e
*) of.test_only to not open file if only stat() is enough
Igor Sysoev <igor@sysoev.ru>
parents:
2629
diff
changeset
|
150 } |
|
09cab3f8d92e
*) of.test_only to not open file if only stat() is enough
Igor Sysoev <igor@sysoev.ru>
parents:
2629
diff
changeset
|
151 |
|
09cab3f8d92e
*) of.test_only to not open file if only stat() is enough
Igor Sysoev <igor@sysoev.ru>
parents:
2629
diff
changeset
|
152 of->uniq = ngx_file_uniq(&fi); |
|
09cab3f8d92e
*) of.test_only to not open file if only stat() is enough
Igor Sysoev <igor@sysoev.ru>
parents:
2629
diff
changeset
|
153 of->mtime = ngx_file_mtime(&fi); |
|
09cab3f8d92e
*) of.test_only to not open file if only stat() is enough
Igor Sysoev <igor@sysoev.ru>
parents:
2629
diff
changeset
|
154 of->size = ngx_file_size(&fi); |
|
09cab3f8d92e
*) of.test_only to not open file if only stat() is enough
Igor Sysoev <igor@sysoev.ru>
parents:
2629
diff
changeset
|
155 of->is_dir = ngx_is_dir(&fi); |
|
09cab3f8d92e
*) of.test_only to not open file if only stat() is enough
Igor Sysoev <igor@sysoev.ru>
parents:
2629
diff
changeset
|
156 of->is_file = ngx_is_file(&fi); |
|
09cab3f8d92e
*) of.test_only to not open file if only stat() is enough
Igor Sysoev <igor@sysoev.ru>
parents:
2629
diff
changeset
|
157 of->is_link = ngx_is_link(&fi); |
|
09cab3f8d92e
*) of.test_only to not open file if only stat() is enough
Igor Sysoev <igor@sysoev.ru>
parents:
2629
diff
changeset
|
158 of->is_exec = ngx_is_exec(&fi); |
|
09cab3f8d92e
*) of.test_only to not open file if only stat() is enough
Igor Sysoev <igor@sysoev.ru>
parents:
2629
diff
changeset
|
159 |
|
09cab3f8d92e
*) of.test_only to not open file if only stat() is enough
Igor Sysoev <igor@sysoev.ru>
parents:
2629
diff
changeset
|
160 return NGX_OK; |
|
09cab3f8d92e
*) of.test_only to not open file if only stat() is enough
Igor Sysoev <igor@sysoev.ru>
parents:
2629
diff
changeset
|
161 } |
|
09cab3f8d92e
*) of.test_only to not open file if only stat() is enough
Igor Sysoev <igor@sysoev.ru>
parents:
2629
diff
changeset
|
162 |
| 1453 | 163 cln = ngx_pool_cleanup_add(pool, sizeof(ngx_pool_cleanup_file_t)); |
| 164 if (cln == NULL) { | |
| 165 return NGX_ERROR; | |
| 166 } | |
| 167 | |
| 168 rc = ngx_open_and_stat_file(name->data, of, pool->log); | |
| 169 | |
| 170 if (rc == NGX_OK && !of->is_dir) { | |
| 171 cln->handler = ngx_pool_cleanup_file; | |
| 172 clnf = cln->data; | |
| 173 | |
| 174 clnf->fd = of->fd; | |
| 175 clnf->name = name->data; | |
| 176 clnf->log = pool->log; | |
| 177 } | |
| 178 | |
| 179 return rc; | |
| 180 } | |
| 181 | |
| 182 cln = ngx_pool_cleanup_add(pool, sizeof(ngx_open_file_cache_cleanup_t)); | |
| 183 if (cln == NULL) { | |
| 184 return NGX_ERROR; | |
| 185 } | |
| 186 | |
| 187 now = ngx_time(); | |
| 188 | |
| 1775 | 189 hash = ngx_crc32_long(name->data, name->len); |
| 190 | |
| 191 file = ngx_open_file_lookup(cache, name, hash); | |
| 192 | |
| 193 if (file) { | |
| 194 | |
| 195 file->uses++; | |
| 196 | |
| 1987 | 197 ngx_queue_remove(&file->queue); |
| 198 | |
| 1775 | 199 if (file->fd == NGX_INVALID_FILE && file->err == 0 && !file->is_dir) { |
| 200 | |
| 201 /* file was not used often enough to keep open */ | |
| 202 | |
| 203 rc = ngx_open_and_stat_file(name->data, of, pool->log); | |
| 1453 | 204 |
| 1775 | 205 if (rc != NGX_OK && (of->err == 0 || !of->errors)) { |
| 206 goto failed; | |
| 207 } | |
| 208 | |
| 209 goto add_event; | |
| 1453 | 210 } |
| 211 | |
| 2070 | 212 if (file->use_event |
|
2063
67a29af877ed
initialize of.uniq in ngx_open_cached_file()
Igor Sysoev <igor@sysoev.ru>
parents:
2006
diff
changeset
|
213 || (file->event == NULL |
|
67a29af877ed
initialize of.uniq in ngx_open_cached_file()
Igor Sysoev <igor@sysoev.ru>
parents:
2006
diff
changeset
|
214 && (of->uniq == 0 || of->uniq == file->uniq) |
|
67a29af877ed
initialize of.uniq in ngx_open_cached_file()
Igor Sysoev <igor@sysoev.ru>
parents:
2006
diff
changeset
|
215 && now - file->created < of->valid)) |
| 1775 | 216 { |
| 217 if (file->err == 0) { | |
| 1772 | 218 |
| 1775 | 219 of->fd = file->fd; |
| 220 of->uniq = file->uniq; | |
| 221 of->mtime = file->mtime; | |
| 222 of->size = file->size; | |
| 1453 | 223 |
| 1775 | 224 of->is_dir = file->is_dir; |
| 225 of->is_file = file->is_file; | |
| 226 of->is_link = file->is_link; | |
| 227 of->is_exec = file->is_exec; | |
|
2246
987831d73bd8
cache directio flag in open file cache
Igor Sysoev <igor@sysoev.ru>
parents:
2231
diff
changeset
|
228 of->is_directio = file->is_directio; |
| 1453 | 229 |
| 1775 | 230 if (!file->is_dir) { |
| 231 file->count++; | |
| 232 ngx_open_file_add_event(cache, file, of, pool->log); | |
| 1453 | 233 } |
| 234 | |
| 1775 | 235 } else { |
| 236 of->err = file->err; | |
| 237 } | |
| 238 | |
| 239 goto found; | |
| 240 } | |
| 241 | |
| 242 ngx_log_debug4(NGX_LOG_DEBUG_CORE, pool->log, 0, | |
| 243 "retest open file: %s, fd:%d, c:%d, e:%d", | |
| 244 file->name, file->fd, file->count, file->err); | |
| 1453 | 245 |
| 1775 | 246 if (file->is_dir) { |
| 247 | |
| 248 /* | |
| 249 * chances that directory became file are very small | |
| 250 * so test_dir flag allows to use a single syscall | |
| 251 * in ngx_file_info() instead of three syscalls | |
| 252 */ | |
| 253 | |
| 254 of->test_dir = 1; | |
| 255 } | |
| 256 | |
|
2069
23930ccd2642
use ngx_file_info() and test uniq if file is already open
Igor Sysoev <igor@sysoev.ru>
parents:
2063
diff
changeset
|
257 of->fd = file->fd; |
|
23930ccd2642
use ngx_file_info() and test uniq if file is already open
Igor Sysoev <igor@sysoev.ru>
parents:
2063
diff
changeset
|
258 of->uniq = file->uniq; |
|
23930ccd2642
use ngx_file_info() and test uniq if file is already open
Igor Sysoev <igor@sysoev.ru>
parents:
2063
diff
changeset
|
259 |
| 1775 | 260 rc = ngx_open_and_stat_file(name->data, of, pool->log); |
| 1453 | 261 |
| 1775 | 262 if (rc != NGX_OK && (of->err == 0 || !of->errors)) { |
| 263 goto failed; | |
| 264 } | |
| 265 | |
| 266 if (of->is_dir) { | |
| 267 | |
| 268 if (file->is_dir || file->err) { | |
| 269 goto update; | |
| 270 } | |
| 271 | |
| 272 /* file became directory */ | |
| 1453 | 273 |
| 1775 | 274 } else if (of->err == 0) { /* file */ |
| 275 | |
| 276 if (file->is_dir || file->err) { | |
| 277 goto add_event; | |
| 278 } | |
| 1453 | 279 |
| 2070 | 280 if (of->uniq == file->uniq) { |
| 1453 | 281 |
| 1775 | 282 file->count++; |
| 1453 | 283 |
| 1775 | 284 if (file->event) { |
| 285 file->use_event = 1; | |
| 1453 | 286 } |
| 287 | |
| 1775 | 288 goto renew; |
| 289 } | |
| 1453 | 290 |
| 1775 | 291 /* file was changed */ |
| 1453 | 292 |
| 1775 | 293 } else { /* error to cache */ |
| 1453 | 294 |
| 1775 | 295 if (file->err || file->is_dir) { |
| 296 goto update; | |
| 1453 | 297 } |
| 298 | |
| 1775 | 299 /* file was removed, etc. */ |
| 300 } | |
| 301 | |
| 302 if (file->count == 0) { | |
| 303 | |
| 304 ngx_open_file_del_event(file); | |
| 1453 | 305 |
| 1775 | 306 if (ngx_close_file(file->fd) == NGX_FILE_ERROR) { |
| 307 ngx_log_error(NGX_LOG_ALERT, pool->log, ngx_errno, | |
| 308 ngx_close_file_n " \"%s\" failed", | |
| 309 name->data); | |
| 310 } | |
| 1453 | 311 |
| 1775 | 312 goto add_event; |
| 313 } | |
| 314 | |
| 315 ngx_rbtree_delete(&cache->rbtree, &file->node); | |
| 316 | |
| 317 cache->current--; | |
| 318 | |
| 319 file->close = 1; | |
| 320 | |
| 321 goto create; | |
| 1453 | 322 } |
| 323 | |
| 324 /* not found */ | |
| 325 | |
| 326 rc = ngx_open_and_stat_file(name->data, of, pool->log); | |
| 327 | |
| 328 if (rc != NGX_OK && (of->err == 0 || !of->errors)) { | |
| 329 goto failed; | |
| 330 } | |
| 331 | |
| 332 create: | |
| 333 | |
| 334 if (cache->current >= cache->max) { | |
| 335 ngx_expire_old_cached_files(cache, 0, pool->log); | |
| 336 } | |
| 337 | |
| 338 file = ngx_alloc(sizeof(ngx_cached_open_file_t), pool->log); | |
| 339 | |
| 340 if (file == NULL) { | |
| 341 goto failed; | |
| 342 } | |
| 343 | |
| 344 file->name = ngx_alloc(name->len + 1, pool->log); | |
| 345 | |
| 346 if (file->name == NULL) { | |
| 347 ngx_free(file); | |
| 348 file = NULL; | |
| 349 goto failed; | |
| 350 } | |
| 351 | |
| 352 ngx_cpystrn(file->name, name->data, name->len + 1); | |
| 353 | |
| 354 file->node.key = hash; | |
| 355 | |
| 356 ngx_rbtree_insert(&cache->rbtree, &file->node); | |
| 357 | |
| 358 cache->current++; | |
| 359 | |
| 1775 | 360 file->uses = 1; |
| 1453 | 361 file->count = 0; |
| 1775 | 362 file->event = NULL; |
| 363 | |
| 364 add_event: | |
| 365 | |
| 366 ngx_open_file_add_event(cache, file, of, pool->log); | |
| 1453 | 367 |
| 368 update: | |
| 369 | |
| 370 file->fd = of->fd; | |
| 371 file->err = of->err; | |
| 372 | |
| 373 if (of->err == 0) { | |
| 374 file->uniq = of->uniq; | |
| 375 file->mtime = of->mtime; | |
| 376 file->size = of->size; | |
| 377 | |
| 378 file->close = 0; | |
| 379 | |
| 380 file->is_dir = of->is_dir; | |
| 381 file->is_file = of->is_file; | |
| 382 file->is_link = of->is_link; | |
| 383 file->is_exec = of->is_exec; | |
|
2246
987831d73bd8
cache directio flag in open file cache
Igor Sysoev <igor@sysoev.ru>
parents:
2231
diff
changeset
|
384 file->is_directio = of->is_directio; |
| 1453 | 385 |
| 386 if (!of->is_dir) { | |
| 387 file->count++; | |
| 388 } | |
| 389 } | |
| 390 | |
| 391 renew: | |
| 392 | |
| 393 file->created = now; | |
| 394 | |
| 395 found: | |
| 396 | |
| 397 file->accessed = now; | |
| 398 | |
| 1765 | 399 ngx_queue_insert_head(&cache->expire_queue, &file->queue); |
| 1453 | 400 |
| 1772 | 401 ngx_log_debug5(NGX_LOG_DEBUG_CORE, pool->log, 0, |
| 402 "cached open file: %s, fd:%d, c:%d, e:%d, u:%d", | |
| 403 file->name, file->fd, file->count, file->err, file->uses); | |
| 1453 | 404 |
| 405 if (of->err == 0) { | |
| 406 | |
| 407 if (!of->is_dir) { | |
| 408 cln->handler = ngx_open_file_cleanup; | |
| 409 ofcln = cln->data; | |
| 410 | |
| 411 ofcln->cache = cache; | |
| 412 ofcln->file = file; | |
| 1772 | 413 ofcln->min_uses = of->min_uses; |
| 1453 | 414 ofcln->log = pool->log; |
| 415 } | |
| 416 | |
| 417 return NGX_OK; | |
| 418 } | |
| 419 | |
| 420 return NGX_ERROR; | |
| 421 | |
| 422 failed: | |
| 423 | |
|
1988
34051c712e41
fix segfault when file is deleted and open_file_cache_errors is off
Igor Sysoev <igor@sysoev.ru>
parents:
1987
diff
changeset
|
424 if (file) { |
| 1453 | 425 ngx_rbtree_delete(&cache->rbtree, &file->node); |
| 426 | |
| 427 cache->current--; | |
| 428 | |
|
1988
34051c712e41
fix segfault when file is deleted and open_file_cache_errors is off
Igor Sysoev <igor@sysoev.ru>
parents:
1987
diff
changeset
|
429 if (file->count == 0) { |
|
34051c712e41
fix segfault when file is deleted and open_file_cache_errors is off
Igor Sysoev <igor@sysoev.ru>
parents:
1987
diff
changeset
|
430 |
|
2006
b52cb9bf2064
style fix: remove tabs and trailing spaces
Igor Sysoev <igor@sysoev.ru>
parents:
1988
diff
changeset
|
431 if (file->fd != NGX_INVALID_FILE) { |
|
b52cb9bf2064
style fix: remove tabs and trailing spaces
Igor Sysoev <igor@sysoev.ru>
parents:
1988
diff
changeset
|
432 if (ngx_close_file(file->fd) == NGX_FILE_ERROR) { |
|
b52cb9bf2064
style fix: remove tabs and trailing spaces
Igor Sysoev <igor@sysoev.ru>
parents:
1988
diff
changeset
|
433 ngx_log_error(NGX_LOG_ALERT, pool->log, ngx_errno, |
|
b52cb9bf2064
style fix: remove tabs and trailing spaces
Igor Sysoev <igor@sysoev.ru>
parents:
1988
diff
changeset
|
434 ngx_close_file_n " \"%s\" failed", |
|
1988
34051c712e41
fix segfault when file is deleted and open_file_cache_errors is off
Igor Sysoev <igor@sysoev.ru>
parents:
1987
diff
changeset
|
435 file->name); |
|
2006
b52cb9bf2064
style fix: remove tabs and trailing spaces
Igor Sysoev <igor@sysoev.ru>
parents:
1988
diff
changeset
|
436 } |
|
b52cb9bf2064
style fix: remove tabs and trailing spaces
Igor Sysoev <igor@sysoev.ru>
parents:
1988
diff
changeset
|
437 } |
|
1988
34051c712e41
fix segfault when file is deleted and open_file_cache_errors is off
Igor Sysoev <igor@sysoev.ru>
parents:
1987
diff
changeset
|
438 |
|
34051c712e41
fix segfault when file is deleted and open_file_cache_errors is off
Igor Sysoev <igor@sysoev.ru>
parents:
1987
diff
changeset
|
439 ngx_free(file->name); |
|
34051c712e41
fix segfault when file is deleted and open_file_cache_errors is off
Igor Sysoev <igor@sysoev.ru>
parents:
1987
diff
changeset
|
440 ngx_free(file); |
|
34051c712e41
fix segfault when file is deleted and open_file_cache_errors is off
Igor Sysoev <igor@sysoev.ru>
parents:
1987
diff
changeset
|
441 |
|
34051c712e41
fix segfault when file is deleted and open_file_cache_errors is off
Igor Sysoev <igor@sysoev.ru>
parents:
1987
diff
changeset
|
442 } else { |
|
34051c712e41
fix segfault when file is deleted and open_file_cache_errors is off
Igor Sysoev <igor@sysoev.ru>
parents:
1987
diff
changeset
|
443 file->close = 1; |
| 1453 | 444 } |
| 445 } | |
| 446 | |
| 447 if (of->fd != NGX_INVALID_FILE) { | |
| 448 if (ngx_close_file(of->fd) == NGX_FILE_ERROR) { | |
| 449 ngx_log_error(NGX_LOG_ALERT, pool->log, ngx_errno, | |
| 450 ngx_close_file_n " \"%s\" failed", name->data); | |
| 451 } | |
| 452 } | |
| 453 | |
| 454 return NGX_ERROR; | |
| 455 } | |
| 456 | |
| 457 | |
| 458 static ngx_int_t | |
| 459 ngx_open_and_stat_file(u_char *name, ngx_open_file_info_t *of, ngx_log_t *log) | |
| 460 { | |
| 461 ngx_fd_t fd; | |
| 462 ngx_file_info_t fi; | |
| 463 | |
| 2070 | 464 if (of->fd != NGX_INVALID_FILE) { |
| 1453 | 465 |
|
2782
4bd7825fab80
uniform ngx_file_info() interface with ngx_fd_info()
Igor Sysoev <igor@sysoev.ru>
parents:
2756
diff
changeset
|
466 if (ngx_file_info(name, &fi) == NGX_FILE_ERROR) { |
|
2756
09cab3f8d92e
*) of.test_only to not open file if only stat() is enough
Igor Sysoev <igor@sysoev.ru>
parents:
2629
diff
changeset
|
467 of->failed = ngx_file_info_n; |
|
2069
23930ccd2642
use ngx_file_info() and test uniq if file is already open
Igor Sysoev <igor@sysoev.ru>
parents:
2063
diff
changeset
|
468 goto failed; |
| 1453 | 469 } |
| 470 | |
| 2070 | 471 if (of->uniq == ngx_file_uniq(&fi)) { |
|
2069
23930ccd2642
use ngx_file_info() and test uniq if file is already open
Igor Sysoev <igor@sysoev.ru>
parents:
2063
diff
changeset
|
472 goto done; |
|
23930ccd2642
use ngx_file_info() and test uniq if file is already open
Igor Sysoev <igor@sysoev.ru>
parents:
2063
diff
changeset
|
473 } |
| 1453 | 474 |
| 2070 | 475 } else if (of->test_dir) { |
| 476 | |
|
2782
4bd7825fab80
uniform ngx_file_info() interface with ngx_fd_info()
Igor Sysoev <igor@sysoev.ru>
parents:
2756
diff
changeset
|
477 if (ngx_file_info(name, &fi) == NGX_FILE_ERROR) { |
|
2756
09cab3f8d92e
*) of.test_only to not open file if only stat() is enough
Igor Sysoev <igor@sysoev.ru>
parents:
2629
diff
changeset
|
478 of->failed = ngx_file_info_n; |
| 2070 | 479 goto failed; |
| 480 } | |
| 481 | |
|
2460
225fa4abd76f
test ngx_file_info() result, the bug has been introduced in r2070
Igor Sysoev <igor@sysoev.ru>
parents:
2246
diff
changeset
|
482 if (ngx_is_dir(&fi)) { |
|
2069
23930ccd2642
use ngx_file_info() and test uniq if file is already open
Igor Sysoev <igor@sysoev.ru>
parents:
2063
diff
changeset
|
483 goto done; |
| 1453 | 484 } |
| 485 } | |
| 486 | |
| 2072 | 487 if (!of->log) { |
| 488 fd = ngx_open_file(name, NGX_FILE_RDONLY, NGX_FILE_OPEN, 0); | |
| 489 | |
| 490 } else { | |
|
2629
367b29612a00
Win32 appends synchronized if only FILE_APPEND_DATA and SYNCHRONIZE are set
Igor Sysoev <igor@sysoev.ru>
parents:
2628
diff
changeset
|
491 fd = ngx_open_file(name, NGX_FILE_APPEND, NGX_FILE_CREATE_OR_OPEN, |
|
367b29612a00
Win32 appends synchronized if only FILE_APPEND_DATA and SYNCHRONIZE are set
Igor Sysoev <igor@sysoev.ru>
parents:
2628
diff
changeset
|
492 NGX_FILE_DEFAULT_ACCESS); |
| 2072 | 493 } |
| 1453 | 494 |
| 495 if (fd == NGX_INVALID_FILE) { | |
|
2756
09cab3f8d92e
*) of.test_only to not open file if only stat() is enough
Igor Sysoev <igor@sysoev.ru>
parents:
2629
diff
changeset
|
496 of->failed = ngx_open_file_n; |
|
2069
23930ccd2642
use ngx_file_info() and test uniq if file is already open
Igor Sysoev <igor@sysoev.ru>
parents:
2063
diff
changeset
|
497 goto failed; |
| 1453 | 498 } |
| 499 | |
| 500 if (ngx_fd_info(fd, &fi) == NGX_FILE_ERROR) { | |
| 501 ngx_log_error(NGX_LOG_CRIT, log, ngx_errno, | |
| 502 ngx_fd_info_n " \"%s\" failed", name); | |
| 503 | |
| 504 if (ngx_close_file(fd) == NGX_FILE_ERROR) { | |
| 505 ngx_log_error(NGX_LOG_ALERT, log, ngx_errno, | |
| 506 ngx_close_file_n " \"%s\" failed", name); | |
| 507 } | |
| 508 | |
|
2069
23930ccd2642
use ngx_file_info() and test uniq if file is already open
Igor Sysoev <igor@sysoev.ru>
parents:
2063
diff
changeset
|
509 of->fd = NGX_INVALID_FILE; |
|
23930ccd2642
use ngx_file_info() and test uniq if file is already open
Igor Sysoev <igor@sysoev.ru>
parents:
2063
diff
changeset
|
510 |
| 1453 | 511 return NGX_ERROR; |
| 512 } | |
| 513 | |
| 514 if (ngx_is_dir(&fi)) { | |
| 515 if (ngx_close_file(fd) == NGX_FILE_ERROR) { | |
| 516 ngx_log_error(NGX_LOG_ALERT, log, ngx_errno, | |
| 517 ngx_close_file_n " \"%s\" failed", name); | |
| 518 } | |
| 519 | |
|
2069
23930ccd2642
use ngx_file_info() and test uniq if file is already open
Igor Sysoev <igor@sysoev.ru>
parents:
2063
diff
changeset
|
520 of->fd = NGX_INVALID_FILE; |
|
23930ccd2642
use ngx_file_info() and test uniq if file is already open
Igor Sysoev <igor@sysoev.ru>
parents:
2063
diff
changeset
|
521 |
|
23930ccd2642
use ngx_file_info() and test uniq if file is already open
Igor Sysoev <igor@sysoev.ru>
parents:
2063
diff
changeset
|
522 } else { |
|
23930ccd2642
use ngx_file_info() and test uniq if file is already open
Igor Sysoev <igor@sysoev.ru>
parents:
2063
diff
changeset
|
523 of->fd = fd; |
| 2129 | 524 |
| 525 if (of->directio <= ngx_file_size(&fi)) { | |
|
2246
987831d73bd8
cache directio flag in open file cache
Igor Sysoev <igor@sysoev.ru>
parents:
2231
diff
changeset
|
526 if (ngx_directio_on(fd) == -1) { |
| 2129 | 527 ngx_log_error(NGX_LOG_ALERT, log, ngx_errno, |
|
2246
987831d73bd8
cache directio flag in open file cache
Igor Sysoev <igor@sysoev.ru>
parents:
2231
diff
changeset
|
528 ngx_directio_on_n " \"%s\" failed", name); |
|
2231
8564129d49b6
*) handle unaligned file part for directio
Igor Sysoev <igor@sysoev.ru>
parents:
2129
diff
changeset
|
529 |
|
8564129d49b6
*) handle unaligned file part for directio
Igor Sysoev <igor@sysoev.ru>
parents:
2129
diff
changeset
|
530 } else { |
|
8564129d49b6
*) handle unaligned file part for directio
Igor Sysoev <igor@sysoev.ru>
parents:
2129
diff
changeset
|
531 of->is_directio = 1; |
| 2129 | 532 } |
| 533 } | |
| 1453 | 534 } |
| 535 | |
|
2069
23930ccd2642
use ngx_file_info() and test uniq if file is already open
Igor Sysoev <igor@sysoev.ru>
parents:
2063
diff
changeset
|
536 done: |
|
23930ccd2642
use ngx_file_info() and test uniq if file is already open
Igor Sysoev <igor@sysoev.ru>
parents:
2063
diff
changeset
|
537 |
| 1453 | 538 of->uniq = ngx_file_uniq(&fi); |
| 539 of->mtime = ngx_file_mtime(&fi); | |
| 540 of->size = ngx_file_size(&fi); | |
| 541 of->is_dir = ngx_is_dir(&fi); | |
| 542 of->is_file = ngx_is_file(&fi); | |
| 543 of->is_link = ngx_is_link(&fi); | |
| 544 of->is_exec = ngx_is_exec(&fi); | |
| 545 | |
| 546 return NGX_OK; | |
|
2069
23930ccd2642
use ngx_file_info() and test uniq if file is already open
Igor Sysoev <igor@sysoev.ru>
parents:
2063
diff
changeset
|
547 |
|
23930ccd2642
use ngx_file_info() and test uniq if file is already open
Igor Sysoev <igor@sysoev.ru>
parents:
2063
diff
changeset
|
548 failed: |
|
23930ccd2642
use ngx_file_info() and test uniq if file is already open
Igor Sysoev <igor@sysoev.ru>
parents:
2063
diff
changeset
|
549 |
|
23930ccd2642
use ngx_file_info() and test uniq if file is already open
Igor Sysoev <igor@sysoev.ru>
parents:
2063
diff
changeset
|
550 of->fd = NGX_INVALID_FILE; |
|
23930ccd2642
use ngx_file_info() and test uniq if file is already open
Igor Sysoev <igor@sysoev.ru>
parents:
2063
diff
changeset
|
551 of->err = ngx_errno; |
|
23930ccd2642
use ngx_file_info() and test uniq if file is already open
Igor Sysoev <igor@sysoev.ru>
parents:
2063
diff
changeset
|
552 |
|
23930ccd2642
use ngx_file_info() and test uniq if file is already open
Igor Sysoev <igor@sysoev.ru>
parents:
2063
diff
changeset
|
553 return NGX_ERROR; |
| 1453 | 554 } |
| 555 | |
| 556 | |
| 1775 | 557 /* |
| 558 * we ignore any possible event setting error and | |
| 559 * fallback to usual periodic file retests | |
| 560 */ | |
| 561 | |
| 562 static void | |
| 563 ngx_open_file_add_event(ngx_open_file_cache_t *cache, | |
| 564 ngx_cached_open_file_t *file, ngx_open_file_info_t *of, ngx_log_t *log) | |
| 565 { | |
| 566 ngx_open_file_cache_event_t *fev; | |
| 567 | |
| 568 if (!(ngx_event_flags & NGX_USE_VNODE_EVENT) | |
| 569 || !of->events | |
| 570 || file->event | |
| 571 || of->fd == NGX_INVALID_FILE | |
| 572 || file->uses < of->min_uses) | |
| 573 { | |
| 574 return; | |
| 575 } | |
| 576 | |
| 2070 | 577 file->use_event = 0; |
| 578 | |
| 1775 | 579 file->event = ngx_calloc(sizeof(ngx_event_t), log); |
| 580 if (file->event== NULL) { | |
| 581 return; | |
| 582 } | |
| 583 | |
| 584 fev = ngx_alloc(sizeof(ngx_open_file_cache_event_t), log); | |
| 585 if (fev == NULL) { | |
| 586 ngx_free(file->event); | |
| 587 file->event = NULL; | |
| 588 return; | |
| 589 } | |
| 590 | |
| 591 fev->fd = of->fd; | |
| 592 fev->file = file; | |
| 593 fev->cache = cache; | |
| 594 | |
| 595 file->event->handler = ngx_open_file_cache_remove; | |
| 596 file->event->data = fev; | |
| 597 | |
| 598 /* | |
| 599 * although vnode event may be called while ngx_cycle->poll | |
| 600 * destruction, however, cleanup procedures are run before any | |
| 601 * memory freeing and events will be canceled. | |
| 602 */ | |
| 603 | |
| 604 file->event->log = ngx_cycle->log; | |
| 605 | |
| 606 if (ngx_add_event(file->event, NGX_VNODE_EVENT, NGX_ONESHOT_EVENT) | |
| 607 != NGX_OK) | |
| 608 { | |
| 609 ngx_free(file->event->data); | |
| 610 ngx_free(file->event); | |
| 611 file->event = NULL; | |
| 612 return; | |
| 613 } | |
| 614 | |
| 615 /* | |
| 2071 | 616 * we do not set file->use_event here because there may be a race |
| 617 * condition: a file may be deleted between opening the file and | |
| 618 * adding event, so we rely upon event notification only after | |
| 619 * one file revalidation on next file access | |
| 1775 | 620 */ |
| 621 | |
| 622 return; | |
| 623 } | |
| 624 | |
| 625 | |
| 1453 | 626 static void |
| 627 ngx_open_file_cleanup(void *data) | |
| 628 { | |
| 629 ngx_open_file_cache_cleanup_t *c = data; | |
| 630 | |
| 631 c->file->count--; | |
| 632 | |
| 1772 | 633 ngx_close_cached_file(c->cache, c->file, c->min_uses, c->log); |
| 1453 | 634 |
| 635 /* drop one or two expired open files */ | |
| 636 ngx_expire_old_cached_files(c->cache, 1, c->log); | |
| 637 } | |
| 638 | |
| 639 | |
| 640 static void | |
| 641 ngx_close_cached_file(ngx_open_file_cache_t *cache, | |
| 1772 | 642 ngx_cached_open_file_t *file, ngx_uint_t min_uses, ngx_log_t *log) |
| 1453 | 643 { |
| 1772 | 644 ngx_log_debug5(NGX_LOG_DEBUG_CORE, log, 0, |
| 645 "close cached open file: %s, fd:%d, c:%d, u:%d, %d", | |
| 646 file->name, file->fd, file->count, file->uses, file->close); | |
| 1453 | 647 |
| 648 if (!file->close) { | |
| 649 | |
| 650 file->accessed = ngx_time(); | |
| 651 | |
| 1765 | 652 ngx_queue_remove(&file->queue); |
| 1453 | 653 |
| 1765 | 654 ngx_queue_insert_head(&cache->expire_queue, &file->queue); |
| 1453 | 655 |
| 1772 | 656 if (file->uses >= min_uses || file->count) { |
| 657 return; | |
| 658 } | |
| 1453 | 659 } |
| 660 | |
| 1775 | 661 ngx_open_file_del_event(file); |
| 1453 | 662 |
| 663 if (file->count) { | |
| 664 return; | |
| 665 } | |
| 666 | |
| 1772 | 667 if (file->fd != NGX_INVALID_FILE) { |
| 668 | |
| 669 if (ngx_close_file(file->fd) == NGX_FILE_ERROR) { | |
| 670 ngx_log_error(NGX_LOG_ALERT, log, ngx_errno, | |
| 671 ngx_close_file_n " \"%s\" failed", file->name); | |
| 672 } | |
| 673 | |
| 674 file->fd = NGX_INVALID_FILE; | |
| 675 } | |
| 676 | |
| 677 if (!file->close) { | |
| 678 return; | |
| 1453 | 679 } |
| 680 | |
| 681 ngx_free(file->name); | |
| 682 ngx_free(file); | |
| 683 } | |
| 684 | |
| 685 | |
| 686 static void | |
| 1775 | 687 ngx_open_file_del_event(ngx_cached_open_file_t *file) |
| 688 { | |
| 689 if (file->event == NULL) { | |
| 690 return; | |
| 691 } | |
| 692 | |
| 693 (void) ngx_del_event(file->event, NGX_VNODE_EVENT, | |
| 694 file->count ? NGX_FLUSH_EVENT : NGX_CLOSE_EVENT); | |
| 695 | |
| 696 ngx_free(file->event->data); | |
| 697 ngx_free(file->event); | |
| 698 file->event = NULL; | |
| 699 file->use_event = 0; | |
| 700 } | |
| 701 | |
| 702 | |
| 703 static void | |
| 1453 | 704 ngx_expire_old_cached_files(ngx_open_file_cache_t *cache, ngx_uint_t n, |
| 705 ngx_log_t *log) | |
| 706 { | |
| 707 time_t now; | |
| 1765 | 708 ngx_queue_t *q; |
| 1453 | 709 ngx_cached_open_file_t *file; |
| 710 | |
| 711 now = ngx_time(); | |
| 712 | |
| 713 /* | |
| 714 * n == 1 deletes one or two inactive files | |
| 715 * n == 0 deletes least recently used file by force | |
| 716 * and one or two inactive files | |
| 717 */ | |
| 718 | |
| 719 while (n < 3) { | |
| 720 | |
| 1765 | 721 if (ngx_queue_empty(&cache->expire_queue)) { |
| 1453 | 722 return; |
| 723 } | |
| 724 | |
| 1765 | 725 q = ngx_queue_last(&cache->expire_queue); |
| 726 | |
| 727 file = ngx_queue_data(q, ngx_cached_open_file_t, queue); | |
| 728 | |
| 1453 | 729 if (n++ != 0 && now - file->accessed <= cache->inactive) { |
| 730 return; | |
| 731 } | |
| 732 | |
| 1765 | 733 ngx_queue_remove(q); |
| 1453 | 734 |
| 735 ngx_rbtree_delete(&cache->rbtree, &file->node); | |
| 736 | |
| 737 cache->current--; | |
| 738 | |
| 739 ngx_log_debug1(NGX_LOG_DEBUG_CORE, log, 0, | |
| 740 "expire cached open file: %s", file->name); | |
| 741 | |
| 742 if (!file->err && !file->is_dir) { | |
| 743 file->close = 1; | |
| 1772 | 744 ngx_close_cached_file(cache, file, 0, log); |
| 1453 | 745 |
| 746 } else { | |
| 747 ngx_free(file->name); | |
| 748 ngx_free(file); | |
| 749 } | |
| 750 } | |
| 751 } | |
| 752 | |
| 753 | |
| 754 static void | |
| 755 ngx_open_file_cache_rbtree_insert_value(ngx_rbtree_node_t *temp, | |
| 756 ngx_rbtree_node_t *node, ngx_rbtree_node_t *sentinel) | |
| 757 { | |
| 758 ngx_rbtree_node_t **p; | |
| 759 ngx_cached_open_file_t *file, *file_temp; | |
| 760 | |
| 761 for ( ;; ) { | |
| 762 | |
| 763 if (node->key < temp->key) { | |
| 764 | |
| 765 p = &temp->left; | |
| 766 | |
| 767 } else if (node->key > temp->key) { | |
| 768 | |
| 769 p = &temp->right; | |
| 770 | |
| 771 } else { /* node->key == temp->key */ | |
| 772 | |
| 773 file = (ngx_cached_open_file_t *) node; | |
| 774 file_temp = (ngx_cached_open_file_t *) temp; | |
| 775 | |
| 776 p = (ngx_strcmp(file->name, file_temp->name) < 0) | |
| 777 ? &temp->left : &temp->right; | |
| 778 } | |
| 779 | |
| 780 if (*p == sentinel) { | |
| 781 break; | |
| 782 } | |
| 783 | |
| 784 temp = *p; | |
| 785 } | |
| 786 | |
| 787 *p = node; | |
| 788 node->parent = temp; | |
| 789 node->left = sentinel; | |
| 790 node->right = sentinel; | |
| 791 ngx_rbt_red(node); | |
| 792 } | |
| 793 | |
| 794 | |
| 1775 | 795 static ngx_cached_open_file_t * |
| 796 ngx_open_file_lookup(ngx_open_file_cache_t *cache, ngx_str_t *name, | |
| 797 uint32_t hash) | |
| 798 { | |
| 799 ngx_int_t rc; | |
| 800 ngx_rbtree_node_t *node, *sentinel; | |
| 801 ngx_cached_open_file_t *file; | |
| 802 | |
| 803 node = cache->rbtree.root; | |
| 804 sentinel = cache->rbtree.sentinel; | |
| 805 | |
| 806 while (node != sentinel) { | |
| 807 | |
| 808 if (hash < node->key) { | |
| 809 node = node->left; | |
| 810 continue; | |
| 811 } | |
| 812 | |
| 813 if (hash > node->key) { | |
| 814 node = node->right; | |
| 815 continue; | |
| 816 } | |
| 817 | |
| 818 /* hash == node->key */ | |
| 819 | |
| 820 do { | |
| 821 file = (ngx_cached_open_file_t *) node; | |
| 822 | |
| 823 rc = ngx_strcmp(name->data, file->name); | |
| 824 | |
| 825 if (rc == 0) { | |
| 826 return file; | |
| 827 } | |
| 828 | |
| 829 node = (rc < 0) ? node->left : node->right; | |
| 830 | |
| 831 } while (node != sentinel && hash == node->key); | |
| 832 | |
| 833 break; | |
| 834 } | |
| 835 | |
| 836 return NULL; | |
| 837 } | |
| 838 | |
| 839 | |
| 1453 | 840 static void |
| 841 ngx_open_file_cache_remove(ngx_event_t *ev) | |
| 842 { | |
| 843 ngx_cached_open_file_t *file; | |
| 844 ngx_open_file_cache_event_t *fev; | |
| 845 | |
| 846 fev = ev->data; | |
| 847 file = fev->file; | |
| 848 | |
| 1765 | 849 ngx_queue_remove(&file->queue); |
| 1453 | 850 |
| 851 ngx_rbtree_delete(&fev->cache->rbtree, &file->node); | |
| 852 | |
| 853 fev->cache->current--; | |
| 854 | |
| 855 /* NGX_ONESHOT_EVENT was already deleted */ | |
| 856 file->event = NULL; | |
| 2070 | 857 file->use_event = 0; |
| 1453 | 858 |
| 859 file->close = 1; | |
| 860 | |
| 1772 | 861 ngx_close_cached_file(fev->cache, file, 0, ev->log); |
| 1453 | 862 |
| 863 /* free memory only when fev->cache and fev->file are already not needed */ | |
| 864 | |
| 865 ngx_free(ev->data); | |
| 866 ngx_free(ev); | |
| 867 } |