Removed unused 'oplock contention limit' config parameter
authorChristof Schmitt <cs@samba.org>
Tue, 17 Oct 2017 21:49:17 +0000 (14:49 -0700)
committerJeremy Allison <jra@samba.org>
Wed, 18 Oct 2017 22:55:24 +0000 (00:55 +0200)
Signed-off-by: Christof Schmitt <cs@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
WHATSNEW.txt
docs-xml/smbdotconf/locking/oplockcontentionlimit.xml [deleted file]
examples/scripts/shares/python/smbparm.py
lib/param/loadparm.c
source3/param/loadparm.c

index 2f2f2f6..cc03f20 100644 (file)
@@ -40,6 +40,7 @@ smb.conf changes
 
   Parameter Name                     Description             Default
   --------------                     -----------             -------
+  oplock contention limit            Removed
 
 NT4-style replication based net commands removed
 ================================================
diff --git a/docs-xml/smbdotconf/locking/oplockcontentionlimit.xml b/docs-xml/smbdotconf/locking/oplockcontentionlimit.xml
deleted file mode 100644 (file)
index edaa7d7..0000000
+++ /dev/null
@@ -1,26 +0,0 @@
-<samba:parameter name="oplock contention limit"
-                 context="S"
-                 type="integer"
-                 xmlns:samba="http://www.samba.org/samba/DTD/samba-doc">
-<description>
-       <para>
-       This is a <emphasis>very</emphasis> advanced <citerefentry><refentrytitle>smbd</refentrytitle>
-       <manvolnum>8</manvolnum></citerefentry> tuning option to improve the efficiency of the 
-       granting of oplocks under multiple client contention for the same file.
-       </para>
-               
-       <para>
-       In brief it specifies a number, which causes <citerefentry><refentrytitle>smbd</refentrytitle>
-       <manvolnum>8</manvolnum></citerefentry>not to grant an oplock even when requested if the 
-       approximate number of clients contending for an oplock on the same file goes over this 
-       limit. This causes <command moreinfo="none">smbd</command> to behave in a similar 
-       way to Windows NT.
-       </para>
-
-       <warning><para>
-       DO NOT CHANGE THIS PARAMETER UNLESS YOU HAVE READ AND UNDERSTOOD THE SAMBA OPLOCK CODE.
-       </para></warning>
-
-</description>
-<value type="default">2</value>
-</samba:parameter>
index f182f57..e0786a4 100644 (file)
@@ -173,7 +173,6 @@ parm_table = {
        "PASSWDCHATDEBUG"        : ("passwd chat debug", SambaParmBool, P_GLOBAL, "No"),
        "SMBPASSWDFILE"          : ("smb passwd file", SambaParmString, P_GLOBAL, "/etc/samba/private/smbpasswd"),
        "GETQUOTACOMMAND"        : ("get quota command", SambaParmString, P_GLOBAL, ""),
-       "OPLOCKCONTENTIONLIMIT"  : ("oplock contention limit", SambaParmString, P_LOCAL, "2"),
        "DOMAINMASTER"           : ("domain master", SambaParmString, P_GLOBAL, "Auto"),
        "DELETESHARECOMMAND"     : ("delete share command", SambaParmString, P_GLOBAL, ""),
        "INVALIDUSERS"           : ("invalid users", SambaParmString, P_LOCAL, ""),
index b91f965..a74a873 100644 (file)
@@ -2935,8 +2935,6 @@ struct loadparm_context *loadparm_init(TALLOC_CTX *mem_ctx)
 
        lpcfg_do_global_parameter(lp_ctx, "blocking locks", "yes");
 
-       lpcfg_do_global_parameter(lp_ctx, "oplock contention limit", "2");
-
        lpcfg_do_global_parameter(lp_ctx, "load printers", "yes");
 
        lpcfg_do_global_parameter(lp_ctx, "idmap cache time", "604800");
index 42e579e..02c3eb6 100644 (file)
@@ -168,7 +168,6 @@ static struct loadparm_service sDefault =
        .max_connections = 0,
        .default_case = CASE_LOWER,
        .printing = DEFAULT_PRINTING,
-       .oplock_contention_limit = 2,
        .csc_policy = 0,
        .block_size = 1024,
        .dfree_cache_time = 0,