Welcome! Log In Create A New Profile

Advanced

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

Maxim Dounin
March 26, 2013 07:28AM
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@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-ru
Subject Author Posted

патч для nginx/win32

Илья Шипицин March 26, 2013 07:04AM

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

Maxim Dounin March 26, 2013 07:28AM

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

Илья Шипицин March 26, 2013 07:44AM

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

Maxim Dounin March 26, 2013 08:10AM

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

Илья Шипицин March 26, 2013 08:24AM

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

Илья Шипицин March 27, 2013 12:00AM

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

Maxim Dounin March 27, 2013 08:22AM

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

Илья Шипицин March 27, 2013 08:26AM

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

Илья Шипицин March 27, 2013 08:28AM

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

Илья Шипицин March 27, 2013 08:58AM

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

Maxim Dounin March 27, 2013 09:40AM

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

Илья Шипицин March 28, 2013 07:24AM

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

Илья Шипицин March 28, 2013 08:04AM

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

Maxim Dounin March 28, 2013 12:22PM



Sorry, only registered users may post in this forum.

Click here to login

Online Users

Guests: 139
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready