On Fri, 2017-10-13 at 09:25 +0100, antonio.fische...@intel.com wrote: > Update documentation with the new commands to Read/Write > ConnTracker configuration parameters. > > CC: Kevin Traynor <ktray...@redhat.com> > CC: Darrell Ball <dlu...@gmail.com> > Signed-off-by: Antonio Fischetti <antonio.fische...@intel.com>
One nit below, but otherwise LGTM. Acked-by: Stephen Finucane <step...@that.guru> > --- > Documentation/intro/install/dpdk.rst | 25 +++++++++++++++++++++++++ > lib/dpctl.man | 10 ++++++++++ > 2 files changed, 35 insertions(+) > > diff --git a/Documentation/intro/install/dpdk.rst > b/Documentation/intro/install/dpdk.rst > index bb69ae5..a1f259c 100644 > --- a/Documentation/intro/install/dpdk.rst > +++ b/Documentation/intro/install/dpdk.rst > @@ -568,6 +568,31 @@ not needed i.e. jumbo frames are not needed, it can be > forced off by adding > chains of descriptors it will make more individual virtio descriptors > available > for rx to the guest using dpdkvhost ports and this can improve performance. > > +Connection Tracker > +~~~~~~~~~~~~~~~~~~ > + > +When the Connection Tracker is enabled the overall performance can be deeply > +affected, even with simple firewall rules and with stateless protocols like > +UDP. In order to find a better tuning, commands like > + > +:: > + > + $ ovs-appctl dpctl/ct-get-glbl-cfg <cfg param> > + $ ovs-appctl dpctl/ct-set-glbl-cfg <cfg param>=<value> > + > +allow respectively to read the current value, or set a new value to a > +configuration parameter. > +For example, to reduce the impact of the Connection Tracker load on the > +system performance, the maximum number of tracked connections can be > +reduced. > + > +The available configuration parameters are: > + > +- maxconn: Maximum number of connections managed by the Connection Tracker > + module. It's both readable and writeable. > +- totconn: Total number of connections currently managed by the Connection > + Tracker module. Readable only. nit: This section would probably read better as a definition list ``maxconn`` Maximum number of connections... ``totconn`` Total number of connections... > + > Limitations > ------------ > > diff --git a/lib/dpctl.man b/lib/dpctl.man > index 675fe5a..64ad105 100644 > --- a/lib/dpctl.man > +++ b/lib/dpctl.man > @@ -235,3 +235,13 @@ For each ConnTracker bucket, displays the number of > connections used > by \fIdp\fR. > If \fBgt=\fIThreshold\fR is specified, bucket numbers are displayed when > the number of connections in a bucket is greater than \fIThreshold\fR. > +. > +.TP > +\*(DX\fBct\-get\-glbl\-cfg\fR [\fIdp\fR] \fBparam\fR > +Read the current value of the specified ConnTracker parameter used > +by \fIdp\fR. > +. > +.TP > +\*(DX\fBct\-set\-glbl\-cfg\fR [\fIdp\fR] \fBparam=\fI..\fR > +Set a value to the specified ConnTracker parameter used > +by \fIdp\fR. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev