Date: Tue, 11 Jan 2005 07:36:23 +0000 (UTC) From: Poul-Henning Kamp <phk@FreeBSD.org> To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org Subject: cvs commit: src/sys/coda coda_venus.c coda_venus.h coda_vfsops.c coda_vnops.c src/sys/fs/msdosfs msdosfs_vfsops.c src/sys/fs/nullfs null_vfsops.c src/sys/fs/unionfs union_vnops.c src/sys/gnu/ext2fs ext2_vfsops.c src/sys/kern vfs_bio.c vfs_default.c vfs_mount.c ... Message-ID: <200501110736.j0B7aN4k048927@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
phk 2005-01-11 07:36:23 UTC FreeBSD src repository Modified files: sys/coda coda_venus.c coda_venus.h coda_vfsops.c coda_vnops.c sys/fs/msdosfs msdosfs_vfsops.c sys/fs/nullfs null_vfsops.c sys/fs/unionfs union_vnops.c sys/gnu/ext2fs ext2_vfsops.c sys/kern vfs_bio.c vfs_default.c vfs_mount.c vfs_subr.c vfs_syscalls.c vfs_vnops.c vnode_if.src sys/nfs4client nfs4_vfsops.c nfs4_vnops.c sys/nfsclient nfs_vfsops.c nfs_vnops.c sys/nfsserver nfs_serv.c sys/sys mount.h sys/ufs/ffs ffs_balloc.c ffs_inode.c ffs_rawread.c ffs_snapshot.c ffs_softdep.c ffs_vfsops.c sys/ufs/ufs ufs_lookup.c sys/vm vm_object.c Log: Remove the unused credential argument from VOP_FSYNC() and VFS_SYNC(). I'm not sure why a credential was added to these in the first place, it is not used anywhere and it doesn't make much sense: The credentials for syncing a file (ability to write to the file) should be checked at the system call level. Credentials for syncing one or more filesystems ("none") should be checked at the system call level as well. If the filesystem implementation needs a particular credential to carry out the syncing it would logically have to the cached mount credential, or a credential cached along with any delayed write data. Discussed with: rwatson Revision Changes Path 1.23 +2 -3 src/sys/coda/coda_venus.c 1.8 +1 -2 src/sys/coda/coda_venus.h 1.57 +1 -2 src/sys/coda/coda_vfsops.c 1.57 +2 -3 src/sys/coda/coda_vnops.c 1.137 +4 -5 src/sys/fs/msdosfs/msdosfs_vfsops.c 1.70 +1 -2 src/sys/fs/nullfs/null_vfsops.c 1.118 +1 -1 src/sys/fs/unionfs/union_vnops.c 1.138 +4 -5 src/sys/gnu/ext2fs/ext2_vfsops.c 1.470 +1 -1 src/sys/kern/vfs_bio.c 1.112 +3 -5 src/sys/kern/vfs_default.c 1.176 +1 -1 src/sys/kern/vfs_mount.c 1.560 +3 -3 src/sys/kern/vfs_subr.c 1.364 +2 -3 src/sys/kern/vfs_syscalls.c 1.219 +1 -1 src/sys/kern/vfs_vnops.c 1.72 +0 -1 src/sys/kern/vnode_if.src 1.18 +2 -2 src/sys/nfs4client/nfs4_vfsops.c 1.21 +6 -6 src/sys/nfs4client/nfs4_vnops.c 1.170 +2 -2 src/sys/nfsclient/nfs_vfsops.c 1.246 +7 -7 src/sys/nfsclient/nfs_vnops.c 1.153 +1 -1 src/sys/nfsserver/nfs_serv.c 1.189 +2 -3 src/sys/sys/mount.h 1.47 +4 -4 src/sys/ufs/ffs/ffs_balloc.c 1.97 +3 -3 src/sys/ufs/ffs/ffs_inode.c 1.19 +1 -1 src/sys/ufs/ffs/ffs_rawread.c 1.93 +9 -9 src/sys/ufs/ffs/ffs_snapshot.c 1.165 +7 -7 src/sys/ufs/ffs/ffs_softdep.c 1.265 +5 -7 src/sys/ufs/ffs/ffs_vfsops.c 1.73 +1 -1 src/sys/ufs/ufs/ufs_lookup.c 1.341 +1 -1 src/sys/vm/vm_object.c
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200501110736.j0B7aN4k048927>