c/mirror_zfs
1
0
Fork 0
mirror of https://git.proxmox.com/git/mirror_zfs.git synced 2025-01-14 04:00:31 +03:00
Code Issues Packages Projects Releases Wiki Activity

FreeBSD: stop using buffer cache-only routines on sync

Browse Source 
Both vop_fsync and vfs_stdsync are effectively just costly no-ops
as they only act on ->v_bufobj.bo_dirty et al, which are unused
by zfs.

Reviewed-by: Ryan Moeller <ryan@iXsystems.com>
Signed-off-by:	Mateusz Guzik <mjguzik@gmail.com>
Closes #14157
This commit is contained in:
Mateusz Guzik 2022-11-29 18:35:25 +01:00 committed by GitHub
parent 4df415aa86
commit 9aea88ba44
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 0 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
module/os/freebsd/zfs/zfs_vfsops.c
View File

@ -438,10 +438,6 @@ zfs_sync(vfs_t *vfsp, int waitfor)
dsl_pool_t *dp; dsl_pool_t *dp;
int error; int error;
error = vfs_stdsync(vfsp, waitfor);
if (error != 0)
return (error);
if ((error = zfs_enter(zfsvfs, FTAG)) != 0) if ((error = zfs_enter(zfsvfs, FTAG)) != 0)
return (error); return (error);
dp = dmu_objset_pool(zfsvfs->z_os); dp = dmu_objset_pool(zfsvfs->z_os);

1
module/os/freebsd/zfs/zfs_vnops_os.c
View File

@ -4724,7 +4724,6 @@ static int
zfs_freebsd_fsync(struct vop_fsync_args *ap) zfs_freebsd_fsync(struct vop_fsync_args *ap)
{ {
vop_stdfsync(ap);
return (zfs_fsync(VTOZ(ap->a_vp), 0, ap->a_td->td_ucred)); return (zfs_fsync(VTOZ(ap->a_vp), 0, ap->a_td->td_ucred));
} }