Re: патч для nginx/win32

Илья Шипицин chipitsine at gmail.com
Tue Mar 26 11:43:21 UTC 2013


давайте разбираться. если запускать nginx в консоли (это штатный режим, так
работают назначенные задания), то завершение задания с точки зрения
мастер-процесса выглядит, как CTRL_CLOSE_EVENT в функции-обработчике
ngx_console_handler

worker-процесс в это время залипает в функции ngx_worker_process_cycle в
цикле "ev=WaitForMultipleObjects()"

соответственно, закрытие мастера путем закрывания не приводит к тому, что в
данном месте возникает какое-то событие.

варианты - либо существенно переделывать логику и протаскивать сюда еще
одно событие, либо жестко закрыть worker через
ngx_terminate_worker_processes.

чем чреват второй вариант ? ну ок, закроются текущие сессии. завершая
задание, мы, вероятно, этого и добиваемся.


26 марта 2013 г., 17:27 пользователь Maxim Dounin <mdounin at mdounin.ru>написал:

> Hello!
>
> On Tue, Mar 26, 2013 at 05:03:30PM +0600, Илья Шипицин wrote:
>
> > Добрый день!
> >
> > мы достаточно плотно используем nginx для Windows, запускаем его через
> > назначенное задание (scheduled tasks). Для этого в конфиге надо сделать
> > "daemon off" и дальше менеджер заданий следит за мастер-процессом,
> > запущенным на терминале.
> >
> > это, кстати, удобнее, чем служба Windows (вообще, назначенные задания
> более
> > удобны и мы чаще используем их, чем службы).
> >
> > в этом сценарии есть один недостаток, при завершении мастер-процесса,
> > остается запущенный worker-процесс.
> >
> > насколько я понял, в случае Windows это штатная ситуация (при такой
> работе
> > с процессами, которая используется в nginx), для исправления предлагаю
> > такой патч (сделан для 1.3.14):
> >
> > --- src/os/win32/ngx_process_cycle.c    2013-03-26 16:57:20.000000000
> +0600
> > +++ src/os/win32/ngx_process_cycle.c.new        2013-03-26
> > 16:57:00.987341331 +0600
> > @@ -303,6 +303,8 @@
> >  ngx_console_handler(u_long type)
> >  {
> >      char  *msg;
> > +       ngx_cycle_t       *cycle;
> > +    cycle = (ngx_cycle_t *) ngx_cycle;
> >
> >      switch (type) {
> >
> > @@ -316,6 +318,7 @@
> >
> >      case CTRL_CLOSE_EVENT:
> >          msg = "console closing, exiting";
> > +               ngx_terminate_worker_processes(cycle);
> >          break;
> >
> >      case CTRL_LOGOFF_EVENT:
>
> Звать ngx_terminate_worker_processes() - это не очень хорошая
> идея, это всё-таки аварийный механизм, и может приводить к
> нехорошему.  Тут имеет смысл как минимум попытаться штатно
> завершить рабочие процессы.
>
> --
> Maxim Dounin
> http://nginx.org/en/donation.html
>
> _______________________________________________
> nginx-ru mailing list
> nginx-ru at nginx.org
> http://mailman.nginx.org/mailman/listinfo/nginx-ru
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.nginx.org/pipermail/nginx-ru/attachments/20130326/9ed4a819/attachment-0001.html>


Подробная информация о списке рассылки nginx-ru