]> git.ipfire.org Git - thirdparty/pdns.git/commitdiff
Apply spelling fixes
authorOtto Moerbeek <otto.moerbeek@open-xchange.com>
Thu, 31 Aug 2023 18:26:21 +0000 (20:26 +0200)
committerOtto Moerbeek <otto.moerbeek@open-xchange.com>
Wed, 13 Sep 2023 11:20:54 +0000 (13:20 +0200)
Co-authored-by: Remi Gacogne <github@coredump.fr>
pdns/recursordist/docs/performance.rst
pdns/recursordist/rec-tcp.cc

index 2176d7d960fb640c62f172f57efc17f450c400f5..9acb87fe65ec58a0e5adc80a4b8e6e75dbea98e2 100644 (file)
@@ -80,13 +80,13 @@ MTasker and MThreads
 PowerDNS Recursor uses a cooperative multitasking in userspace called ``MTasker``, based either on ``boost::context`` if available, or on ``System V ucontexts`` otherwise. For maximum performance, please make sure that your system supports ``boost::context``, as the alternative has been known to be quite slower.
 
 The maximum number of simultaneous MTasker threads, called ``MThreads``, can be tuned via :ref:`setting-max-mthreads`, as the default value of 2048 might not be enough for large-scale installations.
-This number limits the number of mthreads *per physical (Posix) thread*.
+This setting limits the number of mthreads *per physical (Posix) thread*.
 The threads that create mthreads are the distributor and worker threads.
 
 When a ``MThread`` is started, a new stack is dynamically allocated for it on the heap. The size of that stack can be configured via the :ref:`setting-stack-size` parameter, whose default value is 200 kB which should be enough in most cases.
 
 To reduce the cost of allocating a new stack for every query, the recursor can cache a small amount of stacks to make sure that the allocation stays cheap. This can be configured via the :ref:`setting-stack-cache-size` setting.
-This limit is per physcial (Posix) thread.
+This limit is per physical (Posix) thread.
 The only trade-off of enabling this cache is a slightly increased memory consumption, at worst equals to the number of stacks specified by :ref:`setting-stack-cache-size` multiplied by the size of one stack, itself specified via :ref:`setting-stack-size`.
 
 Performance tips
@@ -185,7 +185,7 @@ Before version 5.0.0, TCP queries are processed by either the distributer thread
 Starting with version 5.0.0, :program:`Recursor` has dedicated thread(s) processing TCP queries.
 
 The maximum number of mthreads consumed by TCP queries is :ref:`setting-max-tcp-clients` times :ref:`setting-max-concurrent-requests-per-tcp-connection`.
-If :ref:`setting-pdns-distributes-queries` is true, this number should be (much) lower than :ref:`setting-max-mthreads`, to also allow UDP queries to be handled as these also consume mthrea ds.
+If :ref:`setting-pdns-distributes-queries` is true, this number should be (much) lower than :ref:`setting-max-mthreads`, to also allow UDP queries to be handled as these also consume mthreads.
 Note that :ref:`setting-max-mthreads` is a per Posix thread setting.
 This means that the global maximum number of mthreads  is (#distributor threads + #worker threads) * max-mthreads.
 
@@ -197,7 +197,7 @@ To see the current number of mthreads in use consult the :ref:`stat-concurrent-q
 If a query could not be handled due to mthread shortage, the :ref:`stat-over-capacity-drops` metric is increased.
 
 As an example, if you have typically 200 TCP clients, and the default maximum number of mthreads of 2048, a good number of concurrent requests per TCP connection would be 5. Assuming a worst case packet cache hit ratio, if all 200 TCP clients fill their connections with queries, about half (5 * 200) of the mthreads would be used by incoming TCP queries, leaving the other half for incoming UDP queries.
-Note that starting with versino 5.0.0, TCP queries are processed by dedicated TCP thread(s), so the sharing of mthreads between UDP and TCP queries no longer applies.
+Note that starting with version 5.0.0, TCP queries are processed by dedicated TCP thread(s), so the sharing of mthreads between UDP and TCP queries no longer applies.
 
 The total number of incoming TCP connections is limited by :ref:`setting-max-tcp-clients`.
 There is also a per client address limit: :ref:`setting-max-tcp-per-client` to limit the impact of a single client.
index 3e0cb05a263abf6aec43464c9a29a629a3e146c4..e8b9da98c3797a9bebe13af52f33a96dbb6717f8 100644 (file)
@@ -27,7 +27,7 @@
 #include "mplexer.hh"
 #include "uuid-utils.hh"
 
-// When pdns_distributes query is false with reuseport true (the default since 4.9.0), TCP queries
+// When pdns-distributes-queries is false with reuseport true (the default since 4.9.0), TCP queries
 // are read and handled by worker threads. If the kernel balancing is OK for TCP sockets (observed
 // to be good on Debian bullseye, but not good on e.g. MacOS), the TCP handling is no extra burden.
 // In the case of MacOS all incoming TCP queries are handled by a single worker, while incoming UDP
@@ -47,7 +47,7 @@
 // queries), but the TCP socket must also be passed to a worker thread so it can write its
 // answer. The in-flight bookkeeping also has to be aware of how a query is handled to do the
 // accounting properly. I am not sure if changing the current setup is worth all this trouble,
-// especilly since the default is now to not use pdns-distributes-queries, which works well in many
+// especially since the default is now to not use pdns-distributes-queries, which works well in many
 // cases.
 //
 // The drawback mentioned in https://github.com/PowerDNS/pdns/issues/8394 are not longer true, so an