cvs commit: src/sys/kern vfs_mount.c vfs_subr.c vfs_vnops.c src/sys/sys mount.h vnode.h src/sys/ufs/ffs ffs_extern.h ffs_softdep.c ffs_vfsops.c src/sys/ufs/ufs ufs_inode.c ufs_quota.c ufsmount.h

Pawel Jakub Dawidek pjd at
Fri Mar 10 00:18:34 PST 2006

On Wed, Mar 08, 2006 at 11:43:39PM +0000, Tor Egge wrote:
+> tegge       2006-03-08 23:43:39 UTC
+>   FreeBSD src repository
+>   Modified files:
+>     sys/kern             vfs_mount.c vfs_vnops.c vfs_subr.c 
+>     sys/sys              vnode.h mount.h 
+>     sys/ufs/ffs          ffs_extern.h ffs_softdep.c ffs_vfsops.c 
+>     sys/ufs/ufs          ufs_inode.c ufs_quota.c ufsmount.h 
+>   Log:
+>   Use vn_start_secondary_write() and vn_finished_secondary_write() as a
+>   replacement for vn_write_suspend_wait() to better account for secondary write
+>   processing.
+>   Close race where secondary writes could be started after ffs_sync() returned
+>   but before the file system was marked as suspended.
+>   Detect if secondary writes or softdep processing occurred during vnode sync
+>   loop in ffs_sync() and retry the loop if needed.

When exactly new 'suspended' argument for ffs_sbupdate() should be set
to 1? When I call ffs_sbupdate() after calling vfs_write_suspend(),
should I set it to 1?

Pawel Jakub Dawidek             
pjd at                 
FreeBSD committer                         Am I Evil? Yes, I Am!
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 187 bytes
Desc: not available
Url :

More information about the cvs-src mailing list