From owner-svn-src-projects@freebsd.org Mon May 13 19:47:33 2019 Return-Path: Delivered-To: svn-src-projects@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 4E7E41599A3F for ; Mon, 13 May 2019 19:47:33 +0000 (UTC) (envelope-from asomers@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) server-signature RSA-PSS (4096 bits) client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "Let's Encrypt Authority X3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id DB41A6ACA8; Mon, 13 May 2019 19:47:32 +0000 (UTC) (envelope-from asomers@FreeBSD.org) Received: from repo.freebsd.org (repo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:0]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id C006AE85D; Mon, 13 May 2019 19:47:32 +0000 (UTC) (envelope-from asomers@FreeBSD.org) Received: from repo.freebsd.org ([127.0.1.37]) by repo.freebsd.org (8.15.2/8.15.2) with ESMTP id x4DJlWGF087323; Mon, 13 May 2019 19:47:32 GMT (envelope-from asomers@FreeBSD.org) Received: (from asomers@localhost) by repo.freebsd.org (8.15.2/8.15.2/Submit) id x4DJlVHU087318; Mon, 13 May 2019 19:47:31 GMT (envelope-from asomers@FreeBSD.org) Message-Id: <201905131947.x4DJlVHU087318@repo.freebsd.org> X-Authentication-Warning: repo.freebsd.org: asomers set sender to asomers@FreeBSD.org using -f From: Alan Somers Date: Mon, 13 May 2019 19:47:31 +0000 (UTC) To: src-committers@freebsd.org, svn-src-projects@freebsd.org Subject: svn commit: r347547 - in projects/fuse2: . share/man/man5 sys/fs/fuse X-SVN-Group: projects X-SVN-Commit-Author: asomers X-SVN-Commit-Paths: in projects/fuse2: . share/man/man5 sys/fs/fuse X-SVN-Commit-Revision: 347547 X-SVN-Commit-Repository: base MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Rspamd-Queue-Id: DB41A6ACA8 X-Spamd-Bar: -- Authentication-Results: mx1.freebsd.org X-Spamd-Result: default: False [-2.98 / 15.00]; local_wl_from(0.00)[FreeBSD.org]; NEURAL_HAM_MEDIUM(-1.00)[-0.999,0]; NEURAL_HAM_LONG(-1.00)[-1.000,0]; NEURAL_HAM_SHORT(-0.98)[-0.983,0]; ASN(0.00)[asn:11403, ipnet:2610:1c1:1::/48, country:US] X-BeenThere: svn-src-projects@freebsd.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: "SVN commit messages for the src " projects" tree" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 13 May 2019 19:47:33 -0000 Author: asomers Date: Mon May 13 19:47:31 2019 New Revision: 347547 URL: https://svnweb.freebsd.org/changeset/base/347547 Log: fusefs: remove the vfs.fusefs.sync_resize syctl This sysctl was added > 6.5 years ago for no clear purpose. I'm guessing that it may have had something to do with the incomplete attribute cache. But the attribute cache works now. Since there's no clear motivation for this sysctl, it's best to remove it. Sponsored by: The FreeBSD Foundation Modified: projects/fuse2/UPDATING projects/fuse2/share/man/man5/fusefs.5 projects/fuse2/sys/fs/fuse/fuse_io.c projects/fuse2/sys/fs/fuse/fuse_ipc.h projects/fuse2/sys/fs/fuse/fuse_node.c Modified: projects/fuse2/UPDATING ============================================================================== --- projects/fuse2/UPDATING Mon May 13 19:35:35 2019 (r347546) +++ projects/fuse2/UPDATING Mon May 13 19:47:31 2019 (r347547) @@ -37,9 +37,9 @@ NOTE TO PEOPLE WHO THINK THAT FreeBSD 13.x IS SLOW: been removed from mount_fusefs(8). You can safely remove them from your scripts, because they had no effect. - The vfs.fusefs.fix_broken_io sysctl has been removed. If you felt the - need to set it to a non-default value, please tell asomers@FreeBSD.org - why. + The vfs.fusefs.fix_broken_io and vfs.fusefs.sync_resize sysctls have + been removed. If you felt the need to set either to a non-default + value, please tell asomers@FreeBSD.org why. 20190507: The IPSEC option has been removed from GENERIC. Users requiring Modified: projects/fuse2/share/man/man5/fusefs.5 ============================================================================== --- projects/fuse2/share/man/man5/fusefs.5 Mon May 13 19:35:35 2019 (r347546) +++ projects/fuse2/share/man/man5/fusefs.5 Mon May 13 19:47:31 2019 (r347547) @@ -105,7 +105,6 @@ misbehaving. .\" vfs.fusefs.iov_credit: I don't understand it well enough .\" vfs.fusefs.iov_permanent_bufsize: I don't understand it well enough .\" vfs.fusefs.fix_broken_io: I don't understand it well enough -.\" vfs.fusefs.sync_resize: useless and should be removed .\" vfs.fusefs.refresh_size: probably useless? .\" vfs.fusefs.mmap_enable: why is this optional? .\" vfs.fusefs.data_cache_invalidate: what is this needed for? Modified: projects/fuse2/sys/fs/fuse/fuse_io.c ============================================================================== --- projects/fuse2/sys/fs/fuse/fuse_io.c Mon May 13 19:35:35 2019 (r347546) +++ projects/fuse2/sys/fs/fuse/fuse_io.c Mon May 13 19:47:31 2019 (r347547) @@ -709,7 +709,7 @@ again: break; } while (uio->uio_resid > 0 && n > 0); - if (fuse_sync_resize && (fvdat->flag & FN_SIZECHANGE) != 0) + if ((fvdat->flag & FN_SIZECHANGE) != 0) fuse_vnode_savesize(vp, cred, pid); return (err); Modified: projects/fuse2/sys/fs/fuse/fuse_ipc.h ============================================================================== --- projects/fuse2/sys/fs/fuse/fuse_ipc.h Mon May 13 19:35:35 2019 (r347546) +++ projects/fuse2/sys/fs/fuse/fuse_ipc.h Mon May 13 19:47:31 2019 (r347547) @@ -231,7 +231,6 @@ enum fuse_data_cache_mode { extern int fuse_data_cache_mode; extern int fuse_data_cache_invalidate; extern int fuse_mmap_enable; -extern int fuse_sync_resize; static inline struct fuse_data * fuse_get_mpdata(struct mount *mp) Modified: projects/fuse2/sys/fs/fuse/fuse_node.c ============================================================================== --- projects/fuse2/sys/fs/fuse/fuse_node.c Mon May 13 19:35:35 2019 (r347546) +++ projects/fuse2/sys/fs/fuse/fuse_node.c Mon May 13 19:47:31 2019 (r347547) @@ -135,13 +135,6 @@ SYSCTL_INT(_vfs_fusefs, OID_AUTO, refresh_size, CTLFLA "If non-zero, and no dirty file extension data is buffered, fetch file " "size before write operations"); -int fuse_sync_resize = 1; - -SYSCTL_INT(_vfs_fusefs, OID_AUTO, sync_resize, CTLFLAG_RW, - &fuse_sync_resize, 0, - "If a cached write extended a file, inform FUSE filesystem of the changed" - "size immediately subsequent to the issued writes"); - static int sysctl_fuse_cache_mode(SYSCTL_HANDLER_ARGS) {