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

[Илья Шипицин]

вот такой вариант ?

--- src/os/win32/ngx_process_cycle.c.orig       2013-03-27
09:53:48.000000000 +0600
+++ src/os/win32/ngx_process_cycle.c    2013-03-27 09:48:56.000000000 +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,8 @@

     case CTRL_CLOSE_EVENT:
         msg = "console closing, exiting";
+               ngx_quit = 1;
+        ngx_quit_worker_processes(cycle, 0);
         break;

     case CTRL_LOGOFF_EVENT:






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

> Hello!
>
> On Tue, Mar 26, 2013 at 05:43:21PM +0600, Илья Шипицин wrote:
>
> > давайте разбираться. если запускать nginx в консоли (это штатный режим,
> так
> > работают назначенные задания), то завершение задания с точки зрения
> > мастер-процесса выглядит, как CTRL_CLOSE_EVENT в функции-обработчике
> > ngx_console_handler
> >
> > worker-процесс в это время залипает в функции ngx_worker_process_cycle в
> > цикле "ev=WaitForMultipleObjects()"
> >
> > соответственно, закрытие мастера путем закрывания не приводит к тому,
> что в
> > данном месте возникает какое-то событие.
>
> То, что это плохо - вопросов не вызывает.  По Ctrl-C всё должно
> штатно закрываться, а не висеть вечно.
>
> > варианты - либо существенно переделывать логику и протаскивать сюда еще
> > одно событие, либо жестко закрыть worker через
> > ngx_terminate_worker_processes.
> >
> > чем чреват второй вариант ? ну ок, закроются текущие сессии. завершая
> > задание, мы, вероятно, этого и добиваемся.
>
> Например, могут остаться полусохранённые файлы в кеше/proxy_store -
> если рабочий процесс прервали в процессе копирования временного
> файла в целевой каталог.
>
> (Документация по TerminateProcess() и различные code
> checker'ы любят пугать про "the state of global data maintained
> by dynamic-link libraries (DLLs) may be compromised".  Но это,
> насколько я понимаю, в данном случае к nginx'у неприменимо - по
> крайней мере, в отсутствии сторонних модулей.)
>
> >
> >
> > 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
>
> > _______________________________________________
> > nginx-ru mailing list
> > nginx-ru at nginx.org
> > http://mailman.nginx.org/mailman/listinfo/nginx-ru
>
>
> --
> 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/20130327/bbdb32f9/attachment.html>