On Mon, Sep 26, 2011 at 10:16:41AM -0700, Randy Dunlap wrote: > On 09/25/2011 06:23 PM, Simon Horman wrote: > > Add missing documentation for conntrack, snat_reroute and sync_version. > > > > Also fix up a typo, IPVS_DEBUG should be IP_VS_DEBUG. > > > > Signed-off-by: Simon Horman <horms@xxxxxxxxxxxx> > > > > conntrack > > --- > > Documentation/networking/ipvs-sysctl.txt | 53 +++++++++++++++++++++++++++++- > > 1 files changed, 52 insertions(+), 1 deletions(-) > > > > diff --git a/Documentation/networking/ipvs-sysctl.txt b/Documentation/networking/ipvs-sysctl.txt > > index 4ccdbca..5214339 100644 > > --- a/Documentation/networking/ipvs-sysctl.txt > > +++ b/Documentation/networking/ipvs-sysctl.txt > > @@ -15,6 +15,23 @@ amemthresh - INTEGER > > enabled and the variable is automatically set to 2, otherwise > > the strategy is disabled and the variable is set to 1. > > > > +conntrack - BOOLEAN > > + 0 - disabled (default) > > + not 0 - enabled > > + > > + If set, maintain connection tracking entries for > > + connections handled by IPVS. > > + > > + This should be enabled if connections handled by IPVS are to be > > + also handled by stateful firewall rules. That is, iptables rules > > + that make use of connection tracking. It is a performance > > + optimisation to disable this setting otherwise. > > + > > + Connections handled by the IPVS FTP application module > > + will have connection tracking entries regardless of this setting. > > + > > + Only available when IPVS is compiled with the CONFIG_IP_VS_NFCT > > seems to be missing something. Maybe > > Only available when IPVS is compiled with CONFIG_IP_VS_NFCT enabled. > or > Only available when IPVS is compiled with the CONFIG_IP_VS_NFCT > symbol enabled. > > > + > > cache_bypass - BOOLEAN > > 0 - disabled (default) > > not 0 - enabled > > @@ -39,7 +56,7 @@ debug_level - INTEGER > > 11 - IPVS packet handling (ip_vs_in/ip_vs_out) > > 12 or more - packet traversal > > > > - Only available when IPVS is compiled with the CONFIG_IPVS_DEBUG > > + Only available when IPVS is compiled with the CONFIG_IP_VS_DEBUG > > with CONFIG_IP_VS_DEBUG enabled. > > > > > Higher debugging levels include the messages for lower debugging > > levels, so setting debug level 2, includes level 0, 1 and 2 Thanks Randy, I was just blindly copying the existing language. I'll make a follow up patch which incorporates your suggestions. -- To unsubscribe from this list: send the line "unsubscribe lvs-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
