ПРОЕКТЫ 


  АРХИВ 


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: Tue, 26 Mar 2013 17:43:21 +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=jWERPv9NV+kqMfDqqaKWR7dXFiFlmRI3hlh8WU+zpxw=; b=yu1uH9BUzlnbwMP4s3BMdfua9CE9PJVNLGlyAYwWdH+mRPdNwFCWFQCtsXqvbiloHj lMN2aLtMKPsY60X00Z6gBg4Y/EKbhfihjLj0fQmvXcPjFIUWzEiVPmVYWKdUuhN50nOp NwDOpGQJ9F8kww5mbLBVGd6EVHLNbSHfismNeboWQaZBhmImeYuobVcGZGZpZeVpSaVf vOLhy04NKQZ6nz0c0GKXsI3zQPOkY8EVQacJfyaIesBeQd1thlRRD9t54TuSKrPM+cfO 1OlfHkO6r08BQJVpDNXiFI0+kAcXWJPgroA4VUzDtWr27e4Fxi8jiC+Voodr7OjfkbR3 Jzqg==
  • In-reply-to: <20130326112719.GM62550@mdounin.ru>
  • References: <CAFHpkQEZHAR2E=kU5tzNnHNqeEZebb=O1JcDYaoqwo2o7PPCfw@mail.gmail.com> <20130326112719.GM62550@mdounin.ru>

давайте разбираться. если запускать 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@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


 




Copyright © Lexa Software, 1996-2009.