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