Welcome! Log In Create A New Profile

Advanced

Re: [PATCH] Disabled cloning of sockets without master process (ticket #2403)

Sergey Kandaurov
November 23, 2022 08:58AM
> On 30 Oct 2022, at 06:41, Maxim Dounin <mdounin@mdounin.ru> wrote:
>
> # HG changeset patch
> # User Maxim Dounin <mdounin@mdounin.ru>
> # Date 1667097682 -10800
> # Sun Oct 30 05:41:22 2022 +0300
> # Node ID b73d95226c84b93e51f23f7b35782d98d3b516b9
> # Parent 55bcf8dc4ee35ccf40f5b8a7cffde63e7edb9494
> Disabled cloning of sockets without master process (ticket #2403).
>
> Cloning of listening sockets for each worker process does not make sense
> when working without master process, and causes some of the connections
> not to be accepted if worker_processes is set to more than one and there
> are listening sockets configured with the reuseport flag. Fix is to
> disable cloning when master process is disabled.
>
> diff --git a/src/event/ngx_event.c b/src/event/ngx_event.c
> --- a/src/event/ngx_event.c
> +++ b/src/event/ngx_event.c
> @@ -416,6 +416,7 @@ ngx_event_init_conf(ngx_cycle_t *cycle,
> {
> #if (NGX_HAVE_REUSEPORT)
> ngx_uint_t i;
> + ngx_core_conf_t *ccf;
> ngx_listening_t *ls;
> #endif
>
> @@ -442,7 +443,9 @@ ngx_event_init_conf(ngx_cycle_t *cycle,
>
> #if (NGX_HAVE_REUSEPORT)
>
> - if (!ngx_test_config) {
> + ccf = (ngx_core_conf_t *) ngx_get_conf(cycle->conf_ctx, ngx_core_module);
> +
> + if (!ngx_test_config && ccf->master) {
>
> ls = cycle->listening.elts;
> for (i = 0; i < cycle->listening.nelts; i++) {

Looks good.

--
Sergey Kandaurov

_______________________________________________
nginx-devel mailing list -- nginx-devel@nginx.org
To unsubscribe send an email to nginx-devel-leave@nginx.org
Subject Author Views Posted

[PATCH] Disabled cloning of sockets without master process (ticket #2403)

Maxim Dounin 605 October 29, 2022 10:44PM

Re: [PATCH] Disabled cloning of sockets without master process (ticket #2403)

Maxim Dounin 91 November 18, 2022 08:56AM

Re: [PATCH] Disabled cloning of sockets without master process (ticket #2403)

Sergey Kandaurov 94 November 23, 2022 08:58AM

Re: [PATCH] Disabled cloning of sockets without master process (ticket #2403)

Maxim Dounin 156 November 24, 2022 01:26PM



Sorry, you do not have permission to post/reply in this forum.

Online Users

Guests: 317
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