diff options
author | Björn Jacke <bj@sernet.de> | 2019-06-19 13:04:55 +0200 |
---|---|---|
committer | Bjoern Jacke <bjacke@samba.org> | 2019-07-07 21:32:25 +0000 |
commit | d6b8cbc8f7078b1ee5574204f2cac18d20a206e6 (patch) | |
tree | b0b3a58f87279c54ad76e7b34090cfb7272c2b86 | |
parent | 37b3c6375b43cdbb5d06d194a88835c5637d66a0 (diff) | |
download | samba-d6b8cbc8f7078b1ee5574204f2cac18d20a206e6.tar.gz |
param: change default of "allocation roundup size" to 0
Signed-off-by: Bjoern Jacke <bjacke@samba.org>
Reviewed-by: Christof Schmitt <cs@samba.org>
-rw-r--r-- | docs-xml/smbdotconf/tuning/allocationroundupsize.xml | 14 | ||||
-rw-r--r-- | lib/param/loadparm.c | 2 | ||||
-rw-r--r-- | source3/include/local.h | 3 | ||||
-rw-r--r-- | source3/param/loadparm.c | 1 |
4 files changed, 9 insertions, 11 deletions
diff --git a/docs-xml/smbdotconf/tuning/allocationroundupsize.xml b/docs-xml/smbdotconf/tuning/allocationroundupsize.xml index eaea467bc39..aabbba32655 100644 --- a/docs-xml/smbdotconf/tuning/allocationroundupsize.xml +++ b/docs-xml/smbdotconf/tuning/allocationroundupsize.xml @@ -4,17 +4,21 @@ xmlns:samba="http://www.samba.org/samba/DTD/samba-doc"> <description> <para>This parameter allows an administrator to tune the - allocation size reported to Windows clients. The default - size of 1Mb generally results in improved Windows client - performance. However, rounding the allocation size may cause + allocation size reported to Windows clients. This is only + useful for old SMB1 clients because modern SMB dialects + eliminated that bottleneck and have better performance by + default. Using this parameter may cause difficulties for some applications, e.g. MS Visual Studio. If the MS Visual Studio compiler starts to crash with an internal error, set this parameter to zero for this share. + Settings this parameter to a large value can also cause + small files to allocate more space on the disk than + needed. </para> <para>The integer parameter specifies the roundup size in bytes.</para> </description> -<value type="default">1048576</value> -<value type="example">0<comment>(to disable roundups)</comment></value> +<value type="default">0</value> +<value type="example">1048576<comment>(to set it to the former default of 1 MiB)</comment></value> </samba:parameter> diff --git a/lib/param/loadparm.c b/lib/param/loadparm.c index 413e0237800..239d671803f 100644 --- a/lib/param/loadparm.c +++ b/lib/param/loadparm.c @@ -2889,8 +2889,6 @@ struct loadparm_context *loadparm_init(TALLOC_CTX *mem_ctx) lpcfg_do_global_parameter(lp_ctx, "show add printer wizard", "yes"); - lpcfg_do_global_parameter(lp_ctx, "allocation roundup size", "1048576"); - lpcfg_do_global_parameter(lp_ctx, "ldap page size", "1000"); lpcfg_do_global_parameter(lp_ctx, "kernel share modes", "yes"); diff --git a/source3/include/local.h b/source3/include/local.h index 62700aace3c..9ae1bcf5347 100644 --- a/source3/include/local.h +++ b/source3/include/local.h @@ -151,9 +151,6 @@ /* the maximum age in seconds of a password. Should be a lp_ parameter */ #define MAX_PASSWORD_AGE (21*24*60*60) -/* Default allocation roundup. */ -#define SMB_ROUNDUP_ALLOCATION_SIZE 0x100000 - /* shall we deny oplocks to clients that get timeouts? */ #define FASCIST_OPLOCK_BACKOFF 1 diff --git a/source3/param/loadparm.c b/source3/param/loadparm.c index 5104a3408f1..c2e1f341c57 100644 --- a/source3/param/loadparm.c +++ b/source3/param/loadparm.c @@ -236,7 +236,6 @@ static const struct loadparm_service _sDefault = .acl_map_full_control = true, .acl_group_control = false, .acl_allow_execute_always = false, - .allocation_roundup_size = SMB_ROUNDUP_ALLOCATION_SIZE, .aio_read_size = 1, .aio_write_size = 1, .map_readonly = MAP_READONLY_NO, |