Re: [PATCH 0 of 6] QUIC PATH_CHALLENGE-related series

2023-12-11 Thread Sergey Kandaurov
> On 30 Nov 2023, at 15:05, Roman Arutyunyan wrote: > > Hi, > > A number of patches discussed previously. > A couple more patches to follow. Last two weeks I spent pondering if we have to migrate to per-path congestion control and RTT estimator (as inspired by Alibaba patch https://mailman.n

Re: [PATCH 0 of 6] QUIC PATH_CHALLENGE-related series

2023-12-11 Thread Roman Arutyunyan
Hi, On Mon, Dec 11, 2023 at 01:40:06PM +0400, Sergey Kandaurov wrote: > > > On 30 Nov 2023, at 15:05, Roman Arutyunyan wrote: > > > > Hi, > > > > A number of patches discussed previously. > > > > A couple more patches to follow. > > Last two weeks I spent pondering if we have to migrate to

[nginx] HTTP: removed unused r->port_start and r->port_end.

2023-12-11 Thread Sergey Kandaurov
details: https://hg.nginx.org/nginx/rev/dacad3a9c7b8 branches: changeset: 9187:dacad3a9c7b8 user: Vladimir Khomutov date: Tue Nov 28 12:57:14 2023 +0300 description: HTTP: removed unused r->port_start and r->port_end. Neither r->port_start nor r->port_end were ever used. The r->por

[nginx] HTTP: uniform checks in ngx_http_alloc_large_header_buffer().

2023-12-11 Thread Sergey Kandaurov
details: https://hg.nginx.org/nginx/rev/b05c622715fa branches: changeset: 9188:b05c622715fa user: Vladimir Khomutov date: Wed Nov 29 11:13:05 2023 +0300 description: HTTP: uniform checks in ngx_http_alloc_large_header_buffer(). If URI is not fully parsed yet, some pointers are not s

Re: [PATCH 0 of 6] QUIC PATH_CHALLENGE-related series

2023-12-11 Thread Sergey Kandaurov
> On 11 Dec 2023, at 17:49, Roman Arutyunyan wrote: > > Hi, > > On Mon, Dec 11, 2023 at 01:40:06PM +0400, Sergey Kandaurov wrote: >> >>> On 30 Nov 2023, at 15:05, Roman Arutyunyan wrote: >>> >>> Hi, >>> >>> A number of patches discussed previously. >>> >> >> A couple more patches to follo

Re: [PATCH] Added asm ISB as asm pause for ngx_cpu_pause() for aarch64

2023-12-11 Thread Julio Suarez
Hi Maxim, Nitpicking: Added ISB as ngx_cpu_pause() for aarch64. Yes, we can make that change. Could you please clarify what do you mean by "a bug"? An empty ngx_cpu_pause() is certainly not a bug, it's just a lack of a more optimal solution for the particular architecture. Agree, not a bug.

[PATCH] Linux packages: actualized supported Alpine Linux versions

2023-12-11 Thread Konstantin Pavlov
# HG changeset patch # User Konstantin Pavlov # Date 1702345379 28800 # Mon Dec 11 17:42:59 2023 -0800 # Node ID 55f8ce8a8cb0acf9b360e47fd5d0023f16451a80 # Parent 08533e33d0744bd27bc42d87c47607399903eae5 Linux packages: actualized supported Alpine Linux versions. diff -r 08533e33d074 -r 55f

[njs] Removed not used arg from njs_get_own_ordered_keys().

2023-12-11 Thread Vadim Zhestikov via nginx-devel
details: https://hg.nginx.org/njs/rev/59bd78b060c9 branches: changeset: 2246:59bd78b060c9 user: Vadim Zhestikov date: Mon Dec 11 19:10:36 2023 -0800 description: Removed not used arg from njs_get_own_ordered_keys(). diffstat: src/njs_object.c | 9 - 1 files changed, 4 ins

[njs] Removed not required argument for JSON functions.

2023-12-11 Thread Vadim Zhestikov via nginx-devel
details: https://hg.nginx.org/njs/rev/34df3f0796cf branches: changeset: 2247:34df3f0796cf user: Vadim Zhestikov date: Mon Dec 11 19:10:38 2023 -0800 description: Removed not required argument for JSON functions. diffstat: src/njs_json.c | 59 --