Nginx-ru mailing list archive (nginx-ru@sysoev.ru)
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] http_request: fix a segmentation fault
On Thu, May 28, 2009 at 08:43:24AM +0400, Igor Sysoev wrote:
> On Wed, May 27, 2009 at 01:57:32PM +0400, Kirill A. Korinskiy wrote:
>
> > At Wed, 27 May 2009 04:19:05 +0400,
> > Maxim Dounin <mdounin@xxxxxxxxxx> wrote:
> > >
> > >
> > > Не то чтобы совсем не попал, но не попал в шарик.
> > >
> >
> > В общем третий вариант патча, и итоговый вариант. Да, с ночным
> > хаченьем пора завязывать. Ну или ждать до утра.
>
> Я пару раз наблюдал nginx в ситуации low memory и каждый раз он работал
> продолжительное время без падений (это не относится к nginx'у с перловым
> модулем, потому что перл в таких случаях падает очень легко).
>
> На самом деле, в этом месте проверки косметические, потому что выделения
> небольшие и делаются из только что выделенного пула, в котором по
> умолчанию 16К. Поэтому, чтобы уронить nginx в этом месте, нужно или создать
> около 1000 (для amd64) переменных в конфиге для дефолтного request_pool_size,
> или уменьшать пул: для 4K понадобится 250 переменных, что тоже не мало.
Вернее, для 16К пула нужно тоже 250 переменных, а не 1000.
--
Игорь Сысоев
http://sysoev.ru
|
