<p dir="ltr">Typo : "is probably detrimMENTal"</p>
<div class="gmail_extra"><br><div class="gmail_quote">On Sep 9, 2016 17:17, "Nils Goroll" <<a href="mailto:nils.goroll@uplex.de">nils.goroll@uplex.de</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
commit ca8bd536e58589a208e9b855cdf726<wbr>efd2a55675<br>
Author: Nils Goroll <<a href="mailto:nils.goroll@uplex.de">nils.goroll@uplex.de</a>><br>
Date:   Fri Sep 9 16:31:03 2016 +0200<br>
<br>
    improve thread_pools documentation<br>
<br>
diff --git a/bin/varnishd/mgt/mgt_pool.c b/bin/varnishd/mgt/mgt_pool.c<br>
index 327a4c2..7f0e78f 100644<br>
--- a/bin/varnishd/mgt/mgt_pool.c<br>
+++ b/bin/varnishd/mgt/mgt_pool.c<br>
@@ -84,11 +84,14 @@ struct parspec WRK_parspec[] = {<br>
                "1", NULL,<br>
                "Number of worker thread pools.\n"<br>
                "\n"<br>
-               "Increasing number of worker pools decreases lock "<br>
-               "contention.\n"<br>
+               "Increasing the number of worker pools decreases lock "<br>
+               "contention. Each worker pool also has a thread accepting "<br>
+               "new connections, so for very high rates of incoming new "<br>
+               "connections on systems with many cores, increasing the "<br>
+               "worker pools may be required.\n"<br>
                "\n"<br>
-               "Too many pools waste CPU and RAM resources, and more than "<br>
-               "one pool for each CPU is probably detrimal to performance.\n"<br>
+               "Too many pools waste CPU and RAM resources, and more than one "<br>
+               "pool for each CPU is most likely detrimal to performance.\n"<br>
                "\n"<br>
                "Can be increased on the fly, but decreases require a "<br>
                "restart to take effect.",<br>
diff --git a/include/tbl/params.h b/include/tbl/params.h<br>
index 57bab3a..475086c 100644<br>
--- a/include/tbl/params.h<br>
+++ b/include/tbl/params.h<br>
@@ -1170,10 +1170,14 @@ PARAM(<br>
        /* s-text */<br>
        "Number of worker thread pools.\n"<br>
        "\n"<br>
-       "Increasing number of worker pools decreases lock contention.\n"<br>
+       "Increasing the number of worker pools decreases lock "<br>
+       "contention. Each worker pool also has a thread accepting "<br>
+       "new connections, so for very high rates of incoming new "<br>
+       "connections on systems with many cores, increasing the "<br>
+       "worker pools may be required.\n"<br>
        "\n"<br>
        "Too many pools waste CPU and RAM resources, and more than one "<br>
-       "pool for each CPU is probably detrimal to performance.\n"<br>
+       "pool for each CPU is most likely detrimal to performance.\n"<br>
        "\n"<br>
        "Can be increased on the fly, but decreases require a restart to "<br>
        "take effect.",<br>
<br>
______________________________<wbr>_________________<br>
varnish-commit mailing list<br>
<a href="mailto:varnish-commit@varnish-cache.org">varnish-commit@varnish-cache.<wbr>org</a><br>
<a href="https://www.varnish-cache.org/lists/mailman/listinfo/varnish-commit" rel="noreferrer" target="_blank">https://www.varnish-cache.org/<wbr>lists/mailman/listinfo/<wbr>varnish-commit</a><br>
</blockquote></div></div>