Hello!

On Mon, Sep 12, 2011 at 08:06:36PM +0300, Андрей Василишин wrote:

>
> >А можно ещё для чистоты эксперимента патч откатить, и убедиться
> >что без патча проблема воспроизводится?
> >
> >Потому как я решительно не вижу причин, по которым данный патч мог
> >бы помочь.
> >
>
> В общем опять проблема воспроизвелась, при перемотке прыгает на
> начало (нгинкс тут без патча).

Судя по debug log'у, файл 110912-v.flv писали в процессе его
скачивания клиентами, в результате закешированная длина в
open_file_cache в разных рабочих процессах разная. Соответственно
если запрос со start=<много> попадал в рабочий процесс, считавший,
что длина файла меньше, в ответ отдавался полный файл.

В общем случае совет простой: не пишите файлы неатомарно, а если
пишите - будте готовы к сюрпризам.

В данном случае, судя по всему, ситуация усугублялась проблемой в
open_file_cache: после дописывания файла закешированный размер не
обновлялся. Прилагающийся патч проблему исправляет.

Maxim Dounin
# HG changeset patch
# User Maxim Dounin <mdounin@mdounin.ru>
# Date 1315864171 -14400
# Node ID 0786be1906af99a4ba20edeb8b24348ba6a88bef
# Parent 83b74d7020ba5d02b51f9e2e162bca943d47c486
Bugfix: open_file_cache did not update file info on retest.

If file inode was not changed, cached file information was not updated
on retest. As a result stale information might be cached forever if file
attributes was changed and/or file was extended.

diff --git a/src/core/ngx_open_file_cache.c b/src/core/ngx_open_file_cache.c
--- a/src/core/ngx_open_file_cache.c
+++ b/src/core/ngx_open_file_cache.c
@@ -284,13 +284,11 @@ ngx_open_cached_file(ngx_open_file_cache

if (of->uniq == file->uniq) {

- file->count++;
-
if (file->event) {
file->use_event = 1;
}

- goto renew;
+ goto update;
}

/* file was changed */
@@ -394,8 +392,6 @@ update:
}
}

-renew:
-
file->created = now;

found:
_______________________________________________
nginx-ru mailing list
nginx-ru@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-ru