commit 4536113ad3ae6f16d1638aa4e4d81c94e1e6b357
author Maxim Dounin <mdounin@mdounin.ru> Fri Oct 29 20:21:57 2021 +0300
committer Maxim Dounin <mdounin@mdounin.ru> Fri Oct 29 20:21:57 2021 +0300
tree 50c16e7e52b07684b77f908dd4fadc7a11ffe4f4
parent d77bd28ea7be4edd1f5dfde600f96368ccce55e4

Changed default value of sendfile_max_chunk to 2m.

The "sendfile_max_chunk" directive is important to prevent worker
monopolization by fast connections.  The 2m value implies maximum 200ms
delay with 100 Mbps links, 20ms delay with 1 Gbps links, and 2ms on
10 Gbps links.  It also seems to be a good value for disks.

src/http/ngx_http_core_module.c

diff --git a/src/http/ngx_http_core_module.c b/src/http/ngx_http_core_module.c
index bad43ea..fe1da45 100644
--- a/src/http/ngx_http_core_module.c
+++ b/src/http/ngx_http_core_module.c
@@ -3720,7 +3720,7 @@
     ngx_conf_merge_value(conf->internal, prev->internal, 0);
     ngx_conf_merge_value(conf->sendfile, prev->sendfile, 0);
     ngx_conf_merge_size_value(conf->sendfile_max_chunk,
-                              prev->sendfile_max_chunk, 0);
+                              prev->sendfile_max_chunk, 2 * 1024 * 1024);
     ngx_conf_merge_size_value(conf->subrequest_output_buffer_size,
                               prev->subrequest_output_buffer_size,
                               (size_t) ngx_pagesize);