Sunday, December 6, 2009, 12:18:09 AM, fixxxer wrote:

> И, да, весь этот прыг-скок из fastcgi.c в fpm_* с выделением в
> отдельный sapi как то совсем неактуален. Отрефакторить бы это дело,
> заодно и разобраться, как работает.

:)

> В общем, если из настоящих сварщиков совсем некому, я попробую, но
> ничего не обещаю =)

> Еще конфиг бы переделать, я видел не одного и не двух админов, которым
> резко плохеет от xml-я =) Может, libconfuse? Хотя, лишняя
> зависимость...

Не то, чтобы совсем плохеет, но немного неудобно.

Естественно хочется иметь конфиги c-style более свойственные
unix-системам.

--
Sergey

Здравствуйте.

лог я приводил
а дебажить я не умею
так что все что мог я дал

> stanlee@newmail.ru wrote:
>> Здравствуйте.
>>
>> ничего странного
>> эта ситуация проявилась у одного пула из ста
>> ввести фпм в продакшен была моя идея, я же могу и отказаться
>> к томуже у людей есть динамическая возможность прыгать с фмп на фцги и
>> обратно, что и было сделано
>>
>> а разбираться мне пока некогда, есть другие задачи боле важные
>>
> Ну как-то мне кажется, что ваша позиция странно согласуется с принципами
> OpenSource. Я стараюсь как минимум оставить баг репорт для
> разработчиков, с максимумом полезной информации. А так - баг никогда и
> не будет пофикшен.

On 06.12.2009 01:18, fixxxer wrote:
> И, да, весь этот прыг-скок из fastcgi.c в fpm_* с выделением в
> отдельный sapi как то совсем неактуален. Отрефакторить бы это дело,
> заодно и разобраться, как работает.

You're always welcome.

> В общем, если из настоящих сварщиков совсем некому, я попробую, но
> ничего не обещаю =)

Некому, некому.
У меня не 6 рук.

> Еще конфиг бы переделать, я видел не одного и не двух админов, которым
> резко плохеет от xml-я =) Может, libconfuse? Хотя, лишняя
> зависимость...

Это последняя проблема, которая должна вас сейчас волновать.
Эта часть кода работает нормально, никаких проблем там нет, поэтому трогать её не надо.
Религиозные чувства и персональные предпочтения можно отложить на время.

--
Wbr,
Antony Dovgal
---
http://pinba.org - realtime statistics for PHP

On 06.12.2009 01:05, fixxxer wrote:
> Оххх, как, оказывается, все запущено в этом 0.6.

Здесь надо учитывать, что в 0.6 изначально cgi_main.c и fastcgi.c шли аж из 4-ки.
Поэтому я ручками проапдейтил их до более-менее свежей версии.
Ну и по пути, конечно, сломал всё =) Так что, это не 0.6, эт всё я.

> Патч: http://symbi.org/misc/php_5_3_fpm_lost_request_phases.patch

Спасибо, Фиксер.
Приложил патч, кроме первой его части.

> С сингалами - я не воткнул, в каких случаях таки надо делать
> безусловный exit по sigterm - так что пускай пока делает всегда...
> лучше, чем игнорировать, по крайней мере, а дальше надо смотреть.

Я бы пока не спешил с такими патчами.
Там есть еще куча кода на libevent, которая тоже занимается обработкой сигналов,
и у меня есть впечатоление, что код в fastcgi.c просто мертвый и его надо дропнуть.

> Ну и
> еще конечно реакция на sighup неадекватная (кстати, оно так и было,
> или это тоже нововведение?).

А? См. выше.

--
Wbr,
Antony Dovgal
---
http://pinba.org - realtime statistics for PHP

On 05.12.2009 18:28, jahson wrote:
> Собрал на osx 10.5.8

Вы только там можете протестировать?

> , всё работает, но основной лог стал со скоростью
> света наполняться повторяющимися наборами записей вида
> Dec 05 21:49:02.526089 [NOTICE] fpm_php_trace(), line 170: finished
> trace of 859

Возможно, вам стоит сделать log_level не notice, а warning?
Так, конечно, вы видите все возможные нотисы.

> index.php - индекс symfony (он ещё на что-то ругается в slowlog - в
> виде бэктрейса, видимо из-за нового пхп), а info.php - просто phpinfo
> (). В дальнейшем использовал только info.php.
>
> Ясно, что 2803 секунды (порядка 45 минут) никто не выполняется - всё
> достаточно быстро бегает. slowlog забивается записями вида
> Dec 05 21:53:46.119609 pid 860 (pool default)
> script_filename =·

Окей, принял к сведению.

> и вот что выкопал - основное действо происходит
> в ветке, которая начинается со 128 строки файла fpm_request.c.

Сделайте svn up, плз, что-то я полностью запутался в номерах строк.

> Если отключить slowlog - то никто не ругается, ваш КО.

Понял, до туда я еще не дошел.
Включу и посмотрю, спасибо.

> Да, заметил только во время очередной пересборки - "Installing PHP FPM
> config: NONE/etc/"

Ага, я как раз это долго правил, всё должно исправиться с помощью `svn up && rm configure && ./buildconf && ./config.nice && make`

--
Wbr,
Antony Dovgal
---
http://pinba.org - realtime statistics for PHP

Ага, понял. :)

С сигналами это первый пришедший в голову workaround, щас уже вижу,
что многий код из fastcgi.c не нужен, и надо вернуть все старые патчи
и почикать мертвое. Попробую разгрести, там, похоже, из оригинального
cgi/fcgi sapi надо и не так много.

On 7 дек, 13:18, Antony Dovgal <t...@daylessday.org> wrote:
> On 06.12.2009 01:05, fixxxer wrote:
>
> > Оххх, как, оказывается, все запущено в этом 0.6.
>
> Здесь надо учитывать, что в 0.6 изначально cgi_main.c и fastcgi.c шли аж из 4-ки.
> Поэтому я ручками проапдейтил их до более-менее свежей версии.
> Ну и по пути, конечно, сломал всё =) Так что, это не 0.6, эт всё я.
>
> > Патч:http://symbi.org/misc/php_5_3_fpm_lost_request_phases.patch
>
> Спасибо, Фиксер.
> Приложил патч, кроме первой его части.
>
> > С сингалами - я не воткнул, в каких случаях таки надо делать
> > безусловный exit по sigterm - так что пускай пока делает всегда...
> > лучше, чем игнорировать, по крайней мере, а дальше надо смотреть.
>
> Я бы пока не спешил с такими патчами.
> Там есть еще куча кода на libevent, которая тоже занимается обработкой сигналов,
> и у меня есть впечатоление, что код в fastcgi.c просто мертвый и его надо дропнуть.
>
> > Ну и
> > еще конечно реакция на sighup неадекватная (кстати, оно так и было,
> > или это тоже нововведение?).
>
> А? См. выше.
>
> --
> Wbr,
> Antony Dovgal
> ---http://pinba.org- realtime statistics for PHP

On 07.12.2009 15:35, fixxxer wrote:
> Ага, понял. :)
>
> С сигналами это первый пришедший в голову workaround, щас уже вижу,
> что многий код из fastcgi.c не нужен, и надо вернуть все старые патчи
> и почикать мертвое. Попробую разгрести, там, похоже, из оригинального
> cgi/fcgi sapi надо и не так много.

Вот-вот.
Поэтому оно просто напрашивается на чистку.

--
Wbr,
Antony Dovgal
---
http://pinba.org - realtime statistics for PHP

On 7 дек, 15:52, Antony Dovgal <t...@daylessday.org> wrote:
> > и почикать мертвое. Попробую разгрести, там, похоже, из оригинального
> > cgi/fcgi sapi надо и не так много.
>
> Вот-вот.
> Поэтому оно просто напрашивается на чистку.

Вот такое бы потестить:

http://symbi.org/misc/php_5_3_fpm_sapi_cleanup.patch

(Патч кумулятивный, предыдущее в себя включает).

Зачистил более-менее, починил обработку сигналов (и добавил для
мастера HUP = USR2, ибо, ятд, негоже по sighup-у тупо вываливаться).
Заодно ужаснулся и не смог не переписать init_request_info() - так что
надо особенно потестить, что я ничего не сломал. (Заведомо совсем
кривые конфигурации "сломал", конечно, но на таком вряд ли fpm
запускают). В комментариях там пояснил свою позицию =)

On 8 дек, 05:57, fixxxer <fixxxe...@gmail.com> wrote:
> http://symbi.org/misc/php_5_3_fpm_sapi_cleanup.patch
> (Патч кумулятивный, предыдущее в себя включает).

Обновил. Патч на ревизию 291811:

http://symbi.org/misc/php_5_3_fpm_sapi_cleanup_UPD.patch

On 7 дек, 13:08, Antony Dovgal <t...@daylessday.org> wrote:
>
> > Да, заметил только во время очередной пересборки - "Installing PHP FPM
> > config:        NONE/etc/"
>
> Ага, я как раз это долго правил, всё должно исправиться с помощью `svn up && rm configure && ./buildconf && ./config.nice && make`

Не-а, так и осталось. Проверил на FreeBSD и Linux.

И еще phpize в итоге странно сломался - любой сгенеренный им configure
обламывается на таком:

checking for ld used by gcc... ./configure: s%\\%/%g: not found
no
configure: error: no acceptable ld found in $PATH

On 7 дек, 18:08, Antony Dovgal <t...@daylessday.org> wrote:
> On 05.12.2009 18:28, jahson wrote:
>
> > Собрал на osx 10.5.8
>
> Вы только там можете протестировать?

Это просто то, что всё время под рукой. В принципе могу проверить на
линуксе или фре - но Фиксер уже успел это сделать )

> > , всё работает, но основной лог стал со скоростью
> > света наполняться повторяющимися наборами записей вида
> > Dec 05 21:49:02.526089 [NOTICE] fpm_php_trace(), line 170: finished
> > trace of 859
>
> Возможно, вам стоит сделать log_level не notice, а warning?
> Так, конечно, вы видите все возможные нотисы.

Основной смысл, конечно, был не в нотисах, а в скорости наполнения.

> > и вот что выкопал - основное действо происходит
> > в ветке, которая начинается со 128 строки файла fpm_request.c.
>
> Сделайте svn up, плз, что-то я полностью запутался в номерах строк.

Ну от svn up номера строк там не изменились - только в итоге проблема
не там, насколько я понимаю.

> > Да, заметил только во время очередной пересборки - "Installing PHP FPM
> > config: NONE/etc/"
>
> Ага, я как раз это долго правил, всё должно исправиться с помощью `svn up && rm configure && ./buildconf && ./config.nice && make`
Всё так же и осталось.

On 08.12.2009 08:03, fixxxer wrote:
>> Ага, я как раз это долго правил, всё должно исправиться с помощью `svn up && rm configure && ./buildconf && ./config.nice && make`
>
> Не-а, так и осталось. Проверил на FreeBSD и Linux.

И еще раз пробуй, плз.
Должно исправиться.

> И еще phpize в итоге странно сломался - любой сгенеренный им configure
> обламывается на таком:
>
> checking for ld used by gcc... ./configure: s%\\%/%g: not found
> no
> configure: error: no acceptable ld found in $PATH

# phpize
Configuring for:
PHP Api Version: 20090626
Zend Module Api No: 20090626
Zend Extension Api No: 220090626

Работает. По крайней мере, после моих сегодняшних апдейтов у меня на Линуксе.

--
Wbr,
Antony Dovgal
---
http://pinba.org - realtime statistics for PHP

После недавнего апдейта NONE исчезло, спасибо.

On 08.12.2009 14:54, jahson wrote:
> После недавнего апдейта NONE исчезло, спасибо.

Отлично, спасибо.
А массовые нотисы и слоулог исправились после патча Фиксера?

--
Wbr,
Antony Dovgal
---
http://pinba.org - realtime statistics for PHP

On 08.12.2009 06:03, fixxxer wrote:
> On 8 дек, 05:57, fixxxer <fixxxe...@gmail.com> wrote:
>> http://symbi.org/misc/php_5_3_fpm_sapi_cleanup.patch
>> (Патч кумулятивный, предыдущее в себя включает).
>
> Обновил. Патч на ревизию 291811:
>
> http://symbi.org/misc/php_5_3_fpm_sapi_cleanup_UPD.patch

Фикс, тут речи ведутся о том, что добавить поддержку Win32 в FPM должно быть не очень сложно (учитывая, что libevent работает на винде).
Поэтому я бы не стал радикально удалять из кода всё, что касается винды. По крайней мере, сейчас.

Мне, конечно, сложно представить кому и зачем оно может быть нужно,
но мало ли - может, найдётся какой-нить доброволец, который займется этим.

--
Wbr,
Antony Dovgal
---
http://pinba.org - realtime statistics for PHP

On Tuesday 08 December 2009 15:39, Antony Dovgal wrote:
> On 08.12.2009 06:03, fixxxer wrote:
> > On 8 дек, 05:57, fixxxer <fixxxe...@gmail.com> wrote:
> >> http://symbi.org/misc/php_5_3_fpm_sapi_cleanup.patch
> >> (Патч кумулятивный, предыдущее в себя включает).
> >
> > Обновил. Патч на ревизию 291811:
> >
> > http://symbi.org/misc/php_5_3_fpm_sapi_cleanup_UPD.patch
>
> Фикс, тут речи ведутся о том, что добавить поддержку Win32 в FPM должно
> быть не очень сложно (учитывая, что libevent работает на винде). Поэтому я
> бы не стал радикально удалять из кода всё, что касается винды. По крайней
> мере, сейчас.
>
> Мне, конечно, сложно представить кому и зачем оно может быть нужно,
> но мало ли - может, найдётся какой-нить доброволец, который займется этим.

Уже были успешные попытки -
http://www.softwareprojects.com/resources/programming/t-how-to-install-nginx-php-php-fpm-and-mysql-under-1616.html


--
Andrei Nigmatulin
GPG PUB KEY 6449830D

Now I lay me down to sleep(3)
Pray the OS my core to keep
If I die before I wake
Pray the Disk my core to take

Это надо для разворачивания тестового сервера на машинах разработчиков.

8 декабря 2009 г. 15:39 пользователь Antony Dovgal <tony@daylessday.org>написал:

> On 08.12.2009 06:03, fixxxer wrote:
> > On 8 дек, 05:57, fixxxer <fixxxe...@gmail.com> wrote:
> >> http://symbi.org/misc/php_5_3_fpm_sapi_cleanup.patch
> >> (Патч кумулятивный, предыдущее в себя включает).
> >
> > Обновил. Патч на ревизию 291811:
> >
> > http://symbi.org/misc/php_5_3_fpm_sapi_cleanup_UPD.patch
>
> Фикс, тут речи ведутся о том, что добавить поддержку Win32 в FPM должно
> быть не очень сложно (учитывая, что libevent работает на винде).
> Поэтому я бы не стал радикально удалять из кода всё, что касается винды. По
> крайней мере, сейчас.
>
> Мне, конечно, сложно представить кому и зачем оно может быть нужно,
> но мало ли - может, найдётся какой-нить доброволец, который займется этим.
>
> --
> Wbr,
> Antony Dovgal
> ---
> http://pinba.org - realtime statistics for PHP
>



--
C уважением, Александр Лозовюк
Alpha-Beta-Release Blog
http://abrdev.com

On Dec 8, 8:01 pm, Antony Dovgal <t...@daylessday.org> wrote:
> А массовые нотисы и слоулог исправились после патча Фиксера?

Включил слоулог - он молчит на тех же файлах. Для проверки сделал в
info.php sleep(6); (<value name="request_slowlog_timeout">5s</value>)
- отрапортовало нормально.

On 8 дек, 15:39, Antony Dovgal <t...@daylessday.org> wrote:
> On 08.12.2009 06:03, fixxxer wrote:
>
> > On 8 дек, 05:57, fixxxer <fixxxe...@gmail.com> wrote:
> >>http://symbi.org/misc/php_5_3_fpm_sapi_cleanup.patch
> >> (Патч кумулятивный, предыдущее в себя включает).
>
> > Обновил. Патч на ревизию 291811:
>
> >http://symbi.org/misc/php_5_3_fpm_sapi_cleanup_UPD.patch
>
> Фикс, тут речи ведутся о том, что добавить поддержку Win32 в FPM должно быть не очень сложно (учитывая, что libevent работает на винде).
> Поэтому я бы не стал радикально удалять из кода всё, что касается винды. По крайней мере, сейчас.
>

Честно говоря, win32 код резал просто потому, что без него исходники
сразу понятнее в сто раз =)

Тут надо понимать цель fpm - это sapi для высоконагруженных систем.
Протестировать что-либо под виндой можно и обычным php-cgi -b, а
запускать что-то в производстве на cygwin или виндовом кастрате
селекта - это в лучшем случае баловство. О какой то производительности
и надежности тут даже заикаться смешно.

Производительное же решение для Win32 должно делаться на тредах и i/o
completion ports. Попытки добавить i/o c.p. в libevent были <http://
www.mail-archive.com/libevent-users@monkey.org/msg01160.html>, но я не
очень понимаю, чем это все закончилось - и, если честно, вообще это
себе с трудом представляю - уж шибко архитектурные подходы разные. Но
если кто-то с виндой может проверить и убедиться, что либевент с вот
теми вот патчами работает в достаточных для fpm объемах - то почему бы
и нет.

On 08.12.2009 15:52, Александр Лозовюк wrote:
> Это надо для разворачивания тестового сервера на машинах разработчиков.

Не понял. Вы о чем?

--
Wbr,
Antony Dovgal
---
http://pinba.org - realtime statistics for PHP

On 8 дек, 15:46, Andrei Nigmatulin <andrei.nigmatu...@gmail.com>
wrote:
>
> Уже были успешные попытки -http://www.softwareprojects.com/resources/programming/t-how-to-instal...
>

Дык то cygwin. Там куски с #idfef win32 и не нужны.

phpize работает так же, как и у Фиксера:

checking for ld used by gcc... ./configure: line 2601: s%\\%/%g: No
such file or directory
no
configure: error: no acceptable ld found in $PATH

--
Yours, Oleg Roschupkin
jjahson@gmail.com

Точнее говоря ./configure после phpize.

--
Yours, Oleg Roschupkin
jjahson@gmail.com

On 08.12.2009 16:29, Oleg Roschupkin wrote:
> phpize работает так же, как и у Фиксера:
>
> checking for ld used by gcc... ./configure: line 2601: s%\\%/%g: No
> such file or directory
> no
> configure: error: no acceptable ld found in $PATH
>

FreeBSD?
autoconf какой версии?

А без FPM всё точно так же? (снапшоты тут: http://snaps.php.net/)

--
Wbr,
Antony Dovgal
---
http://pinba.org - realtime statistics for PHP

On 08.12.2009 16:15, jahson wrote:
> On Dec 8, 8:01 pm, Antony Dovgal <t...@daylessday.org> wrote:
>> А массовые нотисы и слоулог исправились после патча Фиксера?
>
> Включил слоулог - он молчит на тех же файлах. Для проверки сделал в
> info.php sleep(6); (<value name="request_slowlog_timeout">5s</value>)
> - отрапортовало нормально.

Супер.
Спасибо Фиксеру за патч и вам за тесты =)

--
Wbr,
Antony Dovgal
---
http://pinba.org - realtime statistics for PHP

On 08.12.2009 16:15, fixxxer wrote:
> Тут надо понимать цель fpm - это sapi для высоконагруженных систем.
> Протестировать что-либо под виндой можно и обычным php-cgi -b, а
> запускать что-то в производстве на cygwin или виндовом кастрате
> селекта - это в лучшем случае баловство. О какой то производительности
> и надежности тут даже заикаться смешно.

Я с тобой совершенно согласен.

--
Wbr,
Antony Dovgal
---
http://pinba.org - realtime statistics for PHP

08.12.2009, в 21:33, Antony Dovgal написал(а):

> On 08.12.2009 16:29, Oleg Roschupkin wrote:
>> phpize работает так же, как и у Фиксера:
>>
>> checking for ld used by gcc... ./configure: line 2601: s%\\%/%g: No
>> such file or directory
>> no
>> configure: error: no acceptable ld found in $PATH
>>
>
> FreeBSD?

Я понял почему вопрос, сейчас объясню - я раньше писал через
интерфейс, потому назывался jahson )
На деле у меня недофря - osx.

> autoconf какой версии?

autoconf проверял 2.13 - через export PHP_AUTOCONF='autoconf213'; от
2.65 всё сразу сходит с ума.

> А без FPM всё точно так же? (снапшоты тут: http://snaps.php.net/)

Попробую попробовать.

--
Yours, Oleg Roschupkin
jjahson@gmail.com

On 8 дек, 16:29, Oleg Roschupkin <jjah...@gmail.com> wrote:
> phpize работает так же, как и у Фиксера:
>
> checking for ld used by gcc... ./configure: line 2601: s%\\%/%g: No  
> such file or directory
> no
> configure: error: no acceptable ld found in $PATH


Есть суровое подозрение, что это проявляется только на 32 битах:

Linux debian 2.6.26-2-amd64
FreeBSD 8.0-RELEASE amd64

тут все нормально;

FreeBSD 7.2-RELEASE i386
Darwin Kernel Version 9.8.0 i386

тут получаем

phpize && ./configure

checking for ld used by gcc... ./configure: s%\\%/%g: not found
no
configure: error: no acceptable ld found in $PATH

Если попытаться схитрить и запустить LD='ld' ./configure - облом
следует дальше

checking command to parse nm output from gcc object... ./configure: s/^
[ABCDGIRSTW][ABCDGIRSTW]* \(.*\) \(.*\)$/ {"\2", (lt_ptr_t) \&\2},/:
not found

но по крайней мере собирается =) Проверял на APC.


Да. Обновил патч (я там перестарался с зачисткой):

http://symbi.org/misc/php_5_3_fpm_sapi_cleanup_UPD2.patch

Накладывать на 291870.

On 08.12.2009 17:12, Oleg Roschupkin wrote:
> Я понял почему вопрос, сейчас объясню - я раньше писал через
> интерфейс, потому назывался jahson )
> На деле у меня недофря - osx.

А, понял =)

>> autoconf какой версии?
>
> autoconf проверял 2.13 - через export PHP_AUTOCONF='autoconf213'; от
> 2.65 всё сразу сходит с ума.
>
>> А без FPM всё точно так же? (снапшоты тут: http://snaps.php.net/)
>
> Попробую попробовать.

Попробуйте только сначала ./vcsclean && ./buildconf && ./config.nice && make install
make install важно, т.к. phpize работает на основе уже установленных файлов.

Вообще, судя по всему, у вас $SED не определена и FPM тут никаким боком..

--
Wbr,
Antony Dovgal
---
http://pinba.org - realtime statistics for PHP

On 08.12.2009 18:20, fixxxer wrote:
> Есть суровое подозрение, что это проявляется только на 32 битах:

Вряд ли.
Ошибка ваша вылазит из build/libtool.m4:2267
Там вот такая строка:
ac_prog=`echo $ac_prog| $SED 's%\\\\%/%g'`

Что ты видишь, когда запускаешь `./config.nice | grep sed` ?

> Да. Обновил патч (я там перестарался с зачисткой):
> http://symbi.org/misc/php_5_3_fpm_sapi_cleanup_UPD2.patch
> Накладывать на 291870.

Ок, гляну.

--
Wbr,
Antony Dovgal
---
http://pinba.org - realtime statistics for PHP