ПРОЕКТЫ 


  АРХИВ 


Apache-Talk @lexa.ru 

Inet-Admins @info.east.ru 

Filmscanners @halftone.co.uk 

Security-alerts @yandex-team.ru 

nginx-ru @sysoev.ru 


  СТАТЬИ 


  ПЕРСОНАЛЬНОЕ 


  ПРОГРАММЫ 



ПИШИТЕ
ПИСЬМА












     АРХИВ :: nginx-ru
Nginx-ru mailing list archive (nginx-ru@sysoev.ru)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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


  • To: nginx-ru@xxxxxxxxx
  • Subject: Re: патч для nginx/win32
  • From: Илья Шипицин <chipitsine@xxxxxxxxx>
  • Date: Wed, 27 Mar 2013 09:58:07 +0600
  • Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:x-received:in-reply-to:references:date:message-id :subject:from:to:content-type; bh=vQkcgar+Inict55hHzbM/imd+8k5+c7FHqSo5K7aaEQ=; b=IJx9Nepkt7nONE8k6CfQ7oGfyQDofP7kS/KIV/v1Lyt6vhIjvyMlQffxN2N9vpWkxY RTsekd9WHb9s+4rU4+xxWpaw0LJYR1eoQXGDDc0GK4fDUSvm9748tEdSr6iz1Q4z+xqL lFS1wGYszAtaQECV7pOMevYIuR3T8gbHXo/k965G7D7Ad8Z7Hdk6DfRiEorKp4d6auRS LYQF7RNsD3wndvjT7JmukqmyChnpqig+kKXvKa9oIBS2JvBNxL3IVya0pKLxti9OMDk3 LxTqZzi0tE29YtTu9INWXZ1Tr9uhg43YwzyjkYkFXvtdHNrbbJIvyujceFFIovWedYKU 27TA==
  • In-reply-to: <20130326120818.GN62550@mdounin.ru>
  • References: <CAFHpkQEZHAR2E=kU5tzNnHNqeEZebb=O1JcDYaoqwo2o7PPCfw@mail.gmail.com> <20130326112719.GM62550@mdounin.ru> <CAFHpkQEW0M2YRew+Rky6C0KeVhPSpEtm+Y3BbLFffC3zut=jSg@mail.gmail.com> <20130326120818.GN62550@mdounin.ru>

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

--- 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@xxxxxxxxxx> написал:
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@xxxxxxxxxx>написал:
>
> > 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@xxxxxxxxx
> > http://mailman.nginx.org/mailman/listinfo/nginx-ru

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


--
Maxim Dounin
http://nginx.org/en/donation.html

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

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


 




Copyright © Lexa Software, 1996-2009.